didericis/bot-bottle
1
0
Fork 0
Code Issues 7 Pull Requests 15 Actions Packages Projects Releases Wiki Activity

fix: migrate integration tests off deprecated APIs and tools #199

Merged
didericis merged 1 commits from fix/integration-test-failures into main 2026-06-06 12:35:04 -04:00
Conversation 0 Commits 1 Files Changed 2
+10 -11
didericis-claude commented 2026-06-06 12:20:09 -04:00
Collaborator
Copy Link
Copy Source

Summary

  • test_sandbox_escape: migrated fixture manifest from deprecated git key to git-gate (PRD 0047) — remotes → repos with url/identity field names
  • test_smolmachines_launch probes: replaced wget (not installed in node:22-slim) with curl -s --show-error --max-time 3, which is explicitly installed in Dockerfile.claude
  • test_smolmachines_launch prompt test: corrected path from /root/.bot-bottle-prompt.txt to /home/node/.bot-bottle-prompt.txt (matches guest_home in smolmachines/prepare.py)

Test plan

  • python3 -m unittest discover -t . -s tests/integration -v — 23/23 pass
## Summary - `test_sandbox_escape`: migrated fixture manifest from deprecated `git` key to `git-gate` (PRD 0047) — `remotes` → `repos` with `url`/`identity` field names - `test_smolmachines_launch` probes: replaced `wget` (not installed in `node:22-slim`) with `curl -s --show-error --max-time 3`, which is explicitly installed in `Dockerfile.claude` - `test_smolmachines_launch` prompt test: corrected path from `/root/.bot-bottle-prompt.txt` to `/home/node/.bot-bottle-prompt.txt` (matches `guest_home` in `smolmachines/prepare.py`) ## Test plan - [x] `python3 -m unittest discover -t . -s tests/integration -v` — 23/23 pass
didericis force-pushed fix/integration-test-failures from c68d0bb701 to 241df1f835 2026-06-06 12:23:36 -04:00 Compare
didericis added 1 commit 2026-06-06 12:29:40 -04:00
fix(tests): fix integration test failures from deprecated git key, missing wget, and wrong prompt path
test / integration (pull_request) Successful in 41s
Details
test / unit (pull_request) Successful in 31s
Details
test / unit (push) Successful in 30s
Details
Update Quality Badges / update-badges (push) Successful in 1m3s
Details
lint / lint (push) Successful in 1m23s
Details
test / integration (push) Successful in 42s
Details
ac9b6d593f 
- test_sandbox_escape: migrate manifest fixture from deprecated `git`
  key to `git-gate` (PRD 0047) — `remotes` → `repos`, field names
  `Name`/`Upstream`/`IdentityFile` → `url`/`identity`
- test_smolmachines_launch probes: replace `wget` (not in node:22-slim)
  with `curl -s --show-error --max-time 3` (installed in Dockerfile.claude)
- test_smolmachines_launch prompt test: correct path /root/ → /home/node/
  to match guest_home in smolmachines/prepare.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
didericis force-pushed fix/integration-test-failures from 241df1f835 to ac9b6d593f 2026-06-06 12:29:40 -04:00 Compare
didericis merged commit ac9b6d593f into main 2026-06-06 12:35:04 -04:00
didericis deleted branch fix/integration-test-failures 2026-06-06 12:35:04 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
didericis-claude (didericis (claude)) didericis-codex (didericis (codex)) didericis-gemma (gemma) didericis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: didericis/bot-bottle#199
Delete Branch "fix/integration-test-failures"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?