From: Tim Otten <totten@civicrm.org>
Date: Tue, 9 Feb 2016 21:05:47 +0000 (-0800)
Subject: Civi\Angular\ManagerTest - Fix false-negative
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=a83af40b048e2c32ddea3503b0575ecbfc9aac08;p=civicrm-core.git

Civi\Angular\ManagerTest - Fix false-negative

The test is a bit overzealous.  To ensure that the 'settings' are being
processed, it checks to ensure that some settings (any settings) are
defined.  However, it looks for 2+ settings, and it's enough look for 1+.
---

diff --git a/tests/phpunit/Civi/Angular/ManagerTest.php b/tests/phpunit/Civi/Angular/ManagerTest.php
index e17e6285f2..5156611406 100644
--- a/tests/phpunit/Civi/Angular/ManagerTest.php
+++ b/tests/phpunit/Civi/Angular/ManagerTest.php
@@ -101,7 +101,7 @@ class ManagerTest extends \CiviUnitTestCase {
     $this->assertTrue($counts['js'] > 0, 'Expect to find at least one JS file');
     $this->assertTrue($counts['css'] > 0, 'Expect to find at least one CSS file');
     $this->assertTrue($counts['partials'] > 0, 'Expect to find at least one partial HTML file');
-    $this->assertTrue($counts['settings'] > 1, 'Expect to find at least one setting');
+    $this->assertTrue($counts['settings'] > 0, 'Expect to find at least one setting');
   }
 
   /**