Skip tests on Windows

nfischer · nfischer · commit 3f79e04be5df · 2016-04-06T00:37:56.000-07:00
diff --git a/test/cp.js b/test/cp.js
@@ -231,36 +231,38 @@ assert.equal(shell.error(), null); // crash test only
 assert.ok(!result.stderr);
 assert.equal(result.code, 0);
 
-// Recursive, everything exists, overwrite a real file with a link (if same name)
-// Because -R implies to not follow links!
-shell.rm('-rf', 'tmp/*');
-shell.cp('-R', 'resources/cp/*', 'tmp');
-assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
-assert.ok(!(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink())); // this one isn't
-assert.notEqual(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
-result = shell.cp('-R', 'tmp/links/*', 'tmp/fakeLinks');
-assert.equal(shell.error(), null);
-assert.ok(!result.stderr);
-assert.equal(result.code, 0);
-assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
-assert.ok(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink()); // this one is now a link
-assert.equal(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
-
-// Recursive, everything exists, overwrite a real file *by following a link*
-// Because missing the -R implies -L.
-shell.rm('-rf', 'tmp/*');
-shell.cp('-R', 'resources/cp/*', 'tmp');
-assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
-assert.ok(!(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink())); // this one isn't
-assert.notEqual(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
-result = shell.cp('tmp/links/*', 'tmp/fakeLinks'); // don't use -R
-assert.equal(shell.error(), null);
-assert.ok(!result.stderr);
-assert.equal(result.code, 0);
-assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
-assert.ok(!fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink()); // this one is still not a link
-// But it still follows the link
-assert.equal(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
+if (process.platform !== 'win32') {
+  // Recursive, everything exists, overwrite a real file with a link (if same name)
+  // Because -R implies to not follow links!
+  shell.rm('-rf', 'tmp/*');
+  shell.cp('-R', 'resources/cp/*', 'tmp');
+  assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
+  assert.ok(!(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink())); // this one isn't
+  assert.notEqual(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
+  result = shell.cp('-R', 'tmp/links/*', 'tmp/fakeLinks');
+  assert.equal(shell.error(), null);
+  assert.ok(!result.stderr);
+  assert.equal(result.code, 0);
+  assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
+  assert.ok(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink()); // this one is now a link
+  assert.equal(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
+
+  // Recursive, everything exists, overwrite a real file *by following a link*
+  // Because missing the -R implies -L.
+  shell.rm('-rf', 'tmp/*');
+  shell.cp('-R', 'resources/cp/*', 'tmp');
+  assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
+  assert.ok(!(fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink())); // this one isn't
+  assert.notEqual(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
+  result = shell.cp('tmp/links/*', 'tmp/fakeLinks'); // don't use -R
+  assert.equal(shell.error(), null);
+  assert.ok(!result.stderr);
+  assert.equal(result.code, 0);
+  assert.ok(fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink()); // this one is a link
+  assert.ok(!fs.lstatSync('tmp/fakeLinks/sym.lnk').isSymbolicLink()); // this one is still not a link
+  // But it still follows the link
+  assert.equal(shell.cat('tmp/links/sym.lnk').toString(), shell.cat('tmp/fakeLinks/sym.lnk').toString());
+}
 
 //recursive, everything exists, with force flag
 shell.rm('-rf', 'tmp/*');
@@ -306,12 +308,12 @@ assert.equal(fs.existsSync('tmp/dest/z'), true);
 
 // On Windows, permission bits are quite different so skip those tests for now
 if (common.platform !== 'win') {
-    //preserve mode bits
-    shell.rm('-rf', 'tmp/*');
-    var execBit = parseInt('001', 8);
-    assert.equal(fs.statSync('resources/cp-mode-bits/executable').mode & execBit, execBit);
-    shell.cp('resources/cp-mode-bits/executable', 'tmp/executable');
-    assert.equal(fs.statSync('resources/cp-mode-bits/executable').mode, fs.statSync('tmp/executable').mode);
+  //preserve mode bits
+  shell.rm('-rf', 'tmp/*');
+  var execBit = parseInt('001', 8);
+  assert.equal(fs.statSync('resources/cp-mode-bits/executable').mode & execBit, execBit);
+  shell.cp('resources/cp-mode-bits/executable', 'tmp/executable');
+  assert.equal(fs.statSync('resources/cp-mode-bits/executable').mode, fs.statSync('tmp/executable').mode);
 }
 
 // Make sure hidden files are copied recursively
@@ -335,7 +337,7 @@ assert.ok(fs.existsSync('tmp/file1.txt'));
 shell.rm('-rf', 'tmp/');
 shell.mkdir('tmp/');
 result = shell.cp('resources/file1.txt', 'resources/file2.txt', 'resources/cp',
-    'resources/ls/', 'tmp/');
+  'resources/ls/', 'tmp/');
 assert.ok(shell.error());
 assert.ok(!fs.existsSync('tmp/.hidden_file')); // doesn't copy dir contents
 assert.ok(!fs.existsSync('tmp/ls')); // doesn't copy dir itself
@@ -344,6 +346,36 @@ assert.ok(!fs.existsSync('tmp/cp')); // doesn't copy dir itself
 assert.ok(fs.existsSync('tmp/file1.txt'));
 assert.ok(fs.existsSync('tmp/file2.txt'));
 
+if (process.platform !== 'win32') {
+  // -R implies -P
+  shell.rm('-rf', 'tmp/*');
+  shell.cp('-R', 'resources/cp/links/sym.lnk', 'tmp');
+  assert.ok(fs.lstatSync('tmp/sym.lnk').isSymbolicLink());
+
+  // using -P explicitly works
+  shell.rm('-rf', 'tmp/*');
+  shell.cp('-P', 'resources/cp/links/sym.lnk', 'tmp');
+  assert.ok(fs.lstatSync('tmp/sym.lnk').isSymbolicLink());
+
+  // using -PR on a link to a folder does not follow the link
+  shell.rm('-rf', 'tmp/*');
+  shell.cp('-PR', 'resources/cp/symFolder', 'tmp');
+  assert.ok(fs.lstatSync('tmp/symFolder').isSymbolicLink());
+
+  // Recursive, copies entire directory with no symlinks and -L option does not cause change in behavior.
+  shell.rm('-rf', 'tmp/*');
+  result = shell.cp('-rL', 'resources/cp/dir_a', 'tmp/dest');
+  assert.equal(shell.error(), null);
+  assert.ok(!result.stderr);
+  assert.equal(result.code, 0);
+  assert.equal(fs.existsSync('tmp/dest/z'), true);
+}
+
+// using -R on a link to a folder *does* follow the link
+shell.rm('-rf', 'tmp/*');
+shell.cp('-R', 'resources/cp/symFolder', 'tmp');
+assert.ok(!fs.lstatSync('tmp/symFolder').isSymbolicLink());
+
 // Without -R, -L is implied
 shell.rm('-rf', 'tmp/*');
 shell.cp('resources/cp/links/sym.lnk', 'tmp');
@@ -359,39 +391,11 @@ shell.rm('-rf', 'tmp/*');
 shell.cp('-LR', 'resources/cp/links/sym.lnk', 'tmp');
 assert.ok(!fs.lstatSync('tmp/sym.lnk').isSymbolicLink());
 
-// -R implies -P
-shell.rm('-rf', 'tmp/*');
-shell.cp('-R', 'resources/cp/links/sym.lnk', 'tmp');
-assert.ok(fs.lstatSync('tmp/sym.lnk').isSymbolicLink());
-
-// using -P explicitly works
-shell.rm('-rf', 'tmp/*');
-shell.cp('-P', 'resources/cp/links/sym.lnk', 'tmp');
-assert.ok(fs.lstatSync('tmp/sym.lnk').isSymbolicLink());
-
 // using -LR also works recursively on directories containing links
 shell.rm('-rf', 'tmp/*');
 shell.cp('-LR', 'resources/cp/links', 'tmp');
 assert.ok(!fs.lstatSync('tmp/links/sym.lnk').isSymbolicLink());
 
-// using -PR on a link to a folder does not follow the link
-shell.rm('-rf', 'tmp/*');
-shell.cp('-PR', 'resources/cp/symFolder', 'tmp');
-assert.ok(fs.lstatSync('tmp/symFolder').isSymbolicLink());
-
-// using -R on a link to a folder *does* follow the link
-shell.rm('-rf', 'tmp/*');
-shell.cp('-R', 'resources/cp/symFolder', 'tmp');
-assert.ok(!fs.lstatSync('tmp/symFolder').isSymbolicLink());
-
-// Recursive, copies entire directory with no symlinks and -L option does not cause change in behavior.
-shell.rm('-rf', 'tmp/*');
-result = shell.cp('-rL', 'resources/cp/dir_a', 'tmp/dest');
-assert.equal(shell.error(), null);
-assert.ok(!result.stderr);
-assert.equal(result.code, 0);
-assert.equal(fs.existsSync('tmp/dest/z'), true);
-
 // Test max depth.
 shell.rm('-rf', 'tmp/');
 shell.mkdir('tmp/');
diff --git a/test/rm.js b/test/rm.js
@@ -226,14 +226,16 @@ assert.equal(fs.existsSync('tmp/rm/link_to_a_dir'), false);
 assert.equal(fs.existsSync('tmp/rm/a_dir'), true);
 
 // remove broken symbolic link
-result = shell.rm('-rf', 'tmp');
-shell.mkdir('tmp');
-shell.cp('-R', 'resources/rm', 'tmp');
-assert.ok(shell.test('-L', 'tmp/rm/fake.lnk'));
-result = shell.rm('tmp/rm/fake.lnk');
-assert.equal(shell.error(), null);
-assert.equal(result.code, 0);
-assert.ok(!shell.test('-L', 'tmp/rm/fake.lnk'));
-assert.equal(fs.existsSync('tmp/rm/fake.lnk'), false);
+if (process.platform !== 'win32') {
+  result = shell.rm('-rf', 'tmp');
+  shell.mkdir('tmp');
+  shell.cp('-R', 'resources/rm', 'tmp');
+  assert.ok(shell.test('-L', 'tmp/rm/fake.lnk'));
+  result = shell.rm('tmp/rm/fake.lnk');
+  assert.equal(shell.error(), null);
+  assert.equal(result.code, 0);
+  assert.ok(!shell.test('-L', 'tmp/rm/fake.lnk'));
+  assert.equal(fs.existsSync('tmp/rm/fake.lnk'), false);
+}
 
 shell.exit(123);
diff --git a/test/touch.js b/test/touch.js
@@ -110,11 +110,13 @@ assert(fs.existsSync(testFile));
 assert(fs.existsSync(testFile2));
 
 // touching broken link creates a new file
-result = shell.touch('resources/badlink');
-assert.equal(result.code, 0);
-assert.ok(!shell.error());
-assert.ok(fs.existsSync('resources/not_existed_file'));
-shell.rm('resources/not_existed_file');
+if (process.platform !== 'win32') {
+  result = shell.touch('resources/badlink');
+  assert.equal(result.code, 0);
+  assert.ok(!shell.error());
+  assert.ok(fs.existsSync('resources/not_existed_file'));
+  shell.rm('resources/not_existed_file');
+}
 
 function resetUtimes(f) {
   var d = new Date();
