Switch from annotations to prefix in unit tests to prepare for phpuni…

…t12 deprecations

tests/Cache/BatchCacheTest.php

@@ -29,9 +29,9 @@ class BatchCacheTest extends TestCase
     private $memory;
 
     /**
-     * @test
+     *
      */
-    public function will_get_multiple_from_memory_if_cells_hold_in_memory()
+    public function test_will_get_multiple_from_memory_if_cells_hold_in_memory()
     {
         $inMemory = [
             'A1' => 'A1-value',
@@ -50,9 +50,9 @@ public function will_get_multiple_from_memory_if_cells_hold_in_memory()
     }
 
     /**
-     * @test
+     *
      */
-    public function will_get_multiple_from_cache_if_cells_are_persisted()
+    public function test_will_get_multiple_from_cache_if_cells_are_persisted()
     {
         $inMemory  = [];
         $persisted = [
@@ -72,9 +72,9 @@ public function will_get_multiple_from_cache_if_cells_are_persisted()
     }
 
     /**
-     * @test
+     *
      */
-    public function will_get_multiple_from_cache_and_persisted()
+    public function test_will_get_multiple_from_cache_and_persisted()
     {
         $inMemory  = [
             'A1' => 'A1-value',
@@ -99,9 +99,9 @@ public function will_get_multiple_from_cache_and_persisted()
     }
 
     /**
-     * @test
+     *
      */
-    public function it_persists_to_cache_when_memory_limit_reached_on_setting_a_value()
+    public function test_it_persists_to_cache_when_memory_limit_reached_on_setting_a_value()
     {
         $memoryLimit = 3;
         $persisted   = [];
@@ -137,9 +137,9 @@ public function it_persists_to_cache_when_memory_limit_reached_on_setting_a_valu
     }
 
     /**
-     * @test
+     *
      */
-    public function it_persists_to_cache_when_memory_limit_reached_on_setting_multiple_values()
+    public function test_it_persists_to_cache_when_memory_limit_reached_on_setting_multiple_values()
     {
         $memoryLimit = 3;
         $persisted   = [];
@@ -180,11 +180,11 @@ public function it_persists_to_cache_when_memory_limit_reached_on_setting_multip
     }
 
     /**
-     * @test
+     *
      *
      * @dataProvider defaultTTLDataProvider
      */
-    public function it_writes_to_cache_with_default_ttl($defaultTTL, $receivedAs)
+    public function test_it_writes_to_cache_with_default_ttl($defaultTTL, $receivedAs)
     {
         config()->set('excel.cache.default_ttl', $defaultTTL);
 
@@ -204,9 +204,9 @@ function (KeyWritten $event) use ($expectedTTL) {
     }
 
     /**
-     * @test
+     *
      */
-    public function it_writes_to_cache_with_a_dateinterval_ttl()
+    public function test_it_writes_to_cache_with_a_dateinterval_ttl()
     {
         // DateInterval is 1 minute
         config()->set('excel.cache.default_ttl', new DateInterval('PT1M'));
@@ -225,9 +225,9 @@ function (KeyWritten $event) {
     }
 
     /**
-     * @test
+     *
      */
-    public function it_can_override_default_ttl()
+    public function test_it_can_override_default_ttl()
     {
         config()->set('excel.cache.default_ttl', 1);
 

tests/CellTest.php

@@ -9,9 +9,9 @@
 class CellTest extends TestCase
 {
     /**
-     * @test
+     *
      */
-    public function can_get_cell_value()
+    public function test_can_get_cell_value()
     {
         config()->set('excel.imports.cells.middleware', []);
 
@@ -24,9 +24,9 @@ public function can_get_cell_value()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_trim_empty_cells()
+    public function test_can_trim_empty_cells()
     {
         config()->set('excel.imports.cells.middleware', [
             TrimCellValue::class,
@@ -40,9 +40,9 @@ public function can_trim_empty_cells()
     }
 
     /**
-     * @test
+     *
      */
-    public function convert_empty_cells_to_null()
+    public function test_convert_empty_cells_to_null()
     {
         config()->set('excel.imports.cells.middleware', [
             TrimCellValue::class,

tests/Concerns/ExportableTest.php

@@ -14,9 +14,9 @@
 class ExportableTest extends TestCase
 {
     /**
-     * @test
+     *
      */
-    public function needs_to_have_a_file_name_when_downloading()
+    public function test_needs_to_have_a_file_name_when_downloading()
     {
         $this->expectException(\Maatwebsite\Excel\Exceptions\NoFilenameGivenException::class);
         $this->expectExceptionMessage('A filename needs to be passed in order to download the export');
@@ -30,9 +30,9 @@ public function needs_to_have_a_file_name_when_downloading()
     }
 
     /**
-     * @test
+     *
      */
-    public function needs_to_have_a_file_name_when_storing()
+    public function test_needs_to_have_a_file_name_when_storing()
     {
         $this->expectException(\Maatwebsite\Excel\Exceptions\NoFilePathGivenException::class);
         $this->expectExceptionMessage('A filepath needs to be passed in order to store the export');
@@ -46,9 +46,9 @@ public function needs_to_have_a_file_name_when_storing()
     }
 
     /**
-     * @test
+     *
      */
-    public function needs_to_have_a_file_name_when_queuing()
+    public function test_needs_to_have_a_file_name_when_queuing()
     {
         $this->expectException(\Maatwebsite\Excel\Exceptions\NoFilePathGivenException::class);
         $this->expectExceptionMessage('A filepath needs to be passed in order to store the export');
@@ -62,9 +62,9 @@ public function needs_to_have_a_file_name_when_queuing()
     }
 
     /**
-     * @test
+     *
      */
-    public function responsable_needs_to_have_file_name_configured_inside_the_export()
+    public function test_responsable_needs_to_have_file_name_configured_inside_the_export()
     {
         $this->expectException(\Maatwebsite\Excel\Exceptions\NoFilenameGivenException::class);
         $this->expectExceptionMessage('A filename needs to be passed in order to download the export');
@@ -78,9 +78,9 @@ public function responsable_needs_to_have_file_name_configured_inside_the_export
     }
 
     /**
-     * @test
+     *
      */
-    public function is_responsable()
+    public function test_is_responsable()
     {
         $export = new class implements Responsable
         {
@@ -97,9 +97,9 @@ public function is_responsable()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_have_customized_header()
+    public function test_can_have_customized_header()
     {
         $export   = new class
         {
@@ -116,9 +116,9 @@ public function can_have_customized_header()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_set_custom_headers_in_export_class()
+    public function test_can_set_custom_headers_in_export_class()
     {
         $export   = new class
         {
@@ -136,9 +136,9 @@ public function can_set_custom_headers_in_export_class()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_get_raw_export_contents()
+    public function test_can_get_raw_export_contents()
     {
         $export = new EmptyExport;
 
@@ -148,9 +148,9 @@ public function can_get_raw_export_contents()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_have_customized_disk_options_when_storing()
+    public function test_can_have_customized_disk_options_when_storing()
     {
         $export = new EmptyExport;
 
@@ -162,9 +162,9 @@ public function can_have_customized_disk_options_when_storing()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_have_customized_disk_options_when_queueing()
+    public function test_can_have_customized_disk_options_when_queueing()
     {
         $export = new EmptyExport;
 
@@ -176,9 +176,9 @@ public function can_have_customized_disk_options_when_queueing()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_set_disk_options_in_export_class_when_storing()
+    public function test_can_set_disk_options_in_export_class_when_storing()
     {
         $export = new class
         {
@@ -197,9 +197,9 @@ public function can_set_disk_options_in_export_class_when_storing()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_set_disk_options_in_export_class_when_queuing()
+    public function test_can_set_disk_options_in_export_class_when_queuing()
     {
         $export = new class
         {
@@ -218,9 +218,9 @@ public function can_set_disk_options_in_export_class_when_queuing()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_override_export_class_disk_options_when_calling_store()
+    public function test_can_override_export_class_disk_options_when_calling_store()
     {
         $export = new class
         {
@@ -237,9 +237,9 @@ public function can_override_export_class_disk_options_when_calling_store()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_override_export_class_disk_options_when_calling_queue()
+    public function test_can_override_export_class_disk_options_when_calling_queue()
     {
         $export = new class
         {
@@ -256,9 +256,9 @@ public function can_override_export_class_disk_options_when_calling_queue()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_have_empty_disk_options_when_storing()
+    public function test_can_have_empty_disk_options_when_storing()
     {
         $export = new EmptyExport;
 
@@ -270,9 +270,9 @@ public function can_have_empty_disk_options_when_storing()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_have_empty_disk_options_when_queueing()
+    public function test_can_have_empty_disk_options_when_queueing()
     {
         $export = new EmptyExport;
 

tests/Concerns/FromArrayTest.php

@@ -9,9 +9,9 @@
 class FromArrayTest extends TestCase
 {
     /**
-     * @test
+     *
      */
-    public function can_export_from_array()
+    public function test_can_export_from_array()
     {
         $export = new class implements FromArray
         {

tests/Concerns/FromCollectionTest.php

@@ -12,9 +12,9 @@
 class FromCollectionTest extends TestCase
 {
     /**
-     * @test
+     *
      */
-    public function can_export_from_collection()
+    public function test_can_export_from_collection()
     {
         $export = new SheetWith100Rows('A');
 
@@ -28,9 +28,9 @@ public function can_export_from_collection()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_export_with_multiple_sheets_from_collection()
+    public function test_can_export_with_multiple_sheets_from_collection()
     {
         $export = new QueuedExport();
 
@@ -52,9 +52,9 @@ public function can_export_with_multiple_sheets_from_collection()
     }
 
     /**
-     * @test
+     *
      */
-    public function can_export_from_lazy_collection()
+    public function test_can_export_from_lazy_collection()
     {
         if (!class_exists('\Illuminate\Support\LazyCollection')) {
             $this->markTestSkipped('Skipping test because LazyCollection is not supported');
@@ -79,9 +79,9 @@ function (array $item) {
     }
 
     /**
-     * @test
+     *
      */
-    public function can_export_from_lazy_collection_with_queue()
+    public function test_can_export_from_lazy_collection_with_queue()
     {
         if (!class_exists('\Illuminate\Support\LazyCollection')) {
             $this->markTestSkipped('Skipping test because LazyCollection is not supported');

tests/Concerns/FromGeneratorTest.php

@@ -10,9 +10,9 @@
 class FromGeneratorTest extends TestCase
 {
     /**
-     * @test
+     *
      */
-    public function can_export_from_generator()
+    public function test_can_export_from_generator()
     {
         $export = new class implements FromGenerator
         {