From 1c4af33b8f55b47ccf9be7a416a98f36cd961802 Mon Sep 17 00:00:00 2001
From: Martin Hettiger <martin@hettiger.com>
Date: Mon, 14 Sep 2020 15:04:03 +0200
Subject: [PATCH] Fix PHPUnit bool server consts result in null (#5409)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

After updating to Laravel 8 I suddenly had my test suite failing because telescope would not be disabled properly by `phpunit.xml` anymore.
This changes fixed my test suite.

I've also created a clean Laravel 8 project and added some tests to demonstrate the issue:
https://github.com/hettiger/laravel-env-demo/commit/908d3405b84e769c12d296e98bef1ca567127254

Maybe this needs to be addressed in PHPUnit. However I'm adding this workaround here because it's a viable solution IMHO.
Maybe should add a note on this in the docs and be done with it…?
---
 phpunit.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/phpunit.xml b/phpunit.xml
index 4ae4d979..8bc0de4e 100644
--- a/phpunit.xml
+++ b/phpunit.xml
@@ -26,6 +26,6 @@
         <server name="MAIL_MAILER" value="array"/>
         <server name="QUEUE_CONNECTION" value="sync"/>
         <server name="SESSION_DRIVER" value="array"/>
-        <server name="TELESCOPE_ENABLED" value="false"/>
+        <server name="TELESCOPE_ENABLED" value="(false)"/>
     </php>
 </phpunit>