Skip to content

Add tests - #54

Merged
Kostis-S-Z merged 10 commits into
mainfrom
add-tests
Dec 4, 2025
Merged

Add tests#54
Kostis-S-Z merged 10 commits into
mainfrom
add-tests

Conversation

@Kostis-S-Z

@Kostis-S-Z Kostis-S-Z commented Dec 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Add simple unit tests (that mock the API)
  • Add two e2e tests (that call the real API)
  • Update the GitHub Action for tests to only run the e2e on a new release OR PR merged to main OR manually
  • Update docs with instructions on how to run the tests

@Kostis-S-Z
Kostis-S-Z marked this pull request as ready for review December 3, 2025 14:15
moz-gzh
moz-gzh approved these changes Dec 3, 2025
View reviewed changes

@moz-gzh moz-gzh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gave the tests a looksee and it all seems very reasonable! 🚀

@Kostis-S-Z
Kostis-S-Z merged commit 65bd888 into main Dec 4, 2025
3 checks passed
@Kostis-S-Z
Kostis-S-Z deleted the add-tests branch December 4, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@moz-gzh moz-gzh moz-gzh approved these changes
@Lguyogiro Lguyogiro Awaiting requested review from Lguyogiro
@umoqnier umoqnier Awaiting requested review from umoqnier

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kostis-S-Z @moz-gzh