Skip to content

Commit ae6bdb9

Browse files
sbrunnerm-kuhn
authored andcommittedMar 22, 2016
Remove no more needed acceptable values in test_qgssipcoverage.py
1 parent fee3197 commit ae6bdb9

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed
 

‎tests/src/python/test_qgssipcoverage.py

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -27,16 +27,6 @@
2727
except:
2828
pass
2929

30-
# BINDING THRESHOLD
31-
#
32-
# The minimum number of unbound functions in QGIS api
33-
#
34-
# DON'T RAISE THIS THRESHOLD!!!
35-
# (changes which lower this threshold are welcomed though!)
36-
37-
ACCEPTABLE_MISSING_CLASSES = 0
38-
ACCEPTABLE_MISSING_MEMBERS = 0
39-
4030

4131
class TestQgsSipCoverage(unittest.TestCase):
4232

@@ -103,7 +93,7 @@ def testCoverage(self):
10393
printImportant("{} total have bindings".format(present_count))
10494
printImportant("Binding coverage by classes {}%".format(coverage))
10595
printImportant("---------------------------------")
106-
printImportant("{} classes missing bindings, out of {} allowed".format(missing_class_count, ACCEPTABLE_MISSING_CLASSES))
96+
printImportant("{} classes missing bindings".format(missing_class_count))
10797
print "---------------------------------"
10898

10999
missing_member_count = len(missing_members)
@@ -115,13 +105,13 @@ def testCoverage(self):
115105
printImportant("{} total have bindings".format(present_count))
116106
printImportant("Binding coverage by members {}%".format(coverage))
117107
printImportant("---------------------------------")
118-
printImportant("{} members missing bindings, out of {} allowed".format(missing_member_count, ACCEPTABLE_MISSING_MEMBERS))
108+
printImportant("{} members missing bindings".format(missing_member_count))
119109

120-
assert missing_class_count <= ACCEPTABLE_MISSING_CLASSES, """\n\nFAIL: new unbound classes have been introduced, please add SIP bindings for these classes
110+
assert missing_class_count <= 0, """\n\nFAIL: new unbound classes have been introduced, please add SIP bindings for these classes
121111
If these classes are not suitable for the Python bindings, please add the Doxygen tag
122112
"@note not available in Python bindings" to the CLASS Doxygen comments"""
123113

124-
assert missing_member_count <= ACCEPTABLE_MISSING_MEMBERS, """\n\nFAIL: new unbound members have been introduced, please add SIP bindings for these members
114+
assert missing_member_count <= 0, """\n\nFAIL: new unbound members have been introduced, please add SIP bindings for these members
125115
If these members are not suitable for the Python bindings, please add the Doxygen tag
126116
"@note not available in Python bindings" to the MEMBER Doxygen comments"""
127117

0 commit comments

Comments
 (0)
Please sign in to comment.