Skip to content

Commit baa3103

Browse files
ai: apply changes for #870 (1 review thread)
Addresses: - #3600339404 at .github/workflows/engineer-bot-followup.yml:107 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent c4f3c76 commit baa3103

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

.bot/prompts/engineer-followup/system.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,16 @@ Your job:
1414
pass: `poetry run python -m pytest tests/unit/<file> -k <name>` (and the
1515
affected file's full set before you finish). Never weaken or skip a test to
1616
go green.
17+
- This runner installs `--all-extras`, so the REAL `databricks-sql-kernel`
18+
wheel is present. The unit suite fakes `databricks_sql_kernel` in
19+
`sys.modules` (`tests/unit/test_kernel_client.py`), which shadows the
20+
real wheel in a shared session — and the `@pytest.mark.realkernel`
21+
routing test (`tests/unit/test_session.py::TestUseKernelRoutesThroughRealWheel`)
22+
`pytest.fail`s loudly on that shadowing. So whenever you run a selection
23+
BROADER than a single `-k` test — a whole file, or `tests/unit` — append
24+
`-m "not realkernel"` (matching how `.github/workflows/code-coverage.yml`
25+
guards the same `--all-extras` install). Skipping this produces a
26+
confusing false red unrelated to your fix.
1727
4. End with a short summary of what changed.
1828

1929
Repo facts you need:

0 commit comments

Comments
 (0)