5428 add automated tests confirming vendor purchase order change

Author: janeewheatley

app/views/vendors/show.html.erb

@@ -76,7 +76,7 @@
               <tbody>
               <% @vendor.purchases.each do |purchase| %>
                 <tr>
-                  <td><%= purchase.issued_at.strftime("%m/%d/%Y") %></td>
+                  <td data-testid="purchase-date"><%= purchase.issued_at.strftime("%m/%d/%Y") %></td>
                   <td><%= purchase.line_items.total %></td>
                   <td><%= view_button_to purchase, { text: "View purchase details" } %></td>
                 </tr>

spec/models/vendor_spec.rb

@@ -56,6 +56,22 @@
     end
   end
 
+  context "Associations" do
+    describe "purchases" do
+      it "orders purchases by issued_at in descending order (newest first)" do
+        vendor = create(:vendor)
+        
+        # Create purchases with different issued_at dates
+        old_purchase = create(:purchase, vendor: vendor, issued_at: 1.week.ago)
+        new_purchase = create(:purchase, vendor: vendor, issued_at: 1.day.ago)
+        middle_purchase = create(:purchase, vendor: vendor, issued_at: 3.days.ago)
+        
+        # Verify they're ordered newest first
+        expect(vendor.purchases.to_a).to eq([new_purchase, middle_purchase, old_purchase])
+      end
+    end
+  end
+
   describe "versioning" do
     it { is_expected.to be_versioned }
   end

spec/system/vendor_system_spec.rb

@@ -130,5 +130,23 @@
       expect(page).to have_xpath("//table/tbody/tr", count: 3)
       expect(page).to have_xpath("//table/tbody/tr/td", text: "10")
     end
+
+    it "displays purchases in reverse chronological order by issued_at date" do
+      # Create purchases with specific issued_at dates  
+      old_purchase = create(:purchase, vendor: @vendor, issued_at: 1.week.ago)
+      new_purchase = create(:purchase, vendor: @vendor, issued_at: 1.day.ago)
+      middle_purchase = create(:purchase, vendor: @vendor, issued_at: 3.days.ago)
+  
+      visit vendor_path(@vendor)
+  
+      # Get all date text using the correct data attribute
+      dates = page.all("[data-testid='purchase-date']").map(&:text)
+  
+      # Get all purchases for this vendor and sort them by issued_at desc
+      all_purchases = @vendor.reload.purchases.order(issued_at: :desc)
+      expected_dates = all_purchases.map { |p| p.issued_at.strftime("%m/%d/%Y") }
+
+      expect(dates).to eq(expected_dates)
+    end
   end
 end