Add a check for None on self.max_version
@@ -149,7 +149,7 @@ class UpgradeTester(Tester):
# Check if a since annotation with a max_version was set on this test.
# The since decorator can only check the starting version of the upgrade,
# so here we check to new version of the upgrade as well.
- if hasattr(self, 'max_version') and new_version_from_build >= self.max_version:
+ if hasattr(self, 'max_version') and self.max_version is not None and new_version_from_build >= self.max_version:
self.skip("Skipping test, new version {} is equal to or higher than max version {}".format(new_version_from_build, self.max_version))
if (new_version_from_build >= '3' and self.protocol_version is not None and self.protocol_version < 3):
