visit: let the initializer choose whether save or not.

Author: crondaemon

app/models/rails_url_shortener/visit.rb

@@ -35,6 +35,7 @@ class Visit < ApplicationRecord
     # Return boolean
     # rubocop:disable Metrics/AbcSize
     def self.parse_and_save(url, request)
+      return false if !RailsUrlShortener.save_visits
       browser(request)
       return false if !RailsUrlShortener.save_bots_visits && @browser.bot?
 

lib/rails_url_shortener.rb

@@ -40,6 +40,8 @@ module RailsUrlShortener
   # so if you put this configuration like false could lose some visits to your link
   # by default saving all requests
   mattr_accessor :save_bots_visits, default: true
+
+  mattr_accessor :save_visits, default: true
 end
 
 ActiveSupport.on_load(:active_record) do

test/models/rails_url_shortener/visit_test.rb

@@ -74,6 +74,26 @@ class VisitTest < ActiveSupport::TestCase
       assert_no_enqueued_jobs do
         assert_not Visit.parse_and_save(rails_url_shortener_urls(:one), request)
       end
+
+      RailsUrlShortener.save_bots_visits = true
+    end
+
+    test "don't save any" do
+      # set the configuration
+      RailsUrlShortener.save_visits = false
+
+      # generate a fake request
+      request = ActionDispatch::TestRequest.create(env = Rack::MockRequest.env_for('/', 'HTTP_HOST' => 'test.host'.b,
+                                                                                        'REMOTE_ADDR' => '1.0.0.0'.b, 'HTTP_USER_AGENT' => 'Rails Testing'.b,
+                                                                                        'HTTP_REFERER' => 'https://example.com'.b))
+      request.user_agent = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 11_3) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.1 Safari/605.1.15'
+
+      # asserts
+      assert_no_enqueued_jobs do
+        assert_not Visit.parse_and_save(rails_url_shortener_urls(:one), request)
+      end
+
+      RailsUrlShortener.save_visits = true
     end
   end
 end