Skip to content
This repository was archived by the owner on Mar 13, 2020. It is now read-only.

Commit 75f4e40

Browse files
authored
Merge pull request #18 from PageUpPeopleOrg/feature/OSC-1247-add-integration-tests
[OSC-1247] Add coverage for all new commands in integration tests - add coverage for all new commands in integration tests - make integration tests re-runnable - apply DRY to assist multiple execution iterations - log passing tests post assertion - move from plain Bourne shell to Bash shell since we now use a 3rd-party gist to generate UUIDs to allow us to re-run integration tests on dev machines
2 parents 6b16f43 + dc37599 commit 75f4e40

File tree

5 files changed

+327
-45
lines changed

5 files changed

+327
-45
lines changed

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,5 +109,4 @@ venv.bak/
109109
# JetBrains
110110
.idea/
111111

112-
test-models/
113112
tests/integration/models/

Pipfile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@ verify_ssl = true
55

66
[dev-packages]
77
pytest = "*"
8+
pep8 = "*"
9+
pylint = "*"
10+
autopep8 = "*"
811

912
[packages]
1013
psycopg2-binary = "==2.7.7"

Pipfile.lock

Lines changed: 128 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ To run integration tests, please ensure the following information is configured
112112

113113
- `tests/integration/test_integration.sh:9`
114114

115-
Please ensure that the database connection string points to a valid PostgreSQL instance.
115+
Please ensure that the database connection string points to a valid PostgreSQL instance with valid parameters.
116116

117117
#### Docker
118118

0 commit comments

Comments
 (0)