Skip to content
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

Draft PHP 8.1 branch 2.2. #305

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

terabytesoftw
Copy link
Member

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

@what-the-diff
Copy link

what-the-diff bot commented Jul 10, 2023

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.

@codecov
Copy link

codecov bot commented Jul 10, 2023

Codecov Report

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

❗ 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           

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

@terabytesoftw terabytesoftw changed the title Draft PHP 8.1. Draft PHP 8.1 branch 2.2. Jul 10, 2023
@terabytesoftw terabytesoftw marked this pull request as ready for review July 11, 2023 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant