Skip to content

Commit 4fff1ba

Browse files
fix(tests): Re-enable MQTT and Postgres tests
Signed-off-by: Karan <karanlokchandani@protonmail.com>
1 parent 50e7c9e commit 4fff1ba

File tree

2 files changed

+19
-3
lines changed

2 files changed

+19
-3
lines changed

.github/workflows/build.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,20 @@ jobs:
191191
# run on a larger runner for more SSD/resource access
192192
runs-on: ubuntu-22.04-4core-spin
193193
if: ${{ github.repository_owner == 'spinframework' }}
194+
services:
195+
postgres:
196+
image: postgres:latest
197+
env:
198+
POSTGRES_PASSWORD: postgres
199+
POSTGRES_USER: postgres
200+
POSTGRES_DB: postgres
201+
ports:
202+
- 5432:5432
203+
options: >-
204+
--health-cmd pg_isready
205+
--health-interval 10s
206+
--health-timeout 5s
207+
--health-retries 5
194208
steps:
195209
- uses: actions/checkout@v3
196210

@@ -217,6 +231,8 @@ jobs:
217231
218232
- name: Run Full Integration Tests
219233
run: make test-integration-full
234+
env:
235+
DB_URL: postgres://postgres:postgres@localhost:5432/postgres
220236

221237
# This runs the same steps as the release build except for signing
222238
ensure-release-builds:

crates/variables/src/env.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ impl EnvVariablesProvider {
5252
/// Creates a new EnvProvider.
5353
///
5454
/// * `prefix` - The string prefix to use to distinguish an environment variable that should be used.
55-
/// If not set, the default prefix is used.
55+
/// If not set, the default prefix is used.
5656
/// * `env_fetcher` - The function to use to fetch an environment variable.
57-
/// * `dotenv_path` - The path to the .env file to load environment variables from. If not set,
58-
/// no .env file is loaded.
57+
/// * `dotenv_path` - The path to the .env file to load environment variables from.
58+
/// If not set, no .env file is loaded.
5959
pub fn new(
6060
prefix: Option<impl Into<String>>,
6161
env_fetcher: impl Fn(&str) -> Result<String, VarError> + Send + Sync + 'static,

0 commit comments

Comments
 (0)