Make all share tests final

stof · stof · commit 03a23c8323af · 2023-02-14T18:24:50.000+01:00
Drivers wanting to run custom tests should define them in their own
tests. They might also decide to skip some of the shared tests. But
there is no reason to customize the shared tests themselves.
diff --git a/tests/Basic/BasicAuthTest.php b/tests/Basic/BasicAuthTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class BasicAuthTest extends TestCase
+final class BasicAuthTest extends TestCase
 {
     /**
      * @dataProvider setBasicAuthDataProvider
diff --git a/tests/Basic/BestPracticesTest.php b/tests/Basic/BestPracticesTest.php
@@ -7,7 +7,7 @@
 /**
  * This testcase ensures that the driver implementation follows recommended practices for drivers.
  */
-class BestPracticesTest extends TestCase
+final class BestPracticesTest extends TestCase
 {
     public function testExtendsCoreDriver()
     {
diff --git a/tests/Basic/ContentTest.php b/tests/Basic/ContentTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class ContentTest extends TestCase
+final class ContentTest extends TestCase
 {
     public function testOuterHtml()
     {
diff --git a/tests/Basic/CookieTest.php b/tests/Basic/CookieTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class CookieTest extends TestCase
+final class CookieTest extends TestCase
 {
     /**
      * test cookie decoding.
diff --git a/tests/Basic/ErrorHandlingTest.php b/tests/Basic/ErrorHandlingTest.php
@@ -7,7 +7,7 @@
 /**
  * @group slow
  */
-class ErrorHandlingTest extends TestCase
+final class ErrorHandlingTest extends TestCase
 {
     const NOT_FOUND_XPATH = '//html/./invalid';
 
diff --git a/tests/Basic/HeaderTest.php b/tests/Basic/HeaderTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class HeaderTest extends TestCase
+final class HeaderTest extends TestCase
 {
     /**
      * test referrer.
diff --git a/tests/Basic/IFrameTest.php b/tests/Basic/IFrameTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class IFrameTest extends TestCase
+final class IFrameTest extends TestCase
 {
     public function testIFrame()
     {
diff --git a/tests/Basic/NavigationTest.php b/tests/Basic/NavigationTest.php
@@ -5,7 +5,7 @@
 use Behat\Mink\Tests\Driver\TestCase;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;
 
-class NavigationTest extends TestCase
+final class NavigationTest extends TestCase
 {
     use AssertionRenames;
 
diff --git a/tests/Basic/ScreenshotTest.php b/tests/Basic/ScreenshotTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class ScreenshotTest extends TestCase
+final class ScreenshotTest extends TestCase
 {
     public function testScreenshot()
     {
diff --git a/tests/Basic/StatusCodeTest.php b/tests/Basic/StatusCodeTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class StatusCodeTest extends TestCase
+final class StatusCodeTest extends TestCase
 {
     public function testStatuses()
     {
diff --git a/tests/Basic/TraversingTest.php b/tests/Basic/TraversingTest.php
@@ -5,7 +5,7 @@
 use Behat\Mink\Tests\Driver\TestCase;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;
 
-class TraversingTest extends TestCase
+final class TraversingTest extends TestCase
 {
     use AssertionRenames;
 
diff --git a/tests/Basic/VisibilityTest.php b/tests/Basic/VisibilityTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class VisibilityTest extends TestCase
+final class VisibilityTest extends TestCase
 {
     public function testVisibility()
     {
diff --git a/tests/Css/HoverTest.php b/tests/Css/HoverTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class HoverTest extends TestCase
+final class HoverTest extends TestCase
 {
     /**
      * @group mouse-events
diff --git a/tests/Form/CheckboxTest.php b/tests/Form/CheckboxTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class CheckboxTest extends TestCase
+final class CheckboxTest extends TestCase
 {
     public function testManipulate()
     {
diff --git a/tests/Form/GeneralTest.php b/tests/Form/GeneralTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class GeneralTest extends TestCase
+final class GeneralTest extends TestCase
 {
     // test multiple submit buttons
     public function testIssue212()
diff --git a/tests/Form/Html5Test.php b/tests/Form/Html5Test.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class Html5Test extends TestCase
+final class Html5Test extends TestCase
 {
     public function testHtml5FormInputAttribute()
     {
diff --git a/tests/Form/RadioTest.php b/tests/Form/RadioTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class RadioTest extends TestCase
+final class RadioTest extends TestCase
 {
     /**
      * @before
diff --git a/tests/Form/SelectTest.php b/tests/Form/SelectTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class SelectTest extends TestCase
+final class SelectTest extends TestCase
 {
     public function testMultiselect()
     {
diff --git a/tests/Js/ChangeEventTest.php b/tests/Js/ChangeEventTest.php
@@ -7,7 +7,7 @@
 /**
  * @group slow
  */
-class ChangeEventTest extends TestCase
+final class ChangeEventTest extends TestCase
 {
     /**
      * 'change' event should be fired after selecting an <option> in a <select>.
diff --git a/tests/Js/EventsTest.php b/tests/Js/EventsTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class EventsTest extends TestCase
+final class EventsTest extends TestCase
 {
     /**
      * @group mouse-events
diff --git a/tests/Js/JavascriptEvaluationTest.php b/tests/Js/JavascriptEvaluationTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class JavascriptEvaluationTest extends TestCase
+final class JavascriptEvaluationTest extends TestCase
 {
     /**
      * Tests, that `wait` method returns check result after exit.
diff --git a/tests/Js/JavascriptTest.php b/tests/Js/JavascriptTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class JavascriptTest extends TestCase
+final class JavascriptTest extends TestCase
 {
     public function testAriaRoles()
     {
diff --git a/tests/Js/WindowTest.php b/tests/Js/WindowTest.php
@@ -4,7 +4,7 @@
 
 use Behat\Mink\Tests\Driver\TestCase;
 
-class WindowTest extends TestCase
+final class WindowTest extends TestCase
 {
     public function testWindow()
     {

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class BasicAuthTest extends TestCase`
	`7`	`+final class BasicAuthTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* @dataProvider setBasicAuthDataProvider`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`/**`
`8`	`8`	`* This testcase ensures that the driver implementation follows recommended practices for drivers.`
`9`	`9`	`*/`
`10`		`-class BestPracticesTest extends TestCase`
	`10`	`+final class BestPracticesTest extends TestCase`
`11`	`11`	`{`
`12`	`12`	`public function testExtendsCoreDriver()`
`13`	`13`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class ContentTest extends TestCase`
	`7`	`+final class ContentTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`public function testOuterHtml()`
`10`	`10`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class CookieTest extends TestCase`
	`7`	`+final class CookieTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* test cookie decoding.`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`/**`
`8`	`8`	`* @group slow`
`9`	`9`	`*/`
`10`		`-class ErrorHandlingTest extends TestCase`
	`10`	`+final class ErrorHandlingTest extends TestCase`
`11`	`11`	`{`
`12`	`12`	`const NOT_FOUND_XPATH = '//html/./invalid';`
`13`	`13`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class HeaderTest extends TestCase`
	`7`	`+final class HeaderTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* test referrer.`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class IFrameTest extends TestCase`
	`7`	`+final class IFrameTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`public function testIFrame()`
`10`	`10`	`{`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`	`use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;`
`7`	`7`
`8`		`-class NavigationTest extends TestCase`
	`8`	`+final class NavigationTest extends TestCase`
`9`	`9`	`{`
`10`	`10`	`use AssertionRenames;`
`11`	`11`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class ScreenshotTest extends TestCase`
	`7`	`+final class ScreenshotTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`public function testScreenshot()`
`10`	`10`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Behat\Mink\Tests\Driver\TestCase;`
`6`	`6`
`7`		`-class StatusCodeTest extends TestCase`
	`7`	`+final class StatusCodeTest extends TestCase`
`8`	`8`	`{`
`9`	`9`	`public function testStatuses()`
`10`	`10`	`{`