fixed the minor issue with url validator

PrincetonUniversity · Jan 29, 2024 · ad43017 · ad43017
1 parent 4132171
commit ad43017
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/synthetic_workload_invoker/WorkloadInvoker.py b/synthetic_workload_invoker/WorkloadInvoker.py
@@ -150,7 +150,7 @@ def HTTPInstanceGeneratorGeneric(instance_times, blocking_cli, url, data):
     """
     if len(instance_times) == 0:
         return False
-    if validators.url(url) is False:
+    if validators.url(url) is not True:
         logger.error("Invalid URL: " + url)
         return False
 

diff --git a/tests/unit_tests/WorkloadInvoker_test.py b/tests/unit_tests/WorkloadInvoker_test.py
@@ -12,7 +12,7 @@ def test_HTTPInstanceGeneratorGenerics(self):
         status = HTTPInstanceGeneratorGeneric(
             instance_times=[1.0, 1.0, 1.0, 1.0],
             blocking_cli=0,
-            url="http://localhost:8080",
+            url="http://127.0.0.1:8080",
             data="",
         )
         self.assertTrue(status)
@@ -21,7 +21,7 @@ def test_HTTPInstanceGeneratorGenerics_with_empty_instance_times(self):
         status = HTTPInstanceGeneratorGeneric(
             instance_times=[],
             blocking_cli=0,
-            url="http://localhost:8080",
+            url="http://127.0.0.1:8080",
             data="",
         )
         self.assertFalse(status)