Skip to content

#1937 - Add tests for Bingo Oracle#3634

Draft
Mantas-Kovarskas wants to merge 12 commits into
masterfrom
issue-1937
Draft

#1937 - Add tests for Bingo Oracle#3634
Mantas-Kovarskas wants to merge 12 commits into
masterfrom
issue-1937

Conversation

@Mantas-Kovarskas
Copy link
Copy Markdown
Collaborator

@Mantas-Kovarskas Mantas-Kovarskas commented May 4, 2026

No description provided.

Copilot AI and others added 12 commits February 20, 2026 14:50
- Create Oracle database adapter (bingo/tests/dbc/OracleDB.py) following
  the same pattern as PostgresSQL.py, with Oracle-specific SQL syntax
- Add Oracle configuration to db_config.ini
- Update base.py to support Oracle connection strings and env var overrides
- Update conftest.py to handle Oracle DB setup/teardown
- Add oracledb dependency to requirements.txt
- Add test_bingo_oracle_linux_x86_64 CI job using gvenzl/oracle-xe service

Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
On Ubuntu 24.04 (current ubuntu-latest), the libaio1 package has been
renamed to libaio1t64. Use fallback to support both Ubuntu versions.

Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
The `PATH: /opt/oracle/instantclient_23_5:${{ env.PATH }}` in the env
block resolved to just `/opt/oracle/instantclient_23_5:` because
`${{ env.PATH }}` doesn't access the system PATH. This caused mkdir,
tar, and other basic commands to not be found. Fixed by using
`export PATH=...` inside the run block where shell $PATH works.

Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
On Ubuntu 24.04, libaio1t64 installs libaio.so.1t64 but sqlplus
expects libaio.so.1. Add a conditional symlink when libaio.so.1
doesn't exist but libaio.so.1t64 does.

Co-authored-by: AlexeyGirin <26869421+AlexeyGirin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants