reworked the tests to make them cleaner

The tests now use simple lists for the types to make it easier to
distinguish between all elements.
The tests themselves are also shared now.

1 files changed, 68 insertions, 34 deletions

diff --git a/spec/unit/zero/request/accept_type/preferred_spec.rb b/spec/unit/zero/request/accept_type/preferred_spec.rb
index daeba86..7d9b881 100644
--- a/spec/unit/zero/request/accept_type/preferred_spec.rb
+++ b/spec/unit/zero/request/accept_type/preferred_spec.rb
@@ -1,41 +1,75 @@
 require 'spec_helper'
 
 describe Zero::Request::AcceptType, '#preferred' do
-  subject { Zero::Request::AcceptType }
+  subject { Zero::Request::AcceptType.new(type_string).preferred }
+  let(:type_string) { types.join(',') }
+
   let(:html) { 'text/html' }
   let(:json) { 'application/json' }
   let(:foo)  { 'text/foo' }
-  let(:lower_quality) { foo + ';q=0.5' }
-  let(:default) { '*/*;q=0.1' }
-  let(:option)  { [foo + ';b=23', html].join(',') }
-  let(:simple_accept)  { [html, json].join(',') }
-  let(:quality_accept) { [html, lower_quality, default].join(',') }
-  let(:random_accept)  { [lower_quality, default, html].join(',') }
-  let(:lower_accept)   { [lower_quality, default].join(',') }
-  
-  context 'without mapping' do
-    it { subject.new(html).preferred.should                    == html  }
-    it { subject.new(json).preferred.should                    == json  }
-    it { subject.new(option).preferred.should                  == foo   }
-    it { subject.new(simple_accept).preferred.should           == html  }
-    it { subject.new(quality_accept).preferred.should          == html  }
-    it { subject.new(random_accept).preferred.should           == html  }
-    it { subject.new(lower_accept).preferred.should            == foo   }
-    it { subject.new(nil).preferred.should                     == '*/*' }
-    it { subject.new('').preferred.should                      == '*/*' }
-    it { subject.new('text / html').preferred.should           == html  }
-    it { subject.new("#{html};q=0.9,#{json}").preferred.should == json  }
-  end
-
-#  context 'with mapping' do
-#    before :all do
-#      Zero::Request::Accept.map = {'text/html' => 'html'}
-#    end
-#
-#    after :all do
-#      Zero::Request::Accept.map = {}
-#    end
-#
-#    it { subject.new(html).preferred.should == 'html' }
-#  end
+  let(:default) { '*/*' }
+  let(:default_q) { '*/*;q=0.1' }
+
+  shared_examples_for 'a preferred type' do
+    it { should eq(result) }
+  end
+
+  context 'with a single type' do
+    let(:types) { [html] }
+    let(:result) { html }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with two types' do
+    let(:types) { [html, json] }
+    let(:result) { html }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with multiple types and quality identifier' do
+    let(:types) { ["#{json};q=0.5", html, default_q] }
+    let(:result) { html }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  # this is for mutant - mutant modifies the default quality from 0 to 1
+  # the 0.9 gets translated to a quality of 1, so that the ordering gets
+  # different than before and that is why we have two tests here
+  context 'with quality ordering' do
+    let(:types) { ["#{json};q=0.9", html, default_q] } 
+    let(:result) { html }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with different kinds of options' do
+    let(:types) { ["#{json};b=foo", html] }
+    let(:result) { json }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with an empty types' do
+    let(:types) { [] }
+    let(:result) { default }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with whitespaces' do
+    let(:types) { ['text / html'] }
+    let(:result) { html }
+
+    it_behaves_like 'a preferred type'
+  end
+
+  context 'with nil' do
+    subject { Zero::Request::AcceptType.new(nil).preferred }
+    let(:result) { default }
+
+    it_behaves_like 'a preferred type'
+  end
 end