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

issue-226 Renamed unit tests #391

Merged
merged 2 commits into from
Dec 3, 2024
Merged

Conversation

Abigael-JT
Copy link
Contributor

@Abigael-JT Abigael-JT commented Oct 24, 2024

renamed unit tests to have descriptive names

Related issue

Related PR

@Abigael-JT Abigael-JT requested a review from eemhu October 28, 2024 08:42
@Abigael-JT Abigael-JT mentioned this pull request Nov 8, 2024
@Abigael-JT Abigael-JT changed the title issue-226 issue-226 Renamed unit tests Nov 28, 2024
@Abigael-JT Abigael-JT marked this pull request as ready for review November 28, 2024 07:39
@Abigael-JT Abigael-JT self-assigned this Nov 28, 2024
Copy link
Contributor

@eemhu eemhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise ok

@Abigael-JT Abigael-JT requested a review from eemhu December 2, 2024 14:03
@Abigael-JT
Copy link
Contributor Author

removed capitalised first letters, unnecessary code and updated testnames

@kortemik kortemik merged commit eb9114a into teragrep:main Dec 3, 2024
1 check passed
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.

3 participants