Skip to content

Commit

Permalink
feat: revert smoke test temp change
Browse files Browse the repository at this point in the history
  • Loading branch information
cbrinson-rise8 committed Jan 17, 2025
1 parent 503b7e2 commit 42885a0
Showing 1 changed file with 13 additions and 14 deletions.
27 changes: 13 additions & 14 deletions .github/workflows/check_smoke_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,14 @@ jobs:
runs-on: ubuntu-latest

# TODO: modify smoke tests to cover multiple db types
# services:
# db:
# image: postgres:17
# env:
# POSTGRES_USER: postgres
# POSTGRES_PASSWORD: pw
# ports:
# - 5432:5432
services:
db:
image: postgres:17
env:
POSTGRES_USER: postgres
POSTGRES_PASSWORD: pw
ports:
- 5432:5432

steps:
- name: Check out repository
Expand All @@ -27,10 +27,10 @@ jobs:
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3

- name: Prepare SQLite Database
run: |
# Ensure the SQLite database file exists
touch test.db
# - name: Prepare SQLite Database
# run: |
# # Ensure the SQLite database file exists
# touch test.db

- name: Build RecordLinker Docker Image
run: |
Expand All @@ -41,8 +41,7 @@ jobs:
# Start Record Linker Service with SQLite
docker run -d --name rl-service \
--network="host" \
-e DB_URI="sqlite:///test.db" \
-v $PWD/test.db:/app/test.db \
-e DB_URI="postgresql+psycopg2://postgres:pw@localhost:5432/postgres" \
-p 8080:8080 \
rl-service-image
Expand Down

0 comments on commit 42885a0

Please sign in to comment.