basic checks

Author: stgm

tests/_basics.py

@@ -25,6 +25,8 @@ def testMethod():
         #     return False, "let op dat je geen min() of max() gebruikt"
         if has_call('sorted'):
             return False, "let op dat je geen sorted() gebruikt"
+        if has_call('map'):
+            return False, "let op dat je geen map() gebruikt"
         if has_import('math'):
             return False, "let op dat je geen import math gebruikt"
 
@@ -39,15 +41,26 @@ def testMethod():
             max_doc_length = 79
         p = subprocess.run([
                 'pycodestyle',
-                '--select=E101,E112,E113,E115,E116,E117,E501,W505',
+                '--select=E101,E112,E113,E115,E116,E117,E501,E502,W505,W291',
                 f"--max-line-length={max_line_length}",
                 f"--max-doc-length={max_doc_length}",
                 test.fileName
             ], capture_output=True, universal_newlines=True)
         if p.returncode != 0:
-            test.fail = lambda info : f"let op juiste indentatie, code >{max_line_length} tekens, comments >{max_doc_length} tekens"
-            return False, p.stdout
-        
+            if "E1" in p.stdout:
+                test.fail = lambda info : f"let op juiste indentatie"
+                return False, p.stdout
+            if "E501" in p.stdout or "W505" in p.stdout:
+                test.fail = lambda info : f"regel(s) te lang, code max {max_line_length} tekens, comments max {max_doc_length} tekens\n  zie boek pagina 24 voor uitleg over splitsen van lange regels"
+                return False, p.stdout
+            if "E502" in p.stdout:
+                test.fail = lambda info: f"gebruik tussen haakjes geen \\ om de regel af te breken"
+                return False, p.stdout
+            if "W291" in p.stdout:
+                pattern = r'[^:\n]+:(\d+):\d+: W291'
+                matches = re.findall(pattern, p.stdout)
+                test.fail = lambda info: f"zorg dat er geen spaties aan het eind van een regel staan (regel {', '.join(matches)})"
+                return False, p.stdout
         return True
     test.test = testMethod
 

tests/_basics_no_listcomp.py

@@ -25,6 +25,8 @@ def testMethod():
         #     return False, "let op dat je geen min() of max() gebruikt"
         if has_call('sorted'):
             return False, "let op dat je geen sorted() gebruikt"
+        if has_call('map'):
+            return False, "let op dat je geen map() gebruikt"
         if has_call('all', 'any'):
             return False, "let op dat je geen all() of any() gebruikt"
         if has_import('math'):