.env для шутки и гнезда не то же самое - PullRequest
Новая
       6

.env для шутки и гнезда не то же самое

0 голосов
/ 21 января 2020

Я хочу протестировать гнездо API, для этого я создаю один файл .env, а для своего разработчика я создаю .env.development файл.

Когда я запускаю тестовый шут, мой первый тест использует хороший файл: .env, но во втором тесте используется неправильный файл .env.development.

Кого я загружаю .env в гнезде: 

import {Module} from '@nestjs/common';
import {ConfigService} from './config.service';

@Module({
  providers: [{
    provide: ConfigService,
    useValue: new ConfigService(`.env${process.env.NODE_ENV}`),
  }],
  exports: [ConfigService],
})

export class ConfigModule {}

Кого я загружаю .env в шутке: 

"test:watch": "NODE_ENV='' jest --watch",

"jest": {
    "displayName": {
      "name": "API",
      "color": "yellow"
    },
    "verbose": true,
    "runner": "jest-serial-runner",
    "testSequencer": "../src/test/sequencer.jest.js",
    "moduleFileExtensions": [
      "js",
      "json",
      "ts"
    ],
    "rootDir": "src",
    "testRegex": ".spec.ts$",
    "transform": {
      "^.+\\.(t|j)s$": "ts-jest"
    },
    "coverageDirectory": "../coverage",
    "testEnvironment": "node",
    "preset": "@shelf/jest-mongodb",
    "setupFiles": [
      "dotenv/config"  <------------------------| here
    ]
  },

Тест на мой .env (все в порядке, правильный .env загружен) 

describe('?️ tests for env file', () => {
  it('should have a db_user', () => {
    expect(process.env.db_user).toBeDefined();
  });
  it('should have a db_pass', () => {
    expect(process.env.db_pass).toBeDefined();
  });
  it('should have a db_uri', () => {
    expect(process.env.db_uri).toBeDefined();
  });
  it('should have a db_port', () => {
    expect(process.env.db_port).toBeDefined();
  });
  it('should have a db_name', () => {
    expect(process.env.db_name).toBeDefined();
  });
});

Тест, который загружает неправильный .env (я поставил неправильный ip для БД в .env.development, поэтому тест не удалось) 

import { Test, TestingModule } from '@nestjs/testing';
import * as request from 'supertest';
import {AppModule} from '../app.module';
// remove console.log
console.log = jest.fn();

describe('?️ UserController', () => {
  let app;

  beforeEach(async () => {
    const moduleFixture: TestingModule = await Test.createTestingModule({
      imports: [AppModule],
    }).compile();

    app = moduleFixture.createNestApplication();
    await app.init();
  });

  /** valid test for create a admin user */
  it('post /users valid test for create a admin user', () => {
    return request(app.getHttpServer())
      .post('/users')
      .send({
        jwt: '',
        firstname: 'admin',
        lastname: 'admin',
        mail: 'admin@gmail.com',
        password: 'test'
      })
      .expect(201);
  });

});

1 Ответ

0 голосов
/ 21 января 2020

Похоже, что в этой строке отсутствует . после .env: 

new ConfigService(`.env${process.env.NODE_ENV}`)

Однако дополнительный период необходим только тогда, когда process.env.NODE_ENV содержит значение, поэтому я хотел бы что-то сделать как: 

const NODE_ENV = process.env.NODE_ENV ? `.${process.env.NODE_ENV}` : '';
//...
new ConfigService(`.env${NODE_ENV}`)

Таким образом, вы либо получите new ConfigService('.env') или new ConfigService('.env.development')

...