Properly check for env vars

Author: ace-n

functions/slack_slash_command/test/DeployTest.php

@@ -50,9 +50,6 @@ public function testFunction(
         $statusCode,
         $headers
     ): void {
-        $this->requireEnv('SLACK_SECRET');
-        $this->requireEnv('KG_API_KEY');
-
         $response = $this->client->request(
             $method,
             '',
@@ -70,11 +67,16 @@ public function testFunction(
         }
     }
 
+    /**
+     * Deploy the Function.
+     *
+     * Overrides CloudFunctionLocalTestTrait::doDeploy().
+     */
     private static function doDeploy()
     {
-        // Forward required env variables to Cloud Functions
-        $envVars = 'SLACK_SECRET=' . getenv('SLACK_SECRET') . ',';
-        $envVars .= 'KG_API_KEY=' . getenv('KG_API_KEY');
+        // Forward required env variables to Cloud Functions.
+        $envVars = 'SLACK_SECRET=' . self::requireEnv('SLACK_SECRET') . ',';
+        $envVars .= 'KG_API_KEY=' . self::requireEnv('KG_API_KEY');
 
         self::$fn->deploy(['--update-env-vars' => $envVars]);
     }

functions/slack_slash_command/test/IntegrationTest.php

@@ -34,6 +34,19 @@ class IntegrationTest extends TestCase
 
     private static $entryPoint = 'receiveRequest';
 
+    /**
+     * Run the PHP server locally for the defined function.
+     *
+     * Overrides CloudFunctionLocalTestTrait::doRun().
+     */
+    private static function doRun()
+    {
+        self::$fn->run([
+            'SLACK_SECRET' => self::requireEnv('SLACK_SECRET'),
+            'KG_API_KEY' => self::requireEnv('KG_API_KEY'),
+        ]);
+    }
+
     /**
       * @dataProvider cases
       */
@@ -45,9 +58,6 @@ public function testFunction(
         $statusCode,
         $headers
     ): void {
-        $this->requireEnv('SLACK_SECRET');
-        $this->requireEnv('KG_API_KEY');
-
         $response = $this->client->request(
             $method,
             '/',