clean up after review

sds · Jan 4, 2024 · f2c70ca · f2c70ca
1 parent 504d2a8
commit f2c70ca
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/Gemfile b/Gemfile
@@ -12,6 +12,6 @@ gem 'rubocop', '1.44.1'
 gem 'simplecov', '~> 0.22.0'
 gem 'simplecov-lcov', '~> 0.8.0'
 
-# added gems
+# TODO: gems added for development, remove before merge
 # gem 'pry'
 gem 'redis', '>= 5.0.0'
diff --git a/lib/mock_redis/set_methods.rb b/lib/mock_redis/set_methods.rb
@@ -126,7 +126,6 @@ def srem(key, members)
           s.delete_if { |m| members.include?(m) }
           orig_size - s.size
         else
-          # !!s.delete?(members.to_s)
           s.delete?(members.to_s) ? 1 : 0
         end
       end

diff --git a/spec/support/redis_multiplexer.rb b/spec/support/redis_multiplexer.rb
@@ -55,25 +55,25 @@ def _gsub_clear
     elsif !equalish?(mock_retval, real_retval) && !mock_error && !real_error
       # no exceptions, just different behavior
       raise MismatchedResponse,
-            "Mock failure: responses not equal.\n" \
-              "Redis.#{method}(#{args.inspect}) returned #{real_retval.inspect}\n" \
-              "MockRedis.#{method}(#{args.inspect}) returned #{mock_retval.inspect}\n"
+        "Mock failure: responses not equal.\n" \
+        "Redis.#{method}(#{args.inspect}) returned #{real_retval.inspect}\n" \
+        "MockRedis.#{method}(#{args.inspect}) returned #{mock_retval.inspect}\n"
     elsif !mock_error && real_error
       raise MismatchedResponse,
-            "Mock failure: didn't raise an error when it should have.\n" \
-              "Redis.#{method}(#{args.inspect}) raised #{real_error.inspect}\n" \
-              "MockRedis.#{method}(#{args.inspect}) raised nothing " \
-              "and returned #{mock_retval.inspect}"
+        "Mock failure: didn't raise an error when it should have.\n" \
+        "Redis.#{method}(#{args.inspect}) raised #{real_error.inspect}\n" \
+        "MockRedis.#{method}(#{args.inspect}) raised nothing " \
+        "and returned #{mock_retval.inspect}"
     elsif !real_error && mock_error
       raise MismatchedResponse,
-            "Mock failure: raised an error when it shouldn't have.\n" \
-              "Redis.#{method}(#{args.inspect}) returned #{real_retval.inspect}\n" \
-              "MockRedis.#{method}(#{args.inspect}) raised #{mock_error.inspect}"
+        "Mock failure: raised an error when it shouldn't have.\n" \
+        "Redis.#{method}(#{args.inspect}) returned #{real_retval.inspect}\n" \
+        "MockRedis.#{method}(#{args.inspect}) raised #{mock_error.inspect}"
     elsif mock_error && real_error && !equalish?(mock_error, real_error)
       raise MismatchedResponse,
-            "Mock failure: raised the wrong error.\n" \
-              "Redis.#{method}(#{args.inspect}) raised #{real_error.inspect}\n" \
-              "MockRedis.#{method}(#{args.inspect}) raised #{mock_error.inspect}"
+        "Mock failure: raised the wrong error.\n" \
+        "Redis.#{method}(#{args.inspect}) raised #{real_error.inspect}\n" \
+        "MockRedis.#{method}(#{args.inspect}) raised #{mock_error.inspect}"
     end
 
     raise mock_error if mock_error