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

[FEATURE] Add parameters to Cat API tests #550

Closed
Jakob3xD opened this issue Sep 4, 2024 · 2 comments · Fixed by #551
Closed

[FEATURE] Add parameters to Cat API tests #550

Jakob3xD opened this issue Sep 4, 2024 · 2 comments · Fixed by #551
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Jakob3xD
Copy link
Contributor

Jakob3xD commented Sep 4, 2024

Is your feature request related to a problem?

Most of the cat API testes lag the parameter h=* which is responsible for returning all fields the endpoint provides.
Beside the bytes and time parameter should also be added to tests as is transforms the returned values.

What solution would you like?

Add the parameters to the cat api tests.

What alternatives have you considered?

None

Do you have any additional context?

None

@Jakob3xD Jakob3xD added enhancement New feature or request untriaged labels Sep 4, 2024
@dblock
Copy link
Member

dblock commented Sep 4, 2024

@Jakob3xD are you going to try and contribute this one?

@dblock dblock added good first issue Good for newcomers and removed untriaged labels Sep 4, 2024
@Jakob3xD
Copy link
Contributor Author

Jakob3xD commented Sep 4, 2024

@Jakob3xD are you going to try and contribute this one?

Yes, MR is nearly finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants