🐛 Set siteUrl to be required

Idearia · Dec 23, 2022 · 82b50c0 · 82b50c0
1 parent dbc95ac
commit 82b50c0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Helpers.php b/src/Helpers.php
@@ -41,6 +41,10 @@ public static function setServerUrl( string $url ): void
 	 */
 	public static function throwIfSiteDoesNotExist( string $siteUrl ): void
 	{
+		if ( ! $siteUrl ) {
+			throw new \Exception( 'You must export the siteUrl environment variable' );
+		}
+
 		if ( static::stripUrl( \get_site_url() ) !== static::stripUrl( $siteUrl ) ) {
 			throw new \Exception( "Given site not found: $siteUrl" );
 		}

diff --git a/src/WordPressTestCase.php b/src/WordPressTestCase.php
@@ -38,8 +38,8 @@ public static function setUpBeforeClass(): void
 
 		static::loadWordPress( $wordPressPath, $_ENV['siteUrl'] ?? '' );
 
-		if ( is_multisite() && $_ENV['siteUrl'] ?? '' ) {
-			Helpers::throwIfSiteDoesNotExist( $_ENV['siteUrl'] );
+		if ( is_multisite() ) {
+			Helpers::throwIfSiteDoesNotExist( $_ENV['siteUrl'] ?? '' );
 		}
 	}