Problem/Motivation

While any MR is created the test run takes ages to complete and hence i think we can run test parallelly to execute pipeline faster.

Issue fork acquia_dam-3514283

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

deepakkm created an issue. See original summary.

deepakkm opened merge request !136

japerry made their first commit to this issue’s fork.

rajeshreeputra made their first commit to this issue’s fork.

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented

Added Drupal 9, 10, 11 in CI:

  1. composer-drupal-core-9 runs for Drupal 9.
  2. Previous major runs fro Drupal 10.
  3. Current runs for Drupal 11 latest.

Next step:

  1. Look into phpunit parallel test execution.
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented

Added Drupal 9, 10, 11 in CI:

  1. composer-drupal-core-9 runs for Drupal 9.
  2. Previous major runs for Drupal 10.
  3. Current runs for Drupal 11 latest.

Next step:

  1. Look into phpunit parallel test execution.
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented

Status:

Unit

  • Drupal 9 ✅
  • Drupal 10 ✅
  • Drupal 11.1 ✅
  • Drupal 11.2 ✅

Kernel

  • Drupal\Tests\acquia_dam\Kernel\ExpiredTagTest::testExpiryTag - This is already fixed in MR!188

    •
  • Got warning, we'll investigate. Option --group cannot be used more than once
  • All test pass but job failed with exit code 1, we'll investigate.

Functional Javascript

  • Got warning, we'll investigate. Option --group cannot be used more than once
  • All test pass but job failed with exit code 1, we'll investigate.

Update:

  • Fixed ExpiredTagTest in this issue.
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Title: Run Tests in Gitlab Parallelly » Configure GitLab to run tests in parallel for faster execution.
rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Needs work » Needs review

Updated CI template to run test in parallel, requesting review.

  • japerry committed 397fe3c7 on 1.1.x authored by deepakkm 
    Resolve #3514283: "Configure GitLab to run tests in parallel for faster...
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Needs work

Committed the gitlab file, but marking NW to try to fix some of these underlying test failures.

rajeshreeputra opened merge request !202

japerry’s picture

japerry
Location KOMK
commented
Priority: Normal » Critical

I fixed the issue with css selector not working. However there are several other issues that need to be resolved:

1) On the configuration page, there is an 'authenticate site' button. This shouldn't be here as pressing 'save' with a new domain should automatically redirect you to authenticate the site. If the key module is working correctly, then pressing save should return a "validating domain - ok!' success message.

2) They Key configuration when pressing save is incorrect and will cause a fatal issue because we don't check to see if key is the auth type before fetching the domain.

3) There is an issue with image styles.. not quite sure what yet, but looking at it.

These are blocking before we can merge in the canvas support.

japerry’s picture

japerry
Location KOMK
commented
Status: Needs work » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.