moved diff testing code to avoid failing snapshot test

attempting to ignore node_modules in diff

.github/workflows/ci.yml

@@ -35,7 +35,7 @@ jobs:
       - name: 📄 Bundle size report
         uses: ./.github/actions/bundle-size # path to composite action
         with:
-          paths: "packages/**/*.min.js"
+          paths: "packages/**/dist/*.min.js"
           onlyDiff: "true"
           # branch: "develop" # branch to compare to
           header: "Bundle size report" # PR comment header

packages/victory-candlestick/src/helper-methods.ts

@@ -488,4 +488,12 @@ export const getBaseProps = (initialProps, fallbackProps) => {
 };
 
 // @TODO: remove this before merging, it's just for verifying the analysis and report.
-const fake = () => "fake";
+
+// @TODO: remove this when analyzer diff is confirmed to work.
+export const bundleSizeIsDifferent = () => {
+  const x = "the bundle size should be different";
+  const y = "the bundle size should be different";
+  const z = "the bundle size should be different";
+
+  return { x, y, z };
+};

packages/victory-candlestick/src/victory-candlestick.tsx

@@ -306,12 +306,3 @@ class VictoryCandlestickBase extends React.Component<VictoryCandlestickProps> {
 }
 
 export const VictoryCandlestick = addEvents(VictoryCandlestickBase, options);
-
-// @TODO: remove this when analyzer diff is confirmed to work.
-export const bundleSizeIsDifferent = () => {
-  const x = "the bundle size should be different";
-  const y = "the bundle size should be different";
-  const z = "the bundle size should be different";
-
-  return { x, y, z };
-};