fix quality problems

Author: Bhupesh-V

coderunner/coderunner.py

@@ -88,8 +88,9 @@ def __readStatus(self, token: str):
         Check Submission status
         """
         while True:
-            req = urllib.request.urlopen(API_URL + token["token"] + FIELDS)
-            req = req.read()
+            req = urllib.request.Request(API_URL + token["token"] + FIELDS)
+            with urllib.request.urlopen(req) as response:
+                req = response.read()
 
             self.__response = json.loads(req.decode("utf-8"))
             self.__memory = self.__response["memory"]
@@ -112,8 +113,9 @@ def __submit(self):
         api_params["source_code"] = self.source
 
         post_data = urllib.parse.urlencode(api_params).encode("ascii")
-        req = urllib.request.urlopen(API_URL, post_data)
-        req = req.read()
+        req = urllib.request.Request(API_URL, post_data)
+        with urllib.request.urlopen(req) as response:
+            req = response.read()
         token = json.loads(req.decode("utf-8"))
 
         return token

testfiles/test_python.py

@@ -1 +1 @@
-print("Hello, World!")
+print("Hello, World!")

tests.py

@@ -127,11 +127,12 @@ def test_run(self):
 class TestRunI(unittest.TestCase):
 
     def test_run(self):
-        with self.assertRaises(ValueError): coderunner.code(
-            "Hello World",
-            "PHP",
-            "Hello World",
-            path = False
+        with self.assertRaises(ValueError):
+            coderunner.code(
+                "Hello World",
+                "PHP",
+                "Hello World",
+                path=False
             )
 
 
@@ -141,10 +142,11 @@ def test_run(self):
 class TestRunJ(unittest.TestCase):
 
     def test_run(self):
-        with self.assertRaises(OSError): coderunner.code(
-            "Hello World",
-            "C++",
-            "Hello World"
+        with self.assertRaises(OSError):
+            coderunner.code(
+                "Hello World",
+                "C++",
+                "Hello World"
             )
 
 
@@ -154,10 +156,11 @@ def test_run(self):
 class TestRunK(unittest.TestCase):
 
     def test_run(self):
-        with self.assertRaises(OSError): coderunner.code(
-            "testfiles/" + "test_c++.cpp",
-            "C++",
-            "Hello World"
+        with self.assertRaises(OSError):
+            coderunner.code(
+                "testfiles/" + "test_c++.cpp",
+                "C++",
+                "Hello World"
             )
 
 
@@ -167,11 +170,12 @@ def test_run(self):
 class TestRunL(unittest.TestCase):
 
     def test_run(self):
-        with self.assertRaises(OSError): coderunner.code(
-            "testfiles/" + "test_c++.cpp",
-            "C++",
-            "testfiles/output/" + "output4.txt",
-            "my_input"
+        with self.assertRaises(OSError):
+            coderunner.code(
+                "testfiles/" + "test_c++.cpp",
+                "C++",
+                "testfiles/output/" + "output4.txt",
+                "my_input"
             )