CorWatts / fasterscale

common/models/LoginForm.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
         if ($this->validate()) {
             $user = $this->getUser();
             if ($user->isVerified()) {
-                return Yii::$app->user->login($user, $this->rememberMe ? 3600 * 24 * 30 : 0);
+                return Yii::$app->user->login($user, $this->rememberMe ? 3600*24*30 : 0);
             } else {
                 Yii::$app->session->setFlash('warning', 'You must verify your account before you can proceed. Please check your email inbox for a verification email and follow the instructions.');
             }

common/models/Category.php

Spacing +8 added lines, -8 removed lines

@@ -12,13 +12,13 @@ 
 class Category extends \yii\base\BaseObject implements \common\interfaces\CategoryInterface
 {
     public static $categories = [
-    [ "id" => 1, "name" => "Restoration"],
-    [ "id" => 2, "name" => "Forgetting Priorities"],
-    [ "id" => 3, "name" => "Anxiety"],
-    [ "id" => 4, "name" => "Speeding Up"],
-    [ "id" => 5, "name" => "Ticked Off"],
-    [ "id" => 6, "name" => "Exhausted"],
-    [ "id" => 7, "name" => "Relapse/Moral Failure"],
+    ["id" => 1, "name" => "Restoration"],
+    ["id" => 2, "name" => "Forgetting Priorities"],
+    ["id" => 3, "name" => "Anxiety"],
+    ["id" => 4, "name" => "Speeding Up"],
+    ["id" => 5, "name" => "Ticked Off"],
+    ["id" => 6, "name" => "Exhausted"],
+    ["id" => 7, "name" => "Relapse/Moral Failure"],
   ];
 
     public static $colors = [
@@ -102,7 +102,7 @@ 
      */
     public static function getCategory($key, $val)
     {
-        $ret = array_values(array_filter(self::$categories, function ($cat) use ($key, $val) {
+        $ret = array_values(array_filter(self::$categories, function($cat) use ($key, $val) {
             return $cat[$key] === $val;
         }));
         return $ret ? $ret[0] : null;

common/models/Behavior.php

Spacing +98 added lines, -98 removed lines

@@ -14,103 +14,103 @@ 
 class Behavior extends \yii\base\BaseObject implements BehaviorInterface
 {
     public static $behaviors = [
-    ['id' => 1,   'name' => 'no current secrets', 'category_id' => 1],
-    ['id' => 2,   'name' => 'resolving problems', 'category_id' => 1],
-    ['id' => 3,   'name' => 'identifying fears and feelings', 'category_id' => 1],
-    ['id' => 4,   'name' => 'keeping commitments to meetings, prayer, family, church, people, goals, and self', 'category_id' => 1],
-    ['id' => 5,   'name' => 'being open', 'category_id' => 1],
-    ['id' => 6,   'name' => 'being honest', 'category_id' => 1],
-    ['id' => 7,   'name' => 'making eye contact', 'category_id' => 1],
-    ['id' => 8,   'name' => 'reaching out to others', 'category_id' => 1],
-    ['id' => 9,   'name' => 'increasing in relationships with God and others', 'category_id' => 1],
-    ['id' => 10,  'name' => 'accountability', 'category_id' => 1],
-    ['id' => 11,  'name' => 'secrets', 'category_id' => 2],
-    ['id' => 12,  'name' => 'bored', 'category_id' => 2],
-    ['id' => 13,  'name' => 'less time/energy for God, meetings, and church', 'category_id' => 2],
-    ['id' => 14,  'name' => 'avoiding support and accountability towards people', 'category_id' => 2],
-    ['id' => 15,  'name' => 'superficial conversations', 'category_id' => 2],
-    ['id' => 16,  'name' => 'sarcasm', 'category_id' => 2],
-    ['id' => 17,  'name' => 'isolating yourself', 'category_id' => 2],
-    ['id' => 18,  'name' => 'changes in goals', 'category_id' => 2],
-    ['id' => 19,  'name' => 'flirting', 'category_id' => 2],
-    ['id' => 20,  'name' => 'obsessed with relationships', 'category_id' => 2],
-    ['id' => 21,  'name' => 'breaking promises/commitments', 'category_id' => 2],
-    ['id' => 22,  'name' => 'neglecting family', 'category_id' => 2],
-    ['id' => 23,  'name' => 'preoccupation with material things, television, or entertainment', 'category_id' => 2],
-    ['id' => 24,  'name' => 'procrastination', 'category_id' => 2],
-    ['id' => 25,  'name' => 'lying', 'category_id' => 2],
-    ['id' => 26,  'name' => 'over-confidence', 'category_id' => 2],
-    ['id' => 27,  'name' => 'hiding money', 'category_id' => 2],
-    ['id' => 28,  'name' => 'worry', 'category_id' => 3],
-    ['id' => 29,  'name' => 'using profanity', 'category_id' => 3],
-    ['id' => 30,  'name' => 'being fearful', 'category_id' => 3],
-    ['id' => 31,  'name' => 'being resentful', 'category_id' => 3],
-    ['id' => 32,  'name' => 'replaying old, negative thoughts', 'category_id' => 3],
-    ['id' => 33,  'name' => 'perfectionism', 'category_id' => 3],
-    ['id' => 34,  'name' => 'judging others\' motives', 'category_id' => 3],
-    ['id' => 35,  'name' => 'making goals and lists you can\'t complete', 'category_id' => 3],
-    ['id' => 36,  'name' => 'poor planning', 'category_id' => 3],
-    ['id' => 37,  'name' => 'mind reading', 'category_id' => 3],
-    ['id' => 38,  'name' => 'fantasy', 'category_id' => 3],
-    ['id' => 41,  'name' => 'co-dependent rescuing', 'category_id' => 3],
-    ['id' => 42,  'name' => 'sleep problems', 'category_id' => 3],
-    ['id' => 43,  'name' => 'trouble concentrating', 'category_id' => 3],
-    ['id' => 44,  'name' => 'seeking/creating drama', 'category_id' => 3],
-    ['id' => 45,  'name' => 'gossip', 'category_id' => 3],
-    ['id' => 46,  'name' => 'using over-the-counter medication for pain, sleep, and weight control', 'category_id' => 3],
-    ['id' => 47,  'name' => 'super busy', 'category_id' => 4],
-    ['id' => 48,  'name' => 'workaholic', 'category_id' => 4],
-    ['id' => 49,  'name' => 'can\'t relax', 'category_id' => 4],
-    ['id' => 50,  'name' => 'driving too fast', 'category_id' => 4],
-    ['id' => 51,  'name' => 'avoiding slowing down', 'category_id' => 4],
-    ['id' => 52,  'name' => 'feeling driven', 'category_id' => 4],
-    ['id' => 53,  'name' => 'in a hurry', 'category_id' => 4],
-    ['id' => 54,  'name' => 'can\'t turn off thoughts', 'category_id' => 4],
-    ['id' => 55,  'name' => 'skipping meals', 'category_id' => 4],
-    ['id' => 56,  'name' => 'binge eating (usually at night)', 'category_id' => 4],
-    ['id' => 57,  'name' => 'overspending', 'category_id' => 4],
-    ['id' => 58,  'name' => 'can\'t identify own feelings/needs', 'category_id' => 4],
-    ['id' => 59,  'name' => 'repetitive, negative thoughts', 'category_id' => 4],
-    ['id' => 60,  'name' => 'irritable', 'category_id' => 4],
-    ['id' => 61,  'name' => 'making excuses for "having to do it all"', 'category_id' => 4],
-    ['id' => 62,  'name' => 'dramatic mood swings', 'category_id' => 4],
-    ['id' => 63,  'name' => 'lust', 'category_id' => 4],
-    ['id' => 64,  'name' => 'too much caffeine', 'category_id' => 4],
-    ['id' => 65,  'name' => 'over exercising', 'category_id' => 4],
-    ['id' => 66,  'name' => 'nervousness', 'category_id' => 4],
-    ['id' => 67,  'name' => 'difficulty being alone or with people', 'category_id' => 4],
-    ['id' => 68,  'name' => 'difficulty listening to others', 'category_id' => 4],
-    ['id' => 69,  'name' => 'avoiding support', 'category_id' => 4],
-    ['id' => 70,  'name' => 'procrastination causing crises in money, work, or relationships', 'category_id' => 5],
-    ['id' => 71,  'name' => 'sarcasm', 'category_id' => 5],
-    ['id' => 72,  'name' => 'black and white, all or nothing thinking', 'category_id' => 5],
-    ['id' => 73,  'name' => 'feeling that no one understands', 'category_id' => 5],
-    ['id' => 74,  'name' => 'overreacting', 'category_id' => 5],
-    ['id' => 75,  'name' => 'road rage', 'category_id' => 5],
-    ['id' => 76,  'name' => 'constant resentments', 'category_id' => 5],
-    ['id' => 77,  'name' => 'pushing others away', 'category_id' => 5],
-    ['id' => 78,  'name' => 'increased isolation', 'category_id' => 5],
-    ['id' => 79,  'name' => 'blaming', 'category_id' => 5],
-    ['id' => 80,  'name' => 'self pity', 'category_id' => 5],
-    ['id' => 81,  'name' => 'arguing', 'category_id' => 5],
-    ['id' => 82,  'name' => 'irrationality', 'category_id' => 5],
-    ['id' => 83,  'name' => 'can\'t handle criticism', 'category_id' => 5],
-    ['id' => 84,  'name' => 'defensive', 'category_id' => 5],
-    ['id' => 85,  'name' => 'people are avoiding you', 'category_id' => 5],
-    ['id' => 86,  'name' => 'having to be right', 'category_id' => 5],
-    ['id' => 87,  'name' => 'digestive problems', 'category_id' => 5],
-    ['id' => 88,  'name' => 'headaches', 'category_id' => 5],
-    ['id' => 89,  'name' => 'obsessive (stuck) thoughts', 'category_id' => 5],
-    ['id' => 90,  'name' => 'can\'t forgive', 'category_id' => 5],
-    ['id' => 91,  'name' => 'feeling grandiose (superior)', 'category_id' => 5],
-    ['id' => 92,  'name' => 'intimidation', 'category_id' => 5],
-    ['id' => 93,  'name' => 'feeling aggressive', 'category_id' => 5],
-    ['id' => 94,  'name' => 'depressed', 'category_id' => 6],
-    ['id' => 95,  'name' => 'panicked', 'category_id' => 6],
-    ['id' => 96,  'name' => 'confused', 'category_id' => 6],
-    ['id' => 97,  'name' => 'hopeless', 'category_id' => 6],
-    ['id' => 98,  'name' => 'sleeping too much or too little', 'category_id' => 6],
-    ['id' => 99,  'name' => 'can\'t cope', 'category_id' => 6],
+    ['id' => 1, 'name' => 'no current secrets', 'category_id' => 1],
+    ['id' => 2, 'name' => 'resolving problems', 'category_id' => 1],
+    ['id' => 3, 'name' => 'identifying fears and feelings', 'category_id' => 1],
+    ['id' => 4, 'name' => 'keeping commitments to meetings, prayer, family, church, people, goals, and self', 'category_id' => 1],
+    ['id' => 5, 'name' => 'being open', 'category_id' => 1],
+    ['id' => 6, 'name' => 'being honest', 'category_id' => 1],
+    ['id' => 7, 'name' => 'making eye contact', 'category_id' => 1],
+    ['id' => 8, 'name' => 'reaching out to others', 'category_id' => 1],
+    ['id' => 9, 'name' => 'increasing in relationships with God and others', 'category_id' => 1],
+    ['id' => 10, 'name' => 'accountability', 'category_id' => 1],
+    ['id' => 11, 'name' => 'secrets', 'category_id' => 2],
+    ['id' => 12, 'name' => 'bored', 'category_id' => 2],
+    ['id' => 13, 'name' => 'less time/energy for God, meetings, and church', 'category_id' => 2],
+    ['id' => 14, 'name' => 'avoiding support and accountability towards people', 'category_id' => 2],
+    ['id' => 15, 'name' => 'superficial conversations', 'category_id' => 2],
+    ['id' => 16, 'name' => 'sarcasm', 'category_id' => 2],
+    ['id' => 17, 'name' => 'isolating yourself', 'category_id' => 2],
+    ['id' => 18, 'name' => 'changes in goals', 'category_id' => 2],
+    ['id' => 19, 'name' => 'flirting', 'category_id' => 2],
+    ['id' => 20, 'name' => 'obsessed with relationships', 'category_id' => 2],
+    ['id' => 21, 'name' => 'breaking promises/commitments', 'category_id' => 2],
+    ['id' => 22, 'name' => 'neglecting family', 'category_id' => 2],
+    ['id' => 23, 'name' => 'preoccupation with material things, television, or entertainment', 'category_id' => 2],
+    ['id' => 24, 'name' => 'procrastination', 'category_id' => 2],
+    ['id' => 25, 'name' => 'lying', 'category_id' => 2],
+    ['id' => 26, 'name' => 'over-confidence', 'category_id' => 2],
+    ['id' => 27, 'name' => 'hiding money', 'category_id' => 2],
+    ['id' => 28, 'name' => 'worry', 'category_id' => 3],
+    ['id' => 29, 'name' => 'using profanity', 'category_id' => 3],
+    ['id' => 30, 'name' => 'being fearful', 'category_id' => 3],
+    ['id' => 31, 'name' => 'being resentful', 'category_id' => 3],
+    ['id' => 32, 'name' => 'replaying old, negative thoughts', 'category_id' => 3],
+    ['id' => 33, 'name' => 'perfectionism', 'category_id' => 3],
+    ['id' => 34, 'name' => 'judging others\' motives', 'category_id' => 3],
+    ['id' => 35, 'name' => 'making goals and lists you can\'t complete', 'category_id' => 3],
+    ['id' => 36, 'name' => 'poor planning', 'category_id' => 3],
+    ['id' => 37, 'name' => 'mind reading', 'category_id' => 3],
+    ['id' => 38, 'name' => 'fantasy', 'category_id' => 3],
+    ['id' => 41, 'name' => 'co-dependent rescuing', 'category_id' => 3],
+    ['id' => 42, 'name' => 'sleep problems', 'category_id' => 3],
+    ['id' => 43, 'name' => 'trouble concentrating', 'category_id' => 3],
+    ['id' => 44, 'name' => 'seeking/creating drama', 'category_id' => 3],
+    ['id' => 45, 'name' => 'gossip', 'category_id' => 3],
+    ['id' => 46, 'name' => 'using over-the-counter medication for pain, sleep, and weight control', 'category_id' => 3],
+    ['id' => 47, 'name' => 'super busy', 'category_id' => 4],
+    ['id' => 48, 'name' => 'workaholic', 'category_id' => 4],
+    ['id' => 49, 'name' => 'can\'t relax', 'category_id' => 4],
+    ['id' => 50, 'name' => 'driving too fast', 'category_id' => 4],
+    ['id' => 51, 'name' => 'avoiding slowing down', 'category_id' => 4],
+    ['id' => 52, 'name' => 'feeling driven', 'category_id' => 4],
+    ['id' => 53, 'name' => 'in a hurry', 'category_id' => 4],
+    ['id' => 54, 'name' => 'can\'t turn off thoughts', 'category_id' => 4],
+    ['id' => 55, 'name' => 'skipping meals', 'category_id' => 4],
+    ['id' => 56, 'name' => 'binge eating (usually at night)', 'category_id' => 4],
+    ['id' => 57, 'name' => 'overspending', 'category_id' => 4],
+    ['id' => 58, 'name' => 'can\'t identify own feelings/needs', 'category_id' => 4],
+    ['id' => 59, 'name' => 'repetitive, negative thoughts', 'category_id' => 4],
+    ['id' => 60, 'name' => 'irritable', 'category_id' => 4],
+    ['id' => 61, 'name' => 'making excuses for "having to do it all"', 'category_id' => 4],
+    ['id' => 62, 'name' => 'dramatic mood swings', 'category_id' => 4],
+    ['id' => 63, 'name' => 'lust', 'category_id' => 4],
+    ['id' => 64, 'name' => 'too much caffeine', 'category_id' => 4],
+    ['id' => 65, 'name' => 'over exercising', 'category_id' => 4],
+    ['id' => 66, 'name' => 'nervousness', 'category_id' => 4],
+    ['id' => 67, 'name' => 'difficulty being alone or with people', 'category_id' => 4],
+    ['id' => 68, 'name' => 'difficulty listening to others', 'category_id' => 4],
+    ['id' => 69, 'name' => 'avoiding support', 'category_id' => 4],
+    ['id' => 70, 'name' => 'procrastination causing crises in money, work, or relationships', 'category_id' => 5],
+    ['id' => 71, 'name' => 'sarcasm', 'category_id' => 5],
+    ['id' => 72, 'name' => 'black and white, all or nothing thinking', 'category_id' => 5],
+    ['id' => 73, 'name' => 'feeling that no one understands', 'category_id' => 5],
+    ['id' => 74, 'name' => 'overreacting', 'category_id' => 5],
+    ['id' => 75, 'name' => 'road rage', 'category_id' => 5],
+    ['id' => 76, 'name' => 'constant resentments', 'category_id' => 5],
+    ['id' => 77, 'name' => 'pushing others away', 'category_id' => 5],
+    ['id' => 78, 'name' => 'increased isolation', 'category_id' => 5],
+    ['id' => 79, 'name' => 'blaming', 'category_id' => 5],
+    ['id' => 80, 'name' => 'self pity', 'category_id' => 5],
+    ['id' => 81, 'name' => 'arguing', 'category_id' => 5],
+    ['id' => 82, 'name' => 'irrationality', 'category_id' => 5],
+    ['id' => 83, 'name' => 'can\'t handle criticism', 'category_id' => 5],
+    ['id' => 84, 'name' => 'defensive', 'category_id' => 5],
+    ['id' => 85, 'name' => 'people are avoiding you', 'category_id' => 5],
+    ['id' => 86, 'name' => 'having to be right', 'category_id' => 5],
+    ['id' => 87, 'name' => 'digestive problems', 'category_id' => 5],
+    ['id' => 88, 'name' => 'headaches', 'category_id' => 5],
+    ['id' => 89, 'name' => 'obsessive (stuck) thoughts', 'category_id' => 5],
+    ['id' => 90, 'name' => 'can\'t forgive', 'category_id' => 5],
+    ['id' => 91, 'name' => 'feeling grandiose (superior)', 'category_id' => 5],
+    ['id' => 92, 'name' => 'intimidation', 'category_id' => 5],
+    ['id' => 93, 'name' => 'feeling aggressive', 'category_id' => 5],
+    ['id' => 94, 'name' => 'depressed', 'category_id' => 6],
+    ['id' => 95, 'name' => 'panicked', 'category_id' => 6],
+    ['id' => 96, 'name' => 'confused', 'category_id' => 6],
+    ['id' => 97, 'name' => 'hopeless', 'category_id' => 6],
+    ['id' => 98, 'name' => 'sleeping too much or too little', 'category_id' => 6],
+    ['id' => 99, 'name' => 'can\'t cope', 'category_id' => 6],
     ['id' => 100, 'name' => 'overwhelmed', 'category_id' => 6],
     ['id' => 101, 'name' => 'crying for "no reason"', 'category_id' => 6],
     ['id' => 102, 'name' => 'can\'t think', 'category_id' => 6],
@@ -206,7 +206,7 @@ 
      */
     public static function getBehavior($key, $val)
     {
-        $ret = array_values(array_filter(self::$behaviors, function ($bvr) use ($key, $val) {
+        $ret = array_values(array_filter(self::$behaviors, function($bvr) use ($key, $val) {
             return $bvr[$key] === $val;
         }));
         return $ret ? $ret[0] : null;

common/tests/unit/components/GraphTest.php

Spacing +4 added lines, -4 removed lines

@@ -14,8 +14,8 @@ 
     use \Codeception\Specify;
 
     private $user;
-    private $filepath = __DIR__.'/../../_output/test_graph.png';
-    private $filepath_extra = __DIR__.'/../../_output/charts/test_graph.png';
+    private $filepath = __DIR__ . '/../../_output/test_graph.png';
+    private $filepath_extra = __DIR__ . '/../../_output/charts/test_graph.png';
 
     public function setUp(): void
     {
@@ -39,7 +39,7 @@ 
       ->willReturn('random1DH4sh');
         $graph = new Graph($this->user);
 
-        expect('the expected graph image filepath will be returned', $this->assertEquals(dirname(dirname(dirname(dirname(__DIR__)))).'/site/web/charts/random1DH4sh.png', $graph->getFilepath()));
+        expect('the expected graph image filepath will be returned', $this->assertEquals(dirname(dirname(dirname(dirname(__DIR__)))) . '/site/web/charts/random1DH4sh.png', $graph->getFilepath()));
     }
 
     public function testGetUrl()
@@ -237,7 +237,7 @@ 
       '2019-02-27' => [],
       '2019-02-28' => [],
       '2019-03-01' => [
-        2 => [ 'name' => 'Forgetting Priorities',
+        2 => ['name' => 'Forgetting Priorities',
         'count' => 6,
         'color' => '#4CA100',
         'highlight' => '#61B219',

common/tests/unit/components/UtilityTest.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
     public function testGetRevHash()
     {
-        $this->specify('getRevHash should function correctly', function () {
+        $this->specify('getRevHash should function correctly', function() {
             expect('getRevHash should return false when the file does not exist', $this->assertFalse(Utility::getRevHash()));
 
             $this->_createRevFile();

common/tests/unit/components/TimeTest.php

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
         $this->container->set('common\interfaces\UserBehaviorInterface', '\site\tests\_support\MockUserBehavior');
         $this->container->set('common\interfaces\QuestionInterface', '\site\tests\_support\MockQuestion');
 
-        $this->container->set('common\interfaces\TimeInterface', function () {
+        $this->container->set('common\interfaces\TimeInterface', function() {
             return new \common\components\Time('America/Los_Angeles');
         });
 
@@ -47,7 +47,7 @@ 
 
     public function testGetLocalTime()
     {
-        $this->specify('getLocalTime should function correctly', function () {
+        $this->specify('getLocalTime should function correctly', function() {
             expect("getLocalTime should work with user's set time", $this->assertEquals($this->time->getLocalTime(), (new DateTime("now", new DateTimeZone("America/Los_Angeles")))->format("Y-m-d H:i:s")));
             expect('getLocalTime should work with a custom timezone', $this->assertEquals($this->time->getLocalTime("UTC"), (new DateTime("now"))->format("Y-m-d H:i:s")));
         });
@@ -55,13 +55,13 @@ 
 
     public function testConvertLocalToUTC()
     {
-        $this->specify('convertLocalToUTC should function correctly', function () {
+        $this->specify('convertLocalToUTC should function correctly', function() {
             $la_tz = (new DateTime("now", new DateTimeZone("America/Los_Angeles")))->format("Y-m-d H:i:s");
 
             expect('convertLocalToUTC should convert a Los Angeles tz to UTC with the included time', $this->assertEquals($this->time->convertLocalToUTC($la_tz), (new DateTime("now"))->format("Y-m-d H:i:s")));
             expect('convertLocalToUTC should convert a Los Angeles tz to UTC without the included time', $this->assertEquals($this->time->convertLocalToUTC($la_tz, false), (new DateTime("now"))->format("Y-m-d")));
             // with UTC
-            $this->container->set('common\interfaces\TimeInterface', function () {
+            $this->container->set('common\interfaces\TimeInterface', function() {
                 return new \common\components\Time('UTC');
             });
             $time = $this->container->get('common\interfaces\TimeInterface');
@@ -74,14 +74,14 @@ 
 
     public function testConvertUTCToLocal()
     {
-        $this->specify('convertUTCToLocal should function correctly', function () {
+        $this->specify('convertUTCToLocal should function correctly', function() {
             $utc_tz = (new DateTime("now"))->format("Y-m-d H:i:s");
 
             expect('convertUTCToLocal should convert a UTC tz to Los Angeles with the included timezone', $this->assertEquals((new DateTime("now", new DateTimeZone("America/Los_Angeles")))->format(DateTime::ATOM), $this->time->convertUTCToLocal($utc_tz)));
             expect('convertUTCToLocal should convert a UTC tz to Los Angeles without the included timezone', $this->assertEquals($this->time->convertUTCToLocal($utc_tz, false), (new DateTime("now", new DateTimeZone("America/Los_Angeles")))->format("Y-m-d H:i:s")));
 
             // with UTC
-            $this->container->set('common\interfaces\TimeInterface', function () {
+            $this->container->set('common\interfaces\TimeInterface', function() {
                 return new \common\components\Time('UTC');
             });
             $time = $this->container->get('common\interfaces\TimeInterface');
@@ -94,7 +94,7 @@ 
 
     public function testGetLocalDate()
     {
-        $this->specify('getLocalDate should function correctly', function () {
+        $this->specify('getLocalDate should function correctly', function() {
             expect("getLocalDate should correctly get the user's local date", $this->assertEquals($this->time->getLocalDate(), (new DateTime("now", new DateTimeZone("America/Los_Angeles")))->format("Y-m-d")));
             expect("getLocalDate should correctly get the local date of a specified timezone", $this->assertEquals($this->time->getLocalDate("UTC"), (new DateTime("now", new DateTimeZone("UTC")))->format("Y-m-d")));
         });
@@ -102,7 +102,7 @@ 
 
     public function testAlterLocalDate()
     {
-        $this->specify('alterLocalDate should function correctly', function () {
+        $this->specify('alterLocalDate should function correctly', function() {
             expect('alterLocalDate should add one day to the local time', $this->assertEquals($this->time->alterLocalDate('2016-05-01 00:00:00', '+1 day'), (new DateTime("2016-05-01 00:00:00 +1 day"))->format("Y-m-d")));
             expect('alterLocalDate should subtract one week to the local time and change the year correctly', $this->assertEquals($this->time->alterLocalDate('2016-01-01 04:03:00', '-1 week'), (new DateTime("2016-01-01 04:03:00 -1 week"))->format("Y-m-d")));
         });
@@ -110,13 +110,13 @@ 
 
     public function testGetUTCBookends()
     {
-        $this->specify('getUTCBookends should function correctly', function () {
+        $this->specify('getUTCBookends should function correctly', function() {
             expect('getUTCBookends should return false if there is a space in the time string', $this->assertFalse($this->time->getUTCBookends('2016-05-30 00:00:00')));
             expect('getUTCBookends should return false if there is a space at the start of the time string', $this->assertFalse($this->time->getUTCBookends(' 2016-05-30 00:00:00')));
             expect('getUTCBookends should return false if there is a space at the end of the time string', $this->assertFalse($this->time->getUTCBookends('2016-05-30 00:00:00 ')));
             expect('getUTCBookends should return UTC bookend times from the Los_Angeles tz', $this->assertEquals($this->time->getUTCBookends('2016-05-30'), ['2016-05-30 07:00:00', '2016-05-31 06:59:59']));
             // with UTC
-            $this->container->set('common\interfaces\TimeInterface', function () {
+            $this->container->set('common\interfaces\TimeInterface', function() {
                 return new \common\components\Time('UTC');
             });
             $time = $this->container->get('common\interfaces\TimeInterface');
@@ -153,14 +153,14 @@ 
         expect('parse should return the default value (false) if the date is not in bounds', $this->assertFalse($observer2->parse('2016-05-05')));
         expect('parse should return the default value if the date is not in bounds', $this->assertEquals($observer2->now()->format('Y-m-d'), $observer2->parse('2016-05-05', $observer2->now())->format('Y-m-d')));
 
-        $this->specify('should all return false', function () {
+        $this->specify('should all return false', function() {
             expect('parse should return false if the date itself is empty', $this->assertFalse($this->time->parse('')));
             expect('parse should return false if the date itself is unacceptable or is in an unacceptable format', $this->assertFalse($this->time->parse('aaaa-aa-aa')));
             expect('parse should return false if the date is not in bounds', $this->assertFalse($this->time->parse('aaaa-aa-aa')));
             expect('parse should return false if the date itself is nonsensical', $this->assertFalse($this->time->parse('2018-22-44')));
         });
 
-        $this->specify('should still work fine with other timezones', function () {
+        $this->specify('should still work fine with other timezones', function() {
             $this->time->timezone = 'UTC';
             $good = new DateTime('2016-05-05', new DateTimeZone('UTC'));
             expect('parse should accept a String time and verify it is in YYYY-MM-DD format, then return it as a \DateTime', $this->assertEquals($this->time->parse('2016-05-05'), $good));
@@ -205,11 +205,11 @@ 
         $tz = 'America/Los_Angeles';
         $this->time->timezone = 'America/Los_Angeles';
 
-        $today =  new \DateTime("now + 1 day", new \DateTimeZone($tz)); // we do an extra day to include today as the last day
+        $today = new \DateTime("now + 1 day", new \DateTimeZone($tz)); // we do an extra day to include today as the last day
         $should_be_today = $this->time->getDateTimesInPeriod()->getEndDate();
         expect('getDateTimesInPeriod should return a list of \DatePeriods, the last being for today', $this->assertEquals($today->format('Y-m-d H'), $should_be_today->format('Y-m-d H')));
 
-        $start =  new \DateTime("30 days ago", new \DateTimeZone($tz));
+        $start = new \DateTime("30 days ago", new \DateTimeZone($tz));
         $should_be_start = $this->time->getDateTimesInPeriod()->getStartDate();
         expect('getDateTimesInPeriod should return a list of \DatePeriods, the first being for 30 days ago (by default)', $this->assertEquals($start->format('Y-m-d H'), $should_be_start->format('Y-m-d H')));
     }

common/tests/unit/models/UserBehaviorTest.php

Spacing +4 added lines, -4 removed lines

@@ -281,7 +281,7 @@ 
     public function setUp(): void
     {
         // pull in test data
-        $data = require(__DIR__.'/../data/checkinData.php');
+        $data = require(__DIR__ . '/../data/checkinData.php');
         $this->singleBhvr = $data['singleBhvr'];
         $this->manyBhvrs = $data['manyBhvrs'];
         $this->allBhvrs = $data['allBhvrs'];
@@ -291,7 +291,7 @@ 
         $this->container->set('common\interfaces\UserInterface', '\site\tests\_support\MockUser');
         $this->container->set('common\interfaces\QuestionInterface', '\site\tests\_support\MockQuestion');
         $this->container->set('common\interfaces\BehaviorInterface', 'common\models\Behavior');
-        $this->container->set('common\interfaces\TimeInterface', function () {
+        $this->container->set('common\interfaces\TimeInterface', function() {
             return new \common\components\Time('America/Los_Angeles');
         });
 
@@ -461,8 +461,8 @@ 
                             ->setMethods(['getIsNewRecord', 'save', 'getBehaviorsWithCounts'])
                             ->getMock();
 
-        $bhvrs = require(__DIR__.'/../data/behaviorsWithCounts.php');
-        $expected = require(__DIR__.'/../data/expected_getCheckinBreakdown.php');
+        $bhvrs = require(__DIR__ . '/../data/behaviorsWithCounts.php');
+        $expected = require(__DIR__ . '/../data/expected_getCheckinBreakdown.php');
         $this->user_behavior->method('getBehaviorsWithCounts')->willReturn(...$bhvrs);
         expect('asdf', $this->assertEquals($expected, $this->user_behavior->getCheckinBreakdown()));
     }

common/tests/unit/models/LoginFormTest.php

Spacing +5 added lines, -5 removed lines

@@ -111,14 +111,14 @@ 
 
     public function testGetUser()
     {
-        $this->specify('getUser shougd function correctly', function () {
+        $this->specify('getUser shougd function correctly', function() {
             expect('getUser should return the user if the user attr on the form object is already set', $this->assertInstanceOf('\common\models\User', $this->form->getUser()));
         });
     }
 
     public function testValidatePassword()
     {
-        $this->specify('validatePassword should function correctly', function () {
+        $this->specify('validatePassword should function correctly', function() {
             expect('validatePassword should always return null', $this->assertNull($this->form->validatePassword()));
             expect('validatePassword should by default in this test class not set errors', $this->assertEmpty($this->form->getErrors()));
 
@@ -130,13 +130,13 @@ 
     }
     public function testLogin()
     {
-        $this->specify('login should function correctly', function () {
+        $this->specify('login should function correctly', function() {
             $this->form->email = 'hunter2@hunter2.com';
             $this->form->password = 'hunter2';
             expect('login should return true if able to log in user', $this->assertTrue($this->form->login()));
         });
 
-        $this->specify('login should fail if account is not verified', function () {
+        $this->specify('login should fail if account is not verified', function() {
             $this->user = $this->mockUser(['isVerified' => false]);
             $this->form = $this->mockForm($this->user);
             $this->form->email = 'hunter2@hunter2.com';
@@ -146,7 +146,7 @@ 
             expect('login should set a flash message if account is not verified', $this->assertNotEmpty(Yii::$app->session->getFlash('warning', null, true)));
         });
 
-        $this->specify('login should fail if credentials are not valid', function () {
+        $this->specify('login should fail if credentials are not valid', function() {
             $this->user = $this->mockUser(['validatePassword' => false]);
             $this->form = $this->mockForm($this->user);
             $this->form->email = 'hunter2@hunter2.com';

common/tests/unit/models/QuestionTest.php

Indentation +4 added lines, -4 removed lines

@@ -159,16 +159,16 @@
     public function setUp(): void
     {
         $this->question = $this->getMockBuilder('\common\models\Question')
-         ->setMethods(['save', 'attributes'])
-         ->getMock();
+          ->setMethods(['save', 'attributes'])
+          ->getMock();
     }
 
     public function testParseQuestionData()
     {
         $questions = array_map(function ($d) {
             $q = $this->getMockBuilder('\common\models\Question')
-         ->setMethods(['save', 'attributes'])
-         ->getMock();
+          ->setMethods(['save', 'attributes'])
+          ->getMock();
             $q->method('save')->willReturn(true);
             $q->method('attributes')
         ->willReturn([

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
 
     public function testParseQuestionData()
     {
-        $questions = array_map(function ($d) {
+        $questions = array_map(function($d) {
             $q = $this->getMockBuilder('\common\models\Question')
          ->setMethods(['save', 'attributes'])
          ->getMock();