Skip to content
This repository has been archived by the owner on Jan 21, 2021. It is now read-only.

Adds testing macros for laravel collections #4

Merged
merged 6 commits into from
Oct 10, 2018
Merged

Adds testing macros for laravel collections #4

merged 6 commits into from
Oct 10, 2018

Conversation

eduarguz
Copy link
Contributor

@eduarguz eduarguz commented Oct 9, 2018 •
edited
Loading

related to #3
so far...

  • Few problems with phpstan, macros are too much magic for the tool. (any advice?)
  • Took the TestCollectionMacros::enable(); approach inspired by carbon business-day. Also, there is no need for the container.
  • Added some very basic tests.

@eduarguz
Copy link
Contributor Author

This PR adds a starting point for Test Collection Macros.

Included Macros are
assertContains and assertNotContains

Other Macros (TestResponse) will be added in a different PR.

@eduarguz eduarguz changed the title [WIP] Adds testing macros for laravel collections Adds testing macros for laravel collections Oct 10, 2018
@svenluijten svenluijten merged commit 4ea1cac into svenluijten:master Oct 10, 2018
@svenluijten svenluijten mentioned this pull request Oct 10, 2018
Open
@svenluijten
Copy link
Owner

Thank you!! 😄

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

@svenluijten svenluijten svenluijten left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@eduarguz @svenluijten