Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Commit

Permalink
Merge pull request #101 from dawagner/xml-coverage-fix
Browse files Browse the repository at this point in the history
xml coverage: fix a copy-paste mistake

Commit ccb164c introduced the possibility to
ignore changes to unknown criterion through an option passed to coverage.py but
a copy-paste mistake swapped the action of this option with the option ignoring
incoherent criterion state changes.

Both flags are set by 'aplog2coverage.sh -f' so the mistake went unnoticed but
it must be fixed nevertheless.

Signed-off-by: David Wagner [email protected]
  • Loading branch information
dawagner committed Apr 20, 2015
2 parents ec4f597 + 77f3390 commit 4c11104
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tools/coverage/coverage.py
Original file line number Diff line number Diff line change
Expand Up @@ -1021,10 +1021,10 @@ def __init__(self):
errorToIgnore.append(Configuration.IneligibleConfigurationAppliedError)

if options.incoherentCriterionFlag:
errorToIgnore.append(ParsePFWlog.ChangeRequestOnUnknownCriterion)
errorToIgnore.append(Criterion.ChangeRequestToNonAccessibleState)

if options.unknwonCriterionFlag:
errorToIgnore.append(Criterion.ChangeRequestToNonAccessibleState)
errorToIgnore.append(ParsePFWlog.ChangeRequestOnUnknownCriterion)

self.errorToIgnore = tuple(errorToIgnore)

Expand Down

0 comments on commit 4c11104

Please sign in to comment.