Skip to content

[PhpUnitBridge] Add polyfill for methods assertNan, assertFinite and assertInfinite #32886

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 3, 2019

Conversation

jderusse
Copy link
Member

@jderusse jderusse commented Aug 2, 2019

Q A
Branch? 4.4
Bug fix? yes
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #32844
License MIT
Doc PR NA

Provide pollyfile for methods assertFinite, assertInfinite and assertNan

@jderusse jderusse force-pushed the polyfill-isMethods branch from ffc44f1 to e4282bd Compare August 2, 2019 21:10
@nicolas-grekas nicolas-grekas added this to the next milestone Aug 2, 2019
@jderusse jderusse force-pushed the polyfill-isMethods branch from e4282bd to 7b95f43 Compare August 2, 2019 21:46
@nicolas-grekas nicolas-grekas changed the title Add polyfill for methods isNan, isFinite and isInfinite [PhpUnitBridge] Add polyfill for methods assertNan, assertFinite and assertInfinite Aug 3, 2019
@nicolas-grekas
Copy link
Member

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit 3450c17 into symfony:4.4 Aug 3, 2019
nicolas-grekas added a commit that referenced this pull request Aug 3, 2019
…rtFinite and assertInfinite (jderusse)

This PR was merged into the 4.4 branch.

Discussion
----------

[PhpUnitBridge] Add polyfill for methods assertNan, assertFinite and assertInfinite

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #32844
| License       | MIT
| Doc PR        | NA

Provide pollyfile for methods `assertFinite`, `assertInfinite` and `assertNan`

Commits
-------

3450c17 [PhpUnitBridge] Add polyfill for methods assertNan, assertFinite and assertInfinite
nicolas-grekas added a commit that referenced this pull request Aug 3, 2019
…las-grekas)

This PR was merged into the 3.4 branch.

Discussion
----------

[PhpUnitBridge] Use PHPUnit75Migration in CS fixer

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes (not yet) need #32882, #32886, #32893
| Fixed tickets | #32844
| License       | MIT
| Doc PR        | NA

Bump php-cs-fixer rule to PHPUnit75Migration:risky which add
- php_unit_dedicate_assert_internal_type
- php_unit_expectation
- php_unit_mock
- php_unit_no_expectation_annotation

which are (or will be) all polyfillyed

- [x] need to polyfill `createPartialMock`

Commits
-------

9fb1c42 Adopt `@PHPUnit55Migration:risky` rule of php-cs-fixer
@jderusse jderusse deleted the polyfill-isMethods branch August 8, 2019 11:37
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.4 Oct 27, 2019
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.

3 participants