Skip to content
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

Feat/integration tests #26

Merged
merged 16 commits into from
Dec 12, 2018
Merged

Feat/integration tests #26

merged 16 commits into from
Dec 12, 2018

Conversation

paasovaara
Copy link

@paasovaara paasovaara commented Nov 8, 2018 •
edited
Loading

Integration tests using testcontainers for Pulsar and self-made main loop for TripUpdateProcessor.

implements #22

known issue:

  • some test-code has been duplicated from commons, because maven doesn't package test-packages into the final jar that we release to bintray. We should create a separate jar from the test-utils.

@paasovaara
Added boilerplate needed for integration tests
036e643 
copied from commons, once works we should publish the boilerplate as separate jar from commons and then use that
@paasovaara
Added DIY integration test framework using MessageRouter as entry point
a5eab56
@paasovaara
Add protobuf schema as pulsar property
92f6763
@paasovaara
Integration test for pulsar properties
692a9a8
@paasovaara
Run integration tests in Travis
04f90de
@paasovaara
it-test: validate payloads for cancellations
05ce38c
@paasovaara
Use dvjId as pulsar message key also with StopTimeUpdates
2ce40c0
@paasovaara
Refactored integration test boilerplate to base class
0c42e8b
@paasovaara
Add another test for cancellations with wrong status
fd327f4
@paasovaara
Added validation for incoming cancellation message direction
baea21d
@paasovaara
Shorten the time for test receive to 500ms
439249a
@paasovaara
Merge branch 'master' into feat/integration-tests
af51a46
@paasovaara
Explicitly ignore cancellation events in timeValidators
c0365a2
@paasovaara
Added integration tests for StopTimeUpdates and fixed found bug relat…
d490e65 
…ed to timestamps
@paasovaara
Merge branch 'master' into feat/integration-tests
662e230
@paasovaara
Minor refactoring
8f0c8be
@paasovaara paasovaara mentioned this pull request Dec 11, 2018
Merged
@laaksma laaksma merged commit b6d2df1 into master Dec 12, 2018
@laaksma laaksma deleted the feat/integration-tests branch December 12, 2018 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@laaksma laaksma laaksma approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paasovaara @laaksma