New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add more integration tests #1050

Open

frankduncan opened this issue Aug 24, 2018 · 2 comments

Assignees

Labels

in progress Z-REQ-PSM-SQ-1.2 Z-REQ-PSM-SQ-1.4

frankduncan commented Aug 24, 2018

This is a catchall issue for generally adding more integration tests. The current codebase has some tests, but could use more to get it to a point where most use cases are at least touched, if not thoroughly explored. For PRs that need groundwork laid for this, this issue can be used.

Author

frankduncan commented Aug 24, 2018

Related to #752

Author

frankduncan commented Aug 24, 2018

Related to #997 (comment)

kfogel added Z-REQ-PSM-SQ-1.4 Z-REQ-PSM-SQ-1.2 labels

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

11bf671

Issue #1050: Add more integration tests

kfogel assigned frankduncan

kfogel added the in progress label

jasonaowen added a commit that referenced this issue


          Add test data to mock out DMF API

f1a0fbb

The DMF API in the cavetl repo is difficult to set up, mostly because it
requires special access and permissions and so on. For the sake of
demos, and to ease testing the PSM side of the DMF integration, add some
test data and document how to use it.

We want this mock data in the PSM repository and not in the CAVETL
repository for a few reasons:

- this is, like the code that consumes the DMF API, documentation of
  what the PSM expects
- these mocks are useful for testing and demonstrating the PSM
- these mocks are useful for writing integration tests
- the PSM's dependency on CAVETL does not include explicit version
  numbers or commit hashes, and if the DMF API implementation in the
  CAVETL repo changes, the PSM should be able to fall back to these
  mocks until it is updated to consume the new API (and these mocks
  should be updated to match)
- as a side effect of the previous point, it may be useful to be able to
  go back in time to previous versions of the PSM, and having these
  mocks eases that

Thanks to Frank for sharing the special SSNs and names returned by the
Veris Test API.

Issue #759 Automatically check SSN against the Death Master File
Issue #1050 Add more integration tests

Co-authored-by: Frank Duncan <[email protected]>

jasonaowen added a commit that referenced this issue


          Add test data to mock out DMF API

63ea90c

The DMF API in the cavetl repo is difficult to set up, mostly because it
requires special access and permissions and so on. For the sake of
demos, and to ease testing the PSM side of the DMF integration, add some
test data and document how to use it.

We want this mock data in the PSM repository and not in the CAVETL
repository for a few reasons:

- this is, like the code that consumes the DMF API, documentation of
  what the PSM expects
- these mocks are useful for testing and demonstrating the PSM
- these mocks are useful for writing integration tests
- the PSM's dependency on CAVETL does not include explicit version
  numbers or commit hashes, and if the DMF API implementation in the
  CAVETL repo changes, the PSM should be able to fall back to these
  mocks until it is updated to consume the new API (and these mocks
  should be updated to match)
- as a side effect of the previous point, it may be useful to be able to
  go back in time to previous versions of the PSM, and having these
  mocks eases that

Thanks to Frank for sharing the special SSNs and names returned by the
Veris Test API.

Issue #759 Automatically check SSN against the Death Master File
Issue #1050 Add more integration tests

Co-authored-by: Frank Duncan <[email protected]>

jasonaowen mentioned this issue

Add test data to mock out DMF API #1056

Merged

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

72fe847

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

6d2fef5

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

ea6c4de

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

16400cc

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

2db6e8f

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

592f077

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

22780dc

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

a40eac9

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

ecf670a

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

5ce3daa

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

20ae36b

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

8fc64c8

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

b8d6c0e

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

b6b4a82

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

5f6859e

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

fe41796

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

e36f907

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

1dce09f

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

ed1f2f8

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

348626d

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

7faad3c

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

13e9f2c

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

53373c0

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

c20280b

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

09a05e5

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

44a40aa

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

7917fef

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

003786b

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

314b388

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add Enrollment Approval Tests.

7d79a2a

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

f32cce1

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

d99b6a0

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

cd60b84

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

9cf7582

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add update enrollment integration tests

6831ab5

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add reject enrollment test.

603bd87

Issue #1050: Add more integration tests #1050

frankduncan pushed a commit that referenced this issue


          Add extra provider type tests.

bb7d184

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

3ba0ca8

Issue #1050: Add more integration tests

PaulMorris pushed a commit that referenced this issue


          Add update enrollment integration tests

9bb654c

Issue #1050: Add more integration tests

PaulMorris pushed a commit that referenced this issue


          Add reject enrollment test.

4b84af3

Issue #1050: Add more integration tests #1050

PaulMorris pushed a commit that referenced this issue


          Add extra provider type tests.

b8ade2a

Issue #1050: Add more integration tests

PaulMorris pushed a commit that referenced this issue


          Add review/approve tests for new enrollments.

6c6c6e1

Issue #1050: Add more integration tests

frankduncan pushed a commit that referenced this issue


          Fix changeset checksum.

099d4e9

When adding #1050: Add integration tests, the hibernate sequence
was updated, but the changeset was changed so that existing databases
would not get the new sequence.  This updates to allow any changsets.

frankduncan mentioned this issue

Fix changeset checksum. #1101

Merged

frankduncan pushed a commit that referenced this issue


          Fix changeset checksum.

22b35d2

When mergin #1052 to add integration tests, the hibernate sequence
was updated, but the changeset was changed so that existing databases
would not get the new sequence.  This updates to allow any changsets.

Related to Issue #1050: Add integration tests

frankduncan pushed a commit that referenced this issue


          Fix changeset checksum.

6508b76

When merging #1052 to add integration tests, the hibernate sequence
was updated, but the changeset was changed so that existing databases
would not get the new sequence.  This updates to allow any changsets.

Related to Issue #1050: Add integration tests

frankduncan pushed a commit that referenced this issue


          Fix changeset checksum.

9cb84fb

When merging #1052 to add integration tests, the hibernate sequence
was updated, but the changeset was changed so that existing databases
would not get the new sequence.  This updates to allow any changesets.

Related to Issue #1050: Add integration tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment