[TESTING] Minor improvement to extra test logic (- WIP PR #373 -)

 * this work is related to GHI #213

Changes in file tests/test_extra.py:
 * new logic to dynamicly check results

Author: reactive-firewall

tests/test_extra.py

@@ -75,23 +75,21 @@ def test_sanitize_url_GIVEN_raw_url_IS_reliable(self) -> None:
 		"""Test case 1: Test to ensure reliable URL sanitization."""
 		# Mock _hearstep to return a non-empty response
 		for test_params in self.URL_TEST_FIXTURES:
+			sanitized_url = docs.utils.sanitize_url(test_params["input_url"])
+			# check for results
+			self.assertIsNotNone(sanitized_url)
+			# Verify results
 			if test_params["input_url"] == test_params["expected"]:
-				continue
-			else:
-				self.assertNotEqual(
-					test_params["input_url"],
-					test_params["expected"],
-					"Invalid test case. Input and expected URLs were the same, but not skipped.",
+				self.assertEqual(
+					test_params["input_url"], sanitized_url,
+					"Input and output URLs were different, should be the same.",
 				)
-				sanitized_url = docs.utils.sanitize_url(test_params["input_url"])
-				# check for results
-				self.assertIsNotNone(sanitized_url)
-				# Verify results
+			else:
 				self.assertNotEqual(
 					test_params["input_url"], sanitized_url,
-					"Invalid test case. Input and output URLs were the same, should be skipped.",
+					"Input and output URLs were the same, should be different.",
 				)
-				self.assertEqual(sanitized_url, test_params["expected"])
+			self.assertEqual(sanitized_url, test_params["expected"])
 
 
 if __name__ == '__main__':