yii2-app-basic icon indicating copy to clipboard operation
yii2-app-basic copied to clipboard

Draft `PHP 8.1` branch `2.2`.

Open terabytesoftw opened this issue 2 years ago • 2 comments

Q A
Is bugfix? no
New feature? yes
Breaks BC? yes
Tests pass? yes

terabytesoftw avatar Jul 10 '23 21:07 terabytesoftw

PR Summary

  • Upgraded PHP Versions and Updated Build Steps In the .github/workflows/build.yml file, this PR updated the supported PHP versions to include 8.1 and 8.2. Furthermore, the step to install dependencies has been expanded to invoke the vendor/bin/codecept run command with the --coverage-xml flag.

  • Updated Test Configuration In the codeception.yml file, the coverage section has been introduced. This means the framework can now gauge the test coverage, essentially determining which parts of the code are exercised by tests and which are not.

  • Updated PHP and Dependency Versions In the composer.json file, the required PHP version was updated to 8.1. Additionally, the yiisoft/yii2 dependency was updated to version 2.0.48.1. Other dependencies for codeception, the testing framework, were also updated.

  • Added New Test Suite Configurations A new configuration file tests/Acceptance.suite.yml has been added. This can be seen as a blueprint for future acceptance test suites.

  • Refactored Test Classes In the tests/acceptance/ directory, several test classes were refactored. Specifically their names and methods were updated to better reflect their functionality.

  • Added New Testers New tester classes were added into both tests/ and tests/Support directories. These tester files help in executing the tests effectively and managing the testing process for different levels of testing i.e., acceptance, functional, and unit tests.

Please review the individual Git diffs for more details. The above changes are steps towards modernizing and optimizing the testing process for our application. With these changes, our tests are more reliable, efficient, and maintainable.

what-the-diff[bot] avatar Jul 10 '23 22:07 what-the-diff[bot]

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@150f77a). Click here to learn what that means. Patch has no changes to coverable lines.

:exclamation: Current head 922d132 differs from pull request most recent head 7b69072. Consider uploading reports for the commit 7b69072 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             master      #305   +/-   ##
==========================================
  Coverage          ?   100.00%           
  Complexity        ?        43           
==========================================
  Files             ?        10           
  Lines             ?       279           
  Branches          ?         0           
==========================================
  Hits              ?       279           
  Misses            ?         0           
  Partials          ?         0           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jul 10 '23 22:07 codecov[bot]