Skip to content

Tests Suite Refactor - #74

Merged
JustSteveKing merged 13 commits into
minicli:mainfrom
WendellAdriel:tests-refactor
May 10, 2023
Merged

Tests Suite Refactor#74
JustSteveKing merged 13 commits into
minicli:mainfrom
WendellAdriel:tests-refactor

Conversation

@WendellAdriel

@WendellAdriel WendellAdriel commented May 10, 2023

Copy link
Copy Markdown
Member

In this PR I'm refactoring the whole test suite to use:

  • High Order Expectations whenever possible
  • Use Expectations instead of asserts to be more compliant with the Pest style

@JustSteveKing

JustSteveKing commented May 10, 2023

Copy link
Copy Markdown
Collaborator

I swear I did something like this already, but will merge it as it seems to have been lost 😅

@JustSteveKing
JustSteveKing merged commit cf8ca6e into minicli:main May 10, 2023
@WendellAdriel
WendellAdriel deleted the tests-refactor branch May 10, 2023 15:13
@WendellAdriel

WendellAdriel commented May 10, 2023

Copy link
Copy Markdown
Member Author

I swear I did something like this already, but will merge it as it seems to have been lost 😅

Oh man, really?
I just worked on this because I saw that were some things that could be updated by checking the main branch.
I hope that I didn't make anything to your changes 😓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

Minicli Development
Status: Done
Minicli Development
Status: ✅ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@WendellAdriel @JustSteveKing