Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Refactoring tests #1011

Merged
merged 1 commit into from
Feb 14, 2018
Merged

Refactoring tests #1011

merged 1 commit into from
Feb 14, 2018

Conversation

carusogabriel
Copy link
Contributor

I've refactored some tests, using:

  • assertInternalType instead of is_* functions;
  • assertCount instead of count and sizeof functions;
  • assertNotSame instead os strict comparisons !==;
  • assertInstanceOf instead of instanceof operator;
  • assertGreaterThan for mathematical comparisons.

@jaypatel512 jaypatel512 merged commit 219390b into paypal:master Feb 14, 2018
@jaypatel512
Copy link
Contributor

Thanks a lot @carusogabriel for your contribution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants