mosaic/stack
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

🔴 [P0] Fix 25 failing tests blocking CI pipeline #306

New Issue
Closed
opened 2026-02-04 02:18:31 +00:00 by jason.woltje · 0 comments
jason.woltje commented 2026-02-04 02:18:31 +00:00
Owner
Copy Link

Summary

After removing || true from the test step in .woodpecker.yml (commit 148121c), tests are now properly blocking CI builds. However, 25 tests are currently failing, which will block all merges to develop.

Impact

  • Severity: P0 - Blocks all CI/CD
  • Scope: 25 test failures across 8 test files
  • Status: Pipeline #118 will fail when tests run
  • Urgency: High - CI/CD is currently blocked

Test Failure Breakdown

Pattern 1: Mock Setup Issues (9 failures) - EASY FIXES

Files:

  • runner-jobs.service.spec.ts (4 failures) - Mock returning undefined
  • stitcher.security.spec.ts (5 failures) - ApiKeyGuard not mocked

Pattern 2: Service Integration Issues (16 failures)

Files:

  • agents.service.spec.ts (2 failures)
  • federation.service.spec.ts (5 failures)
  • query.service.spec.ts (4 failures)
  • integration.service.spec.ts (5 failures)

Full Details

See test output in pipeline #118 or run locally:

pnpm --filter @mosaic/api test

Acceptance Criteria

  • All 25 tests pass locally
  • All tests pass in CI
  • CI/CD pipeline unblocked

Priority

P0 - Critical: Blocking all CI/CD

## Summary After removing `|| true` from the test step in `.woodpecker.yml` (commit 148121c), tests are now properly blocking CI builds. However, **25 tests are currently failing**, which will block all merges to develop. ## Impact - **Severity**: P0 - Blocks all CI/CD - **Scope**: 25 test failures across 8 test files - **Status**: Pipeline #118 will fail when tests run - **Urgency**: High - CI/CD is currently blocked ## Test Failure Breakdown ### Pattern 1: Mock Setup Issues (9 failures) - EASY FIXES **Files:** - runner-jobs.service.spec.ts (4 failures) - Mock returning undefined - stitcher.security.spec.ts (5 failures) - ApiKeyGuard not mocked ### Pattern 2: Service Integration Issues (16 failures) **Files:** - agents.service.spec.ts (2 failures) - federation.service.spec.ts (5 failures) - query.service.spec.ts (4 failures) - integration.service.spec.ts (5 failures) ## Full Details See test output in pipeline #118 or run locally: ```bash pnpm --filter @mosaic/api test ``` ## Acceptance Criteria - [ ] All 25 tests pass locally - [ ] All tests pass in CI - [ ] CI/CD pipeline unblocked ## Priority **P0 - Critical**: Blocking all CI/CD
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai
AI/LLM integration
 api
Backend/API work
 api
auth
Authentication
 database
database
Database/schema work
 devops
Docker/deployment
 docs
Documentation
 frontend
graph
knowledge-module
migration
Data migration
 orchestrator
Orchestrator service (apps/orchestrator/)
 p0
Critical priority
 p1
High priority
 p2
Medium priority
 p3
Low priority
 performance
Performance work
 phase-1
phase-2
phase-3
phase-4
phase-5
plugin
MoltBot plugin work
 search
security
Security-related
 setup
Initial setup
 testing
Testing/QA
 web
Frontend work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaic/stack#306
Delete Branch "%!s()"

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?