Added tests for ported logic of handling negative patterns for DownloadBuildArtifacts and DownloadPipelineArtifacts tasks

[denis-tikhomirov]

Added tests for ported logic of handling negative patterns for DownloadBuildArtifacts and DownloadPipelineArtifacts tasks.

Checklist:

• There’s no risky dependency updates

  • Corrected function is used by 2 tasks. And both these tasks work correctly with the changes.

• Changes has been tested

  • Yes. The changes are tested by 22 cases (2 affected tasks, 11 cases for each task) by build in prepared pipeline. Also test with 7 cases was added in these PR to cover corrected function.

• Enough test coverage for changes, and current test coverage for task doesn't look poor

  • Yes, test with 7 cases was added in these PR to cover corrected function. Most cases of using the function are covered.

• We understand how tasks is working, how changes affect task behavior

  • Yes. The changes allow to collect list of files which should be downloaded from artifact. Previously all files are downloaded.

• There is no breaking changes

  • No breaking changes.

• There is no any other concerns

  • No concerns. Corrected function is used by 2 tasks. And both these tasks work correctly with the changes.

• I have not discovered any new uncovered test/use cases

  • Yes, I don't uncovered test/use cases for corrected function.