@gooddata/test-storybook - npm Package Compare versions

Comparing version 0.1.0-pavelzgdc-ocetnik-doc-test-storybook-2017-12-07T16-32-12-844Z to 0.1.0-pavelzgdc-test-storybook-ignore-no-wrap-2017-12-11T14-13-19-304Z

index.d.ts

@@ -1,1 +0,1 @@
-export const SCREENSHOT_WRAPPER_CLASS: string;
+export const screenshotWrap: Function;

package.json

 {
   "name": "@gooddata/test-storybook",
-  "version": "0.1.0-pavelzgdc-ocetnik-doc-test-storybook-2017-12-07T16-32-12-844Z",
+  "version": "0.1.0-pavelzgdc-test-storybook-ignore-no-wrap-2017-12-11T14-13-19-304Z",
   "description": "Visual regression testing for Storybook",
   "main": "src/index.js",
+  "typings": "./index.d.ts",
   "bin": {
@@ -22,8 +23,7 @@ "run-backstop": "src/run-backstop.js",
     "enzyme": "3.2.0",
-    "enzyme-adapter-react-15.4": "1.0.5",
+    "enzyme-adapter-react-16": "1.1.0",
     "eslint-config-gooddata": "0.0.15",
     "jest": "21.2.1",
-    "react": "15.3.2",
-    "react-addons-test-utils": "15.3.2",
-    "react-dom": "15.3.2"
+    "react": "16.2.0",
+    "react-dom": "16.2.0"
   },
@@ -41,3 +41,6 @@ "dependencies": {
     "react": "*"
+  },
+  "jest": {
+    "setupTestFrameworkScriptFile": "<rootDir>/jest.setup.js"
   }
 }

src/index.js

@@ -14,3 +14,4 @@ const React = require('react');
 module.exports = {
+    SCREENSHOT_WRAPPER_CLASS,
     screenshotWrap
 };

src/run-backstop.js

@@ -36,4 +36,3 @@ #! /usr/bin/env node
 const command = program.reference ? 'reference' : 'test';
-console.log(`running backstopjs("${command}", config)`);
 
 backstopjs(command, { config });

src/test/index.spec.js

 const React = require('react');
 const enzyme = require('enzyme');
-const Adapter = require('enzyme-adapter-react-15.4');
 
 const index = require('../index');
 
-enzyme.configure({ adapter: new Adapter() });
 const InnerComponent = React.createElement('div', { className: 'inner-component' });
 
-describe('index', () => {
+describe('screenshotWrap', () => {
     it('should wrap component', () => {
@@ -12,0 +10,0 @@ const wrappedComponent = enzyme.mount(index.screenshotWrap(InnerComponent));

src/utils.js

@@ -12,6 +12,6 @@ const acornBase = require('acorn');
 
-function getTokens(data) {
+function getTokens(source) {
     try {
         // tokenizer() returns object so we need to convert it to array
-        return [...acorn.tokenizer(data, {
+        return [...acorn.tokenizer(source, {
             plugins: {
@@ -28,18 +28,23 @@ jsx: true,
 
-function isKind(tokens, token, i) {
-    return token.value === 'storiesOf' && tokens[i + 1].type.label === '(';
+function isKind(tokens, i) {
+    return tokens[i].value === 'storiesOf' && tokens[i + 1].type.label === '(';
 }
 
-function isStory(tokens, token, i) {
-    return token.value === 'add' && tokens[i - 1].type.label === '.' && tokens[i + 1].type.label === '(';
+function isStory(tokens, i) {
+    return tokens[i].value === 'add' && tokens[i - 1].type.label === '.' && tokens[i + 1].type.label === '(';
 }
 
-function parseKinds(data) {
+function isScreenshotWrap(tokens, i) {
+    return tokens[i].type.label === 'name' && tokens[i].value === 'screenshotWrap' && tokens[i + 1].type.label === '(';
+
+}
+
+function parseKinds(source) {
     let currentKind = null;
+    let lastStory = null;
 
-    const tokens = getTokens(data);
+    const tokens = getTokens(source);
     const kinds = [];
     for (let i = 0; i < tokens.length; i += 1) {
-        const token = tokens[i];
-        if (isKind(tokens, token, i)) {
+        if (isKind(tokens, i)) {
             currentKind = tokens[i + 2].value;
@@ -53,7 +58,11 @@ kinds.push({
 
-        if (isStory(tokens, token, i)) {
-            const story = tokens[i + 2].value;
-            kinds[kinds.length - 1].stories.push(story);
+        if (isStory(tokens, i)) {
+            lastStory = tokens[i + 2].value;
             i += 2;
         }
+
+        if (lastStory && isScreenshotWrap(tokens, i)) {
+            kinds[kinds.length - 1].stories.push(lastStory);
+            lastStory = null;
+        }
     }
@@ -65,13 +74,11 @@
 function mapKindToScenario(kind, port) {
-    return kind.stories.map((story) => {
-        return {
-            url: `http://localhost:${port}/iframe.html?selectedKind=${encodeURIComponent(kind.kind)}&selectedStory=${encodeURIComponent(story)}`,
-            label: `${kind.kind} - ${story}`,
-            selectors: [`.${SCREENSHOT_WRAPPER_CLASS}`]
-        };
-    });
+    return kind.stories.map((story) => ({
+        url: `http://localhost:${port}/iframe.html?selectedKind=${encodeURIComponent(kind.kind)}&selectedStory=${encodeURIComponent(story)}`,
+        label: `${kind.kind} - ${story}`,
+        selectors: [`.${SCREENSHOT_WRAPPER_CLASS}`]
+    }));
 }
 
 function getScenarios(port) {
-    const fileNames = glob.sync('./stories/**/*.jsx');
+    const fileNames = glob.sync('./stories/**/*.{jsx,tsx}');
 
@@ -81,4 +88,4 @@ const kinds = flatten(
             .map((fileName) => {
-                const data = fs.readFileSync(fileName, 'utf8');
-                return parseKinds(data);
+                const source = fs.readFileSync(fileName, 'utf8');
+                return parseKinds(source);
             })
@@ -85,0 +92,0 @@ .filter(k => k.length > 0)

Fixed alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

180915

increased by409.95%

Dev dependency count

6

decreased by-14.29%

Number of package files

20

increased by17.65%

Lines of code

180

increased by4.65%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes