Reflow

Author: FedericoCeratto

QA/common.py

@@ -9,12 +9,12 @@
 
 
 def execute(args):
-    """ Execute a specified command """
+    """Execute a specified command"""
     subprocess.run(args)
 
 
 def execute_jafar_and_miniooni(ooni_exe, outfile, experiment, tag, args):
-    """ Executes jafar and miniooni. Returns the test keys. """
+    """Executes jafar and miniooni. Returns the test keys."""
     tmpoutfile = "/tmp/{}".format(outfile)
     with contextlib.suppress(FileNotFoundError):
         os.remove(tmpoutfile)  # just in case
@@ -40,17 +40,17 @@ def execute_jafar_and_miniooni(ooni_exe, outfile, experiment, tag, args):
 
 
 def read_result(outfile):
-    """ Reads the result of an experiment """
+    """Reads the result of an experiment"""
     return json.load(open(outfile, "rb"))
 
 
 def test_keys(result):
-    """ Returns just the test keys of a specific result """
+    """Returns just the test keys of a specific result"""
     return result["test_keys"]
 
 
 def check_maybe_binary_value(value):
-    """ Make sure a maybe binary value is correct """
+    """Make sure a maybe binary value is correct"""
     assert isinstance(value, str) or (
         isinstance(value, dict)
         and value["format"] == "base64"
@@ -59,8 +59,8 @@ def check_maybe_binary_value(value):
 
 
 def with_free_port(func):
-    """ This function executes |func| passing it a port number on localhost
-        which is bound but not listening for new connections """
+    """This function executes |func| passing it a port number on localhost
+    which is bound but not listening for new connections"""
     # See <https://stackoverflow.com/a/45690594>
     with contextlib.closing(socket.socket(socket.AF_INET, socket.SOCK_STREAM)) as sock:
         sock.bind(("127.0.0.1", 0))