muster 0.0.10 → 0.0.11

data/spec/muster/strategies/joins_expression_spec.rb

@@ -5,15 +5,14 @@ describe Muster::Strategies::JoinsExpression do
   subject { Muster::Strategies::JoinsExpression.new(options) }
 
   describe '#parse' do
-
     context 'by default' do
       it 'returns empty hash for empty query string' do
-        subject.parse('').should == {}
+        subject.parse('').should eq({})
         subject.parse('').should be_an_instance_of(Muster::Results)
       end
 
       it 'returns hash of all key/value pairs' do
-        subject.parse('joins=author,author.foop').should eq({ "joins" => ["author", {"author" => "foop"}] })
+        subject.parse('joins=author,author.foop').should eq('joins' => ['author', { 'author' => 'foop' }])
       end
 
       it 'hash supports indifferent key access' do
@@ -23,11 +22,11 @@ describe Muster::Strategies::JoinsExpression do
       end
 
       it 'combines multiple expressions into an array' do
-        subject.parse('joins=author,activity').should == { 'joins' => ['author', 'activity'] }
+        subject.parse('joins=author,activity').should eq('joins' => ['author', 'activity'])
       end
 
       it 'discards non unique values' do
-        subject.parse('joins=author&joins=author').should == { 'joins' => ['author'] }
+        subject.parse('joins=author&joins=author').should eq('joins' => ['author'])
       end
     end
 
@@ -38,11 +37,11 @@ describe Muster::Strategies::JoinsExpression do
         end
 
         it 'converts comma separated value into Array' do
-          subject.parse('joins=author|activity').should == { 'joins' => ['author', 'activity'] }
+          subject.parse('joins=author|activity').should eq('joins' => ['author', 'activity'])
         end
 
         it 'ignores spaces after a separator' do
-          subject.parse('joins=author|%20  activity').should == { 'joins' => ['author', 'activity'] }
+          subject.parse('joins=author|%20  activity').should eq('joins' => ['author', 'activity'])
         end
       end
 
@@ -52,7 +51,7 @@ describe Muster::Strategies::JoinsExpression do
         end
 
         it 'converts comma separated value into Array' do
-          subject.parse('joins=author|activity|rule').should == { 'joins' => ['author', 'activity', 'rule'] }
+          subject.parse('joins=author|activity|rule').should eq('joins' => ['author', 'activity', 'rule'])
         end
       end
     end
@@ -62,11 +61,11 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:field_separator] = /\s*:\s*/ }
 
         it 'splits field from values' do
-          subject.parse('joins=author:country:name').should == { 'joins' => [{'author' => {'country' => 'name'}}] }
+          subject.parse('joins=author:country:name').should eq('joins' => [{ 'author' => { 'country' => 'name' } }])
         end
 
         it 'ignores spaces after field' do
-          subject.parse('joins=author : country').should == { 'joins' => [{'author' => 'country'}] }
+          subject.parse('joins=author : country').should eq('joins' => [{ 'author' => 'country' }])
         end
       end
 
@@ -74,7 +73,7 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:field_separator] = ':' }
 
         it 'converts comma separated value into Array' do
-          subject.parse('joins=author:country').should == { 'joins' => [{'author' => 'country'}] }
+          subject.parse('joins=author:country').should eq('joins' => [{ 'author' => 'country' }])
         end
       end
     end
@@ -84,7 +83,7 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:field] = :includes }
 
         it 'fields returns expressions for the key specified' do
-          subject.parse('includes=author').should == { 'includes' => ['author'] }
+          subject.parse('includes=author').should eq('includes' => ['author'])
         end
       end
 
@@ -92,7 +91,7 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:fields] = [:includes, :joins] }
 
         it 'fields returns expressions for the keys specified' do
-          subject.parse('joins=author&includes=activity').should == { 'joins' => ['author'], 'includes' => ['activity'] }
+          subject.parse('joins=author&includes=activity').should eq('joins' => ['author'], 'includes' => ['activity'])
         end
       end
 
@@ -100,7 +99,7 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:field] = 'includes' }
 
         it 'fields returns expressions for the key specified' do
-          subject.parse('includes=author').should == { 'includes' => ['author'] }
+          subject.parse('includes=author').should eq('includes' => ['author'])
         end
       end
 
@@ -108,11 +107,9 @@ describe Muster::Strategies::JoinsExpression do
         before { options[:fields] = ['includes', 'joins'] }
 
         it 'fields returns expressions for the keys specified' do
-          subject.parse('includes=author&joins=activity').should == { 'includes' => ['author'], 'joins' => ['activity'] }
+          subject.parse('includes=author&joins=activity').should eq('includes' => ['author'], 'joins' => ['activity'])
         end
       end
     end
-
   end
 end
-

data/spec/muster/strategies/pagination_spec.rb

@@ -6,25 +6,25 @@ describe Muster::Strategies::Pagination do
 
   describe '#parse' do
     it 'returns a Muster::Results instance' do
-      subject.parse('').should == {"pagination"=>{"page"=>1, "per_page"=>30}, "limit"=>30, "offset"=>nil}
+      subject.parse('').should eq('pagination' => { 'page' => 1, 'per_page' => 30 }, 'limit' => 30, 'offset' => nil)
       subject.parse('').should be_an_instance_of(Muster::Results)
     end
 
     context 'by default' do
       it 'returns default hash for empty query string' do
-        subject.parse('').should == {'pagination' => {'page' => 1, 'per_page' => 30}, 'offset' => nil, 'limit' => 30}
+        subject.parse('').should eq('pagination' => { 'page' => 1, 'per_page' => 30 }, 'offset' => nil, 'limit' => 30)
       end
 
       it 'ensures page is positive integer' do
-        subject.parse('page=foop')[:pagination].should == {'page' => 1, 'per_page' => 30}
+        subject.parse('page=foop')[:pagination].should eq('page' => 1, 'per_page' => 30)
       end
 
       it 'ensures per_page is positive integer' do
-        subject.parse('per_page=foop')[:pagination].should == {'page' => 1, 'per_page' => 30}
+        subject.parse('per_page=foop')[:pagination].should eq('page' => 1, 'per_page' => 30)
       end
 
       it 'accepts page_size instead of per_page' do
-        subject.parse('page_size=10')[:pagination].should == {'page' => 1, 'per_page' => 10}
+        subject.parse('page_size=10')[:pagination].should eq('page' => 1, 'per_page' => 10)
       end
 
       it 'calculates offset from pagination' do
@@ -47,7 +47,7 @@ describe Muster::Strategies::Pagination do
         before { options[:field] = :limit }
 
         it 'fields returns values for the key specified' do
-          subject.parse('per_page=10').should == {'limit' => 10}
+          subject.parse('per_page=10').should eq('limit' => 10)
         end
       end
 
@@ -55,7 +55,7 @@ describe Muster::Strategies::Pagination do
         before { options[:fields] = [:limit, :offset] }
 
         it 'fields returns values for the keys specified' do
-          subject.parse('per_page=10&page=2').should == {'limit' => 10, 'offset' => 10}
+          subject.parse('per_page=10&page=2').should eq('limit' => 10, 'offset' => 10)
         end
       end
 
@@ -63,7 +63,7 @@ describe Muster::Strategies::Pagination do
         before { options[:field] = 'limit' }
 
         it 'fields returns values for the key specified' do
-          subject.parse('per_page=10').should == {'limit' => 10}
+          subject.parse('per_page=10').should eq('limit' => 10)
         end
       end
 
@@ -71,7 +71,7 @@ describe Muster::Strategies::Pagination do
         before { options[:fields] = ['limit', 'offset'] }
 
         it 'fields returns values for the keys specified' do
-          subject.parse('per_page=10&page=2').should == {'limit' => 10, 'offset' => 10}
+          subject.parse('per_page=10&page=2').should eq('limit' => 10, 'offset' => 10)
         end
       end
     end

data/spec/muster/strategies/sort_expression_spec.rb

@@ -5,15 +5,14 @@ describe Muster::Strategies::SortExpression do
   subject { Muster::Strategies::SortExpression.new(options) }
 
   describe '#parse' do
-
     context 'by default' do
       it 'returns empty hash for empty query string' do
-        subject.parse('').should == {}
+        subject.parse('').should eq({})
         subject.parse('').should be_an_instance_of(Muster::Results)
       end
 
       it 'returns hash of all key/value pairs' do
-        subject.parse('sort=id&order=name').should == { 'sort' => 'id asc', 'order' => 'name asc' }
+        subject.parse('sort=id&order=name').should eq('sort' => 'id asc', 'order' => 'name asc')
       end
 
       it 'hash supports indifferent key access' do
@@ -33,7 +32,7 @@ describe Muster::Strategies::SortExpression do
       it 'supports comma separated expressions without directions' do
         subject.parse('sort=id,name').should == { 'sort' => ['id asc', 'name asc'] }
       end
-      
+
       it 'discards non unique values' do
         subject.parse('sort=id&sort=name&sort=id').should == { 'sort' => ['id asc', 'name asc'] }
       end
@@ -90,6 +89,5 @@ describe Muster::Strategies::SortExpression do
         end
       end
     end
-
   end
 end

metadata

@@ -1,111 +1,125 @@
 --- !ruby/object:Gem::Specification
 name: muster
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors:
 - Christopher H. Laco
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-30 00:00:00.000000000 Z
+date: 2015-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.4'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.11.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.11.0
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.33.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.33.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 0.8.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 0.8.2
 description: Muster is a gem that turns query strings of varying formats into data
@@ -116,9 +130,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".rspec"
-- ".yardopts"
+- .gitignore
+- .rspec
+- .rubocop.yml
+- .yardopts
 - Authors
 - Changes
 - Gemfile
@@ -156,17 +171,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Muster various query string formats into a more reusable data structure.