Run rubocop autocorrect

lewispb · lewispb · commit bdfca0ab929b · 2023-07-11T00:14:13.000+01:00
diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 gemspec
diff --git a/README.md b/README.md
@@ -21,11 +21,11 @@ integer.value = 5  # => SET myinteger "5"
 5 == integer.value # => GET myinteger
 
 decimal = Kredis.decimal "mydecimal" # accuracy!
-decimal.value = "%.47f" % (1.0/10) # => SET mydecimal "0.10000000000000000555111512312578270211815834045"
+decimal.value = "%.47f" % (1.0 / 10) # => SET mydecimal "0.10000000000000000555111512312578270211815834045"
 BigDecimal("0.10000000000000000555111512312578270211815834045e0") == decimal.value # => GET mydecimal
 
 float = Kredis.float "myfloat" # speed!
-float.value = 1.0/10 # => SET myfloat "0.1"
+float.value = 1.0 / 10 # => SET myfloat "0.1"
 0.1 == float.value # => GET myfloat
 
 boolean = Kredis.boolean "myboolean"
@@ -228,14 +228,14 @@ If you need to connect to Redis with SSL, the recommended approach is to set you
 
 ```ruby
 Kredis::Connections.connections[:shared] = Redis.new(
-  url: ENV['REDIS_URL'],
+  url: ENV["REDIS_URL"],
   ssl_params: {
     cert_store: OpenSSL::X509::Store.new.tap { |store|
-      store.add_file(Rails.root.join('config', 'ca_cert.pem').to_s)
+      store.add_file(Rails.root.join("config", "ca_cert.pem").to_s)
     },
 
     cert: OpenSSL::X509::Certificate.new(File.read(
-      Rails.root.join('config', 'client.crt')
+      Rails.root.join("config", "client.crt")
     )),
 
     key: OpenSSL::PKey::RSA.new(
diff --git a/bin/console b/bin/console
@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "irb"
 require "bundler/inline"
diff --git a/bin/test b/bin/test
@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 $: << File.expand_path("../test", __dir__)
 
 require "bundler/setup"
diff --git a/kredis.gemspec b/kredis.gemspec
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative "lib/kredis/version"
 
 Gem::Specification.new do |s|
diff --git a/lib/install/install.rb b/lib/install/install.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 yaml_path = Rails.root.join("config/redis/shared.yml")
 unless yaml_path.exist?
   say "Adding `config/redis/shared.yml`"
diff --git a/lib/kredis.rb b/lib/kredis.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support"
 require "active_support/core_ext/module/attribute_accessors"
 require "active_support/core_ext/module/attribute_accessors_per_thread"
diff --git a/lib/kredis/attributes.rb b/lib/kredis/attributes.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kredis::Attributes
   extend ActiveSupport::Concern
 
@@ -106,7 +108,7 @@ def kredis_key_evaluated(key)
     end
 
     def kredis_key_for_attribute(name)
-      "#{self.class.name.tableize.gsub("/", ":")}:#{extract_kredis_id}:#{name}"
+      "#{self.class.name.tableize.tr("/", ":")}:#{extract_kredis_id}:#{name}"
     end
 
     def extract_kredis_id
diff --git a/lib/kredis/connections.rb b/lib/kredis/connections.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "redis"
 
 module Kredis::Connections
@@ -6,11 +8,9 @@ module Kredis::Connections
   mattr_accessor :connector, default: ->(config) { Redis.new(config) }
 
   def configured_for(name)
-    connections[name] ||= begin
-      Kredis.instrument :meta, message: "Connected to #{name}" do
+    connections[name] ||= Kredis.instrument :meta, message: "Connected to #{name}" do
         connector.call configurator.config_for("redis/#{name}")
       end
-    end
   end
 
   def clear_all
diff --git a/lib/kredis/log_subscriber.rb b/lib/kredis/log_subscriber.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/log_subscriber"
 
 class Kredis::LogSubscriber < ActiveSupport::LogSubscriber
diff --git a/lib/kredis/migration.rb b/lib/kredis/migration.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/core_ext/module/delegation"
 
 class Kredis::Migration
diff --git a/lib/kredis/namespace.rb b/lib/kredis/namespace.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kredis::Namespace
   def namespace=(namespace)
     Thread.current[:kredis_namespace] = namespace
diff --git a/lib/kredis/railtie.rb b/lib/kredis/railtie.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Railtie < ::Rails::Railtie
   config.kredis = ActiveSupport::OrderedOptions.new
 
diff --git a/lib/kredis/type_casting.rb b/lib/kredis/type_casting.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "json"
 require "active_model/type"
 require "kredis/type/json"
diff --git a/lib/kredis/types.rb b/lib/kredis/types.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kredis::Types
   autoload :CallbacksProxy, "kredis/types/callbacks_proxy"
 
diff --git a/lib/kredis/types/callbacks_proxy.rb b/lib/kredis/types/callbacks_proxy.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::CallbacksProxy
   attr_reader :type
   delegate :to_s, to: :type
diff --git a/lib/kredis/types/counter.rb b/lib/kredis/types/counter.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Counter < Kredis::Types::Proxying
   proxying :multi, :set, :incrby, :decrby, :get, :del, :exists?
 
diff --git a/lib/kredis/types/cycle.rb b/lib/kredis/types/cycle.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Cycle < Kredis::Types::Counter
   attr_accessor :values
 
diff --git a/lib/kredis/types/enum.rb b/lib/kredis/types/enum.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/core_ext/object/inclusion"
 
 class Kredis::Types::Enum < Kredis::Types::Proxying
diff --git a/lib/kredis/types/flag.rb b/lib/kredis/types/flag.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Flag < Kredis::Types::Proxying
   proxying :set, :exists?, :del
 
diff --git a/lib/kredis/types/hash.rb b/lib/kredis/types/hash.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/core_ext/hash"
 
 class Kredis::Types::Hash < Kredis::Types::Proxying
@@ -14,7 +16,7 @@ def []=(key, value)
   end
 
   def update(**entries)
-    hset entries.transform_values{ |val| type_to_string(val, typed) } if entries.flatten.any?
+    hset entries.transform_values { |val| type_to_string(val, typed) } if entries.flatten.any?
   end
 
   def values_at(*keys)
diff --git a/lib/kredis/types/list.rb b/lib/kredis/types/list.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::List < Kredis::Types::Proxying
   proxying :lrange, :lrem, :lpush, :ltrim, :rpush, :exists?, :del
 
diff --git a/lib/kredis/types/ordered_set.rb b/lib/kredis/types/ordered_set.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::OrderedSet < Kredis::Types::Proxying
   proxying :multi, :zrange, :zrem, :zadd, :zremrangebyrank, :zcard, :exists?, :del
 
@@ -42,7 +44,7 @@ def insert(elements, prepending: false)
           base_score + incremental_score
         end
 
-        [ score , element ]
+        [ score, element ]
       end
 
       multi do
diff --git a/lib/kredis/types/proxy.rb b/lib/kredis/types/proxy.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Proxy
   require_relative "proxy/failsafe"
   include Failsafe
diff --git a/lib/kredis/types/proxy/failsafe.rb b/lib/kredis/types/proxy/failsafe.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kredis::Types::Proxy::Failsafe
   def initialize(*)
     super
diff --git a/lib/kredis/types/proxying.rb b/lib/kredis/types/proxying.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/core_ext/module/delegation"
 
 class Kredis::Types::Proxying
diff --git a/lib/kredis/types/scalar.rb b/lib/kredis/types/scalar.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Scalar < Kredis::Types::Proxying
   proxying :set, :get, :exists?, :del, :expire, :expireat
 
diff --git a/lib/kredis/types/set.rb b/lib/kredis/types/set.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Set < Kredis::Types::Proxying
   proxying :smembers, :sadd, :srem, :multi, :del, :sismember, :scard, :spop, :exists?, :srandmember
 
diff --git a/lib/kredis/types/slots.rb b/lib/kredis/types/slots.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Kredis::Types::Slots < Kredis::Types::Proxying
   class NotAvailable < StandardError; end
 
diff --git a/lib/kredis/types/unique_list.rb b/lib/kredis/types/unique_list.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # You'd normally call this a set, but Redis already has another data type for that
 class Kredis::Types::UniqueList < Kredis::Types::List
   proxying :multi, :ltrim, :exists?
diff --git a/lib/kredis/version.rb b/lib/kredis/version.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Kredis
   VERSION = "1.5.0"
 end
diff --git a/lib/tasks/kredis/install.rake b/lib/tasks/kredis/install.rake
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 namespace :kredis do
   desc "Install kredis"
   task :install do
diff --git a/test/attributes_callbacks_test.rb b/test/attributes_callbacks_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class AttributesCallbacksTest < ActiveSupport::TestCase
diff --git a/test/attributes_test.rb b/test/attributes_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/core_ext/integer"
 
@@ -249,7 +251,7 @@ class AttributesTest < ActiveSupport::TestCase
     assert @person.onboarded.value
 
     @person.onboarded.value = false
-    refute @person.onboarded.value
+    assert_not @person.onboarded.value
   end
 
   test "missing id to constrain key" do
diff --git a/test/callbacks_test.rb b/test/callbacks_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class CallbacksTest < ActiveSupport::TestCase
@@ -30,7 +32,7 @@ class CallbacksTest < ActiveSupport::TestCase
     attention = Kredis.slot "attention", after_change: ->(slot) { @callback_check = slot.available? }
     attention.reserve
 
-    refute @callback_check
+    assert_not @callback_check
   end
 
   test "enum with after_change proc callback" do
diff --git a/test/connections_test.rb b/test/connections_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class ConnectionsTest < ActiveSupport::TestCase
diff --git a/test/log_subscriber_test.rb b/test/log_subscriber_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/log_subscriber/test_helper"
 
diff --git a/test/migration_test.rb b/test/migration_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class MigrationTest < ActiveSupport::TestCase
diff --git a/test/namespace_test.rb b/test/namespace_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class NamespaceTest < ActiveSupport::TestCase
diff --git a/test/proxy_test.rb b/test/proxy_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class ProxyTest < ActiveSupport::TestCase
diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "bundler/setup"
 require "active_support/test_case"
 require "active_support/testing/autorun"
diff --git a/test/types/counter_test.rb b/test/types/counter_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/core_ext/integer"
 
diff --git a/test/types/cycle_test.rb b/test/types/cycle_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/core_ext/integer"
 
diff --git a/test/types/enum_test.rb b/test/types/enum_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class EnumTest < ActiveSupport::TestCase
diff --git a/test/types/flag_test.rb b/test/types/flag_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class FlagTest < ActiveSupport::TestCase
diff --git a/test/types/hash_test.rb b/test/types/hash_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/core_ext/integer"
 
@@ -87,7 +89,7 @@ class HashTest < ActiveSupport::TestCase
   test "exists?" do
     assert_not @hash.exists?
 
-    @hash[:key]  = :value
+    @hash[:key] = :value
     assert @hash.exists?
   end
 end
diff --git a/test/types/list_test.rb b/test/types/list_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "active_support/core_ext/integer"
 
diff --git a/test/types/ordered_set_test.rb b/test/types/ordered_set_test.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class OrderedSetTest < ActiveSupport::TestCase
diff --git a/test/types/scalar_test.rb b/test/types/scalar_test.rb
diff --git a/test/types/set_test.rb b/test/types/set_test.rb
diff --git a/test/types/slots_test.rb b/test/types/slots_test.rb
diff --git a/test/types/unique_list_test.rb b/test/types/unique_list_test.rb

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`source "https://rubygems.org"`
`2`	`4`
`3`	`5`	`gemspec`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`#!/usr/bin/env ruby`
	`2`	`+# frozen_string_literal: true`
`2`	`3`
`3`	`4`	`require "irb"`
`4`	`5`	`require "bundler/inline"`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require_relative "lib/kredis/version"`
`2`	`4`
`3`	`5`	`Gem::Specification.new do \|s\|`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`yaml_path = Rails.root.join("config/redis/shared.yml")`
`2`	`4`	`unless yaml_path.exist?`
`3`	`5`	say "Adding `config/redis/shared.yml`"
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require "active_support"`
`2`	`4`	`require "active_support/core_ext/module/attribute_accessors"`
`3`	`5`	`require "active_support/core_ext/module/attribute_accessors_per_thread"`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require "active_support/log_subscriber"`
`2`	`4`
`3`	`5`	`class Kredis::LogSubscriber < ActiveSupport::LogSubscriber`