test: update tests with dataId check

ccontrols · May 9, 2021 · 44047f7 · 44047f7
1 parent 1867ac2
commit 44047f7
Show file tree

Hide file tree

Showing 39 changed files with 65 additions and 52 deletions.
diff --git a/examples/stories/src/stories/VariantButton/VariantButton.test.ts b/examples/stories/src/stories/VariantButton/VariantButton.test.ts
@@ -33,7 +33,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/examples/stories/tsconfig.json b/examples/stories/tsconfig.json
@@ -0,0 +1,13 @@
+{
+  "extends": "../../tsconfig.json",
+  "compilerOptions": {
+    "module": "esnext",
+    "outDir": "dist",
+    "rootDir": "src",
+    "declaration": true,
+    "types": ["node", "jest"],
+    "typeRoots": ["../../node_modules/@types", "node_modules/@types"]
+  },
+  "include": ["src/**/*", "src/typings.d.ts"],
+  "exclude": ["node_modules/**"]
+}
diff --git a/plugins/cc-cli/templates/document/loop/imports-enzyme-react-17.js b/plugins/cc-cli/templates/document/loop/imports-enzyme-react-17.js
@@ -11,7 +11,7 @@ renderedExamples.forEach(({ name, rendered {{? it.data }}, dataId, values{{?}}})
       {{=it.allytest}}
     {{? it.data }}  
       }
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/templates/document/loop/imports-react-test-renderer.js b/plugins/cc-cli/templates/document/loop/imports-react-test-renderer.js
@@ -23,7 +23,7 @@ renderedExamples.forEach(({ name, rendered{{? it.data }}, dataId, values{{?}}})
       {{=it.allytest}}
       {{? it.data }}  
       }
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/templates/document/loop/imports-react-testing-library.js b/plugins/cc-cli/templates/document/loop/imports-react-testing-library.js
@@ -19,7 +19,7 @@ renderedExamples.forEach(({ name, rendered{{? it.data }}, dataId, values{{?}}})
       {{=it.allytest}}
       {{? it.data }}  
     }
-    if (values) {
+    if (dataId && values) {
       describe(dataId, runTests);
     } else {
       runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/cli-document-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/cli-document-data.test.ts.snap
@@ -65,7 +65,7 @@ describe('Popover', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();
@@ -170,7 +170,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();
@@ -230,7 +230,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();
@@ -294,7 +294,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();
@@ -362,7 +362,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();
@@ -426,7 +426,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-enzyme-cjs-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-enzyme-cjs-data.test.ts.snap
@@ -33,7 +33,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-enzyme-esm-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-enzyme-esm-data.test.ts.snap
@@ -32,7 +32,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-enzyme-ts-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-enzyme-ts-data.test.ts.snap
@@ -32,7 +32,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtl-cjs-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtl-cjs-data.test.ts.snap
@@ -37,7 +37,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtl-esm-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtl-esm-data.test.ts.snap
@@ -36,7 +36,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtl-ts-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtl-ts-data.test.ts.snap
@@ -36,7 +36,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtr-cjs-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtr-cjs-data.test.ts.snap
@@ -45,7 +45,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtr-esm-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtr-esm-data.test.ts.snap
@@ -40,7 +40,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/plugins/cc-cli/test/__snapshots__/document-rtr-ts-data.test.ts.snap b/plugins/cc-cli/test/__snapshots__/document-rtr-ts-data.test.ts.snap
@@ -40,7 +40,7 @@ describe('VariantButton', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/app/src/AppError/AppError.test.ts b/ui/app/src/AppError/AppError.test.ts
@@ -33,7 +33,7 @@ describe('AppError', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/blocks/src/ComponentCommits/ComponentCommits.test.ts b/ui/blocks/src/ComponentCommits/ComponentCommits.test.ts
@@ -33,7 +33,7 @@ describe('ComponentCommits', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/blocks/src/TestsCoverage/TestsCoverage.test.ts b/ui/blocks/src/TestsCoverage/TestsCoverage.test.ts
@@ -33,7 +33,7 @@ describe('TestsCoverage', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/blocks/src/TestsResults/TestsResults.test.ts b/ui/blocks/src/TestsResults/TestsResults.test.ts
@@ -33,7 +33,7 @@ describe('TestsResults', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/ActionBar/ActionBar.test.ts b/ui/components/src/ActionBar/ActionBar.test.ts
@@ -33,7 +33,7 @@ describe('ActionBar', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/BlockContainer/BlockContainer.test.ts b/ui/components/src/BlockContainer/BlockContainer.test.ts
@@ -33,7 +33,7 @@ describe('BlockContainer', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/Collapsible/Collapsible.test.ts b/ui/components/src/Collapsible/Collapsible.test.ts
@@ -33,7 +33,7 @@ describe('Collapsible', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/Donut/Donut.test.ts b/ui/components/src/Donut/Donut.test.ts
@@ -33,7 +33,7 @@ describe('Donut', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/ExternalLink/ExternalLink.test.ts b/ui/components/src/ExternalLink/ExternalLink.test.ts
@@ -33,7 +33,7 @@ describe('ExternalLink', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/GithubAvatar/GithubAvatar.test.ts b/ui/components/src/GithubAvatar/GithubAvatar.test.ts
@@ -33,7 +33,7 @@ describe('GithubAvatar', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/InfoTip/InfoTip.test.ts b/ui/components/src/InfoTip/InfoTip.test.ts
@@ -33,7 +33,7 @@ describe('InfoTip', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/LinkHeading/LinkHeading.test.ts b/ui/components/src/LinkHeading/LinkHeading.test.ts
@@ -33,7 +33,7 @@ describe('LinkHeading', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/LinkHeading/__snapshots__/LinkHeading.test.ts.snap b/ui/components/src/LinkHeading/__snapshots__/LinkHeading.test.ts.snap
@@ -76,11 +76,11 @@ exports[`LinkHeading overview 0 snapshot 1`] = `
         class="emotion-1"
       >
         <a
-          aria-label="navigate to navigate to angelica-howell section"
+          aria-label="navigate to angelica-howell section"
           class="emotion-2"
           data-id="angelica-howell"
           data-level="1"
-          data-title="navigate to angelica-howell"
+          data-title="angelica-howell"
           href="http://localhost/#angelica-howell"
         >
           <svg
@@ -188,11 +188,11 @@ exports[`LinkHeading overview 1 snapshot 1`] = `
         class="emotion-1"
       >
         <a
-          aria-label="navigate to navigate to mrs--rosalyn-moen section"
+          aria-label="navigate to mrs--rosalyn-moen section"
           class="emotion-2"
           data-id="mrs--rosalyn-moen"
           data-level="6"
-          data-title="navigate to mrs--rosalyn-moen"
+          data-title="mrs--rosalyn-moen"
           href="http://localhost/#mrs--rosalyn-moen"
         >
           <svg
@@ -298,11 +298,11 @@ exports[`LinkHeading overview 2 snapshot 1`] = `
         class="emotion-1"
       >
         <a
-          aria-label="navigate to navigate to stuart-rutherford section"
+          aria-label="navigate to stuart-rutherford section"
           class="emotion-2"
           data-id="stuart-rutherford"
           data-level="1"
-          data-title="navigate to stuart-rutherford"
+          data-title="stuart-rutherford"
           href="http://localhost/#stuart-rutherford"
         >
           <svg
@@ -412,11 +412,11 @@ exports[`LinkHeading overview 3 snapshot 1`] = `
         class="emotion-1"
       >
         <a
-          aria-label="navigate to navigate to miss-litzy-bergnaum section"
+          aria-label="navigate to miss-litzy-bergnaum section"
           class="emotion-2"
           data-id="miss-litzy-bergnaum"
           data-level="2"
-          data-title="navigate to miss-litzy-bergnaum"
+          data-title="miss-litzy-bergnaum"
           href="http://localhost/#miss-litzy-bergnaum"
         >
           <svg
@@ -524,11 +524,11 @@ exports[`LinkHeading overview 4 snapshot 1`] = `
         class="emotion-1"
       >
         <a
-          aria-label="navigate to navigate to dr--dariana-murray section"
+          aria-label="navigate to dr--dariana-murray section"
           class="emotion-2"
           data-id="dr--dariana-murray"
           data-level="1"
-          data-title="navigate to dr--dariana-murray"
+          data-title="dr--dariana-murray"
           href="http://localhost/#dr--dariana-murray"
         >
           <svg

diff --git a/ui/components/src/Popover/Popover.test.ts b/ui/components/src/Popover/Popover.test.ts
@@ -33,7 +33,7 @@ describe('Popover', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/ProgressIndicator/ProgressIndicator.test.ts b/ui/components/src/ProgressIndicator/ProgressIndicator.test.ts
@@ -33,7 +33,7 @@ describe('ProgressIndicator', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/Shield/Shield.test.ts b/ui/components/src/Shield/Shield.test.ts
@@ -33,7 +33,7 @@ describe('Shield', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();

diff --git a/ui/components/src/Sidebar/Sidebar.test.ts b/ui/components/src/Sidebar/Sidebar.test.ts
@@ -33,7 +33,7 @@ describe('Sidebar', () => {
           expect(axeResults).toHaveNoAxeViolations();
         });
       };
-      if (values) {
+      if (dataId && values) {
         describe(dataId, runTests);
       } else {
         runTests();