Closed Bug 1320300 Opened 8 years ago Closed 8 years ago

findbugs does not incorporate test code

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: sebastian, Unassigned)

References

Details

Findbugs complains that BookmarksInsertionManager.DEBUG could be final because it's only written once. However we are overriding the value in TestBookmarksInsertionManager.

Findbugs should use the test code when analyzing the app code so that we do not see such false positives.
It turns out that running findbugsAutomationDebug actually uses the test code and it even warned about two issues.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Depends on: 1320316
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.