spec_views 1.1.1 → 2.0.0

data/app/controllers/spec_views/views_controller.rb

@@ -2,7 +2,7 @@
 
 module SpecViews
   class ViewsController < ApplicationController
-    skip_authorization_check
+    skip_authorization_check if respond_to?(:skip_authorization_check)
     layout 'spec_views'
 
     def index
@@ -36,11 +36,21 @@ module SpecViews
       @directory = directory
     end
 
+    def diff
+      if pdf?
+        redirect_to action: :compare
+        return
+      end
+      @champion = get_view(filename('view'), html_safe: false)
+      @challenger = get_view(filename('challenger'), html_safe: false)
+      @directory = directory
+    end
+
     def preview
       @directory = directory
       @next = directories[directories.index(@directory) + 1]
       index = directories.index(@directory)
-      @previous = directories[index - 1] if index > 0
+      @previous = directories[index - 1] if index.positive?
     end
 
     def accept
@@ -72,63 +82,9 @@ module SpecViews
 
     private
 
-    class Directory
-      attr_reader :path
-
-      def initialize(path)
-        @path = path
-      end
-
-      def basename
-        path.basename
-      end
-
-      def to_param
-        basename.to_s
-      end
-
-      def name
-        basename
-      end
-
-      def challenger?
-        File.file?(path.join('challenger.html')) || File.file?(path.join('challenger.pdf'))
-      end
-
-      def controller_name
-        splitted_name.first.gsub(/Controller(_.*)$/, 'Controller').gsub(/Controller$/, '').gsub('_', '::')
-      end
-
-      def method
-        splitted_name.second[0, 3]
-      end
-
-      def description
-        splitted_name.third.gsub(/__pdf$/, '').humanize
-      end
-
-      def last_run
-        @last_run ||= begin
-          Time.zone.parse(File.read(path.join('last_run.txt')))
-        rescue Errno::ENOENT
-          Time.zone.at(0)
-        end
-      end
-
-      def delete!
-        FileUtils.remove_dir(path)
-      end
-
-      private
-
-      def splitted_name
-        @splitted_name ||= name.to_s.split(/_(DELETE|GET|PATCH|POST|PUT)_/)
-      end
-    end
-
     def directories
       @directories ||= Pathname.new(directory_path).children.select(&:directory?).map do |path|
-        Directory.new(path)
+        SpecViews::Directory.new(path)
       end.sort_by(&:basename)
     end
 
@@ -140,9 +96,11 @@ module SpecViews
       directories.detect { |dir| dir.to_param == params[:id] }
     end
 
-    def get_view(filename = nil)
+    def get_view(filename = nil, html_safe: true)
       filename ||= pdf? ? 'view.pdf' : 'view.html'
-      File.read(file_path(filename)).html_safe # rubocop:disable Rails/OutputSafety
+      content = File.read(file_path(filename))
+      content = content.html_safe if html_safe
+      content
     rescue Errno::ENOENT
       ''
     end
@@ -158,7 +116,7 @@ module SpecViews
     end
 
     def pdf?
-      params[:id] && params[:id].end_with?('__pdf')
+      params[:id]&.end_with?('__pdf')
     end
 
     def accept_directory(id)

data/app/models/spec_views/base_matcher.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class BaseMatcher
+    attr_reader :response, :directory, :failure_message, :type
+
+    delegate :champion_html, to: :directory
+
+    def initialize(response, description, run_time:, expected_status: :ok, type: :request)
+      @response = response
+      @type = type
+      @extractor = extractor_class.new(response, expected_status: expected_status)
+      @directory = SpecViews::Directory.for_description(description, content_type: content_type)
+      @extractor_failure = @extractor.extractor_failure?
+      @match = !extractor_failure? && match_challenger
+      @directory.write_last_run(run_time) if champion_html
+      return if match?
+
+      if extractor_failure?
+        @failure_message = @extractor.failure_message 
+        return
+      end
+
+      @failure_message = "#{subject_name} has changed."
+      @failure_message = "#{subject_name} has been added." if champion_html.nil?
+
+      @directory.write_last_run(run_time)
+      @directory.write_description(description)
+      @directory.write_challenger(challenger_body)
+    end
+
+    def match?
+      @match
+    end
+
+    def extractor_failure?
+      @extractor_failure
+    end
+
+    protected
+
+    def subject_name
+      raise NotImplementedError
+    end
+
+    def challenger_body
+      raise NotImplementedError
+    end
+
+    def match_challenger
+      raise NotImplementedError
+    end
+
+    def content_type
+      raise NotImplementedError
+    end
+
+    def extractor_class
+      return MailMessageExtractor if type == :mailer
+
+      HttpResponseExtractor
+    end
+  end
+end

data/app/models/spec_views/directory.rb

@@ -0,0 +1,133 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class Directory
+    attr_reader :path
+
+    def self.for_description(description, content_type: :html)
+      dir_name = description.strip.gsub(/[^0-9A-Za-z.\-]/, '_').gsub('__', '_')
+      dir_name = "#{dir_name}__pdf" if content_type == :pdf
+      new(Rails.root.join(Rails.configuration.spec_views.directory, dir_name))
+    end
+
+    def initialize(path)
+      @path = path
+    end
+
+    def basename
+      path.basename
+    end
+
+    def to_param
+      basename.to_s
+    end
+
+    def name
+      basename
+    end
+
+    def challenger?
+      File.file?(path.join('challenger.html')) || File.file?(path.join('challenger.pdf'))
+    end
+
+    def controller_name
+      splitted_description.first.gsub(/Controller(_.*)$/, 'Controller').gsub(/Controller$/, '').gsub('_', '::')
+    end
+
+    def method
+      splitted_description.second[0, 4]
+    end
+
+    def description_tail
+      splitted_description.third.gsub(/__pdf$/, '')
+    end
+
+    def last_run
+      @last_run ||= begin
+        Time.zone.parse(File.read(path.join('last_run.txt')))
+      rescue Errno::ENOENT
+        Time.zone.at(0)
+      end
+    end
+
+    def delete!
+      FileUtils.remove_dir(path)
+    end
+
+    def champion_path
+      path.join("view.#{file_extension}")
+    end
+
+    def champion_html
+      File.read(champion_path)
+    rescue Errno::ENOENT
+      nil
+    end
+
+    def challenger_path
+      path.join("challenger.#{file_extension}")
+    end
+
+    def write_challenger(content)
+      FileUtils.mkdir_p(path)
+      File.open(challenger_path, binary? ? 'wb' : 'w') { |f| f.write(content) }
+    end
+
+    def last_run_path
+      path.join('last_run.txt')
+    end
+
+    def write_last_run(time)
+      FileUtils.mkdir_p(path)
+      File.write(last_run_path, time)
+    end
+
+    def description_path
+      path.join('description.txt')
+    end
+
+    def write_description(description)
+      FileUtils.mkdir_p(path)
+      File.write(description_path, description)
+    end
+
+    def description
+      File.read(description_path)
+    rescue Errno::ENOENT
+      name
+    end
+
+    def pdf?
+      name.to_s.ends_with?('__pdf')
+    end
+
+    def mailer?
+      @mailer ||= name.to_s.start_with?(/[a-zA-Z0-9:]+Mailer[^a-zA-Z0-9]/)
+    end
+
+    def binary?
+      pdf?
+    end
+
+    private
+
+    def splitted_description
+      @splitted_description ||= begin
+        if mailer?
+          split = description.to_s.match(/(\A[a-zA-Z0-9:]+)(Mailer)(.*)/)
+          split = split[1..3]
+          split[0] += 'Mailer'
+          split[1] = 'MAIL'
+        else
+          split = description.to_s.split(/\s(DELETE|GET|PATCH|POST|PUT)\s/)
+        end
+        split = ['', '', split[0]] if split.size == 1
+        split
+      end
+    end
+
+    def file_extension
+      pdf? ? 'pdf' : 'html'
+    end
+  end
+end

data/app/models/spec_views/html_matcher.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class HtmlMatcher < BaseMatcher
+    delegate :champion_html, to: :directory
+
+    protected
+
+    def subject_name
+      'View'
+    end
+
+    def content_type
+      :html
+    end
+
+    def challenger_body
+      sanitized_body
+    end
+
+    def match_challenger
+      champion_html == challenger_body
+    end
+
+    private
+
+    def sanitized_body
+      remove_pack_digests_from_body(remove_digests_from_body(@extractor.body))
+    end
+
+    def remove_digests_from_body(body)
+      body.gsub(/(-[a-z0-9]{64})(\.css|\.js|\.ico|\.png|\.jpg|\.jpeg|\.svg|\.gif)/, '\2')
+    end
+
+    def remove_pack_digests_from_body(body)
+      body.gsub(%r{(packs.*/js/[a-z0-9_]+)(-[a-z0-9]{20})(\.js)}, '\1\3')
+    end
+  end
+end

data/app/models/spec_views/http_response_extractor.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class HttpResponseExtractor
+    attr_reader :response, :failure_message
+
+    def initialize(response, expected_status:)
+      @response = response
+      @status_match = response_status_match?(response, expected_status)
+      return if @status_match
+
+
+      @failure_message = "Unexpected response status #{response.status}."
+    end
+
+    def extractor_failure?
+      failure_message.present?
+    end
+
+    def body
+      response.body
+    end
+
+    private
+
+    def response_status_match?(response, expected_status)
+      response.status == expected_status ||
+        response.message.parameterize.underscore == expected_status.to_s
+    end
+  end
+end

data/app/models/spec_views/mail_message_extractor.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class MailMessageExtractor
+    attr_reader :mail, :failure_message, :body
+
+    def initialize(mail, expected_status:)
+      @mail = mail
+      @body = extract_body(mail)
+      return if body.present?
+
+      @failure_message = "Failed to find mail part"
+    end
+
+    def extractor_failure?
+      failure_message.present?
+    end
+
+    def extract_body(part)
+      return part.raw_source if part.respond_to?(:raw_source) && part.raw_source.present?
+      return extract_body(part.body) if part.respond_to?(:body)
+      return part if part.is_a?(String)
+      return if nil unless part.respond_to?(:parts)
+
+      part.parts.map do |inner_part|
+        extract_body(inner_part)
+      end.compact.first
+    end
+  end
+end

data/app/models/spec_views/pdf_matcher.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+module SpecViews
+  class PdfMatcher < BaseMatcher
+    protected
+
+    def subject_name
+      'PDF'
+    end
+
+    def content_type
+      :pdf
+    end
+
+    def challenger_body
+      sanitized_body
+    end
+
+    def match_challenger
+      champion_hash == challenger_hash
+    end
+
+    private
+
+    def champion_hash
+      @champion_hash ||= begin
+        Digest::MD5.file(directory.champion_path).hexdigest
+      rescue Errno::ENOENT
+        nil
+      end
+    end
+
+    def challenger_hash
+      @challenger_hash ||= Digest::MD5.hexdigest(sanitized_body)
+    end
+
+    def sanitized_body
+      @sanitized_body ||= remove_headers_from_pdf(@extractor.body)
+    end
+
+    def remove_headers_from_pdf(body)
+      body
+        .force_encoding('BINARY')
+        .gsub(%r{^/CreationDate \(.*\)$}, '')
+        .gsub(%r{^/ModDate \(.*\)$}, '')
+        .gsub(%r{^/ID \[<.+><.+>\]$}, '')
+    end
+  end
+end