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

Add full integration test for tuono new CLI command #405

Open
Valerioageno opened this issue Jan 22, 2025 · 2 comments · May be fixed by #411
Open

Add full integration test for tuono new CLI command #405

Valerioageno opened this issue Jan 22, 2025 · 2 comments · May be fixed by #411
Assignees
Labels
rust Requires rust knowledge

Comments

@Valerioageno
Copy link
Member

Valerioageno commented Jan 22, 2025

This task is the next piece of #398 and covers the creation of the testing suite for the tuono new command.

It requires the followings:

  • New file crates/tuono/tests/cli_new.rs
  • Mock a temporary empty folder (you can use the utility in tests/utils.rs)
  • Mock the GitHub APIs (this two crates look pretty cool -> wiremock | httpmock)
  • Run tuono new and assert the output
@Valerioageno Valerioageno converted this from a draft issue Jan 22, 2025
@marcalexiei marcalexiei added the rust Requires rust knowledge label Jan 22, 2025
@Robert-maker1994
Copy link
Contributor

I can look at this, I was reading about wiremock in a [https://rust-exercises.com/advanced-testing/07_http_mocking/01_basics](Rust book). Looks to provide what we need, but illk at httpmock as well.

I'm unsure how to assign myself in this issue.

@Valerioageno
Copy link
Member Author

Thanks for it! I assigned yourself to it (needs permissions)

@Valerioageno Valerioageno moved this from Ready to In progress in Road to V1 Jan 22, 2025
@jacobhq jacobhq marked this as a duplicate of #412 Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rust Requires rust knowledge
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

3 participants