Skip to content

Fix command that prepares the definitions of tests

Tomasz Maczukin requested to merge fix-prepare-test-definitions-step into master

What does this MR do?

Fixes a bug in parallel tests scripting.

Why was this MR needed?

If there was any compilation error in the test, then whole package was getting excluded from the list of tests to execute. Look at !1008 (comment 106781262) for an example (and search for executors/docker/terminal_test at https://gitlab.com/gitlab-org/gitlab-runner/-/jobs/105055262 to see what happened).

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Merge request reports