Skip to content

Commit 0e3dc8b

Browse files
userclaude
andcommitted
fix(runner): update test for global credential path + make coderabbit install non-fatal
Test: update assertion to match the global credential token URL (/api/ambient/v1/credentials/{id}/token) after removing the project-scoped prefix. Dockerfile: make CodeRabbit CLI install non-fatal so image builds succeed when cli.coderabbit.ai is unavailable. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 23d8958 commit 0e3dc8b

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

components/runners/ambient-runner/Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,8 @@ RUN npm install -g @google/gemini-cli@${GEMINI_CLI_VERSION} && \
4444
npm cache clean --force
4545

4646
# Install CodeRabbit CLI (official install script, binary for current arch)
47-
RUN curl -fsSL https://cli.coderabbit.ai/install.sh | CODERABBIT_INSTALL_DIR=/usr/local/bin sh
47+
RUN curl -fsSL https://cli.coderabbit.ai/install.sh | CODERABBIT_INSTALL_DIR=/usr/local/bin sh \
48+
|| echo "WARNING: CodeRabbit CLI install failed; continuing without it"
4849

4950
# Set environment variables
5051
ENV PYTHONUNBUFFERED=1

components/runners/ambient-runner/tests/test_shared_session_credentials.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -511,7 +511,7 @@ def log_message(self, fmt, *args):
511511

512512
assert result.get("token") == "gh-tok-cp"
513513
assert captured["path"] == (
514-
"/api/ambient/v1/projects/my-project/credentials/cred-abc/token"
514+
"/api/ambient/v1/credentials/cred-abc/token"
515515
)
516516
finally:
517517
server.server_close()

0 commit comments

Comments
 (0)