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 install, run, uninstall #394

Merged
merged 13 commits into from
Jul 21, 2023
Merged

Test install, run, uninstall #394

merged 13 commits into from
Jul 21, 2023

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Jul 19, 2023

This is finally ready for review...

    strategy:
      fail-fast: false
      matrix:
        commands:
          # Enter hackingtool starting from the main menu with \n as the delimiter. 
          - '17\n0\n1\n\n99\n99\n99'  # Install, run, update, update system,      press ENTER to continue, back to main menu, quit
          - '17\n0\n2\n\n99\n99\n99'  # Install, run, update, update hackingtool, press ENTER to continue, back to main menu, quit
          # - '17\n1\n1\n'  # Install, run, uninstall, press ENTER to continue
          - "99"  # Install, run, quit

allows us to walk up and down hackingtool's menu hierarchy to try out different commands.

I will enable the commented-out uninstall test in a follow-up pull request after this one lands.

That test needs the change made to tools/tool_manager.py where after hackingtool has been successfully uninstalled, we do not try to go back to the main menu because it no longer exists!

When merging this, please do Squash & Merge...
Screenshot 2023-07-19 at 18 34 55
...to keep all these little commits out of your repo.

@cclauss cclauss marked this pull request as ready for review July 19, 2023 16:42
@Z4nzu Z4nzu self-requested a review July 21, 2023 17:11
@Z4nzu Z4nzu merged commit 0232498 into Z4nzu:master Jul 21, 2023
@cclauss cclauss deleted the patch-4 branch July 21, 2023 17:30
Menacei pushed a commit to Menacei/hackingtool that referenced this pull request Aug 12, 2023
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.

3 participants