Merge pull request #593 from Gizra/test-drupal-8.8

Require tests to pass on Drupal 8.8.x
Gizra · Dec 9, 2019 · e44ae5a · e44ae5a
2 parents a174a22 + dd66fb1
commit e44ae5a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -24,8 +24,6 @@ matrix:
     - php: 7.4
       env: TEST_SUITE=PHP_CodeSniffer
   allow_failures:
-    - env: TEST_SUITE=8.8.x
-    - env: TEST_SUITE=8.9.x
     - php: 7.4
 
 mysql:

diff --git a/tests/src/Kernel/Entity/GroupAudienceTest.php b/tests/src/Kernel/Entity/GroupAudienceTest.php
@@ -86,8 +86,12 @@ public function testGetAllGroupAudienceFields() {
     Og::createField(OgGroupAudienceHelperInterface::DEFAULT_FIELD, 'entity_test', $bundle, ['field_name' => $field_name1]);
     Og::createField(OgGroupAudienceHelperInterface::DEFAULT_FIELD, 'entity_test', $bundle, ['field_name' => $field_name2]);
 
-    $field_names = $this->groupAudienceHelper->getAllGroupAudienceFields('entity_test', $bundle);
-    $this->assertEquals([$field_name1, $field_name2], array_keys($field_names));
+    $expected_field_names = [$field_name1, $field_name2];
+    $actual_field_names = array_keys($this->groupAudienceHelper->getAllGroupAudienceFields('entity_test', $bundle));
+    sort($expected_field_names);
+    sort($actual_field_names);
+
+    $this->assertEquals($expected_field_names, $actual_field_names);
 
     // Test Og::isGroupContent method, which is just a wrapper around
     // OgGroupAudienceHelper::hasGroupAudienceFields().