Change theme
Help
Press space for more information.
Show links for this issue (Shortcut: i, l)
Copy issue ID
Previous Issue (Shortcut: k)
Next Issue (Shortcut: j)
Sign in to use full features.
Vote: I am impacted
Notification menu
Refresh (Shortcut: Shift+r)
Go home (Shortcut: u)
Pending code changes (auto-populated)
View issue level access limits(Press Alt + Right arrow for more information)
Unintended behavior
View staffing
Description
navigation-lint-common
hosts test helpers that generatesTestFile
.However, b/368070326 . Instead, it should be in test-specific source sets.
TestFile
is for test only and should not reside in production code. It leads toWe should refactor lint-common code into a new structure with common and common-test submodules
This was done in these CLs aosp/3284996 . But this fix is blocked by recent bytecode migration in ag/29027860 . During migration, the existing path workaround , so this new path breaks presubmit. Thread for context.
"navigation:navigation-lint-common"
was hardcoded into aWhile waiting for an updated workaround or a patch from lint infra, we should in the meantime copy paste
TestFile
related code into each lint test artifact so that we restore lint in the mean time.