Merge pull request plotly#630 from monfera/process-json-mocks-only

etpinard · web-flow · commit 2c7c2dba0cb0 · 2016-06-13T10:05:45.000-04:00
Ignore files in the mock directory that aren't .json (e.g. .DS_Store)
diff --git a/test/image/compare_pixels_test.js b/test/image/compare_pixels_test.js
@@ -36,7 +36,10 @@ else runSingle(userFileName);
 function runAll() {
     test('testing mocks', function(t) {
 
-        var allMocks = fs.readdirSync(constants.pathToTestImageMocks);
+        var fileNames = fs.readdirSync(constants.pathToTestImageMocks);
+
+        // eliminate pollutants (e.g .DS_Store) that can accumulate in the mock directory
+        var allMocks = fileNames.filter(function(name) {return name.slice(-5) === '.json';});
 
         /* Test cases:
          *
