From 4940879c052853eca6d2f0029e307a9c70185982 Mon Sep 17 00:00:00 2001
From: Liam Morland <lkmorlan@uwaterloo.ca>
Date: Fri, 22 May 2020 14:52:52 -0400
Subject: [PATCH] Issue #3139195: Add 'void' return type declarations to test
 setUp() methods

---
 tests/src/Functional/FillPdfFormDeleteFormTest.php    | 2 +-
 tests/src/Functional/FillPdfFormDuplicateFormTest.php | 2 +-
 tests/src/Functional/FillPdfFormFormTest.php          | 2 +-
 tests/src/Functional/FillPdfFormImportFormTest.php    | 2 +-
 tests/src/Functional/FillPdfSettingsFormTest.php      | 2 +-
 tests/src/Functional/FillPdfTestBase.php              | 2 +-
 tests/src/Functional/HandlePdfControllerTest.php      | 2 +-
 tests/src/Functional/LinkManipulatorTest.php          | 2 +-
 tests/src/Functional/PdfPopulationTest.php            | 2 +-
 tests/src/Functional/PdfWebformPopulationTest.php     | 2 +-
 tests/src/Functional/UninstallTest.php                | 2 +-
 tests/src/Kernel/FillPdfKernelTestBase.php            | 2 +-
 12 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/tests/src/Functional/FillPdfFormDeleteFormTest.php b/tests/src/Functional/FillPdfFormDeleteFormTest.php
index 4512237..bf43fe8 100644
--- a/tests/src/Functional/FillPdfFormDeleteFormTest.php
+++ b/tests/src/Functional/FillPdfFormDeleteFormTest.php
@@ -29,7 +29,7 @@ class FillPdfFormDeleteFormTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->configureFillPdf();
diff --git a/tests/src/Functional/FillPdfFormDuplicateFormTest.php b/tests/src/Functional/FillPdfFormDuplicateFormTest.php
index b78a000..9d072f9 100644
--- a/tests/src/Functional/FillPdfFormDuplicateFormTest.php
+++ b/tests/src/Functional/FillPdfFormDuplicateFormTest.php
@@ -29,7 +29,7 @@ class FillPdfFormDuplicateFormTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->configureFillPdf();
diff --git a/tests/src/Functional/FillPdfFormFormTest.php b/tests/src/Functional/FillPdfFormFormTest.php
index 098a387..612ba2a 100644
--- a/tests/src/Functional/FillPdfFormFormTest.php
+++ b/tests/src/Functional/FillPdfFormFormTest.php
@@ -16,7 +16,7 @@ class FillPdfFormFormTest extends FillPdfUploadTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->testNode = $this->createNode([
diff --git a/tests/src/Functional/FillPdfFormImportFormTest.php b/tests/src/Functional/FillPdfFormImportFormTest.php
index 2d25e99..ed7db69 100644
--- a/tests/src/Functional/FillPdfFormImportFormTest.php
+++ b/tests/src/Functional/FillPdfFormImportFormTest.php
@@ -27,7 +27,7 @@ class FillPdfFormImportFormTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->configureFillPdf();
diff --git a/tests/src/Functional/FillPdfSettingsFormTest.php b/tests/src/Functional/FillPdfSettingsFormTest.php
index 02cf2f7..c21f77f 100644
--- a/tests/src/Functional/FillPdfSettingsFormTest.php
+++ b/tests/src/Functional/FillPdfSettingsFormTest.php
@@ -35,7 +35,7 @@ class FillPdfSettingsFormTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->initializeUser();
diff --git a/tests/src/Functional/FillPdfTestBase.php b/tests/src/Functional/FillPdfTestBase.php
index a514406..f70081e 100644
--- a/tests/src/Functional/FillPdfTestBase.php
+++ b/tests/src/Functional/FillPdfTestBase.php
@@ -94,7 +94,7 @@ abstract class FillPdfTestBase extends FileFieldTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     // Grant additional permissions to the logged-in admin user.
diff --git a/tests/src/Functional/HandlePdfControllerTest.php b/tests/src/Functional/HandlePdfControllerTest.php
index c8ad872..5208050 100644
--- a/tests/src/Functional/HandlePdfControllerTest.php
+++ b/tests/src/Functional/HandlePdfControllerTest.php
@@ -21,7 +21,7 @@ class HandlePdfControllerTest extends FillPdfUploadTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->testNodes[1] = $this->createNode([
diff --git a/tests/src/Functional/LinkManipulatorTest.php b/tests/src/Functional/LinkManipulatorTest.php
index 62b8483..bf36b46 100644
--- a/tests/src/Functional/LinkManipulatorTest.php
+++ b/tests/src/Functional/LinkManipulatorTest.php
@@ -37,7 +37,7 @@ class LinkManipulatorTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->configureFillPdf();
diff --git a/tests/src/Functional/PdfPopulationTest.php b/tests/src/Functional/PdfPopulationTest.php
index 82cb207..0808aa0 100644
--- a/tests/src/Functional/PdfPopulationTest.php
+++ b/tests/src/Functional/PdfPopulationTest.php
@@ -49,7 +49,7 @@ class PdfPopulationTest extends FillPdfTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     // Grant additional permissions to the logged-in admin user.
diff --git a/tests/src/Functional/PdfWebformPopulationTest.php b/tests/src/Functional/PdfWebformPopulationTest.php
index 7ea9b64..580e0ac 100644
--- a/tests/src/Functional/PdfWebformPopulationTest.php
+++ b/tests/src/Functional/PdfWebformPopulationTest.php
@@ -42,7 +42,7 @@ class PdfWebformPopulationTest extends FillPdfTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     // @todo: Stop skipping this test when Webform has Drupal 9 support.
     if (version_compare(substr(\Drupal::VERSION, 0, 1), 9, '>=')) {
       $this->markTestSkipped('Webform is not yet compatible with Drupal 9, so skipping these tests.');
diff --git a/tests/src/Functional/UninstallTest.php b/tests/src/Functional/UninstallTest.php
index 3df44c9..f7b7e6c 100644
--- a/tests/src/Functional/UninstallTest.php
+++ b/tests/src/Functional/UninstallTest.php
@@ -35,7 +35,7 @@ class UninstallTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->configureFillPdf();
diff --git a/tests/src/Kernel/FillPdfKernelTestBase.php b/tests/src/Kernel/FillPdfKernelTestBase.php
index f2b2b3a..7430355 100644
--- a/tests/src/Kernel/FillPdfKernelTestBase.php
+++ b/tests/src/Kernel/FillPdfKernelTestBase.php
@@ -29,7 +29,7 @@ abstract class FillPdfKernelTestBase extends EntityKernelTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
 
     $this->installEntitySchema('fillpdf_form');
-- 
GitLab