Skip to content

Commit

Permalink
Merge pull request #642 from biolink/go-site-2038-gorule-0000002-no-n…
Browse files Browse the repository at this point in the history
…ot-to-protein-binding

For #2038 Changed to error from warning
  • Loading branch information
mugitty committed Aug 26, 2023
2 parents 95a02f9 + 44aa4a2 commit 6218621
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ontobio/io/qc.py
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ def repair(self, annotation: association.GoAssociation, group=None) -> Tuple[Lis
class GoRule02(GoRule):

def __init__(self):
super().__init__("GORULE:0000002", "No 'NOT' annotations to 'protein binding ; GO:0005515'", FailMode.SOFT)
super().__init__("GORULE:0000002", "No 'NOT' annotations to 'protein binding ; GO:0005515'", FailMode.HARD)

def test(self, annotation: association.GoAssociation, config: assocparser.AssocParserConfig, group=None) -> TestResult:

Expand Down
2 changes: 1 addition & 1 deletion tests/test_qc.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def test_go_rule02():
assoc = make_annotation(qualifier="NOT", goid="GO:0005515").associations[0]

test_result = qc.GoRule02().test(assoc, all_rules_config())
assert test_result.result_type == qc.ResultType.WARNING
assert test_result.result_type == qc.ResultType.ERROR

assoc.negated = False
test_result = qc.GoRule02().test(assoc, all_rules_config())
Expand Down

0 comments on commit 6218621

Please sign in to comment.