Skip to content

Improve test execution

Tomasz Maczukin requested to merge improve-test-execution into master

What does this MR do?

Improves tests execution to use full project as the "context" for coverage report. The second small fix is the default value of CI_NODE_INDEX, which should be 1 not 0.

Why was this MR needed?

Just to make our coverage value more accurate. Before this change when executing tests on package gitlab.com/gitlab-org/gitlab-runner/cache/gcs, the coverage profile was tracked only on executed code parts within this package.

With this MR, the coverage profile will contain information about all parts of gitlab.com/gitlab-org/gitlab-runner code parts. This obviously will increase the coverage value, which is expected - we're interested in tracking whole project on a particular test execution, not only the tested package part :)

The CI_NODE_INDEX fix makes the local usage of scripts/go_test_with_coverage_report easier.

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?

Edited by Tomasz Maczukin

Merge request reports