From e07b145910ca1c61c69727281e012f46b7e510e3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rafa=C5=82=20Pocztarski?= Date: Wed, 2 Dec 2015 18:13:27 +0100 Subject: [PATCH] test: Skip long path tests on non-Windows If not running on Windows it skips the long path tests in: * test-fs-long-path.js * test-require-long-path.js It fixes issue nodejs/node#2255 that is currently: wontfix For context see comments to PRs: nodejs/node#3925 nodejs/node#3929 nodejs/node#4116 --- test/parallel/test-fs-long-path.js | 5 +++++ test/parallel/test-require-long-path.js | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/test/parallel/test-fs-long-path.js b/test/parallel/test-fs-long-path.js index 6ecf54902964fa..eadbcd1f78df98 100644 --- a/test/parallel/test-fs-long-path.js +++ b/test/parallel/test-fs-long-path.js @@ -4,6 +4,11 @@ var fs = require('fs'); var path = require('path'); var assert = require('assert'); +if (!common.isWindows) { + console.log('1..0 # Skipped: this test is Windows-specific.'); + return; +} + var successes = 0; // make a path that will be at least 260 chars long. diff --git a/test/parallel/test-require-long-path.js b/test/parallel/test-require-long-path.js index 1f7e28cba2960c..c30a76a5a9e099 100644 --- a/test/parallel/test-require-long-path.js +++ b/test/parallel/test-require-long-path.js @@ -3,6 +3,11 @@ const common = require('../common'); const fs = require('fs'); const path = require('path'); +if (!common.isWindows) { + console.log('1..0 # Skipped: this test is Windows-specific.'); + return; +} + // make a path that is more than 260 chars long. const dirNameLen = Math.max(260 - common.tmpDir.length, 1); const dirName = path.join(common.tmpDir, 'x'.repeat(dirNameLen));