Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Add tests to check Node & Postgresql versions - Closes #2388 #2392

Merged
merged 4 commits into from
Sep 19, 2018

Conversation

yatki
Copy link
Contributor

@yatki yatki commented Sep 13, 2018

What was the problem?

We didn't have tests to ensure using right node & Postgresql version

How did I fix it?

I added integration tests

Review checklist

@yatki yatki self-assigned this Sep 13, 2018
@yatki yatki force-pushed the 2388-unit-tests-for-node-postgres branch from 5d7f367 to 73d1859 Compare September 13, 2018 10:50
@yatki yatki changed the title WIP: Add tests to check node & postgresql versions Add tests to check node & postgresql versions Sep 13, 2018
@fchavant
Copy link
Contributor

The node version test should pass once #2393 has been merged.

const queriesHelper = require('../common/integration/sql/queriesHelper.js');
const application = require('../common/application.js');

describe('versions', () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we name it more specifically - Dependencies versions?

@yatki yatki force-pushed the 2388-unit-tests-for-node-postgres branch from 73d1859 to 9f71676 Compare September 13, 2018 13:17
@yatki yatki force-pushed the 2388-unit-tests-for-node-postgres branch from 9f71676 to bb1c9ea Compare September 13, 2018 13:19
@MaciejBaj MaciejBaj changed the title Add tests to check node & postgresql versions Add tests to check node & postgresql versions - Closes #2388 Sep 14, 2018
@MaciejBaj MaciejBaj changed the title Add tests to check node & postgresql versions - Closes #2388 Add tests to check Node & Postgresql versions - Closes #2388 Sep 17, 2018
@MaciejBaj MaciejBaj merged commit b3d3bdf into development Sep 19, 2018
@MaciejBaj MaciejBaj deleted the 2388-unit-tests-for-node-postgres branch September 19, 2018 10:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add unit tests to ensure node and postgres versions are supported
4 participants