TestSetUpHelper should use DB_ Postgres instance if available

Description

https://github.com/folio-org/folio-service-tools TestSetUpHelper should use DB_ Postgres instance if available.

PostgresClient constructor, by default, starts Embedded Postgres since RMB v26.
This means that if you have unit tests that do not start Embedded Postgres
explicitly, you might need to terminate Embedded Postgres by calling
PostgresClient.stopEmbeddedPostgres:
https://github.com/folio-org/raml-module-builder/blob/master/doc/upgrading.md#version-26

Purpose

Starting embedded postgres takes very long under Windows.
Many developers want to run the tests against a different postgres instance, the connection parameters are provided in the DB_ enviroment variables:
https://github.com/folio-org/raml-module-builder#environment-variables
The current code ignores the DB_ variables but it shouldn't.

Approach

Let RMB start postgres if needed.

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

relates to

Checklist

hide

TestRail: Results

Activity

Show:

Julian Ladisch May 18, 2020 at 2:38 PM

The pull request is ready for Spitfire team for code review: https://github.com/folio-org/folio-service-tools/pull/30
I don't have write access to the git repository, therefore the Spitfire team needs to merge the pull request.

Done

Details

Assignee

Reporter

Priority

Story Points

Sprint

Development Team

Spitfire

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created May 18, 2020 at 2:36 PM
Updated November 17, 2020 at 7:29 PM
Resolved July 13, 2020 at 4:06 PM
TestRail: Cases
TestRail: Runs