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

test gm commands end to end #48

Merged
merged 17 commits into from
Nov 7, 2023
Merged

test gm commands end to end #48

merged 17 commits into from
Nov 7, 2023

Conversation

amtoine
Copy link
Owner

@amtoine amtoine commented Oct 28, 2023

related to

description

in order to be able to test the gm remove command:

  • add --no-confirm to gm remove

in a new tests/gm.nu test module, adds

  • a run-with-env command to take care of the test environment wrapping and cleaning
  • tests for
    • gm list
    • gm update-cache
    • gm status
    • gm clone
    • gm remove
  • the true expected errors next to assert error in NOTE, for when we'll be able to check the content of an error in Nushell 😏

@amtoine amtoine added the tests Something related to the tests of the library label Oct 28, 2023
@amtoine amtoine force-pushed the test-gm-end-to-end branch 3 times, most recently from 19738ec to 710fa78 Compare October 30, 2023 20:33
@amtoine amtoine mentioned this pull request Nov 2, 2023
4 tasks
@amtoine amtoine added the core Related to the main `nu-git-manager` module label Nov 2, 2023
@amtoine amtoine force-pushed the test-gm-end-to-end branch from 6b65118 to 5f0e0d9 Compare November 5, 2023 09:05
@amtoine amtoine force-pushed the test-gm-end-to-end branch from 5f0e0d9 to 232914f Compare November 5, 2023 09:06
@amtoine

This comment was marked as outdated.

@amtoine amtoine marked this pull request as ready for review November 5, 2023 16:41
@amtoine amtoine force-pushed the test-gm-end-to-end branch from ed443ed to d3f6d6f Compare November 5, 2023 16:50
@amtoine amtoine force-pushed the test-gm-end-to-end branch from aaeea76 to 821d484 Compare November 6, 2023 10:08
This was referenced Nov 6, 2023
it wouldn't remove the repo and yet update the cache...
@amtoine amtoine merged commit 0a7e901 into main Nov 7, 2023
3 checks passed
@amtoine amtoine deleted the test-gm-end-to-end branch November 7, 2023 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Related to the main `nu-git-manager` module tests Something related to the tests of the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant