motion-html-pipeline 0.3 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2bf83e4abe0c69678a81a1861cbb829ec6899410f07436f934c708d595b4a94
-  data.tar.gz: abfafa964af397acada8022caa80579e8114d41f382dc26ac5f462b4dddad88a
+  metadata.gz: 3bcdb20531f70a01428ccebf296156c2983b173e336aaf2d91e13a7140c08058
+  data.tar.gz: b1910202c610ebcc89770226462a00fd80f3d1703cdc02291beab303dba3993a
 SHA512:
-  metadata.gz: '048abc1ca29f81a1c54395eee5ab57ab07905c76ed38801846fab5c1e25ec6799cd9aa546d4850d8fae30bca6a85aae30422b2c39f562aecb7d709a26021f86e'
-  data.tar.gz: d2e83504949146542b7ac984758587dade9cfb2d2895ca824ecd45aec5816db70aeee5039333450a85b7b484bea506624d72500cc5257077fec3451df74f5814
+  metadata.gz: e5c1e1b85ebdf611e50e033c5190f901f5ced19972aa97a83cfd9ac3683dbb2dbc749a4beeea307a5d956118e5fd1121709c4ef59796eb21b0df559978deadd5
+  data.tar.gz: a011de5dac1c3d11f8c9dcdaa3506e63083f3037556350ff194bf03c2dc6a5a504560c0746af305af81c73de5ab900cac96e4d0c73d208257833dd8dc0ce7cf7

data/README.md

@@ -2,6 +2,8 @@
 
 [![Gem Version](https://badge.fury.io/rb/motion-html-pipeline.svg)](http://badge.fury.io/rb/motion-html-pipeline)
 [![Build Status](https://travis-ci.org/digitalmoksha/motion-html-pipeline.svg?branch=master)](https://travis-ci.org/digitalmoksha/motion-html-pipeline)
+[![Platform](https://img.shields.io/badge/platform-macos%20|%20ios-lightgrey.svg)](https://shields.io/)
+[![Language](https://img.shields.io/badge/language-RubyMotion-red.svg)](http://www.rubymotion.com)
 
 _This gem is a port of the [`html-pipeline` gem](https://github.com/jch/html-pipeline) to RubyMotion, for use on iOS and macOS. Currently synced with `html-pipeline` release [`v.2.12.3`](https://github.com/jch/html-pipeline/releases/tag/v2.12.3)_
 
@@ -64,7 +66,7 @@ would output
 
 Filters can be combined into a pipeline which causes each filter to hand its
 output to the next filter's input. So if you wanted to have content be
-filtered through the `ImageFilter` and then wrap it in an `<a>` tag with 
+filtered through the `ImageFilter` and then wrap it in an `<a>` tag with
 a max-width inline style:
 
 ```ruby
@@ -243,7 +245,7 @@ Here are some extensions people have built:
 
 ## Instrumenting
 
-_Although instrumenting was ported, it has not been used real-world, and may not work 
+_Although instrumenting was ported, it has not been used real-world, and may not work
 properly at this time._
 
 Filters and Pipelines can be set up to be instrumented when called. The pipeline

data/lib/motion-html-pipeline.rb

@@ -1,16 +1,14 @@
 # frozen_string_literal: true
 
-unless defined?(Motion::Project::Config)
-  raise "This file must be required within a RubyMotion project Rakefile."
-end
+raise 'This file must be required within a RubyMotion project Rakefile.' unless defined?(Motion::Project::Config)
 
 require 'motion-cocoapods'
 
 lib_dir_path = File.dirname(File.expand_path(__FILE__))
 Motion::Project::App.setup do |app|
-  app.files.unshift(Dir.glob(File.join(lib_dir_path, "motion-html-pipeline/**/*.rb")))
+  app.files.unshift(Dir.glob(File.join(lib_dir_path, 'motion-html-pipeline/**/*.rb')))
 
   app.pods do
-    pod 'HTMLKit', '~> 3.1'
+    pod 'HTMLKit', '~> 4.2'
   end
 end

data/lib/motion-html-pipeline/document_fragment.rb

@@ -24,6 +24,6 @@ module MotionHTMLPipeline
     def to_html
       document.body.innerHTML
     end
-    alias :to_s :to_html
+    alias to_s to_html
   end
 end

data/lib/motion-html-pipeline/pipeline.rb

@@ -42,8 +42,10 @@ module MotionHTMLPipeline
 
     # Public: String name for this Pipeline. Defaults to Class name.
     attr_writer :instrumentation_name
+
     def instrumentation_name
       return @instrumentation_name if defined?(@instrumentation_name)
+
       @instrumentation_name = self.class.name
     end
 
@@ -54,6 +56,7 @@ module MotionHTMLPipeline
 
     def initialize(filters, default_context = {}, result_class = nil)
       raise ArgumentError, 'default_context cannot be nil' if default_context.nil?
+
       @filters = filters.flatten.freeze
       @default_context = default_context.freeze
       @result_class = result_class || Hash
@@ -132,12 +135,11 @@ module MotionHTMLPipeline
     # block, otherwise the block is ran without instrumentation.
     #
     # Returns the result of the provided block.
-    def instrument(event, payload = nil)
+    def instrument(event, payload = nil, &block)
       payload ||= default_payload
       return yield(payload) unless instrumentation_service
-      instrumentation_service.instrument event, payload do |payload|
-        yield payload
-      end
+
+      instrumentation_service.instrument event, payload, &block
     end
 
     # Internal: Default payload for instrumentation.

data/lib/motion-html-pipeline/pipeline/absolute_source_filter.rb

@@ -19,8 +19,10 @@ module MotionHTMLPipeline
       def call
         doc.css('img').each do |element|
           next if element['src'].nil? || element['src'].empty?
+
           src = element['src'].strip
           next if src.start_with? 'http'
+
           base = if src.start_with? '/'
                    image_base_url
                  else

data/lib/motion-html-pipeline/pipeline/disabled/@mention_filter.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# TODO Requires Set (or something similar)
+# TODO: Requires Set (or something similar)
 #------------------------------------------------------------------------------
 # require 'set'
 #

data/lib/motion-html-pipeline/pipeline/disabled/autolink_filter.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# TODO Requires Rinku gem (or something similar)
+# TODO: Requires Rinku gem (or something similar)
 #------------------------------------------------------------------------------
 # module MotionHTMLPipeline
 #   class Pipeline

data/lib/motion-html-pipeline/pipeline/filter.rb

@@ -66,6 +66,7 @@ module MotionHTMLPipeline
       # called.
       def html
         raise InvalidDocumentException if @html.nil? && @doc.nil?
+
         @html || doc.to_html
       end
 

data/lib/motion-html-pipeline/pipeline/image_filter.rb

@@ -10,7 +10,7 @@ module MotionHTMLPipeline
 
     class ImageFilter < TextFilter
       def call
-        @text.gsub(/(https|http)?:\/\/.+\.(jpg|jpeg|bmp|gif|png)(\?\S+)?/i) do |match|
+        @text.gsub(%r{(https|http)?://.+\.(jpg|jpeg|bmp|gif|png)(\?\S+)?}i) do |match|
           %(<img src="#{match}" alt=""/>)
         end
       end

data/lib/motion-html-pipeline/pipeline/text_filter.rb

@@ -7,6 +7,7 @@ module MotionHTMLPipeline
 
       def initialize(text, context = nil, result = nil)
         raise TypeError, 'text cannot be HTML' if text.is_a?(DocumentFragment)
+
         # Ensure that this is always a string
         @text = text.respond_to?(:to_str) ? text.to_str : text.to_s
         super nil, context, result

data/lib/motion-html-pipeline/pipeline/version.rb

@@ -2,6 +2,6 @@
 
 module MotionHTMLPipeline
   class Pipeline
-    VERSION = '0.3'.freeze
+    VERSION = '0.4.2'
   end
 end

data/spec/motion-html-pipeline/_helpers/mock_instumentation_service.rb

@@ -2,6 +2,7 @@
 
 class MockedInstrumentationService
   attr_reader :events
+
   def initialize(event = nil, events = [])
     @events = events
     subscribe event

data/spec/motion-html-pipeline/pipeline/absolute_source_filter_spec.rb

@@ -35,15 +35,15 @@ describe 'MotionHTMLPipeline::Pipeline::AbsoluteSourceFilterTest' do
   end
 
   it 'test_fails_when_context_is_missing' do
-    expect{ AbsoluteSourceFilter.call('<img src="img.png">', {}) }.to raise_error(RuntimeError)
-    expect{ AbsoluteSourceFilter.call('<img src="/img.png">', {}) }.to raise_error(RuntimeError)
+    expect { AbsoluteSourceFilter.call('<img src="img.png">', {}) }.to raise_error(RuntimeError)
+    expect { AbsoluteSourceFilter.call('<img src="/img.png">', {}) }.to raise_error(RuntimeError)
   end
 
   it 'test_tells_you_where_context_is_required' do
-    expect{ AbsoluteSourceFilter.call('<img src="img.png">', {}) }
+    expect { AbsoluteSourceFilter.call('<img src="img.png">', {}) }
       .to raise_error(RuntimeError, 'MotionHTMLPipeline::Pipeline::AbsoluteSourceFilter')
 
-    expect{ AbsoluteSourceFilter.call('<img src="/img.png">', {}) }
+    expect { AbsoluteSourceFilter.call('<img src="/img.png">', {}) }
       .to raise_error(RuntimeError, 'MotionHTMLPipeline::Pipeline::AbsoluteSourceFilter')
   end
 end

data/spec/motion-html-pipeline/pipeline/https_filter_spec.rb

@@ -48,7 +48,7 @@ describe 'MotionHTMLPipeline::Pipeline::HttpsFilter' do
   it 'test_validates_http_url' do
     @options.clear
 
-    expect{ filter('') }
+    expect { filter('') }
       .to raise_error(ArgumentError, 'MotionHTMLPipeline::Pipeline::HttpsFilter: :http_url')
   end
 end

data/spec/spec_helper.rb

@@ -4,10 +4,10 @@
 #
 #   rake spec filter="name of spec"                   # To filter by spec name
 #   rake spec files=foo_spec filter="name of spec"
-#   rake spec filter_context="this doesn't work yet"  # To filter by context name (doesn't work until MacBacon implements it)
+#   rake spec filter_context="this doesn't work yet"  # To filter by context name (doesn't work
+#                                                       until MacBacon implements it)
 #   rake spec hide_backtraces=yes                     # Hide backtraces
 
-
 # From http://chen.do/blog/2013/06/03/running-individual-specs-with-rubymotion/
 #------------------------------------------------------------------------------
 def silence_warnings(&block)
@@ -24,12 +24,12 @@ end
 silence_warnings do
   module Bacon
     if ENV['filter']
-      $stderr.puts "Filtering specs that match: #{ENV['filter']}"
+      warn "Filtering specs that match: #{ENV['filter']}"
       RestrictName = Regexp.new(ENV['filter'])
     end
 
     if ENV['filter_context']
-      $stderr.puts "Filtering contexts that match: #{ENV['filter_context']}"
+      warn "Filtering contexts that match: #{ENV['filter_context']}"
       RestrictContext = Regexp.new(ENV['filter_context'])
     end
 
@@ -40,11 +40,15 @@ end
 #------------------------------------------------------------------------------
 # TODO total hack - without this, the `bacon-expect` gem doesn't get used, and
 # on iOS I get errors like
-# *** Terminating app due to uncaught exception 'NameError', reason: '.../bacon-expect/lib/bacon-expect/matchers/eql.rb:2:in `<main>: uninitialized constant BaconExpect::Matcher::SingleMethod (NameError)
+# *** Terminating app due to uncaught exception 'NameError',
+# reason: '.../bacon-expect/lib/bacon-expect/matchers/eql.rb:2:in `<main>:
+# uninitialized constant BaconExpect::Matcher::SingleMethod (NameError)
 # and on macOS it only show up as `*nil description*`
+# rubocop: disable Lint/EmptyClass
 module BaconExpect
   module Matcher
     class SingleMethod
     end
   end
 end
+# rubocop: enable Lint/EmptyClass

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: motion-html-pipeline
 version: !ruby/object:Gem::Version
-  version: '0.3'
+  version: 0.4.2
 platform: ruby
 authors:
 - Ryan Tomayko
 - Jerry Cheung
 - Garen J. Torikian
 - Brett Walker
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-11 00:00:00.000000000 Z
+date: 2021-05-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: motion-cocoapods
@@ -27,20 +27,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rake
-  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'
 - !ruby/object:Gem::Dependency
   name: motion-expect
   requirement: !ruby/object:Gem::Requirement
@@ -110,7 +96,7 @@ homepage: https://github.com/digitalmoksha/motion-html-pipeline
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -125,8 +111,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Helpers for processing content through a chain of filters (RubyMotion version,
   ported from https://github.com/jch/html-pipeline)