Merge pull request kanwei#44 from keithrbennett/update_dependencies_a…

…nd_fix_spec_warnings Update dependencies and fix spec warnings
GarrisonJ · Apr 4, 2021 · ec54a90 · ec54a90
2 parents da05959 + a610163
commit ec54a90
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 21 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -6,31 +6,31 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    diff-lcs (1.2.5)
-    rake (11.1.1)
-    rake-compiler (0.9.7)
+    diff-lcs (1.4.4)
+    rake (13.0.3)
+    rake-compiler (1.1.1)
       rake
-    rspec (3.4.0)
-      rspec-core (~> 3.4.0)
-      rspec-expectations (~> 3.4.0)
-      rspec-mocks (~> 3.4.0)
-    rspec-core (3.4.4)
-      rspec-support (~> 3.4.0)
-    rspec-expectations (3.4.0)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.4.0)
-    rspec-mocks (3.4.1)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.4.0)
-    rspec-support (3.4.1)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
 
 PLATFORMS
-  ruby
+  x86_64-linux
 
 DEPENDENCIES
   algorithms!
   rake-compiler
   rspec
 
 BUNDLED WITH
-   1.11.2
+   2.2.10
diff --git a/lib/algorithms/sort.rb b/lib/algorithms/sort.rb
@@ -310,7 +310,7 @@ def self.dualpivot(container, left=0, right=container.size-1, div=3)
       pivot2 = container[right]
       # pointers
       less = left + 1
-      great = right -1
+      great = right - 1
       # sorting
       k = less
       while k <= great

diff --git a/spec/deque_spec.rb b/spec/deque_spec.rb
@@ -27,7 +27,7 @@
   end
 
   it "should raise ArgumentError if passed more than one argument" do
-    expect { @deque.class.send("new", Time.now, []) }.to raise_error
+    expect { @deque.class.send("new", Time.now, []) }.to raise_error(ArgumentError)
   end
 end
 

diff --git a/spec/heap_spec.rb b/spec/heap_spec.rb
@@ -7,8 +7,8 @@
   end
 
   it "should not let you merge with non-heaps" do
-    expect { @heap.merge!(nil) }.to raise_error
-    expect { @heap.merge!([]) }.to raise_error
+    expect { @heap.merge!(nil) }.to raise_error(ArgumentError)
+    expect { @heap.merge!([]) }.to raise_error(ArgumentError)
   end
 
   describe "(empty)" do

diff --git a/spec/suffix_array_spec.rb b/spec/suffix_array_spec.rb
@@ -3,7 +3,7 @@
 
 describe "empty suffix array" do
   it "should not initialize with empty string" do
-    expect { Containers::SuffixArray.new("") }.to raise_error
+    expect { Containers::SuffixArray.new("") }.to raise_error(ArgumentError)
   end
 end
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,7 +27,7 @@ @@
       end
       it "should raise ArgumentError if passed more than one argument" do
-        expect { @deque.class.send("new", Time.now, []) }.to raise_error
+        expect { @deque.class.send("new", Time.now, []) }.to raise_error(ArgumentError)
       end
     end
@@ Expand Down @@