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

Changes related to accommodate nightwatch tests in Gitlab CI #483

Merged
merged 30 commits into from
Jan 22, 2024
Merged

Conversation

secretsayan
Copy link
Contributor

@secretsayan secretsayan commented Jan 9, 2024

There are three changes -

  • Nightwatch and Functional Javascript : We have made changes in this PR so that we can use a separate container like drupal.org templates.
  • For tests with Coverage : We are generating report using Cobertura format instead of Clover. Other changes related to names of variable from clover to cobertura will be handled in Change coverage clover to cobertura for Gitlab CI #484 and not here due to time constraints.
  • We are generating junit log for all the phpunit tests so that all tests and reports are shown in the Tests tab of Gitlab CI.

@secretsayan secretsayan added enhancement New feature or request ci Modifications related to CI systems labels Jan 9, 2024
@secretsayan secretsayan force-pushed the wip branch 2 times, most recently from 792b3e7 to ab0e9b4 Compare January 9, 2024 15:25
@secretsayan
Copy link
Contributor Author

@TravisCarden Requesting one more round of review on this ASAP.

Copy link
Contributor

@TravisCarden TravisCarden left a comment

Choose a reason for hiding this comment

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

Just a few minor spelling and consistency issues. (I'll fix them myself quick.) Then it looks great!

bin/ci/_includes.sh Outdated Show resolved Hide resolved
bin/ci/script.sh Outdated Show resolved Hide resolved
bin/ci/self-test/script.sh Outdated Show resolved Hide resolved
bin/ci/self-test/script.sh Outdated Show resolved Hide resolved
bin/ci/_includes.sh Outdated Show resolved Hide resolved
@secretsayan secretsayan merged commit d2b0b5c into develop Jan 22, 2024
49 checks passed
secretsayan pushed a commit to secretsayan/orca that referenced this pull request Feb 15, 2024
)

* tests

* tests

* tests

* tests

* tests

* tests

* go to 4444

* remove core nightwatch

* enable core tests and disable shm

* remove core tests

* change port

* remoce chromedriver

* use port 9515 for gitlab

* minor fix

* Ignore toolbar from nightwatch

* cleanup

* comments

* comments

* comments

* Gitlab CI

* add junit log for all phpunit tests

* minor changes

* minor changes 2

* Cleanup

* change port for gitlab ci

* remove chromedriver

* change port

* change port 2

* Add cobertura format instead of clover

* Apply suggestions from code review

---------

Co-authored-by: Travis Carden <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Modifications related to CI systems enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants