Skip to content

Comments

Refactor tests#17

Open
petrduda wants to merge 5 commits intoVasekPurchart:masterfrom
petrduda:pd-refactor-tests
Open

Refactor tests#17
petrduda wants to merge 5 commits intoVasekPurchart:masterfrom
petrduda:pd-refactor-tests

Conversation

@petrduda
Copy link

@petrduda petrduda commented Mar 28, 2023

  • stop calling static methods in tests as non-static
  • use assertSame() where assertEquals() is not explicitly needed
  • use assertCount() after assertContains() for array comparison
  • add missing Assert::fail() in try+catch blocks when expecting exceptions
  • unify Assert::fail() message
  • replace Consistence\TestCase::ok() with expectNotToPerformAssertions()
  • use try+catch+fail when expecting exceptions with properties
  • rename data providers to singular where applicable
  • use DataProvider suffix for data providers
  • change data provider return type from array to Generator
  • declare data provider values on separate lines
  • isolate data provider cases using Closures
  • name data provider values
  • name data provider cases
  • extract multiple cases from test methods to data providers
  • remove unnecessary private methods for creating mocks

@petrduda petrduda force-pushed the pd-refactor-tests branch from bfe9bbe to 1432f71 Compare April 5, 2023 09:44
@petrduda petrduda force-pushed the pd-refactor-tests branch from 1432f71 to 82916b4 Compare April 5, 2023 20:48
@petrduda petrduda force-pushed the pd-refactor-tests branch from 82916b4 to ddbad06 Compare April 10, 2023 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants