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

Modifying code to add more tests #34

Merged
merged 24 commits into from
Feb 16, 2024
Merged

Modifying code to add more tests #34

merged 24 commits into from
Feb 16, 2024

Conversation

emmahsax
Copy link
Owner

@emmahsax emmahsax commented Feb 12, 2024

Changes

A list of the changes that your pull request will make:

  • Adding a page "..."
  • Refactoring "..."

I've wanted to add some more tests to this project for a while. Unfortunately, the code required a lot of changes to be in a place where it could take tests. Finally... I've been able to try these out!

I will also complete a full manual regression test of all commands, and I want to build this locally and work with it for a while to see how it does before I try to merge it.

Regression Test

  • Test completion (usage)
  • Test change-remote
    • When changing GitLab remote
    • When changing GitHub remote
  • Test checkout-default
  • Test clean-branches
    • When there are changes to pull
    • When there are no changes to pull
    • When already on main branch
  • Test code-request
    • GitHub with main branch, draft, autogenerated title, basic template
    • GitHub with feature branch, non-draft, write your own title, choose a different template
    • GitLab with main branch, draft, autogenerated title, basic template
    • GitLab with feature branch, non-draft, write your own title, choose a different template
    • On either with any base branch, but where branch has jira-123 in the branch
    • On either with any base branch, but where branch has jira_123 in the branch
    • On either with any base branch, but where branch has neither
  • Test completion (command)
  • Test empty-commit
  • Test forget-local-changes
    • While on the main branch
    • While on a feature branch
  • Test forget-local-commits
    • While on the main branch
    • While on a feature branch
  • Test help
    • At root command
    • Within each command
  • Test new-branch
    • Pass in a new branch
    • Provide a new branch when prompted
    • Pass in a bogus branch
    • Provide a bunch of bogus branches when prompted
  • Test set-head-ref
  • Test setup
    • With config file already present, replace with only GitHub, skip plugins, skip completion
    • With config file already present, replace with only GitLab, add plugins, add completion
    • With config file already present, replace with both GitHub and GitLab, add plugins, skip completion
    • With config file already present, opt not to replace, skip plugins, add completion
    • With config file missing, add a new one with both GitHub and GitLab, skip plugins, add completion
  • Test update
  • Test version

Related Pull Requests and Issues

If applicable, a list of related pull requests and issues:

  • Issue URL link
  • Pull Request URL link

Additional Context

Add any other context about the problem here.

@emmahsax emmahsax marked this pull request as ready for review February 16, 2024 22:49
@emmahsax emmahsax merged commit d33a3b2 into main Feb 16, 2024
3 checks passed
@emmahsax emmahsax deleted the adding_tests branch February 16, 2024 22:49
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.

1 participant