-
Notifications
You must be signed in to change notification settings - Fork 63
Closed
Labels
status:doneTask is completeTask is complete
Description
Summary
The next branch has been failing CI consistently since 2026-03-02 with multiple issues.
Objective
Restore green CI on the next branch so the review pipeline can function.
Root Causes
1. Rubocop: 980 Layout/EmptyLineAfterMagicComment offenses (auto-correctable)
- Introduced by: PR Add frozen_string_literal: true to all app Ruby files (WA-NEW-037) #733 (WA-NEW-037: frozen_string_literal), merged 2026-03-02
# frozen_string_literal: truewas added to many files without an empty line after the magic comment- Fix:
bundle exec rubocop --autocorrect(all offenses are auto-correctable)
2. MiddlewareStackTest failures (3 tests)
- Introduced by: PR WA-NEW-040: Fix Rack::Attack compatibility with Rails 7 / Rack 3 #731 (WA-NEW-040: Rack::Attack), merged 2026-03-02
- Tests expect
Rack::Attackat index 3 but it is at index 1 - Actual:
["Rack::Sendfile", "Rack::Attack", "Rack::Timeout", ...] - Fix: Middleware initializer ordering or test expectations need adjustment
- Files:
core/test/integration/workarea/middleware_stack_test.rband Rack::Attack initializer
3. PublishingIntegrationTest / SegmentOverridesIntegrationTest failures
- BSON::ObjectId comparison failures — hardcoded IDs in expected output vs actual
- May be flaky tests or fixtures that changed; needs investigation
Acceptance Criteria
-
bundle exec rubocoppasses with zero offenses onnext -
MiddlewareStackTestall 3 failing tests pass -
PublishingIntegrationTest#test_publishingpasses -
SegmentOverridesIntegrationTest#test_creates_segment_overridespasses - Full CI green on
nextbranch
Verification Plan
bundle exec rubocop --autocorrectbundle exec rails test core/test/integration/workarea/middleware_stack_test.rb- Push to
nextand verify CI passes
Client impact
None — test/lint fixes only, no behavior changes.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
status:doneTask is completeTask is complete