Skip to content

Commit 7cda6a3

Browse files
committed
fix: Skip pgpm tests in regular workflow and add verbose logging
- Exclude tests/test_pgpm_integration.py from regular test workflow (pgpm CLI not installed in that workflow) - Add --verbose flag to pgpm deploy for better debugging - Log stdout and stderr from pgpm deploy for troubleshooting
1 parent 62e8c02 commit 7cda6a3

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,4 +65,4 @@ jobs:
6565
run: poetry run mypy src --ignore-missing-imports
6666

6767
- name: Run tests
68-
run: poetry run pytest -v
68+
run: poetry run pytest -v --ignore=tests/test_pgpm_integration.py

src/pysql_test/seed/pgpm.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ def seed(self, ctx: SeedContext) -> None:
7575
cwd = self._module_path or os.getcwd()
7676

7777
# Build pgpm deploy command
78-
cmd = ["pgpm", "deploy", "--yes"]
78+
cmd = ["pgpm", "deploy", "--yes", "--verbose"]
7979
cmd.extend(self._deploy_args)
8080

8181
logger.info(f"Running pgpm deploy in {cwd}")
@@ -99,7 +99,9 @@ def seed(self, ctx: SeedContext) -> None:
9999

100100
logger.info("pgpm deploy completed successfully")
101101
if result.stdout:
102-
logger.debug(f"pgpm output: {result.stdout}")
102+
logger.info(f"pgpm output: {result.stdout}")
103+
if result.stderr:
104+
logger.info(f"pgpm stderr: {result.stderr}")
103105

104106
except FileNotFoundError as err:
105107
raise RuntimeError(

0 commit comments

Comments
 (0)