ab_spec.rb in kissable-1.0.1

- old
+ new

@@ -3,10 +3,11 @@
 describe Kissable::AB do
   let(:test_name) { "sample_test" }
   let(:groups) { nil }
   let(:ratios) { nil }
   let(:cookies) { {} }
+  let(:login) { 'jsmith@example.com' }
   let(:ab_test) { described_class.new(test_name, groups, ratios) }
 
   describe '#initialize' do
     context "when initialized" do
       context "with no groups" do
@@ -61,70 +62,87 @@
       end
     end
   end
 
   describe '#group' do
-    let(:group) { ab_test.group(cookies) }
+    context "when using cookies" do
+      let(:group) { ab_test.group(cookies) }
 
-    it "is returns Original or Variant" do
-      expect(group).to match(/Original|Variant/)
-    end
-
-    context "when cookie exists" do
-      before :each do
-        ab_test.stub(:cookies).and_return('abid' => 1)
+      it "returns Original or Variant" do
+        expect(group).to match(/Original|Variant/)
       end
 
-      it "doesn't change the cookie" do
-        expect { group }.to_not change { ab_test.cookies }
-      end
-    end
+      context "when cookie exists" do
+        before :each do
+          ab_test.stub(:cookies).and_return('abid' => 1)
+        end
 
-    context "when cookie doesn't exist" do
-      it "sets a cookie" do
-        expect { group }.to change { ab_test.cookies }.from({})
+        it "doesn't change the cookie" do
+          expect { group }.to_not change { ab_test.cookies }
+        end
       end
 
-      describe "the cookie" do
-        let(:cookie) { ab_test.cookies['abid'] }
+      context "when cookie doesn't exist" do
+        it "sets a cookie" do
+          expect { group }.to change { ab_test.cookies }.from({})
+        end
 
-        context "when the domain has been configured" do
-          let(:domain) { 'someaweseomedomain.com' }
+        describe "the cookie" do
+          let(:cookie) { ab_test.cookies['abid'] }
 
-          it "has the domain key set to the correct value" do
-            Kissable.configure do |config|
-              config.domain = domain
+          context "when the domain has been configured" do
+            let(:domain) { 'someaweseomedomain.com' }
+
+            it "has the domain key set to the correct value" do
+              Kissable.configure do |config|
+                config.domain = domain
+              end
+
+              group
+              expect(cookie).to include(:domain => domain)
             end
+          end
 
+          context "when the domain hasn't been configured" do
+            it "doesn't include a domain key" do
+              Kissable.configure do |config|
+                config.domain = nil
+              end
+
+              group
+              expect(cookie).to_not include(:domain)
+            end
+          end
+
+          it "expires" do
             group
-            expect(cookie).to include(:domain => domain)
+            expect(cookie).to include(:expires)
           end
-        end
 
-        context "when the domain hasn't been configured" do
-          it "doesn't include a domain key" do
-            Kissable.configure do |config|
-              config.domain = nil
-            end
+          it "has a path" do
+            group
+            expect(cookie).to include(:path => "/")
+          end
 
+          it "contains a value" do
             group
-            expect(cookie).to_not include(:domain)
+            expect(cookie).to include(:value)
           end
         end
+      end
+    end
 
-        it "expires" do
-          group
-          expect(cookie).to include(:expires)
-        end
+    context "when using login" do
+      let(:group) { ab_test.group(login) }
 
-        it "has a path" do
-          group
-          expect(cookie).to include(:path => "/")
-        end
+      it "returns Original or Variant" do
+        expect(group).to match(/Original|Variant/)
+      end
 
-        it "contains a value" do
-          group
-          expect(cookie).to include(:value)
+      it "always returns the same group" do
+        10.times do
+          test_copy = described_class.new(test_name)
+          expect(test_copy.group(login)).to eq(group)
         end
       end
     end
   end
 