pylint: Enable and fix no-self-use warnings

Author: cameel

scripts/endToEndExtraction/verify-testcases.py

@@ -154,7 +154,8 @@ def diff(self, analyser):
 
         print(len(intersection), "test-cases - ", len(mismatches), " mismatche(s)")
 
-    def check_traces(self, test_name, left, right, mismatches):
+    @classmethod
+    def check_traces(cls, test_name, left, right, mismatches):
         for trace_id, trace in enumerate(left.traces):
             left_trace = trace
             right_trace = right.traces[trace_id]

scripts/pylintrc

@@ -24,7 +24,6 @@ disable=
     invalid-name,
     missing-docstring,
     no-else-return,
-    no-self-use,
     pointless-string-statement,
     redefined-builtin,
     redefined-outer-name,

scripts/regressions.py

@@ -41,7 +41,8 @@ def __init__(self, description, args):
                                          "build/test/tools/ossfuzz")
         self._logpath = os.path.join(self._repo_root, "test_results")
 
-    def parseCmdLine(self, description, args):
+    @classmethod
+    def parseCmdLine(cls, description, args):
         argParser = ArgumentParser(description)
         argParser.add_argument('-o', '--out-dir', required=True, type=str,
                                help="""Directory where test results will be written""")

test/formal/rule.py

@@ -39,6 +39,7 @@ def check(self, _nonopt, _opt):
 			self.error('Rule is incorrect.\nModel: ' + str(m))
 		self.solver.pop()
 
-	def error(self, msg):
+	@classmethod
+	def error(cls, msg):
 		print(msg)
 		sys.exit(1)