Improve cleanup of globals on ajaxTest

civicrm · Nov 21, 2018 · f8b0413 · f8b0413
1 parent d0df87f
commit f8b0413
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/tests/phpunit/CRM/Contact/Page/AjaxTest.php b/tests/phpunit/CRM/Contact/Page/AjaxTest.php
@@ -5,10 +5,24 @@
  */
 class CRM_Contact_Page_AjaxTest extends CiviUnitTestCase {
 
+  /**
+   * Original $_REQUEST
+   *
+   * We are messing with globals so fix afterwards.
+   *
+   * @var array
+   */
+  protected $originalRequest = [];
 
   public function setUp() {
     $this->useTransaction(TRUE);
     parent::setUp();
+    $this->originalRequest = $_REQUEST;
+  }
+
+  public function tearDown() {
+    $_REQUEST = $this->originalRequest;
+    parent::tearDown();
   }
 
   /**