Add warning for intervening audits on distribution edit

app/controllers/distributions_controller.rb

@@ -141,6 +141,9 @@ def edit
       @distribution.line_items.build if @distribution.line_items.size.zero?
       @items = current_organization.items.alphabetized
       @storage_locations = current_organization.storage_locations.active_locations.has_inventory_items.alphabetized
+      @audit_warning = current_organization.audits
+        .where(storage_location_id: @distribution.storage_location_id)
+        .where("updated_at > ?", @distribution.created_at).any?
     else
       redirect_to distributions_path, error: 'To edit a distribution,
       you must be an organization admin or the current date must be later than today.'

app/views/distributions/edit.html.erb

@@ -27,12 +27,20 @@
 <section class="content">
 
   <% unless @distribution.future? %>
-    <div class="alert alert-warning" , role="alert">
+    <div class="alert alert-warning" role="alert">
       The current date is past the date this distribution was scheduled for.
       Please be very careful when editing this record;
       the contents should match what was given to the recipient.
     </div>
   <% end %>
+  <% if @audit_warning %>
+    <div class="alert alert-warning" role="alert">
+      You’ve had an audit since this distribution was started.
+      In the rare case that you are correcting a typo,
+      rather than recording that the physical amounts being distributed have changed,
+      you’ll need to make an adjustment to the inventory as well.
+    </div>
+  <% end %>
 
   <div class="container-fluid">
     <div class="row">
@@ -48,7 +56,8 @@
               <div class="box-body">
                 <%= render 'form', distribution: @distribution, date_place_holder: @distribution.issued_at %>
               </div>
-          </div><!-- /.box -->
+            </div><!-- /.box -->
+          </div>
         </div>
       </div>
     </div>

spec/requests/distributions_requests_spec.rb

@@ -332,6 +332,33 @@
         end
       end
     end
+
+    describe "GET #edit" do
+      let(:location) { create(:storage_location) }
+      let(:partner) { create(:partner) }
+
+      let(:distribution) { create(:distribution, partner: partner) }
+
+      it "should show the distribution" do
+        get edit_distribution_path(default_params.merge(id: distribution.id))
+        expect(response).to be_successful
+        expect(response.body).not_to include("You’ve had an audit since this distribution was started.")
+      end
+
+      it "should show a warning if there is an inteverning audit" do
+        distribution.update!(created_at: 1.week.ago)
+        create(:audit, storage_location: distribution.storage_location)
+        get edit_distribution_path(default_params.merge(id: distribution.id))
+        expect(response.body).to include("You’ve had an audit since this distribution was started.")
+      end
+
+      it "should not show a warning if the audit is for another location" do
+        distribution.update!(created_at: 1.week.ago)
+        create(:audit, storage_location: create(:storage_location))
+        get edit_distribution_path(default_params.merge(id: distribution.id))
+        expect(response.body).not_to include("You’ve had an audit since this distribution was started.")
+      end
+    end
   end
 
   context "While not signed in" do