Rewrite UnitTestsBootstrap::set_server_props to avoid needing to sk…

…ip phpcs checks. Address: #446 (comment)
woocommerce · Jul 1, 2024 · db107f7 · db107f7
1 parent 11df3a9
commit db107f7
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/tests/class-unittestsbootstrap.php b/tests/class-unittestsbootstrap.php
@@ -70,10 +70,12 @@ public function set_path_props() {
 	 * Set server props
 	 */
 	public function set_server_props() {
-		// phpcs:disable WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
-		$_SERVER['REMOTE_ADDR'] = isset( $_SERVER['REMOTE_ADDR'] ) ? $_SERVER['REMOTE_ADDR'] : '';
-		$_SERVER['SERVER_NAME'] = isset( $_SERVER['SERVER_NAME'] ) ? $_SERVER['SERVER_NAME'] : 'ga_integration_test';
-		// phpcs:enable WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
+		if ( ! isset( $_SERVER['REMOTE_ADDR'] ) ) {
+			$_SERVER['REMOTE_ADDR'] = '';
+		}
+		if ( ! isset( $_SERVER['SERVER_NAME'] ) ) {
+			$_SERVER['SERVER_NAME'] = 'ga_integration_test';
+		}
 	}
 
 	/**