Fix a few nits (#107)

* Minor changes

* Gracefully degrade if clang-tidy segfaults

* Typo on returncode

crypto_sign/sphincs-shake256-128f-simple/clean/thash_shake256_simple.c

@@ -1,9 +1,9 @@
 #include <stdint.h>
 #include <string.h>
 
-#include "thash.h"
 #include "address.h"
 #include "params.h"
+#include "thash.h"
 
 #include "fips202.h"
 

crypto_sign/sphincs-shake256-128f-simple/clean/wots.c

@@ -1,11 +1,11 @@
 #include <stdint.h>
 #include <string.h>
 
-#include "utils.h"
 #include "address.h"
 #include "hash.h"
 #include "params.h"
 #include "thash.h"
+#include "utils.h"
 #include "wots.h"
 
 // TODO clarify address expectations, and make them more uniform.

test/helpers.py

@@ -114,8 +114,10 @@ def permit_test(testname, thing, **args):
 
     if isinstance(thing, pqclean.Implementation):
         scheme = thing.scheme
-    else:
+    elif isinstance(thing, pqclean.Scheme):
         scheme = thing
+    else:
+        return True
 
     if 'PQCLEAN_ONLY_TYPES' in os.environ:
         if not(scheme.type.lower() in os.environ['PQCLEAN_ONLY_TYPES'].lower().split(',')):

test/test_common.py

@@ -12,7 +12,7 @@ import helpers
 def test_common():
     for d in os.listdir('common'):
         primitive = re.sub(r"\.c$", "", d)
-        yield check_common, primitive
+        if helpers.permit_test('common', None): yield check_common, primitive
 
 
 def check_common(primitive):