shortener 0.8.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 0ff66256668eb71d5b42010c191233c7346c2651
-  data.tar.gz: f4cf3ffadd4b4f74ae6076be7f9fa13edf55b5ea
+SHA256:
+  metadata.gz: 12594d11b56f84e7c4654bee681887dc084978604ee158c803e384c839b6a937
+  data.tar.gz: 8550a33e8bd06229cd66c5e9f340ba6cd9c49c77acb5481555b23f43ea659883
 SHA512:
-  metadata.gz: c88fd4cda719d6759af9f1faa5fc70829f18ff99662f77dc10ae33f20e9debcfabd051c5fe7157a75b505f45d164752cd40ead26ed110be6c09c1427e81e382e
-  data.tar.gz: 0e4ae113ede69db12a0b9901747bb20bb0ed4a72e51817e619524c166cc8cf1a8159f69c0352af843a48650773c3aa98e62d92a44f5e5ffcbcfd083265a0c629
+  metadata.gz: 911ad7cc429c015d090760690b53aa5151588f3c384090f293cd608484d0804ba3e72034c97bc31cb99730deabf5c5103045e0ffd89b809dd7c12030b732b57e
+  data.tar.gz: 8fe1bd76f2a97dc3a9f2b0c1cfe9b4798591f84654f40eb341724df41e1ed6b4f8ae461d54759b2059163e440cf72b81df8246a7a7715cc68cecc57d3279a1c6

data/.github/workflows/ruby.yml

@@ -0,0 +1,66 @@
+name: Ruby
+
+on:
+  push:
+    branches: [ develop ]
+  pull_request:
+    branches: [ develop ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: true
+      matrix:
+        ruby-version: 
+          - '2.4'
+          - '2.6'
+          - '2.7'
+          - '3.0'
+        gemfile:
+          - rails_4.gemfile
+          - rails_5.0.gemfile
+          - rails_5.1.gemfile
+          - rails_5.2.gemfile
+          - rails_6.0.gemfile
+          - rails_6.1.gemfile
+          - rails_7.0.gemfile
+        include:
+          - ruby-version: '2.4'
+            gemfile: rails_4.gemfile
+            bundler-version: 1
+        exclude:
+        - ruby-version: '2.6'
+          gemfile: rails_4.gemfile
+        - ruby-version: '2.7'
+          gemfile: rails_4.gemfile
+        - ruby-version: '3.0'
+          gemfile: rails_4.gemfile
+        - ruby-version: '3.0'
+          gemfile: rails_5.0.gemfile
+        - ruby-version: '3.0'
+          gemfile: rails_5.1.gemfile
+        - ruby-version: '3.0'
+          gemfile: rails_5.2.gemfile
+        - ruby-version: '2.4'
+          gemfile: rails_6.0.gemfile
+        - ruby-version: '2.4'
+          gemfile: rails_6.1.gemfile
+        - ruby-version: '2.4'
+          gemfile: rails_7.0.gemfile
+        - ruby-version: '2.6'
+          gemfile: rails_7.0.gemfile
+
+    env:
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler: ${{ matrix.bundler-version }}
+        bundler-cache: true
+    - name: Run tests
+      run: bundle exec rake

data/.gitignore

@@ -12,4 +12,6 @@ bin/*
 *.swp
 Gemfile.lock
 .rvmrc
-.redcar/*
+.redcar/*
+gemfiles/*.lock
+spec/dummy/tmp

data/Appraisals

@@ -1,15 +1,32 @@
 appraise "rails_4" do
   gem "rails", "~> 4.2.10"
+  gem "rspec-rails", "~> 3.0"
+  gem "sqlite3", "~> 1.3.6"
 end
 
 appraise "rails_5.0" do
   gem "rails", "~> 5.0.7"
+  gem "rspec-rails", "~> 4.0"
+  gem "sqlite3", "~> 1.3.6"
 end
 
 appraise "rails_5.1" do
   gem "rails", "~> 5.1.6"
+  gem "rspec-rails", "~> 4.0"
 end
 
 appraise "rails_5.2" do
-  gem "rails", "5.2.0"
+  gem "rails", "~> 5.2.0"
+end
+
+appraise "rails_6.0" do
+  gem "rails", "~> 6.0.0"
+end
+
+appraise "rails_6.1" do
+  gem "rails", "~> 6.1.0"
+end
+
+appraise "rails_7.0" do
+  gem "rails", "~> 7.0.0.rc1"
 end

data/Gemfile

@@ -1,4 +1,4 @@
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 # Specify any dependencies in the gemspec
 gemspec

data/README.rdoc

@@ -1,4 +1,6 @@
-{<img src="https://secure.travis-ci.org/jpmcgrath/shortener.png?branch=master" alt="Build Status" />}[http://travis-ci.org/jpmcgrath/shortener]  {<img src="https://codeclimate.com/github/jpmcgrath/shortener/badges/gpa.svg" />}[https://codeclimate.com/github/jpmcgrath/shortener]
+{<img src="https://github.com/jpmcgrath/shortener/actions/workflows/ruby.yml/badge.svg" alt="Build Status" />}[https://github.com/jpmcgrath/shortener/actions]
+{<img src="https://codeclimate.com/github/jpmcgrath/shortener/badges/gpa.svg" />}[https://codeclimate.com/github/jpmcgrath/shortener]
+{<img src="https://badge.fury.io/rb/shortener.svg" alt="Gem version" />}[http://badge.fury.io/rb/shortener]
 
 = Shortener
 
@@ -14,7 +16,7 @@ The majority of the Shortener consists of three parts:
 
 === Dependencies
 
-Shortener is designed to work from within a Rails 3 and Rails 4 applications. It has dependancies Rails core components like ActiveRecord, ActionController, the rails routing engine and more.
+Shortener is designed to work from within a Ruby on Rail  applications. It has dependancies Rails core components like ActiveRecord, ActionController, the rails routing engine and more.
 
 === Ruby Version Support
 
@@ -61,15 +63,9 @@ migration:
 * The link records a count of how many times it has been “un-shortened”;
 * The link can be associated with a user, this allows for stats of the link usage for a particular user and other interesting things;
 
-=== Future improvements:
-
-* There has not been an attempt to remove ambiguous characters (i.e. 1 l and capital i, or 0 and O etc.) from the unique key generated for the link. This means people might copy the link incorrectly if copying the link by hand;
-* The system could pre-generate unique keys in advance, avoiding the database penalty when checking that a newly generated key is unique;
-* The system could store/cache the shortened URL if the url is to be continually rendered;
-
 == Installation
 
-Shortener is compatible with Rails v3, v4 & v5. To install, add to your Gemfile:
+Shortener is compatible with Rails v4, v5, & v6. To install, add to your Gemfile:
 
   gem 'shortener'
 
@@ -101,6 +97,15 @@ the upper and lower case charset, by including the following:
 
   Shortener.charset = :alphanumcase
 
+If you want to use a custom charset, you can create your own combination by creating an array of possible values, such as allowing underscore and dashes:
+
+  Shortener.charset = ("a".."z").to_a + (0..9).to_a + ["-", "_"]
+
+By default, <b>Shortener assumes URLs to be valid web URLs</b> and normalizes them in an effort to make sure there are no duplicate records generated for effectively same URLs with differences of only non-effective slash etc.
+You can control this option if it interferes for any of your logic. One common case is for mobile app links or universal links where normalization can corrupt the URLs of form <tt>appname://some_route</tt>
+
+  Shortener.auto_clean_url = true
+
 == Usage
 
 To generate a Shortened URL object for the URL "http://example.com" within your controller / models do the following:
@@ -143,7 +148,9 @@ And to access those URLs:
 
 You can pass in your own key when generating a shortened URL. This should be unique.
 
-  Shortener::ShortenedUrl.generate("example.com", owner: user, custom_key: "my-key")
+*Important:* Custom keys can't contain characters other than those defined in *Shortener.charset*. Default is numbers and lowercase a-z (See *Configuration*).
+
+  Shortener::ShortenedUrl.generate("example.com", owner: user, custom_key: "mykey")
 
   short_url("http://example.com", custom_key: 'yourkey')
 
@@ -152,11 +159,11 @@ You can pass in your own key when generating a shortened URL. This should be uni
 You can create expirable URLs.
 Probably, most of the time it would be used with owner:
 
-  Shortener::ShortenedUrl.generate("example.com/page", user, expires_at: 24.hours.since)
+  Shortener::ShortenedUrl.generate("example.com/page", owner: user, expires_at: 24.hours.since)
 
-You can omit owner passing nil instead:
+You can omit owner:
 
-  Shortener::ShortenedUrl.generate("example.com/page", nil, expires_at: 24.hours.since)
+  Shortener::ShortenedUrl.generate("example.com/page", expires_at: 24.hours.since)
 
 === Fresh Links
 
@@ -170,7 +177,7 @@ to create a fresh record, you can pass the following argument:
 === Forbidden keys
 
 You can ensure that records with forbidden keys will not be generated.
-In rails you can put next line into config/initializers/shortener.rb
+In Rails you can put next line into config/initializers/shortener.rb
 
   Shortener.forbidden_keys.concat %w(terms promo)
 
@@ -249,6 +256,18 @@ If you want more things to happen when a user accesses one of your short urls, y
 
   *note:* If no shortened URL is found, the url will be `default_redirect` or `/`
 
+=== Configuring a different database for shortened_urls table
+
+You can store a `shortened_urls` table in another database and connecting to it by creating a initializer with the following:
+
+```ruby
+ActiveSupport.on_load(:shortener_record) do
+  connects_to(database: { writing: :dbname, reading: :dbname_replica })
+end
+```
+
+**Note:** Please, replace `dbname` and `dbname_replica` to match your database configuration.
+
 == Origins
 
 For a bit of backstory to Shortener see this {blog post}[http://jamespmcgrath.com/a-simple-link-shortener-in-rails/].
@@ -280,4 +299,4 @@ To contribute:
 6. Create a new Pull Request
 7. Ensure the build is passing
 
-Note: We adhere to the community driven Ruby style guide: https://github.com/bbatsov/ruby-style-guide
+Note: We adhere to the community driven Ruby style guide: https://github.com/bbatsov/ruby-style-guide

data/app/models/shortener/record.rb

@@ -0,0 +1,5 @@
+class Shortener::Record < ActiveRecord::Base #:nodoc:
+  self.abstract_class = true
+end
+
+ActiveSupport.run_load_hooks :shortener_record, Shortener::Record

data/app/models/shortener/shortened_url.rb

@@ -1,10 +1,10 @@
-class Shortener::ShortenedUrl < ActiveRecord::Base
+class Shortener::ShortenedUrl < Shortener::Record
 
   REGEX_LINK_HAS_PROTOCOL = Regexp.new('\Ahttp:\/\/|\Ahttps:\/\/', Regexp::IGNORECASE)
 
   validates :url, presence: true
 
-  before_create :generate_unique_key
+  around_create :generate_unique_key
 
   # allows the shortened link to be associated with a user
   if ActiveRecord::VERSION::MAJOR >= 5
@@ -15,7 +15,7 @@ class Shortener::ShortenedUrl < ActiveRecord::Base
   end
 
   # exclude records in which expiration time is set and expiration time is greater than current time
-  scope :unexpired, -> { where(arel_table[:expires_at].eq(nil).or(arel_table[:expires_at].gt(::Time.current.to_s(:db)))) }
+  scope :unexpired, -> { where(arel_table[:expires_at].eq(nil).or(arel_table[:expires_at].gt(::Time.current))) }
 
   attr_accessor :custom_key
 
@@ -35,7 +35,7 @@ class Shortener::ShortenedUrl < ActiveRecord::Base
   def self.generate!(destination_url, owner: nil, custom_key: nil, expires_at: nil, fresh: false, category: nil)
     # if we get a shortened_url object with a different owner, generate
     # new one for the new owner. Otherwise return same object
-    result = if destination_url.is_a? Shortener::ShortenedUrl
+    if destination_url.is_a? Shortener::ShortenedUrl
       if destination_url.owner == owner
         destination_url
       else
@@ -52,14 +52,13 @@ class Shortener::ShortenedUrl < ActiveRecord::Base
       scope = owner ? owner.shortened_urls : self
       creation_method = fresh ? 'create' : 'first_or_create'
 
-      scope.where(url: clean_url(destination_url), category: category).send(
+      url_to_save = Shortener.auto_clean_url ? clean_url(destination_url) : destination_url
+      scope.where(url: url_to_save, category: category).send(
         creation_method,
         custom_key: custom_key,
         expires_at: expires_at
       )
     end
-
-    result
   end
 
   # return shortened url on success, nil on failure
@@ -127,15 +126,24 @@ class Shortener::ShortenedUrl < ActiveRecord::Base
 
   private
 
-  def generate_unique_key
+  def self.unique_key_candidate
+    charset = ::Shortener.key_chars
+    (0...::Shortener.unique_key_length).map{ charset[rand(charset.size)] }.join
+  end
+
+  def generate_unique_key(retries = Shortener.persist_retries)
     begin
       self.unique_key = custom_key || self.class.unique_key_candidate
       self.custom_key = nil
-    end while self.class.exists?(unique_key: unique_key) && custom_key.blank?
-  end
+    end while self.class.unscoped.exists?(unique_key: unique_key)
 
-  def self.unique_key_candidate
-    charset = ::Shortener.key_chars
-    (0...::Shortener.unique_key_length).map{ charset[rand(charset.size)] }.join
+    yield
+  rescue ActiveRecord::RecordNotUnique
+    if retries <= 0
+      raise
+    else
+      retries -= 1
+      retry
+    end
   end
 end

data/gemfiles/rails_4.gemfile

@@ -1,7 +1,9 @@
 # This file was generated by Appraisal
 
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 gem "rails", "~> 4.2.10"
+gem "rspec-rails", "~> 3.0"
+gem "sqlite3", "~> 1.3.6"
 
 gemspec path: "../"

data/gemfiles/rails_5.0.gemfile

@@ -1,7 +1,9 @@
 # This file was generated by Appraisal
 
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 gem "rails", "~> 5.0.7"
+gem "rspec-rails", "~> 4.0"
+gem "sqlite3", "~> 1.3.6"
 
 gemspec path: "../"

data/gemfiles/rails_5.1.gemfile

@@ -1,7 +1,8 @@
 # This file was generated by Appraisal
 
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 gem "rails", "~> 5.1.6"
+gem "rspec-rails", "~> 4.0"
 
 gemspec path: "../"

data/gemfiles/rails_5.2.gemfile

@@ -1,7 +1,7 @@
 # This file was generated by Appraisal
 
-source "http://rubygems.org"
+source "https://rubygems.org"
 
-gem "rails", "5.2.0"
+gem "rails", "~> 5.2.0"
 
 gemspec path: "../"

data/gemfiles/rails_6.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 6.0.0"
+
+gemspec path: "../"

data/gemfiles/rails_6.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 6.1.0"
+
+gemspec path: "../"

data/gemfiles/rails_7.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.0.0.rc1"
+
+gemspec path: "../"

data/lib/shortener/railtie.rb

@@ -6,8 +6,5 @@ class Shortener::Railtie < ::Rails::Railtie #:nodoc:
     ActiveSupport.on_load :active_record do
       extend Shortener::ActiveRecordExtension
     end
-    ActiveSupport.on_load :action_view do
-      include Shortener::ShortenerHelper
-    end
   end
 end

data/lib/shortener/version.rb

@@ -1,3 +1,3 @@
 module Shortener
-  VERSION = '0.8.0'.freeze
+  VERSION = '1.0.0'.freeze
 end

data/lib/shortener.rb

@@ -1,4 +1,4 @@
-require "active_support/dependencies"
+require "active_support"
 
 module Shortener
 
@@ -19,8 +19,9 @@ module Shortener
   self.unique_key_length = 5
 
   # character set to chose from:
-  #  :alphanum     - a-z0-9     -  has about 60 million possible combos
-  #  :alphanumcase - a-zA-Z0-9  -  has about 900 million possible combos
+  #  :alphanum     // a-z0-9                                       ## has about 60 million possible combos
+  #  :alphanumcase // a-zA-Z0-9                                    ## has about 900 million possible combos
+  #  ("a".."z").to_a + ("A".."Z").to_a + (0..9).to_a + ["-", "_"]  ## define a custom set
   mattr_accessor :charset
   self.charset = :alphanum
 
@@ -36,9 +37,16 @@ module Shortener
   mattr_accessor :ignore_robots
   self.ignore_robots = false
 
+  # persist_retries - number of retries on ActiveRecord::RecordNotUnique error
+  mattr_accessor :persist_retries
+  self.persist_retries = 3
+
+  # auto_clean_url - controls url cleaning mechanism, set it to false to disable
+  mattr_accessor :auto_clean_url
+  self.auto_clean_url = true
 
   def self.key_chars
-    CHARSETS[charset]
+    charset.is_a?(Symbol) ? CHARSETS[charset] : charset
   end
 end
 

data/shortener.gemspec

@@ -12,18 +12,18 @@ Gem::Specification.new do |s|
   s.authors                   = [ "James P. McGrath", "Michael Reinsch" ]
   s.email                     = [ "gems@jamespmcgrath.com", "michael@mobalean.com" ]
   s.homepage                  = "http://jamespmcgrath.com/projects/shortener"
-  s.rubyforge_project         = "shortener"
   s.required_rubygems_version = "> 2.1.0"
 
   s.add_dependency "voight_kampff", '~> 1.1.2'
 
   s.add_development_dependency "rails", '>= 3'
   s.add_development_dependency "sqlite3"
-  s.add_development_dependency "rspec-rails", '~> 3.6.0'
+  s.add_development_dependency "rspec-rails"
   s.add_development_dependency "shoulda-matchers"
   s.add_development_dependency "faker"
   s.add_development_dependency "byebug"
   s.add_development_dependency "appraisal"
+  s.add_development_dependency "capybara"
 
   s.executables = `git ls-files`.split("\n").map{|f| f =~ /^bin\/(.*)/ ? $1 : nil}.compact
   s.require_path = 'lib'

data/spec/controllers/shortened_urls_controller_spec.rb

@@ -132,6 +132,25 @@ describe Shortener::ShortenedUrlsController, type: :controller do
         end
       end
 
+      context "custom charset set" do
+        before do
+          Shortener::ShortenedUrl.delete_all
+          Shortener.charset = ("a".."z").to_a + ("A".."Z").to_a + (0..9).to_a + ["-", "_"]
+        end
+
+        after do
+          Shortener.charset = :alphanum
+        end
+
+        context 'key with valid characters' do
+          let(:key) { "cust-Key_123" }
+          let(:custom_url) { Shortener::ShortenedUrl.generate(Faker::Internet.url, custom_key: key) }
+          it 'allows if in custom charset' do
+            expect(custom_url.unique_key).to eq key
+          end
+        end
+      end
+
       context 'expired code' do
         let(:expired_url) { Shortener::ShortenedUrl.generate(Faker::Internet.url, expires_at: 1.hour.ago) }
         describe "GET show with expired code" do

data/spec/dummy/app/controllers/home_controller.rb

@@ -0,0 +1,3 @@
+class HomeController < ApplicationController
+  def show; end
+end

data/spec/dummy/app/views/home/show.html.erb

@@ -0,0 +1 @@
+Example: <%= short_url("http://example.com") %>

data/spec/dummy/app/views/layouts/application.html.erb

@@ -2,8 +2,6 @@
 <html>
 <head>
   <title>Dummy</title>
-  <%= stylesheet_link_tag :all %>
-  <%= javascript_include_tag :defaults %>
   <%= csrf_meta_tag %>
 </head>
 <body>

data/spec/dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Dummy::Application.routes.draw do
   get '/:id' => "shortener/shortened_urls#show"
-  root to: "application_controller#show"
+  root to: "home#show"
 end

data/spec/features/shortener_feature_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+describe Shortener, type: :feature do
+  it "includes 'short_url'-helper into public namespace" do
+    visit root_path
+    expect(page).to have_content(%r{http://www.example.com/[a-zA-Z0-9]{5}})
+  end
+end

data/spec/models/shortened_url_spec.rb

@@ -2,7 +2,7 @@
 require 'spec_helper'
 
 describe Shortener::ShortenedUrl, type: :model do
-  it { is_expected.to belong_to :owner }
+  it { is_expected.to belong_to(:owner).optional }
   it { is_expected.to validate_presence_of :url }
 
   describe '#generate!' do
@@ -123,10 +123,45 @@ describe Shortener::ShortenedUrl, type: :model do
 
       context "duplicate unique key" do
         let(:duplicate_key) { 'ABCDEF' }
-        it 'should try until it finds a non-dup key' do
+        context 'without retry' do
+          around do |spec|
+            tries = Shortener.persist_retries
+            Shortener.persist_retries = 0
+            spec.run
+            Shortener.persist_retries = tries
+          end
+
+          it 'finds a non-dup key' do
+            Shortener::ShortenedUrl.where(unique_key: duplicate_key).delete_all
+            Shortener::ShortenedUrl.create!(url: Faker::Internet.url, custom_key: duplicate_key)
+            short_url = Shortener::ShortenedUrl.create!(url: Faker::Internet.url, custom_key: duplicate_key)
+            expect(short_url).not_to be_nil
+            expect(short_url.unique_key).not_to be_nil
+            expect(short_url.unique_key).not_to eq duplicate_key
+          end
+
+          it 'unscoped query to finds a non-dup key' do
+            Shortener::ShortenedUrl.where(unique_key: duplicate_key).delete_all
+            Shortener::ShortenedUrl.create!(url: Faker::Internet.url, custom_key: duplicate_key)
+            short_url = Shortener::ShortenedUrl.where(
+              url: Faker::Internet.url, category: :test
+            ).create!(url: Faker::Internet.url, custom_key: duplicate_key)
+            expect(short_url).not_to be_nil
+            expect(short_url.unique_key).not_to be_nil
+            expect(short_url.unique_key).not_to eq duplicate_key
+          end
+        end
+
+        it 'use retry in case with DB unique constraint exception' do
           Shortener::ShortenedUrl.where(unique_key: duplicate_key).delete_all
           Shortener::ShortenedUrl.create!(url: Faker::Internet.url, custom_key: duplicate_key)
-          short_url = Shortener::ShortenedUrl.create!(url: Faker::Internet.url, unique_key: duplicate_key)
+
+          query = double
+          allow(query).to receive(:exists?).and_return(false)
+          allow(Shortener::ShortenedUrl).to receive(:unscoped).and_return(query).once
+          allow(Shortener::ShortenedUrl).to receive(:unscoped).and_call_original
+
+          short_url = Shortener::ShortenedUrl.create!(url: Faker::Internet.url, custom_key: duplicate_key)
           expect(short_url).not_to be_nil
           expect(short_url.unique_key).not_to be_nil
           expect(short_url.unique_key).not_to eq duplicate_key
@@ -142,8 +177,26 @@ describe Shortener::ShortenedUrl, type: :model do
         it 'finds the shortened url from slashless oath' do
           expect(Shortener::ShortenedUrl.generate!(path)).to eq existing_shortened_url
         end
-        it "should look up exsiting URL" do
-          expect(Shortener::ShortenedUrl.generate!("/#{path}")).to eq existing_shortened_url
+
+        context 'with auto_clean_url enabled by default' do
+          it "looks up existing cleaned URL" do
+            expect(Shortener::ShortenedUrl.generate!("/#{path}")).to eq existing_shortened_url
+          end
+        end
+
+        context 'with auto_clean_url disabled' do
+          around do |spec|
+            tries = Shortener.auto_clean_url
+            Shortener.auto_clean_url = false
+            spec.run
+            Shortener.auto_clean_url = tries
+          end
+
+          it "does not look up existing cleaned URL" do
+            shortened_url = Shortener::ShortenedUrl.generate!("/#{path}")
+            expect(shortened_url).not_to eq existing_shortened_url
+            expect(shortened_url.url).to eq "/#{path}"
+          end
         end
       end
     end

data/spec/spec_helper.rb

@@ -7,8 +7,10 @@ require 'rspec/rails'
 require 'shoulda/matchers'
 require 'byebug'
 require 'faker'
+require 'capybara/rspec'
 
 Rails.backtrace_cleaner.remove_silencers!
+ActiveRecord::Base.logger = Logger.new(STDOUT) if ENV['LOG_STDOUT']
 
 Shoulda::Matchers.configure do |config|
   config.integrate do |with|
@@ -18,8 +20,11 @@ Shoulda::Matchers.configure do |config|
 end
 
 # Run any available migration
+migration_path = File.expand_path("../dummy/db/migrate/", __FILE__)
 if ActiveRecord::Migrator.respond_to?(:migrate)
-  ActiveRecord::Migrator.migrate File.expand_path("../dummy/db/migrate/", __FILE__)
+  ActiveRecord::Migrator.migrate(migration_path)
+elsif Rails::VERSION::MAJOR < 6
+  ActiveRecord::MigrationContext.new(migration_path).migrate
 else
-  ActiveRecord::MigrationContext.new(File.expand_path("../dummy/db/migrate/", __FILE__)).migrate
+  ActiveRecord::MigrationContext.new(migration_path, ActiveRecord::Base.connection.schema_migration).migrate
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: shortener
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 1.0.0
 platform: ruby
 authors:
 - James P. McGrath
 - Michael Reinsch
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-07-27 00:00:00.000000000 Z
+date: 2022-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: voight_kampff
@@ -57,16 +57,16 @@ dependencies:
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.6.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.6.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: shoulda-matchers
   requirement: !ruby/object:Gem::Requirement
@@ -123,6 +123,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Shortener is a Rails Engine Gem that makes it easy to create and interpret
   shortened URLs on your own domain from within your Rails application. Once installed
   Shortener will generate, store URLS and "unshorten" shortened URLs for your applications
@@ -134,8 +148,8 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
-- ".travis.yml"
 - Appraisals
 - Gemfile
 - MIT-LICENSE
@@ -143,11 +157,15 @@ files:
 - Rakefile
 - app/controllers/shortener/shortened_urls_controller.rb
 - app/helpers/shortener/shortener_helper.rb
+- app/models/shortener/record.rb
 - app/models/shortener/shortened_url.rb
 - gemfiles/rails_4.gemfile
 - gemfiles/rails_5.0.gemfile
 - gemfiles/rails_5.1.gemfile
 - gemfiles/rails_5.2.gemfile
+- gemfiles/rails_6.0.gemfile
+- gemfiles/rails_6.1.gemfile
+- gemfiles/rails_7.0.gemfile
 - lib/generators/shortener/shortener_generator.rb
 - lib/generators/shortener/templates/migration.rb
 - lib/shortener.rb
@@ -160,9 +178,12 @@ files:
 - spec/controllers/shortened_urls_controller_spec.rb
 - spec/dummy/.gitignore
 - spec/dummy/Rakefile
+- spec/dummy/app/assets/config/manifest.js
 - spec/dummy/app/controllers/application_controller.rb
+- spec/dummy/app/controllers/home_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/models/user.rb
+- spec/dummy/app/views/home/show.html.erb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/config.ru
 - spec/dummy/config/application.rb
@@ -188,6 +209,7 @@ files:
 - spec/dummy/db/schema.rb
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/features/shortener_feature_spec.rb
 - spec/helpers/shortener_helper_spec.rb
 - spec/models/shortened_url_spec.rb
 - spec/models/user_spec.rb
@@ -196,7 +218,7 @@ files:
 homepage: http://jamespmcgrath.com/projects/shortener
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -211,9 +233,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 2.1.0
 requirements: []
-rubyforge_project: shortener
-rubygems_version: 2.5.1
-signing_key: 
+rubyforge_project:
+rubygems_version: 2.7.6
+signing_key:
 specification_version: 4
 summary: Shortener is a Rails Engine that makes it easy to create shortened URLs for
   your rails application.

data/.travis.yml

@@ -1,16 +0,0 @@
-language: ruby
-
-rvm:
-  - 2.2.10
-  - 2.3
-  - 2.4
-  - 2.5
-
-matrix:
-  fast_finish: true
-
-gemfile:
-  - gemfiles/rails_4.gemfile
-  - gemfiles/rails_5.0.gemfile
-  - gemfiles/rails_5.1.gemfile
-  - gemfiles/rails_5.2.gemfile