Skip to content

Commit

Permalink
Fix node-sdk-unit-tests target
Browse files Browse the repository at this point in the history
Change 899 introduced a new unit test for the node sdk
which causes the node sdk unit tests to fail.  The fix was to
change the chaincodeID from mycc3 to mycc4 for the new test.
This amends FAB-285

Fixes FAB-285

Change-Id: I8bdfec16f7e46727995cb755c9e51a5e27658de0
Signed-off-by: Gari Singh <gari.r.singh@gmail.com>
  • Loading branch information
mastersingh24 committed Sep 12, 2016
1 parent 491eb19 commit 7f5f727
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion sdk/node/bin/run-unit-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ runAssetMgmtWithRolesTests() {

runAssetMgmtWithDynamicRolesTests() {
echo "BEGIN running asset management with dynamic roles tests ..."
preExample asset_management_with_roles mycc3
preExample asset_management_with_roles mycc4
node $UNITTEST/asset-mgmt-with-dynamic-roles.js
if [ $? -ne 0 ]; then
echo "ERROR running asset management with dynamic roles tests!"
Expand Down
2 changes: 1 addition & 1 deletion sdk/node/test/unit/asset-mgmt-with-dynamic-roles.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ var util = require('util');
var crypto = require('../../lib/crypto');

var chain, chaincodeID;
var chaincodeName = "mycc3";
var chaincodeName = "mycc4";
var deployer, alice, bob, assigner;
var aliceAccount = "12345-56789";
var bobAccount = "23456-67890";
Expand Down

0 comments on commit 7f5f727

Please sign in to comment.