moleculerjs / moleculer-template-project

:mortar_board: Common project template for Moleculer-based projects
MIT License
21 stars 21 forks source link

Using `process.env.NODE_ENV` instead of `process.env.TEST` #23

Closed oanhnn closed 3 years ago

oanhnn commented 4 years ago

Default this template using jest for testing. It will auto set process.env.NODE_ENV to 'test' when run. So that you don't need add process.env.TEST = true; in test files. Update in db.mixin.js

  if (process.env.NODE_ENV === 'test') {
    // NeDB memory adapter for testing
    schema.adapter = new DbService.MemoryAdapter()
  } else if (process.env.MONGO_URI) {
    // Mongo adapter
    const MongoAdapter = require('moleculer-db-adapter-mongo')

    schema.adapter = new MongoAdapter(process.env.MONGO_URI)
    schema.collection = collection
  } else {
    // NeDB file DB adapter

    // Create data folder
    if (!fs.existsSync('./data')) {
      fs.mkdirSync('./data')
    }

    schema.adapter = new DbService.MemoryAdapter({ filename: `./data/${collection}.db` })
  }
icebob commented 3 years ago

Could you create a PR?

oanhnn commented 3 years ago

I made PR #28 and https://github.com/moleculerjs/moleculer-template-project-typescript/pull/43

icebob commented 3 years ago

Awesome, thanks!