Skip to content

Remove flaky tests from the dependency proxy suite

🌲 Context

In Maven Dependency proxy: cache miss path (!130549 - merged), we expanded the workhorse dependency proxy with options to customize how the remote file is uploaded.

At that time, we introduced a set of tests that started failing on master after a while. See #425738 (closed).

Fixing those tests will take a bit of time: we need to put some proper validation in place. To quickly fix master, we suggest in this MR to simply remove those tests.

In a subsequent MR, we will add them again along with the validation.

Removing tests seem not a great idea but actually, things are quite covered in this area. The feature that will actively use this "extension" of the dependency proxy is the dependency proxy for Maven. That part is:

🤔 What does this MR do and why?

  • Remove invalid_x tests from the dependency proxy suite in workhorse.

🖼 Screenshots or screen recordings

none

gear How to set up and validate locally

Run all the workhorse tests with:

$ make test

They should all pass.

🏁 MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports