versioncake 4.0.1 → 4.1.1

data/lib/versioncake/view_additions_rails7.rb

@@ -0,0 +1,155 @@
+require 'action_view'
+
+# register an addition detail for the lookup context to understand,
+# this will allow us to have the versions available upon lookup in
+# the resolver.
+ActionView::LookupContext.register_detail(:versions){ [] }
+
+ActionView::Resolver::PathParser.class_eval do
+    def build_path_regex
+      handlers = ActionView::Template::Handlers.extensions.map { |x| Regexp.escape(x) }.join("|")
+      formats = ActionView::Template::Types.symbols.map { |x| Regexp.escape(x) }.join("|")
+      locales = "[a-z]{2}(?:-[A-Z]{2})?"
+      variants = "[^.]*"
+
+      %r{
+          \A
+          (?:(?<prefix>.*)/)?
+          (?<partial>_)?
+          (?<action>.*?)
+          (?:\.(?<locale>#{locales}))??
+          (?:\.(?<format>#{formats}))??
+          (?:\+(?<variant>#{variants}))??
+          (?:\.(?<versions>v[0-9]+))??
+          (?:\.(?<handler>#{handlers}))?
+          \z
+        }x
+    end
+
+    def parse(path)
+      @regex ||= build_path_regex
+      match = @regex.match(path)
+      path = ActionView::TemplatePath.build(match[:action], match[:prefix] || "", !!match[:partial])
+      details = ActionView::TemplateDetails.new(
+        match[:locale]&.to_sym,
+        match[:handler]&.to_sym,
+        match[:format]&.to_sym,
+        match[:variant]&.to_sym,
+        match[:versions]&.to_sym,
+      )
+
+      ActionView::Resolver::PathParser::ParsedPath.new(path, details)
+    end
+  end
+
+ActionView::TemplateDetails::Requested.class_eval do
+  attr_reader :locale, :handlers, :formats, :variants, :versions
+  attr_reader :locale_idx, :handlers_idx, :formats_idx, :variants_idx, :versions_idx
+
+  def initialize(locale:, handlers:, formats:, variants:, versions:)
+    @locale = locale
+    @handlers = handlers
+    @formats = formats
+    @variants = variants
+    @versions = versions
+
+    @locale_idx   = build_idx_hash(locale)
+    @handlers_idx = build_idx_hash(handlers)
+    @formats_idx  = build_idx_hash(formats)
+    @versions_idx  = build_idx_hash(versions)
+    if variants == :any
+      @variants_idx = ActionView::TemplateDetails::Requested::ANY_HASH
+    else
+      @variants_idx = build_idx_hash(variants)
+    end
+  end
+end
+
+ActionView::TemplateDetails.class_eval do
+  attr_reader :locale, :handler, :format, :variant, :version
+
+  def initialize(locale, handler, format, variant, version)
+    @locale = locale
+    @handler = handler
+    @format = format
+    @variant = variant
+    @version = version
+  end
+
+  def matches?(requested)
+    requested.formats_idx[@format] &&
+      requested.locale_idx[@locale] &&
+      requested.versions_idx[@version] &&
+      requested.variants_idx[@variant] &&
+      requested.handlers_idx[@handler]
+  end
+
+  def sort_key_for(requested)
+    [
+      requested.formats_idx[@format],
+      requested.locale_idx[@locale],
+      requested.versions_idx[@version],
+      requested.variants_idx[@variant],
+      requested.handlers_idx[@handler]
+    ]
+  end
+end
+
+ActionView::Template.class_eval do
+  def initialize(source, identifier, handler, locals:, format: nil, variant: nil, virtual_path: nil, version: nil)
+    @source            = source
+    @identifier        = identifier
+    @handler           = handler
+    @compiled          = false
+    @locals            = locals
+    @virtual_path      = virtual_path
+
+    @variable = if @virtual_path
+      base = @virtual_path.end_with?("/") ? "" : ::File.basename(@virtual_path)
+      base =~ /\A_?(.*?)(?:\.\w+)*\z/
+      $1.to_sym
+    end
+
+    @format            = format
+    @version           = version
+    @variant           = variant
+    @compile_mutex     = Mutex.new
+  end
+
+  def marshal_dump # :nodoc:
+    [ @source, @identifier, @handler, @compiled, @locals, @virtual_path, @format, @variant, @version ]
+  end
+
+  def marshal_load(array) # :nodoc:
+    @source, @identifier, @handler, @compiled, @locals, @virtual_path, @format, @variant, @version = *array
+    @compile_mutex = Mutex.new
+  end
+
+  # the identifier method name filters out numbers,
+  # but we want to preserve them for v1 etc.
+  # TODO: Consider updating to match current implementation:
+  # short_identifier.tr("^a-z_", "_")
+  def identifier_method_name #:nodoc:
+    short_identifier.gsub(/[^a-z0-9_]/, '_')
+  end
+end
+
+ActionView::UnboundTemplate.class_eval do
+  delegate :version, to: :@details
+
+  def build_template(locals)
+    ActionView::Template.new(
+      @source,
+      @identifier,
+      details.handler_class,
+
+      format: details.format_or_default,
+      variant: variant&.to_s,
+      virtual_path: @virtual_path,
+      version: version&.to_s,
+
+      locals: locals.map(&:to_s)
+    )
+  end
+end
+

data/lib/versioncake.rb

@@ -26,6 +26,7 @@ if defined?(Rails)
   require 'versioncake/controller_additions'
   require 'versioncake/view_additions'
   require 'versioncake/engine'
+  require 'versioncake/railtie'
 end
 
 module VersionCake
@@ -38,4 +39,5 @@ module VersionCake
   def self.setup
     yield self.config
   end
-end
+end
+

data/spec/integration/controller/unversioned_controller_spec.rb

@@ -6,7 +6,7 @@ describe UnversionedController, type: :controller do
   context '#index' do
     render_views
 
-    it { expect(response).to be_success }
+    it { expect(response).to be_successful }
     it { expect(response.body).to eq 'unversioned index' }
   end
 end

data/spec/integration/view/render_spec.rb

@@ -3,7 +3,14 @@ require './spec/rails_helper'
 describe ActionView::Base do
   let(:path) { ActionView::FileSystemResolver.new('./spec/fixtures') }
   let(:view_paths) { ActionView::PathSet.new([path]) }
-  let(:view) { ActionView::Base.new(view_paths) }
+  let(:view) do
+    clazz = if ActionPack::VERSION::MAJOR >= 7
+      ActionView::Base.with_empty_template_cache
+    else
+      ActionView::Base
+    end
+    clazz.new(ActionView::LookupContext.new(view_paths), [], nil)
+  end
   let(:version_override) { nil }
   subject { view.render(template: 'templates/versioned', versions: version_override) }
 

data/spec/integration/view/view_additions_rails5_spec.rb

@@ -0,0 +1,67 @@
+require './spec/rails_helper'
+
+if ActionPack::VERSION::MAJOR == 5
+  describe ActionView::PathResolver do
+    let(:resolver) { ActionView::PathResolver.new }
+
+    context '#extract_handler_and_format_and_variant' do
+      subject do
+        resolver.extract_handler_and_format_and_variant("application.#{extension}")
+      end
+
+      let(:variant) { subject[2].to_s }
+      let(:format) { subject[1].to_s }
+      let(:handler) { subject[0] }
+
+      context 'when only handler and format are present' do
+        let(:extension) { 'html.erb' }
+
+        it do
+          expect(format).to eq 'text/html'
+          expect(variant).to be_empty
+          expect(handler).to be_a ActionView::Template::Handlers::ERB
+        end
+      end
+
+      context 'when handler, format and version are present' do
+        let(:extension) { 'json.v1.jbuilder' }
+
+        it do
+          expect(format).to eq 'application/json'
+          expect(variant).to be_empty
+          expect(handler).to be_a ActionView::Template::Handlers::Raw
+        end
+      end
+
+      context 'when handler, format and locale are present' do
+        let(:extension) { 'en.json.jbuilder' }
+
+        it do
+          expect(format).to eq 'application/json'
+          expect(variant).to be_empty
+          expect(handler).to be_a ActionView::Template::Handlers::Raw
+        end
+      end
+
+      context 'when handler, format, locale and version are present' do
+        let(:extension) { 'en.json.v1.jbuilder' }
+
+        it do
+          expect(format).to eq 'application/json'
+          expect(variant).to be_empty
+          expect(handler).to be_a ActionView::Template::Handlers::Raw
+        end
+      end
+
+      context 'when handler, format, variant and version are present' do
+        let(:extension) { 'json+tablet.v1.jbuilder' }
+
+        it do
+          expect(format).to eq 'application/json'
+          expect(variant).to eq 'tablet'
+          expect(handler).to be_a ActionView::Template::Handlers::Raw
+        end
+      end
+    end
+  end
+end

data/spec/integration/view/view_additions_rails6_spec.rb

@@ -0,0 +1,44 @@
+require './spec/rails_helper'
+
+if ActionPack::VERSION::MAJOR == 6
+  describe ActionView::PathResolver do
+    let(:resolver) { ActionView::PathResolver.new }
+
+    context '#extract_handler_and_format' do
+      subject(:template_format) do
+        _, format = resolver.extract_handler_and_format("application.#{template_extension}", nil)
+        format.to_s
+      end
+
+      context 'when only handler and format are present' do
+        let(:template_extension) { 'html.erb' }
+
+        it { expect(template_format).to eq 'text/html' }
+      end
+
+      context 'when handler, format and version are present' do
+        let(:template_extension) { 'json.v1.jbuilder' }
+
+        it { expect(template_format).to eq 'application/json' }
+      end
+
+      context 'when handler, format and locale are present' do
+        let(:template_extension) { 'en.json.jbuilder' }
+
+        it { expect(template_format).to eq 'application/json' }
+      end
+
+      context 'when handler, format, locale and version are present' do
+        let(:template_extension) { 'en.json.v1.jbuilder' }
+
+        it { expect(template_format).to eq 'application/json' }
+      end
+
+      context 'when handler, format, variant and version are present' do
+        let(:template_extension) { 'application.json+tablet.v1.jbuilder' }
+
+        it { expect(template_format).to eq 'application/json' }
+      end
+    end
+  end
+end

data/spec/integration/view/view_additions_rails7_spec.rb

@@ -0,0 +1,76 @@
+require './spec/rails_helper'
+
+if ActionPack::VERSION::MAJOR == 7
+  describe ActionView::Resolver::PathParser do
+    let(:resolver) { ActionView::Resolver::PathParser.new }
+
+    context '#extract_handler_and_format' do
+      subject(:parsed_path) do
+        resolver.parse("application.#{template_extension}")
+      end
+
+      context 'when only handler and format are present' do
+        let(:template_extension) { 'html.erb' }
+
+        it do
+          expect(parsed_path.details.format).to eq :html
+          expect(parsed_path.details.handler).to eq :erb
+        end
+      end
+
+      context 'when handler, format and version are present' do
+        let(:template_extension) { 'json.v1.builder' }
+
+        it do
+          expect(parsed_path.details.format).to eq :json
+          expect(parsed_path.details.handler).to eq :builder
+          expect(parsed_path.details.version).to eq :v1
+        end
+      end
+
+      context 'when handler, format and locale are present' do
+        let(:template_extension) { 'en.json.builder' }
+
+        it do
+          expect(parsed_path.details.locale).to eq :en
+          expect(parsed_path.details.format).to eq :json
+          expect(parsed_path.details.handler).to eq :builder
+          expect(parsed_path.details.version).to eq nil
+        end
+      end
+
+      context 'when handler, format, locale and version are present' do
+        let(:template_extension) { 'en.json.v1.builder' }
+
+        it do
+          expect(parsed_path.details.format).to eq :json
+          expect(parsed_path.details.handler).to eq :builder
+          expect(parsed_path.details.version).to eq :v1
+        end
+      end
+
+      context 'when handler, format, variant and version are present' do
+        let(:template_extension) { 'json+tablet.v1.builder' }
+
+        it do
+          expect(parsed_path.details.variant).to eq :tablet
+          expect(parsed_path.details.format).to eq :json
+          expect(parsed_path.details.handler).to eq :builder
+          expect(parsed_path.details.version).to eq :v1
+        end
+      end
+
+      context 'when handler, format, variant, locale and version are present' do
+        let(:template_extension) { 'en.json+tablet.v1.builder' }
+
+        it do
+          expect(parsed_path.details.locale).to eq :en
+          expect(parsed_path.details.variant).to eq :tablet
+          expect(parsed_path.details.format).to eq :json
+          expect(parsed_path.details.handler).to eq :builder
+          expect(parsed_path.details.version).to eq :v1
+        end
+      end
+    end
+  end
+end

data/spec/unit/strategies/extraction_strategy_spec.rb

@@ -60,13 +60,13 @@ describe VersionCake::ExtractionStrategy do
   it "it fails to create a custom strategy for a proc with no parameters" do
     expect do
       VersionCake::ExtractionStrategy.lookup(lambda{})
-    end.to raise_error(Exception)
+    end.to raise_error(VersionCake::ExtractionStrategy::InvalidStrategyError)
   end
 
   it "it raises error when no strategy is found" do
     expect do
       VersionCake::ExtractionStrategy.lookup(:fake_extraction)
-    end.to raise_error(Exception)
+    end.to raise_error(VersionCake::ExtractionStrategy::InvalidStrategyError)
   end
 
   describe '.list' do

data/spec/unit/version_checker_spec.rb

@@ -10,7 +10,7 @@ describe VersionCake::VersionChecker do
   end
   let(:version) { }
   subject do
-    checker = VersionCake::VersionChecker.new(resource, version)
+    checker = VersionCake::VersionChecker.new(version, resource)
     checker.execute
   end
 

data/spec/unit/versioned_request_spec.rb

@@ -24,12 +24,5 @@ describe VersionCake::VersionedRequest do
       it { expect(versioned_request.version).to be_nil }
       it { expect(versioned_request.failed).to be_falsey }
     end
-
-    context 'with a strategy failure' do
-      let(:strategies) { [ lambda { raise 'Failed extraction' } ] }
-
-      it { expect(versioned_request.version).to be_nil }
-      it { expect(versioned_request.failed).to be_truthy }
-    end
   end
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: versioncake
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.1.1
 platform: ruby
 authors:
 - Jim Jones
 - Ben Willis
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-03-12 00:00:00.000000000 Z
+date: 2022-04-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -146,9 +146,9 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
 - Appraisals
 - CHANGELOG.md
 - CONTRIBUTING.md
@@ -157,6 +157,7 @@ files:
 - README.md
 - RELEASE.md
 - Rakefile
+- SECURITY.md
 - bin/versioncake
 - gemfiles/.bundle/config
 - gemfiles/rails5.0.gemfile
@@ -165,6 +166,8 @@ files:
 - gemfiles/rails5.2.gemfile.lock
 - gemfiles/rails6.0.gemfile
 - gemfiles/rails6.0.gemfile.lock
+- gemfiles/rails7.0.gemfile
+- gemfiles/rails7.0.gemfile.lock
 - images/versioncake-logo450x100.png
 - lib/generators/templates/versioncake.rb
 - lib/generators/versioncake/install_generator.rb
@@ -175,6 +178,7 @@ files:
 - lib/versioncake/engine.rb
 - lib/versioncake/exceptions.rb
 - lib/versioncake/rack/middleware.rb
+- lib/versioncake/railtie.rb
 - lib/versioncake/response_strategy/base.rb
 - lib/versioncake/response_strategy/http_content_type_strategy.rb
 - lib/versioncake/response_strategy/http_header_strategy.rb
@@ -194,6 +198,9 @@ files:
 - lib/versioncake/versioned_resource.rb
 - lib/versioncake/versioned_response_service.rb
 - lib/versioncake/view_additions.rb
+- lib/versioncake/view_additions_rails5.rb
+- lib/versioncake/view_additions_rails6.rb
+- lib/versioncake/view_additions_rails7.rb
 - spec/fixtures/partials/_versioned.erb
 - spec/fixtures/partials/_versioned.v1.erb
 - spec/fixtures/partials/_versioned.v2.erb
@@ -212,7 +219,9 @@ files:
 - spec/integration/controller/unversioned_controller_spec.rb
 - spec/integration/rack/middleware_regression_spec.rb
 - spec/integration/view/render_spec.rb
-- spec/integration/view/view_additions_spec.rb
+- spec/integration/view/view_additions_rails5_spec.rb
+- spec/integration/view/view_additions_rails6_spec.rb
+- spec/integration/view/view_additions_rails7_spec.rb
 - spec/rails_helper.rb
 - spec/spec_helper.rb
 - spec/test_app/Rakefile
@@ -252,7 +261,7 @@ homepage: http://bwillis.github.io/versioncake
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -267,8 +276,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Easily render versions of your rails views.
 test_files: []

data/.travis.yml

@@ -1,20 +0,0 @@
-language: ruby
-
-before_install: gem install bundler && bundle update --bundler
-
-rvm:
-  - 2.3.3
-  - 2.4.7
-  - 2.5.6
-
-gemfile:
-  - gemfiles/rails5.0.gemfile
-  - gemfiles/rails5.2.gemfile
-  - gemfiles/rails6.0.gemfile
-
-matrix:
-  exclude:
-    - rvm: 2.3.3
-      gemfile: gemfiles/rails6.0.gemfile
-    - rvm: 2.4.7
-      gemfile: gemfiles/rails6.0.gemfile

data/spec/integration/view/view_additions_spec.rb

@@ -1,42 +0,0 @@
-require './spec/rails_helper'
-
-describe ActionView::PathResolver do
-  let(:resolver) { ActionView::PathResolver.new }
-
-  context '#extract_handler_and_format' do
-    subject(:template_format) do
-      _, format = resolver.extract_handler_and_format("application.#{template_extension}", nil)
-      format.to_s
-    end
-
-    context 'when only handler and format are present' do
-      let(:template_extension) { 'html.erb' }
-
-      it { expect(template_format).to eq 'text/html' }
-    end
-
-    context 'when handler, format and version are present' do
-      let(:template_extension) { 'json.v1.jbuilder' }
-
-      it { expect(template_format).to eq 'application/json' }
-    end
-
-    context 'when handler, format and locale are present' do
-      let(:template_extension) { 'en.json.jbuilder' }
-
-      it { expect(template_format).to eq 'application/json' }
-    end
-
-    context 'when handler, format, locale and version are present' do
-      let(:template_extension) { 'en.json.v1.jbuilder' }
-
-      it { expect(template_format).to eq 'application/json' }
-    end
-
-    context 'when handler, format, variant and version are present' do
-      let(:template_extension) { 'application.json+tablet.v1.jbuilder' }
-
-      it { expect(template_format).to eq 'application/json' }
-    end
-  end
-end