slimmer 14.0.0 → 15.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5e29edb1500e56cda58cd5b43463f19beb2b8e1e53a2a0c0c7bf4f88b1542c3d
-  data.tar.gz: 90a425692e649c7f941f9f45a8966babff2f31dc888ada70d47ea86b7b365002
+  metadata.gz: 69efc7036b8f43e79f0a658b8cb7a4d1b8a3a6d6e7083adc4f0d7c5f94660de4
+  data.tar.gz: 1ea0fc004cc8eaf7b51db1071aa6d681bd178aa4f28e311a5c8c896330a2a0d0
 SHA512:
-  metadata.gz: 2132180bb938d0db3a8dfe094a45b173254b52ec6847977480ae14a9ceec8a2c44da0dd54bfc7c4f062a2e050710ac5cdf3137b46a238993c40b148df631032f
-  data.tar.gz: cb45e29c4f0d6d3874352a89f29146cc154279430a58535fe3e9ff6d46e53900171534a29209e5432f203073fe1708f578181d800f595d07844be52eda8d15f4
+  metadata.gz: 526e412c02dec4300125ef6047212538dea07fecf9842b0cfb2716e7f962c582b76ff921107ca3b039c053718a310adc29fe6d7035fb3544c5b0f66b73b83b4e
+  data.tar.gz: d414a38146b156fc6ba454798fe79741bff8b3f3015d42dad4b991e8f5f48e1bf9f0c46b2de7cb079c78b9beb62eeb9925c23ed5514c5a67d80b5e249b31fa83

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+# 15.0.0
+
+* BREAKING: Test templates no longer embed links to production JavaScript files,
+  apps should provide their own dependencies.
+
 # 14.0.0
 
 * Remove handling of non-200 responses (#245)

data/README.md

@@ -96,3 +96,7 @@ Add the following code to spec/spec_helper:
 ```rb
 require 'slimmer/rspec'
 ```
+
+## Licence
+
+[MIT License](LICENCE)

data/Rakefile

@@ -1,8 +1,9 @@
-# -*- encoding: utf-8 -*-
-
 require "bundler/gem_tasks"
 require "rdoc/task"
 require "rake/testtask"
+require "rubocop/rake_task"
+
+RuboCop::RakeTask.new
 
 Dir.glob("lib/tasks/*.rake").each { |r| import r }
 
@@ -18,4 +19,4 @@ Rake::TestTask.new("test") do |t|
   t.verbose = true
 end
 
-task default: %i[test]
+task default: %i[test rubocop]

data/lib/slimmer/app.rb

@@ -35,7 +35,7 @@ module Slimmer
       if response_can_be_rewritten?(status, headers)
         response = Rack::Response.new(body, status, headers)
 
-        if !skip_slimmer?(env, response)
+        unless skip_slimmer?(env, response)
           status, headers, body = rewrite_response(env, response)
         end
       end
@@ -61,7 +61,7 @@ module Slimmer
     end
 
     def skip_slimmer_header?(response)
-      !!response.headers[Headers::SKIP_HEADER]
+      response.headers.key?(Headers::SKIP_HEADER)
     end
 
     def s(body)

data/lib/slimmer/headers.rb

@@ -9,17 +9,17 @@ module Slimmer
 
     # @private
     SLIMMER_HEADER_MAPPING = {
-      application_name:     "Application-Name",
-      format:               "Format",
-      page_owner:           "Page-Owner",
-      organisations:        "Organisations",
-      world_locations:      "World-Locations",
-      result_count:         "Result-Count",
-      search_parameters:    "Search-Parameters",
-      section:              "Section",
-      skip:                 "Skip",
-      template:             "Template",
-      remove_search:        "Remove-Search",
+      application_name: "Application-Name",
+      format: "Format",
+      page_owner: "Page-Owner",
+      organisations: "Organisations",
+      world_locations: "World-Locations",
+      result_count: "Result-Count",
+      search_parameters: "Search-Parameters",
+      section: "Section",
+      skip: "Skip",
+      template: "Template",
+      remove_search: "Remove-Search",
     }.freeze
 
     # @private

data/lib/slimmer/processors/tag_mover.rb

@@ -1,10 +1,10 @@
 module Slimmer::Processors
   class TagMover
     def filter(src, dest)
-      move_tags(src, dest, "script", dest_node: "body", keys: %w(src inner_html))
+      move_tags(src, dest, "script", dest_node: "body", keys: %w[src inner_html])
       move_tags(src, dest, "link",   must_have: %w[href])
-      move_tags(src, dest, "meta",   must_have: %w(name content), keys: %w[name content http-equiv], insertion_location: :top)
-      move_tags(src, dest, "meta",   must_have: %w(property content), keys: %w(property content), insertion_location: :top)
+      move_tags(src, dest, "meta",   must_have: %w[name content], keys: %w[name content http-equiv], insertion_location: :top)
+      move_tags(src, dest, "meta",   must_have: %w[property content], keys: %w[property content], insertion_location: :top)
     end
 
     def include_tag?(node, min_attrs)
@@ -39,15 +39,15 @@ module Slimmer::Processors
       dest_node = opts[:dest_node] || "head"
 
       src.css(type).each do |node|
-        if include_tag?(node, min_attrs) && !already_there.include?(tag_fingerprint(node, comparison_attrs))
-          node = wrap_node(src, node)
-          node.remove
-
-          if opts[:insertion_location] == :top
-            dest.at_xpath("/html/#{dest_node}").prepend_child(node)
-          else
-            dest.at_xpath("/html/#{dest_node}") << node
-          end
+        next unless include_tag?(node, min_attrs) && !already_there.include?(tag_fingerprint(node, comparison_attrs))
+
+        node = wrap_node(src, node)
+        node.remove
+
+        if opts[:insertion_location] == :top
+          dest.at_xpath("/html/#{dest_node}").prepend_child(node)
+        else
+          dest.at_xpath("/html/#{dest_node}") << node
         end
       end
     end

data/lib/slimmer/skin.rb

@@ -5,12 +5,12 @@ module Slimmer
   class Skin
     attr_accessor :asset_host, :logger, :strict, :options
 
-    def initialize options = {}
+    def initialize(options = {})
       @options = options
       @asset_host = options[:asset_host]
 
       @logger = options[:logger] || NullLogger.instance
-      @strict = options[:strict] || %w{development test}.include?(ENV["RACK_ENV"])
+      @strict = options[:strict] || %w[development test].include?(ENV["RACK_ENV"])
     end
 
     def template(template_name)
@@ -42,7 +42,7 @@ module Slimmer
       doc = Nokogiri::HTML.parse(html)
       if strict
         errors = doc.errors.select(&:error?).reject { |e| ignorable?(e) }
-        if !errors.empty?
+        unless errors.empty?
           error = errors.first
           message = "In #{description_for_error_message}: '#{error.message}' at line #{error.line} col #{error.column} (code #{error.code}).\n"
           message << "Add ?skip_slimmer=1 to the url to show the raw backend request.\n\n"
@@ -59,7 +59,7 @@ module Slimmer
       lines = [""] + html.split("\n")
       from = [1, error.line - context_size].max
       to = [lines.size - 1, error.line + context_size].min
-      context = (from..to).zip(lines[from..to]).map { |lineno, line| "%4d: %s" % [lineno, line] }
+      context = (from..to).zip(lines[from..to]).map { |lineno, line| sprintf("%4d: %s", lineno, line) }
       marker = " " * (error.column - 1) + "-----v"
       context.insert(context_size, marker)
       context.join("\n")

data/lib/slimmer/template.rb

@@ -1,16 +1,15 @@
 module Slimmer
   module Template
-    def self.included into
+    def self.included(into)
       into.extend ClassMethods
     end
 
-    def slimmer_template template_name
+    def slimmer_template(template_name)
       response.headers[Slimmer::Headers::TEMPLATE_HEADER] = template_name.to_s
     end
 
-
     module ClassMethods
-      def slimmer_template template_name
+      def slimmer_template(template_name)
         after_action do
           response.headers[Slimmer::Headers::TEMPLATE_HEADER] ||= template_name.to_s
         end

data/lib/slimmer/test.rb

@@ -2,10 +2,10 @@ require "slimmer/skin"
 
 module Slimmer
   class Skin
-    def load_template name
+    def load_template(name)
       logger.debug "Slimmer: TEST MODE - Loading fixture template from #{__FILE__}"
       if name =~ /\A(.*)\.raw\z/
-        %{<div id="test-#{$1}"></div>}
+        %(<div id="test-#{Regexp.last_match(1)}"></div>)
       elsif File.exist?(template_path = File.join(File.dirname(__FILE__), "test_templates", "#{name}.html"))
         File.read(template_path)
       else

data/lib/slimmer/test_templates/header_footer_only.html

@@ -19,9 +19,5 @@
     <div id="wrapper"></div>
 
     <footer id="footer"></footer>
-
-    <script src="https://assets.publishing.service.gov.uk/static/govuk-template.js" type="text/javascript"></script>
-    <script src="https://assets.publishing.service.gov.uk/static/libs/jquery/jquery-1.7.2.js" type="text/javascript"></script>
-    <script src="https://assets.publishing.service.gov.uk/static/header-footer-only.js" type="text/javascript"></script>
   </body>
 </html>

data/lib/slimmer/test_templates/wrapper.html

@@ -19,9 +19,5 @@
     <div id="wrapper"></div>
 
     <footer id="footer"></footer>
-
-    <script src="https://assets.publishing.service.gov.uk/static/govuk-template.js" type="text/javascript"></script>
-    <script src="https://assets.publishing.service.gov.uk/static/libs/jquery/jquery-1.7.2.js" type="text/javascript"></script>
-    <script src="https://assets.publishing.service.gov.uk/static/application.js" type="text/javascript"></script>
   </body>
 </html>

data/lib/slimmer/version.rb

@@ -1,3 +1,3 @@
 module Slimmer
-  VERSION = "14.0.0".freeze
+  VERSION = "15.0.0".freeze
 end

data/lib/tasks/slimmer.rake

@@ -6,9 +6,9 @@ namespace :slimmer do
     path_to_static = "../static/public"
     path_to_public = "public"
     commands = ["cd #{path_to_public}"]
-    Dir.glob("../static/public/*") { |f|
+    Dir.glob("../static/public/*") do |f|
       commands << "ln -s #{path_to_static}/#{f}"
-    }
+    end
     commands << ["cd .."]
     run commands.join(" && ")
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slimmer
 version: !ruby/object:Gem::Version
-  version: 14.0.0
+  version: 15.0.0
 platform: ruby
 authors:
 - GOV.UK Dev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-14 00:00:00.000000000 Z
+date: 2020-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -184,14 +184,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement