From e8f046f02e3d6d2225bcf10628357406bfc1deee Mon Sep 17 00:00:00 2001
From: radhu587 <makwanar587@gmail.com>
Date: Mon, 22 Jun 2020 17:07:35 +0530
Subject: [PATCH] refactor factories

---
 database/factories/CompanyFactory.php       | 13 +++++++++++++
 database/factories/EstimateFactory.php      |  8 ++++----
 database/factories/InvoiceFactory.php       |  8 ++++----
 database/factories/PaymentFactory.php       |  7 ++++---
 database/factories/PaymentMethodFactory.php | 14 ++++++++++++++
 5 files changed, 39 insertions(+), 11 deletions(-)
 create mode 100644 database/factories/CompanyFactory.php
 create mode 100644 database/factories/PaymentMethodFactory.php

diff --git a/database/factories/CompanyFactory.php b/database/factories/CompanyFactory.php
new file mode 100644
index 00000000..64813a49
--- /dev/null
+++ b/database/factories/CompanyFactory.php
@@ -0,0 +1,13 @@
+<?php
+
+/** @var \Illuminate\Database\Eloquent\Factory $factory */
+
+use Crater\Company;
+use Faker\Generator as Faker;
+
+$factory->define(Company::class, function (Faker $faker) {
+    return [
+        'unique_hash' => str_random(60),
+        'name' => $faker->name
+    ];
+});
diff --git a/database/factories/EstimateFactory.php b/database/factories/EstimateFactory.php
index 6c35626d..be9108c9 100644
--- a/database/factories/EstimateFactory.php
+++ b/database/factories/EstimateFactory.php
@@ -11,10 +11,10 @@ use Crater\EstimateTemplate;
 
 $factory->define(Estimate::class, function (Faker $faker) {
     return [
-        'estimate_date' => $faker->date($format = 'd/m/Y', $max = 'now'),
-        'expiry_date' => $faker->date($format = 'd/m/Y', $max = 'now'),
-        'estimate_number' => 'EST-'.Estimate::getNextEstimateNumber(),
-        'reference_number' => Estimate::getNextEstimateNumber(),
+        'estimate_date' => $faker->date($format = 'Y-m-d', $max = 'now'),
+        'expiry_date' => $faker->date($format = 'Y-m-d', $max = 'now'),
+        'estimate_number' => 'EST-'.Estimate::getNextEstimateNumber('EST'),
+        'reference_number' => Estimate::getNextEstimateNumber('EST'),
         'company_id' => User::find(1)->company_id,
         'user_id' => function () {
             return factory(User::class)->create(['role' => 'customer'])->id;
diff --git a/database/factories/InvoiceFactory.php b/database/factories/InvoiceFactory.php
index c3b13d00..2fcc6aa3 100644
--- a/database/factories/InvoiceFactory.php
+++ b/database/factories/InvoiceFactory.php
@@ -11,10 +11,10 @@ use Faker\Generator as Faker;
 
 $factory->define(Invoice::class, function (Faker $faker) {
     return [
-        'invoice_date' => $faker->date($format = 'd/m/Y', $max = 'now'),
-        'due_date' => $faker->date($format = 'd/m/Y', $max = 'now'),
-        'invoice_number' => 'INV-'.Invoice::getNextInvoiceNumber(),
-        'reference_number' => Invoice::getNextInvoiceNumber(),
+        'invoice_date' => $faker->date($format = 'Y-m-d', $max = 'now'),
+        'due_date' => $faker->date($format = 'Y-m-d', $max = 'now'),
+        'invoice_number' => 'INV-'.Invoice::getNextInvoiceNumber('INV'),
+        'reference_number' => Invoice::getNextInvoiceNumber('INV'),
         'user_id' => function () {
             return factory(User::class)->create(['role' => 'customer'])->id;
         },
diff --git a/database/factories/PaymentFactory.php b/database/factories/PaymentFactory.php
index 4e09cd9e..bd4ec5b1 100644
--- a/database/factories/PaymentFactory.php
+++ b/database/factories/PaymentFactory.php
@@ -3,7 +3,9 @@
 /** @var \Illuminate\Database\Eloquent\Factory $factory */
 
 use Crater\Payment;
+use Crater\PaymentMethod;
 use Crater\User;
+use Crater\Invoice;
 use Faker\Generator as Faker;
 
 $factory->define(Payment::class, function (Faker $faker) {
@@ -11,11 +13,10 @@ $factory->define(Payment::class, function (Faker $faker) {
         'user_id' => function () {
             return factory(User::class)->create(['role' => 'customer'])->id;
         },
-        'payment_date' => $faker->date($format = 'd/m/Y', $max = 'now'),
+        'payment_date' => $faker->date($format = 'Y-m-d', $max = 'now'),
         'company_id' => User::find(1)->company_id,
         'notes' => $faker->text(80),
         'amount' => $faker->randomDigitNotNull,
-        'payment_number' => 'PAY-'.Payment::getNextPaymentNumber(),
-        'payment_mode' => 'OTHER'
+        'payment_number' => 'PAY-'.Payment::getNextPaymentNumber('PAY'),
     ];
 });
diff --git a/database/factories/PaymentMethodFactory.php b/database/factories/PaymentMethodFactory.php
new file mode 100644
index 00000000..c2093a5f
--- /dev/null
+++ b/database/factories/PaymentMethodFactory.php
@@ -0,0 +1,14 @@
+<?php
+
+/** @var \Illuminate\Database\Eloquent\Factory $factory */
+
+use Crater\PaymentMethod;
+use Crater\User;
+use Faker\Generator as Faker;
+
+$factory->define(PaymentMethod::class, function (Faker $faker) {
+    return [
+        'name' => $faker->name,
+        'company_id' => User::find(1)->company_id,
+    ];
+});