[FAB-6612] Handle Windows tempdir in node-sdk test

This CR escapes the filepath used for the debug logger on
Windows so that HFC_LOGGING contains properly formatted JSON
and can be parsed successfully.

Before:
$ gulp test-headless
...
[17:15:43] Starting 'test-headless'...
[17:15:43] Starting 'test-headless'...
warn: Failed to parse environment variable "HFC_LOGGING". Returned
 a winston logger with default configurations. Error:
SyntaxError: Unexpected token U in JSON at position 13
    at Object.parse (native)
...

$ ll ~/AppData/Local/Temp/hfc/test-log/debug.log
-rwx------+ 1 lehors None 0 Oct 13 17:15 /home/lehors/AppData/Local
/Temp/hfc/test-log/debug.log

After:
$ gulp test-headless
...
[17:17:34] Starting 'test-headless'...

TAP version 13

*** BlockDecoder test for readwrite sets
ok 1 Successfully build a results proto with read and write sets
...

$ ll ~/AppData/Local/Temp/hfc/test-log/debug.log
-rw-r--r-- 1 lehors 197121 462970 Oct 13 17:18 /c/Users/lehors/AppD
ata/Local/Temp/hfc/test-log/debug.log

This CR also fixes a typo in the fabric-ca-client tests.

Change-Id: Ia767aa135eb8803608de0322d7e4675326026b08
Signed-off-by: Arnaud J Le Hors <[email protected]>

build/tasks/test.js

@@ -20,7 +20,13 @@ var testConstants = require('../../test/unit/constants.js');
 
 // by default for running the tests print debug to a file
 var debugPath = path.join(testConstants.tempdir, 'test-log/debug.log');
-process.env.HFC_LOGGING = util.format('{"debug":"%s"}', debugPath);
+process.env.HFC_LOGGING = util.format('{"debug":"%s"}', escapeWindowsPath(debugPath));
+
+function escapeWindowsPath(p) {
+	if (path.sep == "/") return p
+	return p.replace(/\\/g, "\\\\")
+}
+
 console.log('\n####################################################');
 console.log(util.format('# debug log: %s', debugPath));
 console.log('####################################################\n');

test/unit/fabric-ca-client.js

@@ -269,8 +269,8 @@ test('FabricCAServices:  Test newCryptoKeyStore() function', function(t) {
 	}
 	t.end();
 });
-// Test optional consturctor
-test('FabricCAServices:  Test optional consturctor', function(t) {
+// Test optional constructor
+test('FabricCAServices:  Test optional constructor', function(t) {
 	let opts = {
 		url : 'http://localhost:7054'
 	}
@@ -280,7 +280,7 @@ test('FabricCAServices:  Test optional consturctor', function(t) {
 			let service = new FabricCAServices(opts);
 		},
 		null,
-		'FabricCAServices consturctor with object and just url'
+		'FabricCAServices constructor with object and just url'
 	);
 
 	opts.caName = 'someca';
@@ -290,7 +290,7 @@ test('FabricCAServices:  Test optional consturctor', function(t) {
 			t.equals(service._fabricCAClient._caName,'someca','Caname should have been passed correctly');
 		},
 		null,
-		'FabricCAServices consturctor with object and just url'
+		'FabricCAServices constructor with object and just url'
 	);
 
 	opts.cryptoSuite = 'somesuite';
@@ -300,7 +300,7 @@ test('FabricCAServices:  Test optional consturctor', function(t) {
 			t.equals(service._fabricCAClient._cryptoPrimitives,'somesuite','CryptoSuite should have been passed correctly');
 		},
 		null,
-		'FabricCAServices consturctor with object and just url'
+		'FabricCAServices constructor with object and just url'
 	);
 
 	opts.tlsOptions = {verify : 'someverify'};
@@ -310,7 +310,7 @@ test('FabricCAServices:  Test optional consturctor', function(t) {
 			t.equals(service._fabricCAClient._tlsOptions.verify,'someverify','tlsOptions should have been passed correctly');
 		},
 		null,
-		'FabricCAServices consturctor with object and just url'
+		'FabricCAServices constructor with object and just url'
 	);
 
 	t.end();