[BUG] Code coverage metrics aren't uploaded when build fails due to low code coverage

[dbwiddis]

What is the bug?

I'm trying to improve code coverage on #2430. The build is passing all the unit and integ tests, but is failing on lack of coverage:

> Task :opensearch-ml-plugin:test
> Task :opensearch-ml-plugin:jacocoTestReport
[ant:jacocoReport] Rule violated for class org.opensearch.ml.sdkclient.RemoteClusterIndicesClient: branches covered ratio is 0.5, but expected minimum is 0.7
[ant:jacocoReport] Rule violated for class org.opensearch.ml.sdkclient.LocalClusterIndicesClient: branches covered ratio is 0.5, but expected minimum is 0.7

I'm covering about half of my branches, but honestly I don't know which ones I'm missing.

The code coverage report would tell me, but it doesn't run, because it's placed after the test in the CI yaml.

      - name: Build and Run Tests
        id: step-build-test-linux
        run: |
          // unit and integ tests

      - name: Upload Coverage Report
        uses: codecov/codecov-action@v1

This presents an extremely frustrating experience for a developer.

How can one reproduce the bug?
Steps to reproduce the behavior:

1. Submit a PR with a file that does not meet minimum code coverage requirements but otherwise passes tests.
2. Observe a lack of coverage reports.

What is the expected behavior?

When the specific reason for the build failure is lack of coverage, the specific lines/branches missed should be available to the developer somewhere.

What is your host/environment?

• OS: Both Linux and WIndows CI

Do you have any additional context?

If codecov was placed in a separate workflow, it would always run. This gives the added benefit of ensuring the coverage comment is (one of) the first one on the PR and easier for reviewers to find. It would still fail that other workflow, although not quite as visibly as the tests.

Alternately if the desire is to keep the entire test failing on low coverage, the jacocoTestReport gradle task could be moved until after the coverage run in the workflow.

[dhrubo-os]

Cut an issue to the infra team for token: opensearch-project/.github#270

Raised the PR to downgrade codecov action: #3409

Now we can see the codecoveage report: Example PR

Closing the issue.