Merge pull request #5650 from plotly/reduce-image-test-artifacts

archmoj · web-flow · commit a47c398f7597 · 2021-05-10T18:27:07.000-04:00
Reduce unnecessary artifacts during image test
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -93,7 +93,7 @@ jobs:
             npm run docker -- setup
       - run:
           name: Run image tests (part A)
-          command: ./.circleci/test.sh stable-image
+          command: ./.circleci/test.sh stable-image ; find build -maxdepth 1 -type f -delete
       - store_artifacts:
           path: build
           destination: /
diff --git a/test/image/compare_pixels_test.js b/test/image/compare_pixels_test.js
@@ -270,6 +270,7 @@ function comparePixels(mockName, cb) {
         }
         if(isEqual) {
             fs.unlinkSync(imagePaths.diff);
+            fs.unlinkSync(imagePaths.test);
         }
 
         cb(isEqual, mockName);

Original file line number	Diff line number	Diff line change
`@@ -270,6 +270,7 @@ function comparePixels(mockName, cb) {`
`270`	`270`	`}`
`271`	`271`	`if(isEqual) {`
`272`	`272`	`fs.unlinkSync(imagePaths.diff);`
	`273`	`+ fs.unlinkSync(imagePaths.test);`
`273`	`274`	`}`
`274`	`275`
`275`	`276`	`cb(isEqual, mockName);`