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

[DEX-943] Send phpunit tests to Report Portal #154

Merged

Conversation

carine-bonnafous
Copy link
Contributor

@carine-bonnafous carine-bonnafous commented Nov 18, 2024

Reason for change

Closes DEX-943

Code changes

  • For PHP versions 7.1, 7.2, 7.3 and 7.4, only in the CI, the script ./tests/reportportal/reportportal_conf.sh is run to configure PHP unit to send test results to ReportPortal
  • reportportal/basic library is patched to use ReportPortal api/v2

How to test

As a reviewer, you are encouraged to test the PR locally.

@carine-bonnafous carine-bonnafous force-pushed the devx/dex-943-send-phpunit-tests-to-reportportal branch 3 times, most recently from a1fe042 to 3265e6b Compare November 19, 2024 11:04
@carine-bonnafous carine-bonnafous force-pushed the devx/dex-943-send-phpunit-tests-to-reportportal branch from 9d0fbb5 to f132dd9 Compare November 19, 2024 13:16
@carine-bonnafous carine-bonnafous force-pushed the devx/dex-943-send-phpunit-tests-to-reportportal branch from a6f4021 to 574d06b Compare November 28, 2024 16:51
@carine-bonnafous carine-bonnafous force-pushed the devx/dex-943-send-phpunit-tests-to-reportportal branch from df07bb3 to 0c686f2 Compare November 29, 2024 10:08
@carine-bonnafous carine-bonnafous marked this pull request as ready for review November 29, 2024 11:07
@carine-bonnafous carine-bonnafous requested review from a team as code owners November 29, 2024 11:07
Copy link
Member

@hyahiaoui hyahiaoui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing job!

@carine-bonnafous carine-bonnafous merged commit e2df0d8 into main Dec 13, 2024
13 checks passed
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