Feature/dynamic tests

readme.md

@@ -24,14 +24,9 @@ herbs new
 
 ```bash
 .
-├── node_modules
 ├── package-lock.json
 ├── package.json
 └── src
-    ├── config
-    │   ├── api.js
-    │   ├── index.js
-    │   └── mongo.js
     ├── domain
     │   ├── entities
     │   │   ├── index.js
@@ -40,13 +35,13 @@ herbs new
     │       ├── index.js
     │       └── user
     │           ├── createUser.js
-    │           ├── createUser.test.js
     │           ├── deleteUser.js
-    │           ├── deleteUser.test.js    
     │           ├── getByIdUser.js
-    │           ├── getByIdUser.test.js    
-    │           └── updateUser.js
-    │           └── updateUser.test.js    
+    │           ├── updateUser.js
+    │           ├── createUser.test.js
+    │           ├── deleteUser.test.js
+    │           ├── getByIdUser.test.js
+    │           └── updateUser.test.js
     ├── index.js
     └── infra
         ├── api
@@ -60,11 +55,16 @@ herbs new
         │   ├── rest
         │   │   └── index.js
         │   └── server.js
+        ├── config
+        │   ├── api.js
+        │   ├── index.js
+        │   └── postgres.js
         └── data
             ├── database
-            │   └── index.js
+            │   ├── index.js
+            │   └── migrations
+            │       └── 20211005010714_users.js
             └── repositories
-                ├── baseRepository.js
                 ├── index.js
                 └── userRepository.js
 ```
@@ -88,7 +88,7 @@ Generate a new project based on your entities(or not)
 | Generate graphql layer?                       | This will generate a graphql layer for your domain.                                                                            | Y                              | n                             |
 | Generate rest layer?                          | Description                                                                                                                    | Y                              | n                             |
 | What database do you want to use?             | Use arrow keys to choose your Database.                                                                                        | --                             | --                            |
-| Specify your entities folder path? (optional) | If you have [Gotu](https://github.com/herbsjs/gotu) entities, specify the folder path for CLI generate the project around that | empty                          | ./example/my-custom-entities/ |
+| Specify your entities folder path? (optional) | If you have [Gotu](https://github.com/herbsjs/gotu) entities, specify the folder path for CLI generate the project around that | empty                          | ./example-entities/ |
 
 ### herbs update
 

src/generators/index.js

@@ -8,24 +8,24 @@ module.exports = async (tools) => {
 
   return {
     new: {
-      packageJson: await require('./new/packagejson')(tools),
-      entities: await require('./new/domain/entities')(tools),
-      connection: await require('./new/infra/database/connection')(tools),
-      repositories: await require('./new/infra/database/repositories/repositories')(tools),
-      migrations: (options.postgres || options.sqlserver) ? await require('./new/infra/database/migrations/migrations')(tools) : ignore,
-      useCases: await require('./new/domain/useCases')(tools),
-      useCasesTests: (options.entities === '') ? await require('./new/domain/useCasesTests')(tools) : ignore,
-      graphql: options.graphql ? await require('./new/infra/graphql')(tools) : ignore,
-      rest: options.rest ? await require('./new/infra/rest')(tools) : ignore,
-      config: infra ? await require('./new/infra/config')(tools) : ignore,
-      server: infra ? await require('./new/infra/server')(tools) : ignore,
-      index: await require('./new/index')(tools)
+      packageJson: await require('./src/packagejson')(tools),
+      entities: await require('./src/domain/entities')(tools),
+      connection: await require('./src/infra/database/connection')(tools),
+      repositories: await require('./src/infra/database/repositories/repositories')(tools),
+      migrations: (options.postgres || options.sqlserver) ? await require('./src/infra/database/migrations/migrations')(tools) : ignore,
+      useCases: await require('./src/domain/useCases/useCases')(tools),
+      useCasesTests: await require('./src/domain/usecases/unitTests')(tools),
+      graphql: options.graphql ? await require('./src/infra/graphql')(tools) : ignore,
+      rest: options.rest ? await require('./src/infra/rest')(tools) : ignore,
+      config: infra ? await require('./src/infra/config')(tools) : ignore,
+      server: infra ? await require('./src/infra/server')(tools) : ignore,
+      index: await require('./src/index')(tools)
     },
     update: {
-      entities: await require('./new/domain/entities')(tools, true),
-      useCases: await require('./new/domain/useCases')(tools, true),
-      migrations: updateMigration ? await require('./new/infra/database/migrations/migrations')(tools) : ignore,
-      repositories: await require('./new/infra/database/repositories/repositories')(tools, true)
+      entities: await require('./src/domain/entities')(tools, true),
+      useCases: await require('./src/domain/usecases/useCases')(tools, true),
+      migrations: updateMigration ? await require('./src/infra/database/migrations/migrations')(tools) : ignore,
+      repositories: await require('./src/infra/database/repositories/repositories')(tools, true)
     }
   }
 }

src/generators/src/domain/usecases/unitTests.js

@@ -0,0 +1,94 @@
+
+const camelCase = require('lodash.camelcase')
+const { objToString } = require('../../../utils')
+const fs = require('fs')
+
+function invertObjValues (obj) {
+  for (const key of Object.keys(obj)) {
+    switch (typeof obj[key]) {
+      case 'String':
+        obj[key] = 123
+        break
+      case 'Number':
+      case 'Boolean':
+        obj[key] = '123'
+        break
+      default:
+        obj[key] = true
+    }
+  }
+  return obj
+}
+
+const validUseCaseRequests = {
+  create: (obj) => {
+    delete obj.id
+    return obj
+  },
+  update: (obj) => obj,
+  delete: (obj) => { return { id: obj.id } },
+  getById: (obj) => { return { id: obj.id } }
+}
+const invalidUseCaseRequests = {
+  create: (obj) => {
+    delete obj.id
+    obj = invertObjValues(obj)
+    return obj
+  },
+  update: (obj) => invertObjValues(obj),
+  delete: () => { return { id: null } },
+  getById: () => { return { id: null } }
+}
+
+const useCases = Object.keys(validUseCaseRequests)
+
+const valueType = {
+  String: "'string'",
+  Number: 99,
+  Boolean: true,
+  Array: []
+}
+
+function generateMockObj (schema) {
+  const obj = {}
+  for (const key of Object.keys(schema)) {
+    obj[key] = valueType[schema[key].type.name]
+  }
+  return obj
+}
+
+function generateRequestObject (schema, action, validReq) {
+  const obj = generateMockObj(schema)
+  if (validReq) return validUseCaseRequests[action](obj)
+  return invalidUseCaseRequests[action](obj)
+}
+
+module.exports = async ({ template: { generate }, filesystem }) => async () => {
+  const entities = require(`${filesystem.cwd()}/src/domain/entities`)
+
+  for (const entity of Object.keys(entities)) {
+    const { name, schema } = entities[entity].prototype.meta
+    for (const action of useCases) {
+      const useCaseName = `${action}${name}`
+      const ucPath = `${filesystem.cwd()}/src/domain/usecases/${camelCase(name)}/${useCaseName}.test.js`
+
+      if (fs.existsSync(ucPath)) continue
+
+      await generate({
+        template: `domain/useCases/tests/${action}.test.ejs`,
+        target: ucPath,
+        props: {
+          name: {
+            pascalCase: name,
+            camelCase: camelCase(name)
+          },
+          request: {
+            valid: objToString(generateRequestObject(schema, action, true)),
+            invalid: objToString(generateRequestObject(schema, action, false))
+          },
+          mock: objToString(generateRequestObject(schema, action))
+        }
+      })
+    }
+  }
+}

src/generators/new/domain/useCases.js → src/generators/src/domain/usecases/useCases.js

@@ -1,10 +1,10 @@
 const useCases = ['create', 'update', 'delete', 'getById']
-const { objToString } = require('../../utils')
+const { objToString } = require('../../../utils')
 const camelCase = require('lodash.camelcase')
 const startCase = require('lodash.startCase')
 const fs = require('fs')
 
-async function generateRequest (schema) {
+async function generateRequestschema (schema) {
   // schema to plain JSON
   const obj = Object.keys(schema).reduce((obj, key) => {
     const { name, type } = schema[key]
@@ -54,7 +54,7 @@ module.exports = async ({ template: { generate }, filesystem }) => async () => {
             pascalCase: startCase(camelCase(name)).replace(/ /g, ''),
             camelCase: camelCase(name)
           },
-          request: await generateRequest(schema)
+          request: await generateRequestschema(schema)
         }
       })
     }

src/templates/domain/useCases/tests/create.test.ejs

@@ -3,9 +3,7 @@ const assert = require('assert')
 
 
 describe('Create <%= props.name.pascalCase %>', () => {
-  function aUser({ hasAccess }) {
-    return { hasAccess }
-  }
+  const authorizedUser = { hasAccess: true }
 
   describe('Valid <%= props.name.pascalCase %>', () => {
 
@@ -16,13 +14,12 @@ describe('Create <%= props.name.pascalCase %>', () => {
           async insert(<%= props.name.camelCase %>) { return (<%= props.name.camelCase %>) }
         })
       }
-      const user = aUser({ hasAccess: true })
-      /*{ <%= props.request %> }*/
-      const req = {  nickname: 'herbsUser', password: 'V&eryStr0ngP@$$'}
+
+      const req = <%- props.request.valid %>
 
       // When
       const uc = create<%= props.name.pascalCase %>(injection)()
-      await uc.authorize(<%= props.name.camelCase %>)
+      await uc.authorize(authorizedUser)
       const ret = await uc.run(req)
 
       // Then
@@ -37,18 +34,16 @@ describe('Create <%= props.name.pascalCase %>', () => {
     it('should not create invalid <%= props.name.pascalCase %>', async () => {
       // Given
       const injection = {}
-      const user = aUser({ hasAccess: true })
-      /*{ <%= props.request %> }*/
-      const req =  {  nickname: 'herbsUser', password: 96587422 }
+
+      const req = <%- props.request.invalid %>
 
       // When
       const uc = create<%= props.name.pascalCase %>(injection)()
-      await uc.authorize(<%= props.name.camelCase %>)
+      await uc.authorize(authorizedUser)
       const ret = await uc.run(req)
 
       // Then
       assert.ok(ret.isErr)
-      assert.deepStrictEqual(ret.err, {request :[{password:[{wrongType:"String"}]}]})
     })
   })
 })

src/templates/domain/useCases/tests/delete.test.ejs

@@ -16,13 +16,12 @@ describe('Delete the <%= props.name.pascalCase %>', () => {
           async deleteByID(id) { return true }
         })
       }
-      const user = aUser({ hasAccess: true })
-      /*{ <%= props.request %> }*/
-      const req = {  id : 5 }
+
+      const req = <%- props.request.valid %>
 
       // When
       const uc = delete<%= props.name.pascalCase %>(injection)()
-      await uc.authorize(<%= props.name.camelCase %>)
+      await uc.authorize(authorizedUser)
       const ret = await uc.run(req)
 
       // Then