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

feat(api): Add tests to the api #651

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat(api): Add tests to the api #651

wants to merge 1 commit into from

Conversation

cb0s
Copy link
Member

@cb0s cb0s commented Oct 27, 2022

Summary

Add tests to the telestion-api to make it more robust.

Details

  • Add tests for provided configurations TelestionConfiguration, NoConfiguration, and UntypedConfiguration
  • Add tests for TelestionVerticle
  • Add tests for Timing and DecodedMessage
  • Add tests for all Verticle Traits

Additional information

As I am not sure, yet, how much time I will find in the next weeks, this branch could also be used as a rolling release branch, where a merge into the main branch happens after some amount of progress.

Related links

CLA

  • I have signed the individual contributor's license agreement and sent it to the board of the WüSpace e. V. organization.

@cb0s cb0s added 🌷 enhancement New feature or request 📄 java Pull requests that update Java code 👓 tests Tests for the project labels Oct 27, 2022
@cb0s cb0s requested a review from fussel178 October 27, 2022 21:30
…ion`, `NoConfiguration`, and `UntypedConfiguration`
@pklaschka pklaschka force-pushed the main branch 2 times, most recently from 8871be5 to b78e9a0 Compare December 23, 2022 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👓 tests Tests for the project 📄 java Pull requests that update Java code 🌷 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant