Skip to content

Eric/cus 9 add tests for all modes #33

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

Closed
wants to merge 16 commits into from

Conversation

flowstate
Copy link
Contributor

@flowstate flowstate commented Nov 12, 2024

Adding unit, integration, and smoke testing for the CLI.

Tasks

  • Core class unit tested
  • Core class integration tested
  • Github SCM tested
  • Gitlab SCM tested

note the code isn't yet functional, since modifications have been made to the Core class to allow for proper testing and to fix discovered issues. I'll update here once that's ameliorated.

@flowstate flowstate requested a review from dacoburn November 12, 2024 23:32
Copy link

socket-security bot commented Nov 12, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/[email protected] environment, filesystem, shell 0 63.2 kB joeyespo
pypi/[email protected] environment, eval, filesystem, network, shell, unsafe 0 36.4 MB astral-bot, crmarsh, zanie

View full report↗︎

@socket-security-staging
Copy link

socket-security-staging bot commented Nov 12, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/[email protected] environment, filesystem, shell 0 63.2 kB joeyespo
pypi/[email protected] None 0 0 B

View full report↗︎

@flowstate flowstate force-pushed the eric/cus-9-add-tests-for-all-modes branch from ee3e646 to e5a9c95 Compare January 8, 2025 21:39
@flowstate flowstate marked this pull request as ready for review February 6, 2025 20:10
@flowstate flowstate requested a review from a team as a code owner February 6, 2025 20:10
@flowstate flowstate requested review from tmpvar and trevnorris and removed request for a team February 6, 2025 20:10
@flowstate
Copy link
Contributor Author

Closing and reopening so the workflows will actually trigger

@flowstate flowstate closed this Feb 6, 2025
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.

2 participants