Merge pull request #9 from roberts/drewroberts/feature/6-lead-businesses

Rename Lead Business #6

database/factories/WcBusinessFactory.php → database/factories/LeadBusinessFactory.php

@@ -2,23 +2,13 @@
 
 namespace Roberts\Leads\Database\Factories;
 
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 use Illuminate\Database\Eloquent\Factories\Factory;
 
-class WcBusinessFactory extends Factory
+class LeadBusinessFactory extends Factory
 {
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = WcBusiness::class;
+    protected $model = LeadBusiness::class;
 
-    /**
-     * Define the model's default state.
-     *
-     * @return array
-     */
     public function definition()
     {
         return [

database/factories/LeadFactory.php

@@ -3,28 +3,18 @@
 namespace Roberts\Leads\Database\Factories;
 
 use Illuminate\Database\Eloquent\Factories\Factory;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 use Roberts\Leads\Models\Lead;
 
 class LeadFactory extends Factory
 {
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
     protected $model = Lead::class;
 
-    /**
-     * Define the model's default state.
-     *
-     * @return array
-     */
     public function definition()
     {
         return [
             'email' => $this->faker->email,
-            'wc_business_id' => WcBusiness::factory(),
+            'lead_business_id' => LeadBusiness::factory(),
         ];
     }
 
@@ -39,7 +29,7 @@ public function withNullableFields()
                 'current_plan_under_cancellation' => $this->faker->boolean,
                 'current_plan_expires_at' => $this->faker->date,
                 'past_comp_claims' => $this->faker->paragraph,
-                'wc_business_id' => WcBusiness::factory(),
+                'lead_business_id' => LeadBusiness::factory(),
             ];
         });
     }

database/migrations/2021_01_01_100000_create_leads_table.php

@@ -3,7 +3,7 @@
 use Illuminate\Database\Migrations\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 
 class CreateLeadsTable extends Migration
 {
@@ -13,13 +13,13 @@ public function up()
             $table->id();
             $table->string('lead_number')->index()->unique(); // Generated by system. This is identifier used to communicate about the quote request & for additional form pages.
             $table->string('email');
-            $table->foreignIdFor(WcBusiness::class);
+            $table->foreignIdFor(LeadBusiness::class);
             $table->string('first_name')->nullable();
             $table->string('last_name')->nullable();
             $table->string('position')->nullable();
             $table->string('phone_number')->nullable(); // Will be replaced later with relationship to Phone model in tipoff/addresses
 
-            // Move these 3 fields to WcBusiness table
+            // Move these 3 fields to LeadBusiness table
             $table->boolean('current_plan_under_cancellation')->nullable();
             $table->date('current_plan_expires_at')->nullable();
             $table->text('past_comp_claims')->nullable();

database/migrations/2021_03_13_032312_create_wc_businesses_table.php → database/migrations/2021_01_01_110000_create_lead_businesses_table.php

@@ -4,11 +4,11 @@
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-class CreateWcBusinessesTable extends Migration
+class CreateLeadBusinessesTable extends Migration
 {
     public function up()
     {
-        Schema::create('wc_businesses', function (Blueprint $table) {
+        Schema::create('lead_businesses', function (Blueprint $table) {
             $table->id();
             $table->string('name');
             $table->string('nature')->nullable();

src/Livewire/OnboardingForm/Presentation.php

@@ -4,7 +4,7 @@
 
 use Roberts\Leads\Enums\OnboardingFormStep;
 use Roberts\Leads\Models\Lead;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 
 class Presentation extends OnboardingFormStepComponent
 {
@@ -22,11 +22,11 @@ public function render()
 
     public function processLead(array $data)
     {
-        $business = WcBusiness::create($data['business']);
+        $business = LeadBusiness::create($data['business']);
 
         return Lead::create([
             'email' => $data['email'],
-            'wc_business_id' => $business->id,
+            'lead_business_id' => $business->id,
         ]);
     }
 

src/Models/Lead.php

@@ -48,7 +48,7 @@ protected function generateLeadNumber(): string
 
     public function business()
     {
-        return $this->belongsTo(WcBusiness::class, 'wc_business_id');
+        return $this->belongsTo(LeadBusiness::class, 'lead_business_id');
     }
 
     public function payrollClassifications()

src/Models/WcBusiness.php → src/Models/LeadBusiness.php

@@ -5,9 +5,7 @@
 use Tipoff\Support\Models\BaseModel;
 use Tipoff\Support\Traits\HasPackageFactory;
 
-class WcBusiness extends BaseModel
+class LeadBusiness extends BaseModel
 {
     use HasPackageFactory;
-
-    protected $guarded = [];
 }

tests/Feature/LivewireComponents/BusinessDetailsTest.php

@@ -6,7 +6,7 @@
 use Livewire\Livewire;
 use Roberts\Leads\Livewire\OnboardingForm\BusinessDetails;
 use Roberts\Leads\Models\Lead;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 use Roberts\Leads\Tests\TestCase;
 
 class BusinessDetailsTest extends TestCase
@@ -16,10 +16,10 @@ class BusinessDetailsTest extends TestCase
     /** @test */
     public function a_user_can_add_business_details()
     {
-        $business = WcBusiness::factory()->create();
-        $lead = Lead::factory()->create(['wc_business_id' => $business->id]);
+        $business = LeadBusiness::factory()->create();
+        $lead = Lead::factory()->create(['lead_business_id' => $business->id]);
 
-        $attributes = WcBusiness::factory()
+        $attributes = LeadBusiness::factory()
             ->withNullableFields()
             ->raw(['name' => $business->name]);
 
@@ -31,7 +31,7 @@ public function a_user_can_add_business_details()
             ->set('attributes.year_of_establishment', $attributes['year_of_establishment'])
             ->call('submit');
 
-        $this->assertDatabaseHas('wc_businesses', $attributes);
+        $this->assertDatabaseHas('lead_businesses', $attributes);
     }
 
     /** @test */

tests/Feature/LivewireComponents/PresentationTest.php

@@ -26,13 +26,13 @@ public function a_user_can_create_a_lead()
             ->set('attributes.business.name', $attributes['business_name'])
             ->call('submit');
 
-        $this->assertDatabaseHas('wc_businesses', [
+        $this->assertDatabaseHas('lead_businesses', [
             'name' => $attributes['business_name'],
         ]);
 
         $this->assertDatabaseHas('leads', [
             'email' => $attributes['email'],
-            'wc_business_id' => 1,
+            'lead_business_id' => 1,
         ]);
     }
 

tests/Unit/Models/WcBusinessTest.php → tests/Unit/Models/LeadBusinessTest.php

@@ -4,10 +4,10 @@
 
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Foundation\Testing\WithFaker;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 use Roberts\Leads\Tests\TestCase;
 
-class WcBusinessTest extends TestCase
+class LeadBusinessTest extends TestCase
 {
     use RefreshDatabase;
     use WithFaker;
@@ -16,7 +16,7 @@ class WcBusinessTest extends TestCase
     public function it_has_a_name()
     {
         $name = $this->faker->company;
-        $business = WcBusiness::factory()->create(['name' => $name]);
+        $business = LeadBusiness::factory()->create(['name' => $name]);
 
         $this->assertEquals($name, $business->name);
     }
@@ -25,7 +25,7 @@ public function it_has_a_name()
     public function it_has_a_nature()
     {
         $nature = $this->faker->word;
-        $business = WcBusiness::factory()->create(['nature' => $nature]);
+        $business = LeadBusiness::factory()->create(['nature' => $nature]);
 
         $this->assertEquals($nature, $business->nature);
     }
@@ -34,7 +34,7 @@ public function it_has_a_nature()
     public function it_has_a_fein()
     {
         $fein = $this->faker->uuid;
-        $business = WcBusiness::factory()->create(['fein' => $fein]);
+        $business = LeadBusiness::factory()->create(['fein' => $fein]);
 
         $this->assertEquals($fein, $business->fein);
     }
@@ -43,7 +43,7 @@ public function it_has_a_fein()
     public function it_has_the_year_of_establishment()
     {
         $year = $this->faker->numberBetween(1900, 2010);
-        $business = WcBusiness::factory()->create(['year_of_establishment' => $year]);
+        $business = LeadBusiness::factory()->create(['year_of_establishment' => $year]);
 
         $this->assertEquals($year, $business->year_of_establishment);
     }
@@ -52,7 +52,7 @@ public function it_has_the_year_of_establishment()
     public function it_has_a_legal_entity_type()
     {
         $type = $this->faker->word;
-        $business = WcBusiness::factory()->create(['legal_entity_type' => $type]);
+        $business = LeadBusiness::factory()->create(['legal_entity_type' => $type]);
 
         $this->assertEquals($type, $business->legal_entity_type);
     }

tests/Unit/Models/LeadTest.php

@@ -6,7 +6,7 @@
 use Illuminate\Foundation\Testing\WithFaker;
 use Illuminate\Support\Carbon;
 use Roberts\Leads\Models\Lead;
-use Roberts\Leads\Models\WcBusiness;
+use Roberts\Leads\Models\LeadBusiness;
 use Roberts\Leads\Models\WcPayrollClassification;
 use Roberts\Leads\Tests\TestCase;
 
@@ -91,7 +91,7 @@ public function it_is_associated_with_a_business()
     {
         $lead = Lead::factory()->create();
 
-        $this->assertInstanceOf(WcBusiness::class, $lead->business);
+        $this->assertInstanceOf(LeadBusiness::class, $lead->business);
     }
 
     /** @test */