Skip to content

test(project): Mark dependency content change test as failing - #1599

Open
matz3 wants to merge 1 commit into
mainfrom
test/dependency-change-failing
Open

matz3 wants to merge 1 commit into
mainfrom
test/dependency-change-failing

Conversation

@matz3

@matz3 matz3 commented Sep 17, 2026

Copy link
Copy Markdown
Member

Convert the skipped "Build application.a (dependency content changes)" integration test from test.serial.skip to test.serial.failing. Its assertions already encode the desired behavior: a dependency content change should invalidate the application's result cache and trigger a rebuild.

It currently fails because the custom task reads dependencies via taskUtil.getProject().getReader(), a path not tracked by the caching system's ResourceRequestManager, so dependency changes do not invalidate the cache. Marking it failing (instead of skipping) keeps CI green while turning the gap into a live signal that flips to a hard error once the behavior is fixed.

Convert the skipped "Build application.a (dependency content changes)"
integration test from test.serial.skip to test.serial.failing. Its
assertions already encode the desired behavior: a dependency content
change should invalidate the application's result cache and trigger a
rebuild.

It currently fails because the custom task reads dependencies via
taskUtil.getProject().getReader(), a path not tracked by the caching
system's ResourceRequestManager, so dependency changes do not invalidate
the cache. Marking it failing (instead of skipping) keeps CI green while
turning the gap into a live signal that flips to a hard error once the
behavior is fixed.
@matz3
matz3 requested a review from a team September 17, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants