From 1b2a9a8926ca22fb658d29842539b68929ff823d Mon Sep 17 00:00:00 2001
From: Steve Boyd <emteknetnz@gmail.com>
Date: Fri, 5 Nov 2021 13:23:15 +1300
Subject: [PATCH] MNT Remove polyfill

---
 composer.json             |  1 -
 tests/ReportAdminTest.php | 35 +++++++++++++----------------------
 2 files changed, 13 insertions(+), 23 deletions(-)

diff --git a/composer.json b/composer.json
index 9c5d18b2..fcb9c183 100644
--- a/composer.json
+++ b/composer.json
@@ -30,7 +30,6 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5",
-        "dms/phpunit-arraysubset-asserts": "^0.3.0",
         "squizlabs/php_codesniffer": "^3.0"
     },
     "extra": {
diff --git a/tests/ReportAdminTest.php b/tests/ReportAdminTest.php
index 3da725df..f84576b3 100644
--- a/tests/ReportAdminTest.php
+++ b/tests/ReportAdminTest.php
@@ -5,7 +5,6 @@ use ReflectionClass;
 use SilverStripe\Control\Controller;
 use SilverStripe\Control\HTTPRequest;
 use SilverStripe\Dev\SapphireTest;
-use DMS\PHPUnitExtensions\ArraySubset\ArraySubsetAsserts;
 use SilverStripe\Reports\Report;
 use SilverStripe\Reports\ReportAdmin;
 use SilverStripe\Reports\Tests\ReportAdminTest\FakeReport;
@@ -13,8 +12,6 @@ use SilverStripe\Reports\Tests\ReportAdminTest\FakeReport2;
 
 class ReportAdminTest extends SapphireTest
 {
-    use ArraySubsetAsserts;
-
     public function testBreadcrumbsAreGenerated()
     {
         $noExtraCrumbs = FakeReport::create();
@@ -23,15 +20,12 @@ class ReportAdminTest extends SapphireTest
         $breadcrumbs = $controller->BreadCrumbs();
 
         $this->assertCount(2, $breadcrumbs);
+        $map = $breadcrumbs[0]->toMap();
+        $this->assertSame('Reports', $map['Title']);
+        $this->assertSame('admin/reports/', $map['Link']);
 
-        $this->assertArraySubset([
-            'Title' => 'Reports',
-            'Link' => 'admin/reports/',
-        ], $breadcrumbs[0]->toMap(), true, 'Link to top level reports is within breadcrumbs');
-
-        $this->assertArraySubset([
-            'Title' => 'Fake report'
-        ], $breadcrumbs[1]->toMap(), true, 'Current report is within breadcrumbs');
+        $map = $breadcrumbs[1]->toMap();
+        $this->assertSame('Fake report', $map['Title']);
 
         $extraCrumbs = FakeReport2::create();
         $controller = $this->mockController($extraCrumbs);
@@ -39,19 +33,16 @@ class ReportAdminTest extends SapphireTest
 
         $this->assertCount(3, $breadcrumbs);
 
-        $this->assertArraySubset([
-            'Title' => 'Reports',
-            'Link' => 'admin/reports/',
-        ], $breadcrumbs[0]->toMap(), true, 'Link to top level reports is within breadcrumbs (again)');
+        $map = $breadcrumbs[0]->toMap();
+        $this->assertSame('Reports', $map['Title']);
+        $this->assertSame('admin/reports/', $map['Link']);
 
-        $this->assertArraySubset([
-            'Title' => 'Fake report title',
-            'Link' => 'admin/reports/show/SilverStripe-Reports-Tests-ReportAdminTest-FakeReport',
-        ], $breadcrumbs[1]->toMap(), true, 'Custom breadcrumb appears');
+        $map = $breadcrumbs[1]->toMap();
+        $this->assertSame('Fake report title', $map['Title']);
+        $this->assertSame('admin/reports/show/SilverStripe-Reports-Tests-ReportAdminTest-FakeReport', $map['Link']);
 
-        $this->assertArraySubset([
-            'Title' => 'Fake report two'
-        ], $breadcrumbs[2]->toMap(), true, 'Current report is still within breadcrumbs');
+        $map = $breadcrumbs[2]->toMap();
+        $this->assertSame('Fake report two', $map['Title']);
     }
 
     /**