Robustness improvement we can submit repo source files one-by-one, Shoes visualization still WIP

Author: Laszlo-JFLMTCO

.DS_Store

@@ -1,4 +1,3 @@
-require 'pry'
 require_relative 'calculator'
 
 #Colection of methods used for Merchant related

lib/merchant_analyst.rb

@@ -7,7 +7,6 @@
 require_relative 'sales_engine_relations'
 require_relative 'sales_engine_grouping'
 require_relative 'sales_engine_create'
-require 'pry'
 
 #Support for Sales Analyst layer, connecting Sales Analyst and
 #various repositories with each other
@@ -24,13 +23,21 @@ class SalesEngine
                 :transactions
 
   def initialize(files)
+    file_loader(files)
+  end
+
+  def from_csv(files)
+    file_loader(files)
+  end
+
+  def file_loader(files)
     raise "Please enter a valid file name" unless valid?(files)
-    @items = create_item_repository(files)
-    @merchants = create_merchants_repository(files)
-    @invoices = create_invoices_repository(files)
-    @invoice_items = create_invoice_item_repository(files)
-    @transactions = create_transaction_repository(files)
-    @customers = create_customer_repository(files)
+    @items = create_item_repo(files) if files[:items]
+    @invoices = create_invoices_repo(files) if files[:invoices]
+    @invoice_items = create_invoice_item_repo(files) if files[:invoice_items]
+    @transactions = create_transaction_repo(files) if files[:transactions]
+    @customers = create_customer_repo(files) if files[:customers]
+    @merchants = create_merchants_repo(files) if files[:merchants]
   end
 
   def self.from_csv(files)
@@ -39,7 +46,6 @@ def self.from_csv(files)
 
   def valid?(paths)
     paths.values.all? do |path|
-      #binding.pry
       File.exist?(path)
     end
   end

lib/sales_engine.rb

@@ -1,37 +1,37 @@
 #Collection of methods used for Sales Engine repository creation
 module SalesEngineCreate
 
-  def create_item_repository(files)
+  def create_item_repo(files)
     if files.include?(:items)
       ItemRepository.new(files[:items], self)
     end
   end
 
-  def create_merchants_repository(files)
+  def create_merchants_repo(files)
     if files.include?(:merchants)
       MerchantRepository.new(files[:merchants], self)
     end
   end
 
-  def create_invoices_repository(files)
+  def create_invoices_repo(files)
     if files.include?(:invoices)
       InvoiceRepository.new(files[:invoices], self)
     end
   end
 
-  def create_invoice_item_repository(files)
+  def create_invoice_item_repo(files)
     if files.include?(:invoice_items)
       InvoiceItemRepository.new(files[:invoice_items], self)
     end
   end
 
-  def create_transaction_repository(files)
+  def create_transaction_repo(files)
     if files.include?(:transactions)
       TransactionRepository.new(files[:transactions], self)
     end
   end
 
-  def create_customer_repository(files)
+  def create_customer_repo(files)
     if files.include?(:customers)
       CustomerRepository.new(files[:customers], self)
     end

lib/sales_engine_create.rb

@@ -1,8 +1,6 @@
 require_relative 'test_helper'
 require './lib/sales_analyst'
 require './lib/sales_engine'
-require 'pry'
-
 
 class InvoiceAnalystTest < MiniTest::Test
 

test/invoice_analyst_test.rb

@@ -1,11 +1,11 @@
 require_relative 'test_helper'
 require './lib/sales_engine'
 
-
 class SalesEngineTest < Minitest::Test
 
   attr_reader   :se,
-                :test_magic_file_list
+                :test_magic_file_list,
+                :one_test_file
 
   def setup
     test_file_list = {:merchants => "./data/merchants_fixture.csv",
@@ -19,7 +19,8 @@ def setup
                     :invoices => "./data/invoices_magic.csv",
                     :invoice_items => "./data/invoice_items_magic.csv",
                     :transactions => "./data/transactions_magic.csv",
-                    :customers => "./data/customers_magic.csv"}                    
+                    :customers => "./data/customers_magic.csv"}
+    @one_test_file = {:merchants => "./data/merchants_fixture.csv"}
     @se = SalesEngine.from_csv(test_file_list)
   end
 
@@ -29,6 +30,16 @@ def test_with_non_existing_csv
     end
   end
 
+  def test_updating_single_repository
+    test_object_id_before = se.object_id
+    test_merchants_repo_object_id_before = se.merchants.object_id
+    test_items_repo_object_id_before = se.items.object_id
+    se.from_csv(one_test_file)
+    assert_equal test_object_id_before, se.object_id
+    refute test_merchants_repo_object_id_before == se.merchants.object_id
+    assert_equal test_items_repo_object_id_before, se.items.object_id
+  end
+
   def test_merchant_repository_exists
     assert se.merchants    
   end

test/sales_engine_test.rb

@@ -3,8 +3,6 @@
 require 'minitest/autorun'
 require './lib/transaction'
 
-require 'pry'
-
 class TransactionTest < MiniTest::Test
 
   attr_reader     :transaction_info,