Skip to content
Snippets Groups Projects

Issue #92: Improve test coverage

Merged Issue #92: Improve test coverage
All threads resolved!
Merged Rafael Dias requested to merge issue/92 into develop
All threads resolved!

Foi adicionado testes para almentar a cobertura, os arquivos modificados para suportar as mudanças: config/ci_test.yaml.example, config/market_dimensions.yaml.example, config/market_enum.yaml.example, config/market_views.yaml.example, config/source_test_only.yaml.example, config/test/postgres/fixtures/sellerstatus.json, src/api/controllers/data.spec.ts, src/api/middlewares/error.spec.ts, src/core/engine.spec.ts, test/postgres/fixtures/sellerstatus.json, test/scenario.ts.

Alguns trecho de código foram removido por não estarem sendo utilizado: src/adapter/monet.ts, src/adapter/postgres.ts, src/api/controllers/collect.ts, src/core/adapter.ts.

A execução dos testes é simultânea em relação aos banco de dados, os arquivos foram modificados para incluir ambos os bancos postgres e monet: .gitlab-ci.yml, config/ci_database_test.env.example, config/ci_monet_test.env.example, config/ci_postgres_test.env.example, src/adapter/postgres.spec.ts, src/api/controllers/collect.spec.ts, src/util/configParser.ts, src/main.ts.

Signed-off-by: rafaelatc3sl rpd17@c3sl

Merge request reports

Pipeline #18958 passed

Pipeline passed for 3b0a5eec on issue/92

Test coverage 96.23% (2.96%) from 1 job
Approval is optional

Merged by Lucas Fernandes de OliveiraLucas Fernandes de Oliveira 6 years ago (Jan 9, 2019 10:24am UTC)

Merge details

Pipeline #18965 passed

Pipeline passed for d2c88631 on develop

Test coverage 96.23% (2.96%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Lucas Fernandes de Oliveira
  • Rafael Dias added 1 commit

    added 1 commit

    Compare with previous version

  • Rafael Dias resolved all discussions

    resolved all discussions

  • Author Developer

    Terminei as modificações.

  • mentioned in commit d2c88631

  • Please register or sign in to reply
    Loading