[SYNTH-15675] Fix bug where timed out retries are not reported

[AntoineDona]

What and why?

We recently discovered a bug where timed out fast retries were not reported as fast retries. This lead to executeTests exiting without the reporter spinner being stopped.

This was due to the fact that getResultsToReport didn’t handle still non-final results after the end of a batch, resulting in an empty resultsToReport array. This PR patches this issue.

How?

• Add the still non-final fast retry results to the residualResults

Review checklist

• [x] Feature or bugfix MUST have appropriate tests (unit, integration)

[Drarig29]

@AntoineDona @etnbrd this PR is more of a draft of a solution we thought was working. But I now think we missed something, and we'll probably need to add a property in the backend.

For now, consider this PR on-hold.

[datadog-datadog-prod-us1[bot]]

Datadog Report

Branch report: antoine.donascimento/SYNTH-15675/fix-non-reported-timing-out-fast-retries Commit report: 6d23211 Test service: datadog-ci-tests

:x: 41 Failed (0 Known Flaky), 3533 Passed, 0 Skipped, 2m 29.74s Total duration (1m 2.63s time saved)

:x: Failed Tests (41)

This report shows up to 5 failed tests.

• Proxy configuration use proxy defined in configuration - src/commands/synthetics/__tests__/tunnel/websocket.test.ts - Details

<details>
<summary>Expand for error</summary>

```
Failed to get batch: could not query base/synthetics/ci/batch/bid
connect ECONNREFUSED 127.0.0.1:80

```
</details>

• utils waitForResults correct number of passed and timed out results - src/commands/synthetics/__tests__/utils/public.test.ts - Details

  Expand for error

  ``` expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 3 @@ -60,10 +60,11 @@ "subtype": "http", "tags": Array [], "type": "api", }, ... ```

• utils waitForResults correct number of passed and timed out results - src/commands/synthetics/__tests__/utils/public.test.ts - Details

  Expand for error

  ``` expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 3 @@ -60,10 +60,11 @@ "subtype": "http", "tags": Array [], "type": "api", }, ... ```

• utils waitForResults correct number of passed and timed out results - src/commands/synthetics/__tests__/utils/public.test.ts - Details

  Expand for error

  ``` expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 3 @@ -60,10 +60,11 @@ "subtype": "http", "tags": Array [], "type": "api", }, ... ```

• utils waitForResults correct number of passed and timed out results - src/commands/synthetics/__tests__/utils/public.test.ts - Details

  Expand for error

  ``` expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 3 @@ -60,10 +60,11 @@ "subtype": "http", "tags": Array [], "type": "api", }, ... ```