From 5ae7863d8f53950972dfcc53ceee207a69dbb6b3 Mon Sep 17 00:00:00 2001 From: Ayush Sharma Date: Sat, 2 Dec 2017 19:31:06 +0530 Subject: [PATCH] webui snapshot testing deps fixes for ui tests snapshot fixes --- .gitignore | 1 - jest.config.js | 3 +++ package.json | 2 -- .../components/__snapshots__/header.spec.js.snap | 3 +++ .../components/__snapshots__/help.spec.js.snap | 5 +++++ .../components/__snapshots__/noitems.spec.js.snap | 3 +++ .../components/__snapshots__/notfound.spec.js.snap | 3 +++ .../components/__snapshots__/package.spec.js.snap | 3 +++ .../__snapshots__/packagedetail.spec.js.snap | 3 +++ .../__snapshots__/packagelist.spec.js.snap | 3 +++ .../components/__snapshots__/readme.spec.js.snap | 3 +++ .../components/__snapshots__/search.spec.js.snap | 3 +++ test/webui/components/header.spec.js | 13 ++++++++++++- test/webui/components/help.spec.js | 2 ++ test/webui/components/noitems.spec.js | 1 + test/webui/components/notfound.spec.js | 1 + test/webui/components/package.spec.js | 1 + test/webui/components/packagedetail.spec.js | 1 + test/webui/components/packagelist.spec.js | 2 ++ test/webui/components/readme.spec.js | 1 + test/webui/components/search.spec.js | 10 +++++++--- 21 files changed, 60 insertions(+), 7 deletions(-) create mode 100644 test/webui/components/__snapshots__/header.spec.js.snap create mode 100644 test/webui/components/__snapshots__/help.spec.js.snap create mode 100644 test/webui/components/__snapshots__/noitems.spec.js.snap create mode 100644 test/webui/components/__snapshots__/notfound.spec.js.snap create mode 100644 test/webui/components/__snapshots__/package.spec.js.snap create mode 100644 test/webui/components/__snapshots__/packagedetail.spec.js.snap create mode 100644 test/webui/components/__snapshots__/packagelist.spec.js.snap create mode 100644 test/webui/components/__snapshots__/readme.spec.js.snap create mode 100644 test/webui/components/__snapshots__/search.spec.js.snap diff --git a/.gitignore b/.gitignore index 220045757..6dbf2a18b 100644 --- a/.gitignore +++ b/.gitignore @@ -26,7 +26,6 @@ coverage/ bundle.js bundle.js.map __tests__ -__snapshots__ # Compiled script static/ diff --git a/jest.config.js b/jest.config.js index 71f15ca6f..bc19b4ba2 100644 --- a/jest.config.js +++ b/jest.config.js @@ -16,6 +16,9 @@ module.exports = { 'modulePathIgnorePatterns': [ 'global.js' ], + 'testPathIgnorePatterns': [ + 'test/webui/components/__snapshots__/' + ], 'moduleNameMapper': { '\\.(scss)$': '/node_modules/identity-obj-proxy', 'github-markdown-css': '/node_modules/identity-obj-proxy', diff --git a/package.json b/package.json index 5bc753160..836ba2184 100644 --- a/package.json +++ b/package.json @@ -98,8 +98,6 @@ "identity-obj-proxy": "^3.0.0", "in-publish": "2.0.0", "jest": "^21.2.1", - "jest-css-modules": "^1.1.0", - "jest-serializer-enzyme": "^1.0.0", "localstorage-memory": "1.0.2", "node-sass": "4.7.2", "normalize.css": "7.0.0", diff --git a/test/webui/components/__snapshots__/header.spec.js.snap b/test/webui/components/__snapshots__/header.spec.js.snap new file mode 100644 index 000000000..2d13c0cc4 --- /dev/null +++ b/test/webui/components/__snapshots__/header.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`
snapshot test shoud match snapshot 1`] = `"
npm set registry nullblank
npm adduser --registry nullblank
Login



"`; diff --git a/test/webui/components/__snapshots__/help.spec.js.snap b/test/webui/components/__snapshots__/help.spec.js.snap new file mode 100644 index 000000000..6ad719437 --- /dev/null +++ b/test/webui/components/__snapshots__/help.spec.js.snap @@ -0,0 +1,5 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should set html from props with / base path 1`] = `"

  • No Package Published Yet

    To publish your first package just:

    1. Login
    npm adduser --registry  http://example.com
    2. Publish
    npm publish --registry http://example.com
    3. Refresh this page!
    • "`; + +exports[` component should set html from props with someOtherPath 1`] = `"

  • No Package Published Yet

    To publish your first package just:

    1. Login
    npm adduser --registry  http://example.com/someOtherPath
    2. Publish
    npm publish --registry http://example.com/someOtherPath
    3. Refresh this page!
    • "`; diff --git a/test/webui/components/__snapshots__/noitems.spec.js.snap b/test/webui/components/__snapshots__/noitems.spec.js.snap new file mode 100644 index 000000000..cfb8d0a5b --- /dev/null +++ b/test/webui/components/__snapshots__/noitems.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should set html from props 1`] = `"

    This is a test string

    "`; diff --git a/test/webui/components/__snapshots__/notfound.spec.js.snap b/test/webui/components/__snapshots__/notfound.spec.js.snap new file mode 100644 index 000000000..f84e66ceb --- /dev/null +++ b/test/webui/components/__snapshots__/notfound.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should set html from props 1`] = `"

    Error 404 - verdaccio

    Oops, The package you are trying to access does not exist.

    "`; diff --git a/test/webui/components/__snapshots__/package.spec.js.snap b/test/webui/components/__snapshots__/package.spec.js.snap new file mode 100644 index 000000000..cd97395d6 --- /dev/null +++ b/test/webui/components/__snapshots__/package.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should load the component 1`] = `"

    verdacciov1.0.0

    By: Sam

    Private NPM repository

    "`; diff --git a/test/webui/components/__snapshots__/packagedetail.spec.js.snap b/test/webui/components/__snapshots__/packagedetail.spec.js.snap new file mode 100644 index 000000000..055e7c2f2 --- /dev/null +++ b/test/webui/components/__snapshots__/packagedetail.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should load the component 1`] = `"

    Verdaccio

    Test readme
    "`; diff --git a/test/webui/components/__snapshots__/packagelist.spec.js.snap b/test/webui/components/__snapshots__/packagelist.spec.js.snap new file mode 100644 index 000000000..58638ce7d --- /dev/null +++ b/test/webui/components/__snapshots__/packagelist.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should load the component with packages 1`] = `""`; diff --git a/test/webui/components/__snapshots__/readme.spec.js.snap b/test/webui/components/__snapshots__/readme.spec.js.snap new file mode 100644 index 000000000..5ad638277 --- /dev/null +++ b/test/webui/components/__snapshots__/readme.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should dangerously set html 1`] = `"

    This is a test string

    "`; diff --git a/test/webui/components/__snapshots__/search.spec.js.snap b/test/webui/components/__snapshots__/search.spec.js.snap new file mode 100644 index 000000000..db744e042 --- /dev/null +++ b/test/webui/components/__snapshots__/search.spec.js.snap @@ -0,0 +1,3 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[` component should match the snapshot 1`] = `""`; diff --git a/test/webui/components/header.spec.js b/test/webui/components/header.spec.js index af96b4c0d..efb966371 100644 --- a/test/webui/components/header.spec.js +++ b/test/webui/components/header.spec.js @@ -2,7 +2,7 @@ * Header component */ import React from 'react'; -import { shallow } from 'enzyme'; +import { shallow, mount } from 'enzyme'; import Header from '../../../src/webui/src/components/Header'; import { BrowserRouter } from 'react-router-dom'; import storage from '../../../src/webui/utils/storage'; @@ -103,3 +103,14 @@ describe('
    component shallow', () => { }); }); }); + +describe('
    snapshot test', () => { + it('shoud match snapshot', () => { + const wrapper = mount( + +
    + + ); + expect(wrapper.html()).toMatchSnapshot(); + }); +}); diff --git a/test/webui/components/help.spec.js b/test/webui/components/help.spec.js index e3e04c905..852273319 100644 --- a/test/webui/components/help.spec.js +++ b/test/webui/components/help.spec.js @@ -29,6 +29,7 @@ describe(' component', () => { .dive() .text() ).toEqual('npm adduser --registry http://example.com'); + expect(wrapper.html()).toMatchSnapshot(); }); it('should set html from props with someOtherPath', () => { @@ -44,5 +45,6 @@ describe(' component', () => { .dive() .text() ).toEqual('npm publish --registry http://example.com/someOtherPath'); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/noitems.spec.js b/test/webui/components/noitems.spec.js index 515e135a5..8c270273b 100644 --- a/test/webui/components/noitems.spec.js +++ b/test/webui/components/noitems.spec.js @@ -20,5 +20,6 @@ describe(' component', () => { }; const wrapper = shallow(); expect(wrapper.find('h2').text()).toEqual('This is a test string'); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/notfound.spec.js b/test/webui/components/notfound.spec.js index 826ff9ea0..026c7248e 100644 --- a/test/webui/components/notfound.spec.js +++ b/test/webui/components/notfound.spec.js @@ -23,5 +23,6 @@ describe(' component', () => { expect(wrapper.find('p').text()).toEqual( 'Oops, The package you are trying to access does not exist.' ); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/package.spec.js b/test/webui/components/package.spec.js index bfd391620..7affa8ff1 100644 --- a/test/webui/components/package.spec.js +++ b/test/webui/components/package.spec.js @@ -37,5 +37,6 @@ describe(' component', () => { .text() ).toEqual('By: Sam'); expect(wrapper.find('p').text()).toEqual('Private NPM repository'); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/packagedetail.spec.js b/test/webui/components/packagedetail.spec.js index 44aec9a7b..5858bb90d 100644 --- a/test/webui/components/packagedetail.spec.js +++ b/test/webui/components/packagedetail.spec.js @@ -28,5 +28,6 @@ describe(' component', () => { .dive() .html() ).toEqual('
    Test readme
    '); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/packagelist.spec.js b/test/webui/components/packagelist.spec.js index 31a28ad53..f3617a3ef 100644 --- a/test/webui/components/packagelist.spec.js +++ b/test/webui/components/packagelist.spec.js @@ -69,5 +69,7 @@ describe(' component', () => { ); // package count expect(wrapper.find('Package')).toHaveLength(3); + // match snapshot + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/readme.spec.js b/test/webui/components/readme.spec.js index d1f6ed7d8..6f53b0d0e 100644 --- a/test/webui/components/readme.spec.js +++ b/test/webui/components/readme.spec.js @@ -22,5 +22,6 @@ describe(' component', () => { expect(wrapper.html()).toEqual( '

    This is a test string

    ' ); + expect(wrapper.html()).toMatchSnapshot(); }); }); diff --git a/test/webui/components/search.spec.js b/test/webui/components/search.spec.js index 0a7730713..e277819e0 100644 --- a/test/webui/components/search.spec.js +++ b/test/webui/components/search.spec.js @@ -5,7 +5,6 @@ import React from 'react'; import { shallow } from 'enzyme'; import Search from '../../../src/webui/src/components/Search/index'; - console.error = jest.fn(); describe(' component', () => { @@ -32,11 +31,16 @@ describe(' component', () => { it('should call the handleSearchInput function', () => { const props = { - handleSearchInput: jest.fn(), - placeHolder: 'Test placeholder' + handleSearchInput: jest.fn() }; const wrapper = shallow(); wrapper.find('input').simulate('change'); expect(props.handleSearchInput).toBeCalled(); }); + + it('should match the snapshot', () => { + const props = { handleSearchInput: () => {} }; + const wrapper = shallow(); + expect(wrapper.html()).toMatchSnapshot(); + }); });