From 6f36c0463c0dd80a985366016f1956d6bcc15ec1 Mon Sep 17 00:00:00 2001
From: Rohan Mitchell <rohan@rohanmitchell.com>
Date: Fri, 9 Jan 2015 11:02:00 +1100
Subject: [PATCH] Separate data preparation from test conditions

---
 .../user_balance_calculator_spec.rb           | 30 +++++++++++--------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/spec/lib/open_food_network/user_balance_calculator_spec.rb b/spec/lib/open_food_network/user_balance_calculator_spec.rb
index 9bb8b0bef2..5117a44aed 100644
--- a/spec/lib/open_food_network/user_balance_calculator_spec.rb
+++ b/spec/lib/open_food_network/user_balance_calculator_spec.rb
@@ -18,20 +18,26 @@ module OpenFoodNetwork
         UserBalanceCalculator.new(user1, hub1).balance.to_i.should == 5
       end
 
-      it "does not find the balance for other enterprises" do
-        hub2 = create(:distributor_enterprise)
-        o3 = create(:order_with_totals_and_distribution, 
-                    user: user1, distributor: hub2) #total=10
-        p3 = create(:payment, order: o3, amount: 10.00)
-        UserBalanceCalculator.new(user1, hub2).balance.to_i.should == 0
+      context "with another hub" do
+        let!(:hub2) { create(:distributor_enterprise) }
+        let!(:o3) { create(:order_with_totals_and_distribution,
+                           user: user1, distributor: hub2) } #total=10
+        let!(:p3) { create(:payment, order: o3, amount: 10.00) }
+
+        it "does not find the balance for other enterprises" do
+          UserBalanceCalculator.new(user1, hub2).balance.to_i.should == 0
+        end
       end
 
-      it "does not find the balance for other users" do
-        user2 = create(:user)
-        o4 = create(:order_with_totals_and_distribution, 
-                    user: user2, distributor: hub1) #total=10
-        p3 = create(:payment, order: o4, amount: 20.00)
-        UserBalanceCalculator.new(user2, hub1).balance.to_i.should == 10
+      context "with another user" do
+        let!(:user2) { create(:user) }
+        let!(:o4) { create(:order_with_totals_and_distribution,
+                           user: user2, distributor: hub1) } #total=10
+        let!(:p3) { create(:payment, order: o4, amount: 20.00) }
+
+        it "does not find the balance for other users" do
+          UserBalanceCalculator.new(user2, hub1).balance.to_i.should == 10
+        end
       end
     end 
   end