From 3f77763dd3663928ddc471d638b8c5c0c0bf7bf4 Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Sat, 1 Apr 2023 16:50:13 +0200
Subject: [PATCH] Tests: remove some unused fixtures

PR 102 removed support for the `ExpectException` polyfill.

These test fixtures belonged with that polyfill and were missed in the removal. They are now unused and should be removed.
---
 .../Fixtures/InvalidExceptionCodeTestCase.php | 25 -----------------
 .../InvalidExceptionMessageTestCase.php       | 28 -------------------
 .../Fixtures/ThrowExceptionTestCase.php       | 28 -------------------
 3 files changed, 81 deletions(-)
 delete mode 100644 tests/Polyfills/Fixtures/InvalidExceptionCodeTestCase.php
 delete mode 100644 tests/Polyfills/Fixtures/InvalidExceptionMessageTestCase.php
 delete mode 100644 tests/Polyfills/Fixtures/ThrowExceptionTestCase.php

diff --git a/tests/Polyfills/Fixtures/InvalidExceptionCodeTestCase.php b/tests/Polyfills/Fixtures/InvalidExceptionCodeTestCase.php
deleted file mode 100644
index 6ae0cd4..0000000
--- a/tests/Polyfills/Fixtures/InvalidExceptionCodeTestCase.php
+++ /dev/null
@@ -1,25 +0,0 @@
-<?php
-
-namespace Yoast\PHPUnitPolyfills\Tests\Polyfills\Fixtures;
-
-use Exception;
-use PHPUnit\Framework\TestCase;
-
-/**
- * Fixture to test the exception method polyfills for correctly failing on an invalid argument.
- */
-class InvalidExceptionCodeTestCase extends TestCase {
-
-	/**
-	 * Test resulting in an "invalid argument type" exception for the expectExceptionCode() call.
-	 *
-	 * @return void
-	 *
-	 * @throws Exception For test purposes.
-	 */
-	public function test() {
-		$this->expectExceptionCode( null );
-
-		throw new Exception( 'A runtime error occurred', 999 );
-	}
-}
diff --git a/tests/Polyfills/Fixtures/InvalidExceptionMessageTestCase.php b/tests/Polyfills/Fixtures/InvalidExceptionMessageTestCase.php
deleted file mode 100644
index 99a25eb..0000000
--- a/tests/Polyfills/Fixtures/InvalidExceptionMessageTestCase.php
+++ /dev/null
@@ -1,28 +0,0 @@
-<?php
-
-namespace Yoast\PHPUnitPolyfills\Tests\Polyfills\Fixtures;
-
-use Exception;
-use PHPUnit\Framework\TestCase;
-use Yoast\PHPUnitPolyfills\Polyfills\ExpectException;
-
-/**
- * Fixture to test the exception method polyfills for correctly failing on an invalid argument.
- */
-class InvalidExceptionMessageTestCase extends TestCase {
-
-	use ExpectException;
-
-	/**
-	 * Test resulting in an "invalid argument type" exception for the expectExceptionMessage() call.
-	 *
-	 * @return void
-	 *
-	 * @throws Exception For test purposes.
-	 */
-	public function test() {
-		$this->expectExceptionMessage( [ 1, 2, 3 ] );
-
-		throw new Exception( 'A runtime error occurred', 999 );
-	}
-}
diff --git a/tests/Polyfills/Fixtures/ThrowExceptionTestCase.php b/tests/Polyfills/Fixtures/ThrowExceptionTestCase.php
deleted file mode 100644
index 794bb73..0000000
--- a/tests/Polyfills/Fixtures/ThrowExceptionTestCase.php
+++ /dev/null
@@ -1,28 +0,0 @@
-<?php
-
-namespace Yoast\PHPUnitPolyfills\Tests\Polyfills\Fixtures;
-
-use Exception;
-use PHPUnit\Framework\TestCase;
-use Yoast\PHPUnitPolyfills\Polyfills\ExpectException;
-
-/**
- * Fixture to test the exception method polyfills for correctly failing when expectations
- * have been set for both the message as well as the code.
- */
-class ThrowExceptionTestCase extends TestCase {
-
-	// Needed for PHPUnit 4.x.
-	use ExpectException;
-
-	/**
-	 * Test resulting in an exception.
-	 *
-	 * @return void
-	 *
-	 * @throws Exception For test purposes.
-	 */
-	public function test() {
-		throw new Exception( 'A runtime error occurred', 999 );
-	}
-}