Skip to content

Conversation

@ybelenko
Copy link
Contributor

@ybelenko ybelenko commented Jun 9, 2020

Update PHP client package and PHPUnit to meet PHP 7.2 version.
Related issue #6522

Test results from samples/client/petstore/php/OpenAPIClient-php:

PHPUnit 8.5.5 by Sebastian Bergmann and contributors.

RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR  63 / 239 ( 26%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 126 / 239 ( 52%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 189 / 239 ( 79%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR              239 / 239 (100%)

Time: 411 ms, Memory: 10.00 MB
(...)
OK, but incomplete, skipped, or risky tests!
Tests: 239, Assertions: 0, Risky: 239.
$  vendor/bin/phpunit tests
PHPUnit 8.5.5 by Sebastian Bergmann and contributors.

................................................................. 65 / 69 ( 94%)
....                                                              69 / 69 (100%)

Time: 10.88 seconds, Memory: 14.00 MB

OK (69 tests, 1110 assertions)
$  vendor/bin/php-cs-fixer fix --allow-risky=yes
Loaded config default from "/Users/ybelenko/Sites/openapi-generator/samples/client/petstore/php/OpenAPIClient-php/.php_cs".

Fixed all files in 1.370 seconds, 22.000 MB memory used

With PHPUnit 9:

$  vendor/bin/phpunit
PHPUnit 9.2.2 by Sebastian Bergmann and contributors.

RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR  63 / 239 ( 26%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 126 / 239 ( 52%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 189 / 239 ( 79%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR              239 / 239 (100%)

Time: 00:00.156, Memory: 12.00 MB
(...)

OK, but incomplete, skipped, or risky tests!
Tests: 239, Assertions: 0, Risky: 239.
$  vendor/bin/phpunit tests
PHPUnit 9.2.2 by Sebastian Bergmann and contributors.

................................................................. 65 / 69 ( 94%)
....                                                              69 / 69 (100%)

Time: 00:09.746, Memory: 14.00 MB

OK (69 tests, 1111 assertions)

Tests results from samples/openapi3/client/petstore/php/OpenAPIClient-php:

$  vendor/bin/phpunit
PHPUnit 8.5.5 by Sebastian Bergmann and contributors.

RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR  63 / 227 ( 27%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 126 / 227 ( 55%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 189 / 227 ( 83%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR                          227 / 227 (100%)

Time: 323 ms, Memory: 10.00 MB
(...)
OK, but incomplete, skipped, or risky tests!
Tests: 227, Assertions: 0, Risky: 227.
$  vendor/bin/phpunit tests
PHPUnit 8.5.5 by Sebastian Bergmann and contributors.

................................................................. 65 / 70 ( 92%)
.....                                                             70 / 70 (100%)

Time: 10.3 seconds, Memory: 14.00 MB

OK (70 tests, 1116 assertions)
$  vendor/bin/php-cs-fixer fix --allow-risky=yes
Loaded config default from "/Users/ybelenko/Sites/openapi-generator/samples/openapi3/client/petstore/php/OpenAPIClient-php/.php_cs".

Fixed all files in 1.438 seconds, 22.000 MB memory used

Files that were not fixed due to errors reported during linting before fixing:
   1) /Users/ybelenko/Sites/openapi-generator/samples/openapi3/client/petstore/php/OpenAPIClient-php/lib/Model/OuterEnumIntegerDefaultValue.php
   2) /Users/ybelenko/Sites/openapi-generator/samples/openapi3/client/petstore/php/OpenAPIClient-php/lib/Model/OuterEnumInteger.php

Fixer detects broken PHP syntax in enum models, but it's obviously not a job for this PR. Should be fixed in future.

With PHPUnit 9:

$  vendor/bin/phpunit
PHPUnit 9.2.2 by Sebastian Bergmann and contributors.

RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR  63 / 227 ( 27%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 126 / 227 ( 55%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR 189 / 227 ( 83%)
RRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRRR                          227 / 227 (100%)

Time: 00:00.154, Memory: 12.00 MB
(...)
OK, but incomplete, skipped, or risky tests!
Tests: 227, Assertions: 0, Risky: 227.
$  vendor/bin/phpunit tests
PHPUnit 9.2.2 by Sebastian Bergmann and contributors.

................................................................. 65 / 70 ( 92%)
.....                                                             70 / 70 (100%)

Time: 00:13.903, Memory: 14.00 MB

OK (70 tests, 1121 assertions)

PR checklist

  • Read the contribution guidelines.
  • If contributing template-only or documentation-only changes which will change sample output, build the project beforehand.
  • Run the shell script ./bin/generate-samples.shto update all Petstore samples related to your fix. This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master. These must match the expectations made by your contribution. You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/config/java*.
  • File the PR against the correct branch: master, 4.3.x, 5.0.x. Default: master.
  • Copy the technical committee to review the pull request if your PR is targeting a particular programming language.

cc @jebentier @dkarlovi @mandrean @jfastnacht @ackintosh @renepardon

@ybelenko
Copy link
Contributor Author

ybelenko commented Jun 9, 2020

I guess this PR won't pass CI, because root Travis config of this repo set to PHP 7.1.

@wing328 wing328 merged commit 4f1d7c0 into OpenAPITools:master Jun 14, 2020
@ybelenko ybelenko deleted the 6522_php_client branch June 14, 2020 14:48
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