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

github action tests #18208

Closed
wants to merge 116 commits into from
Closed

github action tests #18208

wants to merge 116 commits into from

Conversation

peterhashair
Copy link
Contributor

@peterhashair peterhashair commented Oct 24, 2021

Description:

add GitHub action test.

PHP Tests

  1. UnitTests ✅. (Modified one tests, fully passed)
  2. SystemTestsCore
  3. SystemTestsPlugins
  4. IntegrationTestsCore
  5. IntegrationTestsPlugins

Js Tests

  1. JavascriptTests. ✅ (Not sure I did it correctly, but currenly shown as passed)
  2. AngularJSTests. ✅ (Not sure I did it correctly, but currenly shown as passed)

UI tests

  1. Group 0
  2. Group 1
  3. Group 2

around 22min runing all the php tests, we can reduce the run time a little more by add @group in tests split into smaller groups. Not sure the test faileds, it seems like the object in the function is not displayed. Could be xdebug not enable. About to set up the UI test, and js test.

Review

Peter Zhang added 4 commits October 24, 2021 19:44
add github action test
add config file
update file name
update branch name
.github/workflows/tests.yml Outdated Show resolved Hide resolved
Peter Zhang added 11 commits October 25, 2021 10:59
update git lfs
add ui tests
update node js
update directory
update tests
update tests
update all tests
alter a php test
update database container
update db port
@peterhashair
Copy link
Contributor Author

@sgiehl for the php test, it's actually got coupe errors, do you have any idea if there are any configure I did wrong?

update UI tests
@sgiehl
Copy link
Member

sgiehl commented Oct 25, 2021

I guess one point might be that we were actually using port 80 and not port 3000 if sudo was available on travis. See https://github.com/matomo-org/travis-scripts/blob/ca0dfcfa2ca0e552fc65b77649dffbf101a6a17e/setup_webserver.sh#L63-L64

Peter Zhang and others added 11 commits October 25, 2021 20:36
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
Peter Zhang added 29 commits October 27, 2021 15:47
use php-fpm
remove update php
remove a config not in use
disable fast cgi
copy config file
update UI test to apache
update ui and php config
enable UI tests
update cache permission
update folder permission for apache
update file access permission
update redis and update gloable ini
update apache perimssion
remove permission on fonts etc
use php instead of package
remove apache, set nginx
only test UI
make it fast only test one
only test nginx
check apache status
tests nginx
typo
add full path
remove error conf
update bracks
add events fields
use php 8.0
apache ui 0 test
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

2 participants