Skip to content

Commit

Permalink
[receiver/sqlquery] Decrease time of running integration tests (#37624)
Browse files Browse the repository at this point in the history
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Improvements to integration test:
1. Reduced test execution time by changing the behavior from starting
and shutting down the database container for each test case to starting
the container **once** and then running all tests for the given engine.
Gain ~22s:

[Main
job](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/13077774968/job/36494091929):
 ```
DONE 47 tests, 8 skipped in 136.005s
```
[PR job](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/13077857640/job/36494335331?pr=37624)
```
DONE 44 tests, 4 skipped in 114.374s
```  

2. Updated the Oracle container from [gvenzl/oracle-xe:21-slim-faststart](https://hub.docker.com/r/gvenzl/oracle-xe) to [gvenzl/oracle-free:slim-faststart](https://hub.docker.com/r/gvenzl/oracle-free), primarily to support ARM architecture.


<!--Describe what testing was performed and which tests were added.-->
#### Testing
* Ran tests for Oracle and SQL Server locally to ensure they are not broken.

Co-authored-by: Curtis Robert <[email protected]>
  • Loading branch information
Grandys and crobert-1 authored Feb 7, 2025
1 parent 4244166 commit 769c4c9
Show file tree
Hide file tree
Showing 6 changed files with 304 additions and 240 deletions.
Loading

0 comments on commit 769c4c9

Please sign in to comment.