Added tests for everything.

Author: Mario Basic

app/Occurrences/OccurrenceCreator.php

@@ -8,6 +8,14 @@
 
 class OccurrenceCreator
 {
+    /**
+     * Creates a new occurrence that occurs at given date
+     * and associates it with given service.
+     *
+     * @param  Carbon $date
+     * @param  Service $service
+     * @return Occurrence
+     */
     public function create(Carbon $date, Service $service)
     {
         $occurrence = new Occurrence;
@@ -17,6 +25,8 @@ public function create(Carbon $date, Service $service)
         $occurrence->receipt_sent = false;
         $occurrence->service()->associate($service);
         $occurrence->save();
+
+        return $occurrence;
     }
 
 }

app/Service.php

@@ -51,13 +51,12 @@ public function occurrences()
     }
 
     /**
-     * Converts 112345 to 1.123,45.
+     * Converts 112345 to 1123,45.
      * For forms only.
      *
-     * @param  integer $value
      * @return string
      */
-    public function formCostAttribute($value)
+    public function formCostAttribute()
     {
         return number_format($this->cost / 100, 2, ',', '');
     }

tests/TestCase.php

@@ -29,6 +29,6 @@ public function setUp()
 
         // Avoid using external service to fetch quotes
         $rate = new \Swap\Model\Rate(1);
-        Swap::shouldReceive('quote')->andReturn($rate);
+        \Swap::shouldReceive('quote')->andReturn($rate);
     }
 }

tests/CategoryTest.php renamed to tests/functional/CategoryTest.php

@@ -1,19 +1,19 @@
-<?php
+<?php namespace Tests\Functional;
 
 use Illuminate\Foundation\Testing\WithoutMiddleware;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
 use Illuminate\Foundation\Testing\DatabaseTransactions;
 
-class CategoryTest extends TestCase
+class CategoryTest extends \TestCase
 {
     use DatabaseTransactions;
 
     /** @test */
     public function it_views_all_categories()
     {
-        factory(App\Category::class, 10)->create();
+        factory(\App\Category::class, 10)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/categories')
@@ -23,7 +23,7 @@ public function it_views_all_categories()
     /** @test */
     public function it_creates_a_new_category()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $name = 'Hosting';
 
@@ -42,9 +42,9 @@ public function it_creates_a_new_category()
     /** @test */
     public function it_updates_an_existing_category()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
-        $category = factory(App\Category::class)->create([
+        $category = factory(\App\Category::class)->create([
             'name' => 'Hostingg'
         ]);
 
@@ -64,9 +64,9 @@ public function it_updates_an_existing_category()
     /** @test */
     public function it_deletes_a_category()
     {
-        $category = factory(App\Category::class)->create();
+        $category = factory(\App\Category::class)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/categories')

tests/ClientTest.php renamed to tests/functional/ClientTest.php

@@ -1,19 +1,19 @@
-<?php
+<?php namespace Tests\Functional;
 
 use Illuminate\Foundation\Testing\WithoutMiddleware;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
 use Illuminate\Foundation\Testing\DatabaseTransactions;
 
-class ClientTest extends TestCase
+class ClientTest extends \TestCase
 {
     use DatabaseTransactions;
 
     /** @test */
     public function it_views_all_clients()
     {
-        factory(App\Client::class, 10)->create();
+        factory(\App\Client::class, 10)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/clients')
@@ -23,7 +23,7 @@ public function it_views_all_clients()
     /** @test */
     public function it_creates_a_new_client()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $name = 'Pastor';
         $tax_number = '12345678963';
@@ -50,9 +50,9 @@ public function it_creates_a_new_client()
     /** @test */
     public function it_updates_an_existing_client()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
-        $client = factory(App\Client::class)->create([
+        $client = factory(\App\Client::class)->create([
             'name' => 'Acme building'
         ]);
 
@@ -80,9 +80,9 @@ public function it_updates_an_existing_client()
     /** @test */
     public function it_deletes_a_client()
     {
-        $client = factory(App\Client::class)->create();
+        $client = factory(\App\Client::class)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/clients')

tests/functional/OverviewTest.php

@@ -0,0 +1,20 @@
+<?php namespace Tests\Functional;
+
+use Illuminate\Foundation\Testing\WithoutMiddleware;
+use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+
+class OverviewTest extends \TestCase
+{
+    use DatabaseTransactions;
+
+    /** @test */
+    public function it_views_overview_page()
+    {
+        $user = factory(\App\User::class)->create();
+
+        $this->actingAs($user)
+            ->visit('/')
+            ->see('Overview');
+    }
+}

tests/functional/ReportTest.php

@@ -0,0 +1,20 @@
+<?php namespace Tests\Functional;
+
+use Illuminate\Foundation\Testing\WithoutMiddleware;
+use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+
+class ReportTest extends \TestCase
+{
+    use DatabaseTransactions;
+
+    /** @test */
+    public function it_views_report_page()
+    {
+        $user = factory(\App\User::class)->create();
+
+        $this->actingAs($user)
+            ->visit('/report')
+            ->see('Report');
+    }
+}

tests/ServiceTest.php renamed to tests/functional/ServiceTest.php

@@ -1,19 +1,19 @@
-<?php
+<?php namespace Tests\Functional;
 
 use Illuminate\Foundation\Testing\WithoutMiddleware;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
 use Illuminate\Foundation\Testing\DatabaseTransactions;
 
-class ServiceTest extends TestCase
+class ServiceTest extends \TestCase
 {
     use DatabaseTransactions;
 
     /** @test */
     public function it_views_all_services()
     {
-        factory(App\Service::class, 10)->create();
+        factory(\App\Service::class, 10)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/services')
@@ -23,10 +23,10 @@ public function it_views_all_services()
     /** @test */
     public function it_creates_a_new_service()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
-        $category = factory(App\Category::class)->create();
-        $client = factory(App\Client::class)->create();
+        $category = factory(\App\Category::class)->create();
+        $client = factory(\App\Client::class)->create();
 
         $title = 'Basic hosting plan';
         $category_id = $category->id;
@@ -62,14 +62,14 @@ public function it_creates_a_new_service()
     /** @test */
     public function it_updates_an_existing_service()
     {
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
-        $service = factory(App\Service::class)->create([
+        $service = factory(\App\Service::class)->create([
             'title' => 'Some strange plan'
         ]);
 
-        $category = factory(App\Category::class)->create();
-        $client = factory(App\Client::class)->create();
+        $category = factory(\App\Category::class)->create();
+        $client = factory(\App\Client::class)->create();
 
         $title = 'Basic hosting plan';
         $category_id = $category->id;
@@ -104,9 +104,9 @@ public function it_updates_an_existing_service()
     /** @test */
     public function it_deletes_a_service()
     {
-        $service = factory(App\Service::class)->create();
+        $service = factory(\App\Service::class)->create();
 
-        $user = factory(App\User::class)->create();
+        $user = factory(\App\User::class)->create();
 
         $this->actingAs($user)
             ->visit('/services')

tests/functional/SettingsTest.php

@@ -0,0 +1,41 @@
+<?php namespace Tests\Functional;
+
+use Illuminate\Foundation\Testing\WithoutMiddleware;
+use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+
+class SettingsTest extends \TestCase
+{
+    use DatabaseTransactions;
+
+    /** @test */
+    public function it_updates_settings()
+    {
+        $user = factory(\App\User::class)->create([
+            'name' => 'Test Dummy',
+            'email' => '[email protected]',
+            'preferred_currency' => 'hrk',
+            'email_notifications' => false
+        ]);
+
+        $name = 'Not A Test Dummy';
+        $email = '[email protected]';
+        $preferred_currency = 'usd';
+
+        $this->actingAs($user)
+            ->visit('/settings')
+            ->see('Settings')
+            ->type($name, 'name')
+            ->type($email, 'email')
+            ->select($preferred_currency, 'preferred_currency')
+            ->check('email_notifications')
+            ->press('Update')
+            ->see('Settings updated!')
+            ->seeInDatabase('users', [
+                'name' => $name,
+                'email' => $email,
+                'preferred_currency' => $preferred_currency,
+                'email_notifications' => 1
+            ]);
+    }
+}

tests/models/OccurrenceTest.php

@@ -0,0 +1,52 @@
+<?php namespace Tests\Models;
+
+use Illuminate\Foundation\Testing\WithoutMiddleware;
+use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+
+class OccurrenceTest extends \TestCase
+{
+    use DatabaseTransactions;
+
+    /** @test */
+    public function it_gets_future_offer_state()
+    {
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'offer_sent' => true
+        ]);
+        $this->assertEquals(0, $occurrence->getFutureOfferState());
+
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'offer_sent' => false
+        ]);
+        $this->assertEquals(1, $occurrence->getFutureOfferState());
+    }
+
+    /** @test */
+    public function it_gets_future_payment_state()
+    {
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'payment_received' => true
+        ]);
+        $this->assertEquals(0, $occurrence->getFuturePaymentState());
+
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'payment_received' => false
+        ]);
+        $this->assertEquals(1, $occurrence->getFuturePaymentState());
+    }
+
+    /** @test */
+    public function it_gets_future_receipt_state()
+    {
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'receipt_sent' => true
+        ]);
+        $this->assertEquals(0, $occurrence->getFutureReceiptState());
+
+        $occurrence = factory(\App\Occurrence::class)->make([
+            'receipt_sent' => false
+        ]);
+        $this->assertEquals(1, $occurrence->getFutureReceiptState());
+    }
+}