3430 add item quantities to storage location export (#3516)

Author: kev-kev

app/controllers/storage_locations_controller.rb

@@ -18,9 +18,20 @@ def index
       @storage_locations = @storage_locations.kept
     end
 
+    active_inventory_item_names = []
+    @storage_locations.each do |storage_location|
+      active_inventory_item_names <<
+        storage_location
+        .active_inventory_items
+        .joins(:item)
+        .select('distinct items.name')
+        .pluck(:name)
+    end
+    active_inventory_item_names = active_inventory_item_names.flatten.uniq.sort
+
     respond_to do |format|
       format.html
-      format.csv { send_data StorageLocation.generate_csv(@storage_locations), filename: "StorageLocations-#{Time.zone.today}.csv" }
+      format.csv { send_data StorageLocation.generate_csv(@storage_locations, active_inventory_item_names), filename: "StorageLocations-#{Time.zone.today}.csv" }
     end
   end
 

app/models/storage_location.rb

@@ -81,6 +81,12 @@ def size
     inventory_items.sum(:quantity)
   end
 
+  def total_active_inventory_count
+    active_inventory_items
+    .select('items.quantity')
+    .sum(:quantity)
+  end
+
   def inventory_total_value_in_dollars
     inventory_total_value = inventory_items.joins(:item).map do |inventory_item|
       value_in_cents = inventory_item.item.try(:value_in_cents)
@@ -211,10 +217,18 @@ def self.csv_export_headers
   end
 
   def csv_export_attributes
-    [name, address, square_footage, warehouse_type, size]
+    attributes = [name, address, square_footage, warehouse_type, total_active_inventory_count]
+    active_inventory_items.sort_by { |inv_item| inv_item.item.name }.each { |item| attributes << item.quantity }
+    attributes
   end
 
   def empty_inventory_items?
     inventory_items.map(&:quantity).uniq.reject(&:zero?).empty?
   end
+
+  def active_inventory_items
+    inventory_items
+    .includes(:item)
+    .where(items: { active: true })
+  end
 end

spec/models/storage_location_spec.rb

@@ -228,5 +228,28 @@
         expect(storage_location.longitude).not_to eq(nil)
       end
     end
+
+    describe "csv_export_attributes" do
+      it "returns an array of storage location attributes, followed by inventory item quantities that are sorted by alphabetized item names" do
+        item1 = create(:item, name: "C")
+        item2 = create(:item, name: "B")
+        item3 = create(:item, name: "A")
+        inactive_item = create(:item, name: "inactive item", active: false)
+        name = "New Storage Location"
+        address = "1500 Remount Road, Front Royal, VA 22630"
+        warehouse_type = "Warehouse with loading bay"
+        square_footage = rand(1000..10000)
+        storage_location = create(:storage_location, name: name, address: address, warehouse_type: warehouse_type, square_footage: square_footage)
+        quantity1 = rand(100..1000)
+        quantity2 = rand(100..1000)
+        quantity3 = rand(100..1000)
+        create(:inventory_item, storage_location_id: storage_location.id, item_id: item1.id, quantity: quantity1)
+        create(:inventory_item, storage_location_id: storage_location.id, item_id: item2.id, quantity: quantity2)
+        create(:inventory_item, storage_location_id: storage_location.id, item_id: item3.id, quantity: quantity3)
+        create(:inventory_item, storage_location_id: storage_location.id, item_id: inactive_item.id, quantity: 1)
+        sum = quantity1 + quantity2 + quantity3
+        expect(storage_location.csv_export_attributes).to eq([name, address, square_footage, warehouse_type, sum, quantity3, quantity2, quantity1])
+      end
+    end
   end
 end

spec/requests/storage_locations_requests_spec.rb

@@ -38,11 +38,26 @@
 
       context "csv" do
         let(:response_format) { 'csv' }
-
         it "succeeds" do
           get storage_locations_path(default_params.merge(format: response_format))
           expect(response).to be_successful
         end
+
+        it "includes headers followed by alphabetized item names" do
+          storage_location_with_items = create(:storage_location)
+          item1 = create(:item, name: 'C')
+          item2 = create(:item, name: 'B')
+          item3 = create(:item, name: 'A')
+          create(:item, name: 'inactive item', active: false)
+          Item.last(4).each { |item| create(:inventory_item, storage_location_id: storage_location_with_items.id, item_id: item.id, quantity: 1) }
+
+          storage_location_with_duplicate_item = create(:storage_location)
+          create(:inventory_item, storage_location_id: storage_location_with_duplicate_item.id, item_id: item3.id, quantity: 1)
+
+          get storage_locations_path(default_params.merge(format: response_format))
+
+          expect(response.body.split("\n")[0]).to eq([StorageLocation.csv_export_headers, item3.name, item2.name, item1.name].join(','))
+        end
       end
     end