Skip to content

Navigation Menu

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

doctrine / annotations Public

Notifications You must be signed in to change notification settings
Fork 234
Star 6.7k

Code
Issues 26
Pull requests
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Security
Insights

Cleanup: move to correct phpunit assertions#111

Merged

lcobucci merged 32 commits intomasterdoctrine/annotations:masterfrom cleanup/move-to-correct-phpunit-assertionsdoctrine/annotations:cleanup/move-to-correct-phpunit-assertionsCopy head branch name to clipboard

Dec 30, 2016

+496-476

Conversation Commits32 (32)Checks Files changed

Commits

Commits on Dec 30, 2016

Replacing assertEquals() + count() with assertCount()
Ocramius
committed
Replacing assertEquals() + sizeof() with assertCount()
Ocramius
committed
Using ::class where applicable
Ocramius
committed
Declaring access modifiers
Ocramius
committed
Using single quotes where applicable, since we don't rely on interpolation
Ocramius
committed
Importing used classes
Ocramius
committed
Removing unnecessary parentheses
Ocramius
committed
Using ::class where applicable
Ocramius
committed
Using explicit type casts rather than cast functions
Ocramius
committed
Using assertArrayNotHasKey rather than manual isset() checking
Ocramius
committed
Using assertInstanceOf rather than manual instanceof checking
Ocramius
committed
Removing unsafe uniqid() usage
Ocramius
committed
Removing non-absolute path include
Ocramius
committed
Typo fix in class name s/contructor/constructor
Ocramius
committed
Typo fix s/inteface/interface
Ocramius
committed
Documenting possible mock types
Ocramius
committed
Correcting documented parameter type
Ocramius
committed
Removing double quotes (loads of weird escape sequences otherwise)
Ocramius
committed
Corrected iterations count comparison
Ocramius
committed
Corrected namespacing of some docblock variable declarations
Ocramius
committed
Adding hinting about retrieved annotation type
Ocramius
committed
Removed possible undefined variable failure
Ocramius
committed
Better error matching via assertStringMatchesFormat()
Ocramius
committed
Removed unused imports
Ocramius
committed
Removed unused variables
Ocramius
committed
Documenting possible mock types
Ocramius
committed
Replacing useless FQCN
Ocramius
committed
Replacing manual is_array checks with assertInternalType('array', ...)
Ocramius
committed
s/$this->assert/self::assert
Ocramius
committed
EOF EOL
Ocramius
committed
EOF EOL
Ocramius
committed
CS (spacing)
Ocramius
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.