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

Tests GitHub action #18254

Closed
wants to merge 260 commits into from
Closed

Tests GitHub action #18254

wants to merge 260 commits into from

Conversation

peterhashair
Copy link
Contributor

@peterhashair peterhashair commented Nov 2, 2021

Description:

The pipeline test should auto-cancel the previous build, restart the new build job, the test contains PHP, UI, and javascript tests.

Split test into more parts, it reduces 10mins. Can do more, but I haven't looked at it

Server using php -S 80 and php -S 3000. keep 2 ports is because PHP can only start on one port curl exec the other. To pass some send HTTP-related test.

Currently using Mariadb and tmpf memory setup to speed the process, but seems like failed some of the MySQL-related tests. some are being skipped, some are fixed.

Still have quite a few screenshots that failed. Currently, the failed UI is in the artifact, can be downloaded from here,
image
would be great, if we can config the build.matomo.org to do a comparison. so can easily identify the failed reason.

Review

Peter Zhang and others added 30 commits October 25, 2021 14:57
update tests
update all tests
alter a php test
update database container
update db port
update UI tests
update tests
update test to port 80
check 80 port
user port 3000 for all tests
test tmux 3000 is runing
test 80
update test to 80 ports
update tmp to 755
test new pipeline
update ui tests
update folder permission
command line up
update command to sudo
update ui folder permission
update UI test to sudo
run sudo ui tests
update tests
update php packages and mysql config
update tests lib
display expected screen shot
add git lfs pull screen shot
add ui texxt
remove travisyml
@github-actions
Copy link
Contributor

If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'.

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Nov 25, 2021
@sgiehl sgiehl deleted the tests-github-action branch April 5, 2023 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale The label used by the Close Stale Issues action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants