Parsoid SiteConfig: only perform schema validation when running tests

In Ib4c94eab0f3091d224c334b92fcb79215c519121 Parsoid gains a JSON
schema for its extension module configuration array.  This patch
ensures that, for efficiency, we only validate against this schema
when running tests.

Follows-Up: Ib4c94eab0f3091d224c334b92fcb79215c519121
Change-Id: I5d81a7d95237bd749f8f4f8a98c62802dee298dd

1 files changed, 7 insertions, 0 deletions

diff --git a/includes/parser/Parsoid/Config/SiteConfig.php b/includes/parser/Parsoid/Config/SiteConfig.php
index 24a6eef635b4..4e91f19aa96d 100644
--- a/includes/parser/Parsoid/Config/SiteConfig.php
+++ b/includes/parser/Parsoid/Config/SiteConfig.php
@@ -749,6 +749,13 @@ class SiteConfig extends ISiteConfig {
 	}
 
 	/** @inheritDoc */
+	protected function shouldValidateExtConfig(): bool {
+		// Only perform json schema validation for extension module
+		// configurations when running tests.
+		return defined( 'MW_PHPUNIT_TEST' ) || defined( 'MW_PARSER_TEST' );
+	}
+
+	/** @inheritDoc */
 	public function getMaxTemplateDepth(): int {
 		return (int)$this->config->get( MainConfigNames::MaxTemplateDepth );
 	}