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.

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

Refactor TTL tests for KV memory connector #1078

Merged
merged 1 commit into from Sep 6, 2016

Conversation

superkhau
Copy link
Contributor

@superkhau superkhau commented Sep 4, 2016

Reduce the complixity in the TTL test suite for the kv-memory connector to
get CI to pass. This change makes 3.x tests equivalent to 2.x tests (which are already merged and passing on CI).

  • Tests messages were reworded and reorganized to make the scope of each test clearer
  • Some of the code refactoring was done to make CI pass (the original tests were failing due to some expiry issue)

Forwardport of #1077

@superkhau superkhau self-assigned this Sep 4, 2016
@superkhau superkhau force-pushed the forwardport/refactor-ttl-suite branch 2 times, most recently from 950b032 to 54bee15 Compare September 4, 2016 19:26
Reduce the complixity in the TTL test suite for the kv-memory connector
to get CI to pass.
@superkhau superkhau force-pushed the forwardport/refactor-ttl-suite branch from 54bee15 to 095745c Compare September 4, 2016 19:27
@superkhau
Copy link
Contributor Author

Unrelated failures, merging.

cc @bajtos

@superkhau superkhau merged commit 19dae30 into master Sep 6, 2016
@superkhau superkhau deleted the forwardport/refactor-ttl-suite branch September 6, 2016 23:02
@superkhau superkhau removed the #review label Sep 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants