From a83af40b048e2c32ddea3503b0575ecbfc9aac08 Mon Sep 17 00:00:00 2001 From: Tim Otten Date: Tue, 9 Feb 2016 13:05:47 -0800 Subject: [PATCH] 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+. --- tests/phpunit/Civi/Angular/ManagerTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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'); } /** -- 2.25.1