Test RelationForm

tangzhen · Mar 27, 2019 · b98905a · b98905a
1 parent a3419da
commit b98905a
Show file tree

Hide file tree

Showing 10 changed files with 845 additions and 49 deletions.
diff --git a/jest.config.front.js b/jest.config.front.js
@@ -2,41 +2,16 @@ const jest = require('jest');
 
 module.exports = {
   collectCoverageFrom: [
-    // 'packages/strapi-admin/admin/src/**/**/*.{js,jsx}',
     'packages/strapi-plugin-content-type-builder/admin/src/**/**/*.js',
     'packages/strapi-plugin-content-type-builder/admin/src/InjectedComponents/*.js',
     'packages/strapi-plugin-content-type-builder/admin/src/InjectedComponents/tests/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/containers/AppOld/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/containers/HomePageOld/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/containers/ModelPageOld/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/containers/Form/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/ContentHeader/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/AttributeCard/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/AttributeRow/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/InputCheckboxWithNestedInputs/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/List/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PluginLeftMenu/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PluginLeftMenuLink/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PluginLeftMenuSection/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PopUpForm/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PopUpHeaderNavLink/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/PopUpRelations/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/RelationBox/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/RelationIco/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/components/RelationNaturePicker/*.js',
     '!packages/strapi-plugin-content-type-builder/admin/src/components/TableList/*.js',
     '!packages/strapi-plugin-content-type-builder/admin/src/components/TableListRow/*.js',
-    '!packages/strapi-plugin-content-type-builder/admin/src/containers/Form/utils/*.js',
     '!packages/strapi-plugin-content-type-builder/admin/src/utils/*.js',
-    // 'packages/strapi-plugin-**/**/admin/src/**/**/*.{js,jsx}',
-    '!packages/strapi-admin/admin/src/*.{js,jsx}',
-    '!packages/strapi-plugin-**/**/admin/src/*.{js,jsx}',
-    '!packages/strapi-admin/admin/src/**/**/tests/*.test.{js,jsx}',
     '!packages/strapi-plugin-*/admin/src/**/**/tests/*.test.{js,jsx}',
   ],
   coverageThreshold: {
     global: {
-      // NOTE: This should be increased at some point
       statements: 90,
       branches: 90,
       functions: 90,
@@ -47,10 +22,8 @@ module.exports = {
     __webpack_public_path__: 'http://localhost:4000',
     strapi: {},
   },
-  // NOTE: Should be dynamic
   moduleDirectories: [
     'node_modules',
-    // '<rootDir>/packages/strapi-admin/admin/src',
     '<rootDir>/packages/strapi-helper-plugin/node_modules',
     '<rootDir>/packages/strapi-helper-plugin',
     '<rootDir>/packages/strapi-helper-plugin/lib/src',

diff --git a/packages/strapi-plugin-content-type-builder/admin/src/containers/App/index.js b/packages/strapi-plugin-content-type-builder/admin/src/containers/App/index.js
@@ -64,6 +64,7 @@ export class App extends React.Component {
     this.props.getData();
   }
 
+  /* istanbul ignore next */
   componentDidUpdate(prevProps) {
     if (prevProps.shouldRefetchData !== this.props.shouldRefetchData) {
       this.props.getData();

diff --git a/packages/strapi-plugin-content-type-builder/admin/src/containers/App/reducer.js b/packages/strapi-plugin-content-type-builder/admin/src/containers/App/reducer.js
@@ -383,9 +383,6 @@ function appReducer(state = initialState, action) {
       }
 
       return state
-        .updateIn(['temporaryAttributeRelation', 'target'], () => action.target)
-        .updateIn(['temporaryAttributeRelation', 'name'], () => action.target)
-        .updateIn(['temporaryAttributeRelation', 'plugin'], () => action.source || '')
         .updateIn(['temporaryAttributeRelation', 'target'], () => action.target)
         .updateIn(['temporaryAttributeRelation', 'name'], () => action.target)
         .updateIn(['temporaryAttributeRelation', 'plugin'], () => action.source || '');