Skip to content

Conversation

gasperzgonec
Copy link
Contributor

Summary

Added instructions for what tests should look like and how they should be implemented. This should help all developers working on this project to use consistent testing schema.

Connected Issues

Copy link
Contributor

@patricijabrecko patricijabrecko left a comment

Choose a reason for hiding this comment

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

Do we also want to follow the AAA (Arrange, Act, Assert) pattern?

@gasperzgonec
Copy link
Contributor Author

I've added AAA. Checking the current compliance, the AAA is being followed in about 85% of tests.
I'll update the remaining ones.

Copy link
Collaborator

@radovanjorgic radovanjorgic left a comment

Choose a reason for hiding this comment

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

Check&resolve @Mesko89's comments and merge once you think it's ready. Let's see how this works for us in long run. We will surely have some more changes/suggestions.

@gasperzgonec gasperzgonec enabled auto-merge (squash) August 22, 2025 08:29
@gasperzgonec gasperzgonec merged commit 283a90f into main Aug 22, 2025
4 checks passed
@gasperzgonec gasperzgonec deleted the gasperz/ISS-202840 branch August 22, 2025 08:32
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.

4 participants