Commit 6da6172
Fix CI: use find_dotenv() and fall back to DATABASE_* vars
connect_to_db() in test_postgres_real_data.py had a hardcoded .env path
that doesn't work inside Docker containers, and required DATABASE_URL
with no fallback. Now uses find_dotenv() to search up the directory tree,
and falls back to DATABASE_HOST/PORT/NAME/USER/PASSWORD (which the
docker-compose.test.yml delphi service already provides).
Also removes dead POSTGRES_* env vars from the CI exec command — nothing
in Delphi reads those anymore.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>1 parent 5afa60d commit 6da6172
2 files changed
Lines changed: 15 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | 93 | | |
97 | 94 | | |
98 | 95 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
102 | 102 | | |
103 | 103 | | |
104 | 104 | | |
105 | | - | |
106 | | - | |
107 | | - | |
| 105 | + | |
| 106 | + | |
108 | 107 | | |
109 | | - | |
110 | | - | |
| 108 | + | |
| 109 | + | |
111 | 110 | | |
112 | 111 | | |
113 | 112 | | |
114 | | - | |
115 | | - | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
116 | 124 | | |
117 | 125 | | |
118 | 126 | | |
| |||
0 commit comments