UrlUtils: Simplify and increase test coverage

Follows-up I31f1403ab40c79ab270c417.

* Make the test cases less dynamic. It was approaching the same
  kind of complexity as the source code, thus reducing confidence
  in the test.

* Make the test cases more explicit. A good test can be read without
  executing it and show plainly what the expected output are.
  Use little to no indirection, favouring small amounts of
  duplication when in doubt.

* Increase coverage of the "non-standard HTTPS port" configuration,
  by removing the two hardcoded assertions for it, in favour of
  including it in the overal test matrix and applying the same set
  of default cases to it as the other configurations under test.

* There are now 306 instead of 2355 generated test cases, but
  the code coverage of UrlUtils.php remains identical
  at 98.28% (171/174 lines).

Change-Id: I755c017964757b9aab5ebeabd80be5664be48798

1 files changed, 2 insertions, 1 deletions

diff --git a/includes/utils/UrlUtils.php b/includes/utils/UrlUtils.php
index 55b89e400ee1..f21b9e734d59 100644
--- a/includes/utils/UrlUtils.php
+++ b/includes/utils/UrlUtils.php
@@ -144,7 +144,8 @@ class UrlUtils {
 				$defaultProto = $serverProto ?? PROTO_HTTP;
 			}
 
-			$defaultProtoWithoutSlashes = $defaultProto === null ? '' : substr( $defaultProto, 0, -2 );
+			// @phan-suppress-next-line PhanTypeMismatchArgumentNullableInternal T308355
+			$defaultProtoWithoutSlashes = $defaultProto === PROTO_FALLBACK ? '' : substr( $defaultProto, 0, -2 );
 
 			if ( substr( $url, 0, 2 ) == '//' ) {
 				$url = $defaultProtoWithoutSlashes . $url;