Add returns on tests, just because.

Author: Crell

tests/FlexBridgeDatabaseTest.php

@@ -41,14 +41,14 @@ public function setUp()
         );
     }
 
-    public function testNotOnPlatformshDoesNotSetDatabase()
+    public function testNotOnPlatformshDoesNotSetDatabase() : void
     {
         mapPlatformShEnvironment();
 
         $this->assertArrayNotHasKey('DATABASE_URL', $_SERVER);
     }
 
-    public function testNoRelationships()
+    public function testNoRelationships() : void
     {
         // We assume no relationships array, but a PLATFORM_APPLICATION env var,
         // means we're in a build hook.
@@ -62,7 +62,7 @@ public function testNoRelationships()
         $this->assertEquals($this->defaultDbUrl, $_SERVER['DATABASE_URL']);
     }
 
-    public function testNoDatabaseRelationship()
+    public function testNoDatabaseRelationship() : void
     {
         putenv('PLATFORM_APPLICATION=test');
 
@@ -76,7 +76,7 @@ public function testNoDatabaseRelationship()
         $this->assertEquals($this->defaultDbUrl, $_SERVER['DATABASE_URL']);
     }
 
-    public function testDatabaseRelationshipSet()
+    public function testDatabaseRelationshipSet() : void
     {
         putenv('PLATFORM_APPLICATION=test');
         putenv(sprintf('PLATFORM_RELATIONSHIPS=%s', base64_encode(json_encode($this->relationships))));

tests/FlexBridgeTest.php

@@ -8,14 +8,14 @@
 class FlexBridgeTest extends TestCase
 {
 
-    public function testDoesNotRunWithoutPlatformshVariables()
+    public function testDoesNotRunWithoutPlatformshVariables() : void
     {
         mapPlatformShEnvironment();
 
         $this->assertFalse(getenv('APP_SECRET'));
     }
 
-    public function testSetAppSecret()
+    public function testSetAppSecret() : void
     {
         putenv('PLATFORM_APPLICATION=test');
         putenv('PLATFORM_PROJECT_ENTROPY=test');
@@ -25,7 +25,7 @@ public function testSetAppSecret()
         $this->assertEquals('test', $_SERVER['APP_SECRET']);
     }
 
-    public function testDontChangeAppSecret()
+    public function testDontChangeAppSecret() : void
     {
         putenv('PLATFORM_APPLICATION=test');
         putenv('PLATFORM_PROJECT_ENTROPY=test');
@@ -36,7 +36,7 @@ public function testDontChangeAppSecret()
         $this->assertEquals('original', $_SERVER['APP_SECRET']);
     }
 
-    public function testAppEnvAlreadySetInServer()
+    public function testAppEnvAlreadySetInServer() : void
     {
         putenv('PLATFORM_APPLICATION=test');
         $_SERVER['APP_ENV'] = 'dev';
@@ -46,7 +46,7 @@ public function testAppEnvAlreadySetInServer()
         $this->assertEquals('dev', $_SERVER['APP_ENV']);
     }
 
-    public function testAppEnvAlreadySetInEnv()
+    public function testAppEnvAlreadySetInEnv() : void
     {
         putenv('PLATFORM_APPLICATION=test');
         putenv('APP_ENV=dev');
@@ -56,7 +56,7 @@ public function testAppEnvAlreadySetInEnv()
         $this->assertEquals('dev', $_SERVER['APP_ENV']);
     }
 
-    public function testAppEnvNeedsDefault()
+    public function testAppEnvNeedsDefault() : void
     {
         putenv('PLATFORM_APPLICATION=test');