rubocop --only Style/NegatedIf -a

Author: BookOfGreg

lib/generators/react/install_generator.rb

@@ -24,7 +24,7 @@ def create_directory
           javascript_dir
         end
         empty_directory File.join(components_dir, 'components')
-        if !options[:skip_git]
+        unless options[:skip_git]
           create_file File.join(components_dir, 'components/.gitkeep')
         end
       end

lib/react/rails/railtie.rb

@@ -78,7 +78,7 @@ class Railtie < ::Rails::Railtie
         })
 
         sprockets_env = app.assets || app.config.try(:assets) # sprockets-rails 3.x attaches this at a different config
-        if !sprockets_env.nil?
+        unless sprockets_env.nil?
           sprockets_env.version = [sprockets_env.version, "react-#{asset_variant.react_build}",].compact.join('-')
         end
 

test/generators/install_generator_sprockets_test.rb

@@ -2,7 +2,7 @@
 require 'generators/react/install_generator'
 
 # If webpacker is available, its setup is preferred
-if !WebpackerHelpers.available?
+unless WebpackerHelpers.available?
   class InstallGeneratorSprocketsTest < Rails::Generators::TestCase
     destination File.join(Rails.root, 'tmp', 'generator_test_output')
     tests React::Generators::InstallGenerator

test/react/rails/component_mount_test.rb

@@ -4,7 +4,7 @@
   class ComponentMountTest < ActionDispatch::IntegrationTest
     compiled_once = false
     setup do
-      if !compiled_once
+      unless compiled_once
         WebpackerHelpers.clear_webpacker_packs
         WebpackerHelpers.compile
       end

test/react/rails/controller_lifecycle_test.rb

@@ -25,7 +25,7 @@ def react_component(*args)
 class ControllerLifecycleTest < ActionDispatch::IntegrationTest
   compiled = false
   setup do
-    if !compiled
+    unless compiled
       compile = true
       WebpackerHelpers.compile
     end

test/react/rails/react_rails_ujs_test.rb

@@ -6,7 +6,7 @@ class ReactRailsUJSTest < ActionDispatch::IntegrationTest
 
     compiled = false
     setup do
-      if !compiled
+      unless compiled
         React::ServerRendering.reset_pool
         WebpackerHelpers.compile
       end

test/react/server_rendering/bundle_renderer_test.rb

@@ -90,7 +90,7 @@ class BundleRendererTest < ActiveSupport::TestCase
       assert_match(/console.error.apply\(console, \["setTimeout #{message}"\]\);$/, result)
     end
 
-    if !WebpackerHelpers.available?
+    unless WebpackerHelpers.available?
       # This doesn't work with webpacker since finding components is based on filename
       test '.new accepts additional code to add to the JS context' do
         additional_code = File.read(File.expand_path('../../../helper_files/WithoutSprockets.js', __FILE__))

test/support/webpacker_helpers.rb

@@ -18,7 +18,7 @@ def when_webpacker_available
   end
 
   def compile
-    return if !available?
+    return unless available?
     clear_webpacker_packs
     Dir.chdir("./test/#{DUMMY_LOCATION}") do
       # capture_io do
@@ -31,7 +31,7 @@ def compile
   end
 
   def compile_if_missing
-    if !File.exist?(PACKS_DIRECTORY)
+    unless File.exist?(PACKS_DIRECTORY)
       compile
     end
   end
@@ -127,7 +127,7 @@ def with_dev_server
     end
 
     # If we didn't hook up with a dev server after waiting, fail loudly.
-    if !detected_dev_server
+    unless detected_dev_server
       raise 'Failed to start dev server'
     end