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

Increase test coverage to 80% #40

Open
rhinof opened this issue Apr 24, 2019 · 1 comment
Open

Increase test coverage to 80% #40

rhinof opened this issue Apr 24, 2019 · 1 comment
Labels
good first issue Good for newcomers Priority tech debt technical debt that should be payed off

Comments

@rhinof
Copy link
Contributor

rhinof commented Apr 24, 2019

test coverage as reported by coveralls should be at least 80%

@rhinof rhinof added tech debt technical debt that should be payed off Priority good first issue Good for newcomers labels Apr 24, 2019
@rhinof
Copy link
Contributor Author

rhinof commented May 7, 2019

quick wins:

  • remove leftover methods from builder interface and implementation will increase coverage Cleanup bus builder interface #34
  • add simple unit tests to policies (ttl, durability)
  • Add unit test for bus builder that validates the ConfigureHealthChecks sets values properly on the builder
  • Set integration tests to configure bus to run with multiple workers

Not so quick wins but needed:

  • Test protobuff and avro serializers (@vladshub )
  • Test RPC calls from with in a Saga
  • Test transactional outbox scavenging capabilities
  • Test error scenarios of Glue component

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Priority tech debt technical debt that should be payed off
Projects
None yet
Development

No branches or pull requests

1 participant