From 5e63584b0157da1f8ab2b46c4dae6b3db0fed654 Mon Sep 17 00:00:00 2001 From: Rajeshreeputra Date: Fri, 26 Apr 2024 14:37:05 +0530 Subject: [PATCH] ACMS-3680: Allow strictConfigSchema check in tests. --- .../src/ExistingSite/ArticleImageDamTest.php | 4 ---- .../tests/src/Functional/ArticleTest.php | 15 --------------- .../tests/src/Functional/BlogTest.php | 15 --------------- .../tests/src/Functional/AudioTest.php | 14 -------------- .../src/Functional/AcquiaCmsConfigSyncTest.php | 15 --------------- .../tests/src/Functional/BasicPermissionsTest.php | 15 --------------- .../src/Functional/EntityPermissionsTestBase.php | 15 --------------- .../Ckeditor5ConfigurationTestBase.php | 15 --------------- .../tests/src/Functional/DocumentTest.php | 15 --------------- .../FunctionalJavascript/DocumentEmbedTest.php | 15 --------------- .../tests/src/ExistingSite/EventImageDamTest.php | 4 ---- .../tests/src/Functional/EventTest.php | 15 --------------- .../tests/src/Functional/ImageTest.php | 15 --------------- .../src/FunctionalJavascript/ImageEmbedTest.php | 15 --------------- .../tests/src/Kernel/SiteLogoTest.php | 15 --------------- .../tests/src/ExistingSite/PageImageDamTest.php | 4 ---- .../tests/src/Functional/PageTest.php | 15 --------------- .../src/Kernel/PageWithLayoutCanvasFieldTest.php | 3 --- .../tests/src/ExistingSite/PersonImageDamTest.php | 4 ---- .../tests/src/Functional/PersonTest.php | 15 --------------- .../tests/src/ExistingSite/PlaceImageDamTest.php | 4 ---- .../tests/src/Functional/PlaceTest.php | 15 --------------- .../tests/src/Functional/AcquiaConnectorTest.php | 15 --------------- .../AcquiaSearchIntegrationFromTourPageTest.php | 15 --------------- .../AcquiaFacetSearchTest.php | 3 --- .../tests/src/Kernel/AcquiaFacetFacadeTest.php | 3 --- .../Functional/FilterFormatFilteredHtmlTest.php | 15 --------------- .../tests/src/Functional/SiteStudioCoreTest.php | 15 --------------- .../src/Functional/SiteStudioPermissionsTest.php | 15 --------------- .../tests/src/Functional/AcquiaGoogleMapsTest.php | 15 --------------- .../tests/src/Functional/GoogleAnalyticsTest.php | 15 --------------- .../tests/src/Functional/GoogleTagManager.php | 15 --------------- .../tests/src/Functional/RecaptchaTest.php | 15 --------------- .../tests/src/Kernel/AcquiaTourDashboardTest.php | 15 --------------- .../tests/src/Functional/VideoTest.php | 15 --------------- .../src/FunctionalJavascript/VideoEmbedTest.php | 15 --------------- 36 files changed, 448 deletions(-) diff --git a/modules/acquia_cms_article/tests/src/ExistingSite/ArticleImageDamTest.php b/modules/acquia_cms_article/tests/src/ExistingSite/ArticleImageDamTest.php index 8a2fdb909..d18d7b768 100644 --- a/modules/acquia_cms_article/tests/src/ExistingSite/ArticleImageDamTest.php +++ b/modules/acquia_cms_article/tests/src/ExistingSite/ArticleImageDamTest.php @@ -19,10 +19,6 @@ class ArticleImageDamTest extends ExistingSiteBase { use ConfigurationTraits; - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_article/tests/src/Functional/ArticleTest.php b/modules/acquia_cms_article/tests/src/Functional/ArticleTest.php index 3e723e690..28517af76 100644 --- a/modules/acquia_cms_article/tests/src/Functional/ArticleTest.php +++ b/modules/acquia_cms_article/tests/src/Functional/ArticleTest.php @@ -34,21 +34,6 @@ class ArticleTest extends ContentTypeTestBase { 'schema_article', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_article/tests/src/Functional/BlogTest.php b/modules/acquia_cms_article/tests/src/Functional/BlogTest.php index 68bb8948e..f5e024431 100644 --- a/modules/acquia_cms_article/tests/src/Functional/BlogTest.php +++ b/modules/acquia_cms_article/tests/src/Functional/BlogTest.php @@ -41,21 +41,6 @@ class BlogTest extends BrowserTestBase { 'schema_article', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_audio/tests/src/Functional/AudioTest.php b/modules/acquia_cms_audio/tests/src/Functional/AudioTest.php index 9951ff2c5..a79eebfed 100644 --- a/modules/acquia_cms_audio/tests/src/Functional/AudioTest.php +++ b/modules/acquia_cms_audio/tests/src/Functional/AudioTest.php @@ -21,20 +21,6 @@ class AudioTest extends MediaTypeTestBase { */ protected static $modules = ['acquia_cms_audio']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd /** * {@inheritdoc} diff --git a/modules/acquia_cms_common/modules/acquia_cms_support/tests/src/Functional/AcquiaCmsConfigSyncTest.php b/modules/acquia_cms_common/modules/acquia_cms_support/tests/src/Functional/AcquiaCmsConfigSyncTest.php index f7200d4ee..c7a6fc6fb 100644 --- a/modules/acquia_cms_common/modules/acquia_cms_support/tests/src/Functional/AcquiaCmsConfigSyncTest.php +++ b/modules/acquia_cms_common/modules/acquia_cms_support/tests/src/Functional/AcquiaCmsConfigSyncTest.php @@ -20,21 +20,6 @@ class AcquiaCmsConfigSyncTest extends BrowserTestBase { */ protected $defaultTheme = 'stark'; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_common/tests/src/Functional/BasicPermissionsTest.php b/modules/acquia_cms_common/tests/src/Functional/BasicPermissionsTest.php index 37874c885..5973f84c0 100644 --- a/modules/acquia_cms_common/tests/src/Functional/BasicPermissionsTest.php +++ b/modules/acquia_cms_common/tests/src/Functional/BasicPermissionsTest.php @@ -31,21 +31,6 @@ class BasicPermissionsTest extends BrowserTestBase { 'views', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the basic module permissions. * diff --git a/modules/acquia_cms_common/tests/src/Functional/EntityPermissionsTestBase.php b/modules/acquia_cms_common/tests/src/Functional/EntityPermissionsTestBase.php index ac55fe85a..0bcd0f679 100644 --- a/modules/acquia_cms_common/tests/src/Functional/EntityPermissionsTestBase.php +++ b/modules/acquia_cms_common/tests/src/Functional/EntityPermissionsTestBase.php @@ -17,21 +17,6 @@ abstract class EntityPermissionsTestBase extends BrowserTestBase { */ protected $defaultTheme = 'stark'; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Defines an array of role which should & shouldn't exist. */ diff --git a/modules/acquia_cms_common/tests/src/FunctionalJavascript/Ckeditor5ConfigurationTestBase.php b/modules/acquia_cms_common/tests/src/FunctionalJavascript/Ckeditor5ConfigurationTestBase.php index 5a0363fe6..c669096f6 100644 --- a/modules/acquia_cms_common/tests/src/FunctionalJavascript/Ckeditor5ConfigurationTestBase.php +++ b/modules/acquia_cms_common/tests/src/FunctionalJavascript/Ckeditor5ConfigurationTestBase.php @@ -31,21 +31,6 @@ abstract class Ckeditor5ConfigurationTestBase extends WebDriverTestBase { 'acquia_cms_common', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_document/tests/src/Functional/DocumentTest.php b/modules/acquia_cms_document/tests/src/Functional/DocumentTest.php index 76e0e20d2..cf1419ddd 100644 --- a/modules/acquia_cms_document/tests/src/Functional/DocumentTest.php +++ b/modules/acquia_cms_document/tests/src/Functional/DocumentTest.php @@ -21,21 +21,6 @@ class DocumentTest extends MediaTypeTestBase { */ protected static $modules = ['acquia_cms_document']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_document/tests/src/FunctionalJavascript/DocumentEmbedTest.php b/modules/acquia_cms_document/tests/src/FunctionalJavascript/DocumentEmbedTest.php index e075ead3a..1077e8e51 100644 --- a/modules/acquia_cms_document/tests/src/FunctionalJavascript/DocumentEmbedTest.php +++ b/modules/acquia_cms_document/tests/src/FunctionalJavascript/DocumentEmbedTest.php @@ -20,21 +20,6 @@ class DocumentEmbedTest extends MediaEmbedTestBase { */ protected static $modules = ['acquia_cms_document']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_event/tests/src/ExistingSite/EventImageDamTest.php b/modules/acquia_cms_event/tests/src/ExistingSite/EventImageDamTest.php index e1b5f9993..35d5d962a 100644 --- a/modules/acquia_cms_event/tests/src/ExistingSite/EventImageDamTest.php +++ b/modules/acquia_cms_event/tests/src/ExistingSite/EventImageDamTest.php @@ -19,10 +19,6 @@ class EventImageDamTest extends ExistingSiteBase { use ConfigurationTraits; - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_event/tests/src/Functional/EventTest.php b/modules/acquia_cms_event/tests/src/Functional/EventTest.php index c5a999085..1b67c20a8 100644 --- a/modules/acquia_cms_event/tests/src/Functional/EventTest.php +++ b/modules/acquia_cms_event/tests/src/Functional/EventTest.php @@ -37,21 +37,6 @@ class EventTest extends ContentTypeTestBase { private $defaultTime; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_image/tests/src/Functional/ImageTest.php b/modules/acquia_cms_image/tests/src/Functional/ImageTest.php index 38b7580da..00968fd80 100644 --- a/modules/acquia_cms_image/tests/src/Functional/ImageTest.php +++ b/modules/acquia_cms_image/tests/src/Functional/ImageTest.php @@ -20,21 +20,6 @@ class ImageTest extends MediaTypeTestBase { */ protected static $modules = ['acquia_cms_image']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_image/tests/src/FunctionalJavascript/ImageEmbedTest.php b/modules/acquia_cms_image/tests/src/FunctionalJavascript/ImageEmbedTest.php index a48728426..6c619699e 100644 --- a/modules/acquia_cms_image/tests/src/FunctionalJavascript/ImageEmbedTest.php +++ b/modules/acquia_cms_image/tests/src/FunctionalJavascript/ImageEmbedTest.php @@ -20,21 +20,6 @@ class ImageEmbedTest extends MediaEmbedTestBase { */ protected static $modules = ['acquia_cms_common', 'acquia_cms_image', 'focal_point']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_image/tests/src/Kernel/SiteLogoTest.php b/modules/acquia_cms_image/tests/src/Kernel/SiteLogoTest.php index fdf678f28..3af864a19 100644 --- a/modules/acquia_cms_image/tests/src/Kernel/SiteLogoTest.php +++ b/modules/acquia_cms_image/tests/src/Kernel/SiteLogoTest.php @@ -63,21 +63,6 @@ class SiteLogoTest extends KernelTestBase { */ protected $siteLogo; - /** - * Disable strict config schema checks in this test. - * - * There are some config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_page/tests/src/ExistingSite/PageImageDamTest.php b/modules/acquia_cms_page/tests/src/ExistingSite/PageImageDamTest.php index bbb6e8068..1ed0c435e 100644 --- a/modules/acquia_cms_page/tests/src/ExistingSite/PageImageDamTest.php +++ b/modules/acquia_cms_page/tests/src/ExistingSite/PageImageDamTest.php @@ -19,10 +19,6 @@ class PageImageDamTest extends ExistingSiteBase { use ConfigurationTraits; - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_page/tests/src/Functional/PageTest.php b/modules/acquia_cms_page/tests/src/Functional/PageTest.php index 0bb2512a5..8663a9679 100644 --- a/modules/acquia_cms_page/tests/src/Functional/PageTest.php +++ b/modules/acquia_cms_page/tests/src/Functional/PageTest.php @@ -32,21 +32,6 @@ class PageTest extends ContentTypeTestBase { 'schema_article', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_page/tests/src/Kernel/PageWithLayoutCanvasFieldTest.php b/modules/acquia_cms_page/tests/src/Kernel/PageWithLayoutCanvasFieldTest.php index 2ffdfb1bf..ada0b8284 100644 --- a/modules/acquia_cms_page/tests/src/Kernel/PageWithLayoutCanvasFieldTest.php +++ b/modules/acquia_cms_page/tests/src/Kernel/PageWithLayoutCanvasFieldTest.php @@ -29,9 +29,6 @@ class PageWithLayoutCanvasFieldTest extends FieldKernelTestBase { /** * {@inheritdoc} */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd /** * The field_config object. diff --git a/modules/acquia_cms_person/tests/src/ExistingSite/PersonImageDamTest.php b/modules/acquia_cms_person/tests/src/ExistingSite/PersonImageDamTest.php index 4535e959a..7c39cdb63 100644 --- a/modules/acquia_cms_person/tests/src/ExistingSite/PersonImageDamTest.php +++ b/modules/acquia_cms_person/tests/src/ExistingSite/PersonImageDamTest.php @@ -19,10 +19,6 @@ class PersonImageDamTest extends ExistingSiteBase { use ConfigurationTraits; - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_person/tests/src/Functional/PersonTest.php b/modules/acquia_cms_person/tests/src/Functional/PersonTest.php index 32984614b..93f4866bc 100644 --- a/modules/acquia_cms_person/tests/src/Functional/PersonTest.php +++ b/modules/acquia_cms_person/tests/src/Functional/PersonTest.php @@ -35,21 +35,6 @@ class PersonTest extends ContentTypeTestBase { 'schema_person', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_place/tests/src/ExistingSite/PlaceImageDamTest.php b/modules/acquia_cms_place/tests/src/ExistingSite/PlaceImageDamTest.php index ba71043ae..74ad12f93 100644 --- a/modules/acquia_cms_place/tests/src/ExistingSite/PlaceImageDamTest.php +++ b/modules/acquia_cms_place/tests/src/ExistingSite/PlaceImageDamTest.php @@ -19,10 +19,6 @@ class PlaceImageDamTest extends ExistingSiteBase { use ConfigurationTraits; - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_place/tests/src/Functional/PlaceTest.php b/modules/acquia_cms_place/tests/src/Functional/PlaceTest.php index 921682c35..88222d9bd 100644 --- a/modules/acquia_cms_place/tests/src/Functional/PlaceTest.php +++ b/modules/acquia_cms_place/tests/src/Functional/PlaceTest.php @@ -37,21 +37,6 @@ class PlaceTest extends ContentTypeTestBase { 'schema_place', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_search/tests/src/Functional/AcquiaConnectorTest.php b/modules/acquia_cms_search/tests/src/Functional/AcquiaConnectorTest.php index 9fc87fe27..b6a70e563 100644 --- a/modules/acquia_cms_search/tests/src/Functional/AcquiaConnectorTest.php +++ b/modules/acquia_cms_search/tests/src/Functional/AcquiaConnectorTest.php @@ -27,21 +27,6 @@ class AcquiaConnectorTest extends BrowserTestBase { 'acquia_connector', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the Acquia CMS Connector form. */ diff --git a/modules/acquia_cms_search/tests/src/Functional/AcquiaSearchIntegrationFromTourPageTest.php b/modules/acquia_cms_search/tests/src/Functional/AcquiaSearchIntegrationFromTourPageTest.php index 720aab13f..69cc73ffd 100644 --- a/modules/acquia_cms_search/tests/src/Functional/AcquiaSearchIntegrationFromTourPageTest.php +++ b/modules/acquia_cms_search/tests/src/Functional/AcquiaSearchIntegrationFromTourPageTest.php @@ -31,21 +31,6 @@ class AcquiaSearchIntegrationFromTourPageTest extends BrowserTestBase { 'acquia_search', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_search/tests/src/FunctionalJavascript/AcquiaFacetSearchTest.php b/modules/acquia_cms_search/tests/src/FunctionalJavascript/AcquiaFacetSearchTest.php index 1d8e0db0c..d6806937d 100644 --- a/modules/acquia_cms_search/tests/src/FunctionalJavascript/AcquiaFacetSearchTest.php +++ b/modules/acquia_cms_search/tests/src/FunctionalJavascript/AcquiaFacetSearchTest.php @@ -35,9 +35,6 @@ class AcquiaFacetSearchTest extends BrowserTestBase { * * @todo Fix config schema for fallback_view & main_listing_pages_view plugin. */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd /** * The module installer object. diff --git a/modules/acquia_cms_search/tests/src/Kernel/AcquiaFacetFacadeTest.php b/modules/acquia_cms_search/tests/src/Kernel/AcquiaFacetFacadeTest.php index c70ea63e9..1dca23ef3 100644 --- a/modules/acquia_cms_search/tests/src/Kernel/AcquiaFacetFacadeTest.php +++ b/modules/acquia_cms_search/tests/src/Kernel/AcquiaFacetFacadeTest.php @@ -43,9 +43,6 @@ class AcquiaFacetFacadeTest extends KernelTestBase { * * @todo Fix config schema for fallback_view & main_listing_pages_view plugin. */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd /** * {@inheritdoc} diff --git a/modules/acquia_cms_site_studio/tests/src/Functional/FilterFormatFilteredHtmlTest.php b/modules/acquia_cms_site_studio/tests/src/Functional/FilterFormatFilteredHtmlTest.php index 7de3b4afd..e473bf6d0 100644 --- a/modules/acquia_cms_site_studio/tests/src/Functional/FilterFormatFilteredHtmlTest.php +++ b/modules/acquia_cms_site_studio/tests/src/Functional/FilterFormatFilteredHtmlTest.php @@ -24,21 +24,6 @@ class FilterFormatFilteredHtmlTest extends BrowserTestBase { 'acquia_cms_site_studio', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the filter black_list_html_tags. */ diff --git a/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioCoreTest.php b/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioCoreTest.php index 55e1189cb..188be2d6f 100644 --- a/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioCoreTest.php +++ b/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioCoreTest.php @@ -26,21 +26,6 @@ class SiteStudioCoreTest extends BrowserTestBase { 'acquia_cms_site_studio', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the Site Studio Core Form. */ diff --git a/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioPermissionsTest.php b/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioPermissionsTest.php index 1fbed2df3..d22f5aacc 100644 --- a/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioPermissionsTest.php +++ b/modules/acquia_cms_site_studio/tests/src/Functional/SiteStudioPermissionsTest.php @@ -28,21 +28,6 @@ class SiteStudioPermissionsTest extends BrowserTestBase { 'acquia_cms_site_studio', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_tour/tests/src/Functional/AcquiaGoogleMapsTest.php b/modules/acquia_cms_tour/tests/src/Functional/AcquiaGoogleMapsTest.php index f4eadac42..b6fe43237 100644 --- a/modules/acquia_cms_tour/tests/src/Functional/AcquiaGoogleMapsTest.php +++ b/modules/acquia_cms_tour/tests/src/Functional/AcquiaGoogleMapsTest.php @@ -27,21 +27,6 @@ class AcquiaGoogleMapsTest extends BrowserTestBase { 'acquia_cms_place', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests that the Google Maps API key can be set on the tour page. */ diff --git a/modules/acquia_cms_tour/tests/src/Functional/GoogleAnalyticsTest.php b/modules/acquia_cms_tour/tests/src/Functional/GoogleAnalyticsTest.php index 445fe7ad1..dfd403463 100644 --- a/modules/acquia_cms_tour/tests/src/Functional/GoogleAnalyticsTest.php +++ b/modules/acquia_cms_tour/tests/src/Functional/GoogleAnalyticsTest.php @@ -25,21 +25,6 @@ class GoogleAnalyticsTest extends BrowserTestBase { 'google_analytics', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the Google Analytics Form. */ diff --git a/modules/acquia_cms_tour/tests/src/Functional/GoogleTagManager.php b/modules/acquia_cms_tour/tests/src/Functional/GoogleTagManager.php index 3420376e1..2bd381579 100644 --- a/modules/acquia_cms_tour/tests/src/Functional/GoogleTagManager.php +++ b/modules/acquia_cms_tour/tests/src/Functional/GoogleTagManager.php @@ -25,21 +25,6 @@ class GoogleTagManager extends BrowserTestBase { 'google_tag', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the Google Tag Manager Form. */ diff --git a/modules/acquia_cms_tour/tests/src/Functional/RecaptchaTest.php b/modules/acquia_cms_tour/tests/src/Functional/RecaptchaTest.php index d6a9d45fa..6f5c7c228 100644 --- a/modules/acquia_cms_tour/tests/src/Functional/RecaptchaTest.php +++ b/modules/acquia_cms_tour/tests/src/Functional/RecaptchaTest.php @@ -25,21 +25,6 @@ class RecaptchaTest extends BrowserTestBase { 'recaptcha', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests the Recaptcha Form. */ diff --git a/modules/acquia_cms_tour/tests/src/Kernel/AcquiaTourDashboardTest.php b/modules/acquia_cms_tour/tests/src/Kernel/AcquiaTourDashboardTest.php index 0c5965156..8dd344420 100644 --- a/modules/acquia_cms_tour/tests/src/Kernel/AcquiaTourDashboardTest.php +++ b/modules/acquia_cms_tour/tests/src/Kernel/AcquiaTourDashboardTest.php @@ -22,21 +22,6 @@ class AcquiaTourDashboardTest extends KernelTestBase { 'acquia_cms_tour', ]; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * Tests AcquiaCMSTour plugins and make sure they are sorted per weights. */ diff --git a/modules/acquia_cms_video/tests/src/Functional/VideoTest.php b/modules/acquia_cms_video/tests/src/Functional/VideoTest.php index 70bbe499d..55229d2b9 100644 --- a/modules/acquia_cms_video/tests/src/Functional/VideoTest.php +++ b/modules/acquia_cms_video/tests/src/Functional/VideoTest.php @@ -22,21 +22,6 @@ class VideoTest extends MediaTypeTestBase { */ protected static $modules = ['acquia_cms_video']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */ diff --git a/modules/acquia_cms_video/tests/src/FunctionalJavascript/VideoEmbedTest.php b/modules/acquia_cms_video/tests/src/FunctionalJavascript/VideoEmbedTest.php index cad79ecb9..22ec75a79 100644 --- a/modules/acquia_cms_video/tests/src/FunctionalJavascript/VideoEmbedTest.php +++ b/modules/acquia_cms_video/tests/src/FunctionalJavascript/VideoEmbedTest.php @@ -20,21 +20,6 @@ class VideoEmbedTest extends MediaEmbedTestBase { */ protected static $modules = ['acquia_cms_video']; - /** - * Disable strict config schema checks in this test. - * - * Cohesion has a lot of config schema errors, and until they are all fixed, - * this test cannot pass unless we disable strict config schema checking - * altogether. Since strict config schema isn't critically important in - * testing this functionality, it's okay to disable it for now, but it should - * be re-enabled (i.e., this property should be removed) as soon as possible. - * - * @var bool - */ - // @codingStandardsIgnoreStart - protected $strictConfigSchema = FALSE; - // @codingStandardsIgnoreEnd - /** * {@inheritdoc} */