middleman-core 4.3.0.rc.1 → 4.3.0.rc.2

data/lib/middleman-core/sources.rb

@@ -24,7 +24,7 @@ module Middleman
     include Contracts
 
     # Types which could cause output to change.
-    OUTPUT_TYPES = [:source, :locales, :data].freeze
+    OUTPUT_TYPES = %i[source locales data].freeze
 
     # Types which require a reload to eval ruby
     CODE_TYPES = [:reload].freeze
@@ -56,7 +56,7 @@ module Middleman
     # @param [Hash] options Global options.
     # @param [Array] watchers Default watchers.
     Contract IsA['Middleman::Application'], Maybe[Hash], Maybe[Array] => Any
-    def initialize(app, _options={}, watchers=[])
+    def initialize(app, _options = {}, watchers = [])
       @app = app
       @watchers = watchers
       @sorted_watchers = @watchers.dup.freeze
@@ -88,7 +88,7 @@ module Middleman
     # @param [Proc] block Ignore by block evaluation.
     # @return [void]
     Contract Symbol, Symbol, Or[Regexp, Proc] => Any
-    def ignore(name, type, regex=nil, &block)
+    def ignore(name, type, regex = nil, &block)
       @ignores = @ignores.put(name, type: type,
                                     validator: (block_given? ? block : regex))
 
@@ -116,13 +116,13 @@ module Middleman
     # @param [Hash] options The watcher options.
     # @return [#changed, #deleted]
     Contract Or[Symbol, HANDLER], Maybe[Hash] => HANDLER
-    def watch(type_or_handler, options={})
+    def watch(type_or_handler, options = {})
       handler = if type_or_handler.is_a? Symbol
-        path = File.expand_path(options.delete(:path), app.root)
-        SourceWatcher.new(self, type_or_handler, path, options)
-      else
-        type_or_handler
-      end
+                  path = File.expand_path(options.delete(:path), app.root)
+                  SourceWatcher.new(self, type_or_handler, path, options)
+                else
+                  type_or_handler
+                end
 
       @watchers << handler
 
@@ -188,7 +188,7 @@ module Middleman
     # @param [Boolean] glob If the path contains wildcard or glob characters.
     # @return [Middleman::SourceFile, nil]
     Contract Or[Symbol, ArrayOf[Symbol], SetOf[Symbol]], Or[Pathname, String], Maybe[Bool] => Maybe[SourceFile]
-    def find(types, path, glob=false)
+    def find(types, path, glob = false)
       array_of_types = Array(types)
 
       watchers
@@ -277,7 +277,7 @@ module Middleman
     #
     # @param [nil,Regexp] matcher A Regexp to match the change path against
     Contract Maybe[Matcher] => Any
-    def changed(matcher=nil, &_block)
+    def changed(matcher = nil, &_block)
       on_change OUTPUT_TYPES do |updated, _removed|
         updated
           .select { |f| matcher.nil? ? true : matches?(matcher, f) }
@@ -289,7 +289,7 @@ module Middleman
     #
     # @param [nil,Regexp] matcher A Regexp to match the change path against
     Contract Maybe[Matcher] => Any
-    def deleted(matcher=nil, &_block)
+    def deleted(matcher = nil, &_block)
       on_change OUTPUT_TYPES do |_updated, removed|
         removed
           .select { |f| matcher.nil? ? true : matches?(matcher, f) }

data/lib/middleman-core/sources/source_watcher.rb

@@ -15,7 +15,7 @@ module Listen
       #   return true unless only_patterns.any? { |pattern| path =~ pattern }
       # end
 
-      return !only_patterns.any? { |pattern| path =~ pattern } if only_patterns
+      return only_patterns.none? { |pattern| path =~ pattern } if only_patterns
 
       ignore_patterns.any? { |pattern| path =~ pattern }
     end
@@ -50,7 +50,7 @@ module Middleman
     # Reference to lower level listener
     attr_reader :listener
 
-    IGNORED_DIRECTORIES = Set.new(%w(.git node_modules .sass-cache vendor/bundle .bundle))
+    IGNORED_DIRECTORIES = Set.new(%w[.git node_modules .sass-cache vendor/bundle .bundle])
 
     # Construct a new SourceWatcher
     #
@@ -59,7 +59,7 @@ module Middleman
     # @param [String] directory The on-disk path to watch.
     # @param [Hash] options Configuration options.
     Contract IsA['Middleman::Sources'], Symbol, String, Hash => Any
-    def initialize(parent, type, directory, options={})
+    def initialize(parent, type, directory, options = {})
       @parent = parent
       @options = options
 
@@ -103,7 +103,7 @@ module Middleman
       poll_once!
     end
 
-    def update_config(options={})
+    def update_config(options = {})
       without_listener_running do
         @disable_watcher = options.fetch(:disable_watcher, false)
         @force_polling = options.fetch(:force_polling, false)
@@ -134,13 +134,19 @@ module Middleman
     # @param [Boolean] glob If the path contains wildcard characters.
     # @return [Middleman::SourceFile, nil]
     Contract Or[String, Pathname], Maybe[Bool] => Maybe[IsA['Middleman::SourceFile']]
-    def find(path, glob=false)
+    def find(path, glob = false)
       path = path.to_s.encode!('UTF-8', 'UTF-8-MAC') if RUBY_PLATFORM =~ /darwin/
 
       p = Pathname(path)
 
       return nil if p.absolute? && !p.to_s.start_with?(@directory.to_s)
 
+      destination_dir = @options[:destination_dir]
+      if destination_dir.present? && p.to_s.start_with?(destination_dir)
+        path_without_destination_dir = p.to_s[destination_dir.to_s.length + 1..-1]
+        p = Pathname(path_without_destination_dir)
+      end
+
       p = @directory + p if p.relative?
 
       if glob
@@ -357,16 +363,16 @@ module Middleman
     private
 
     def without_listener_running
-      listener_running = @listener && @listener.processing?
+      listener_running = @listener&.processing?
 
       stop_listener! if listener_running
 
       yield
 
-      if listener_running
-        poll_once!
-        listen!
-      end
+      return unless listener_running
+
+      poll_once!
+      listen!
     end
   end
 end

data/lib/middleman-core/step_definitions/builder_steps.rb

@@ -1,57 +1,57 @@
 require 'fileutils'
 
 Before do
-  @modification_times = Hash.new
+  @modification_times = {}
 end
 
 Given /^a built app at "([^\"]*)"$/ do |path|
-  step %Q{a fixture app "#{path}"}
+  step %(a fixture app "#{path}")
 
   cwd = File.expand_path(aruba.current_directory)
-  step %Q{I set the environment variable "MM_ROOT" to "#{cwd}"}
+  step %(I set the environment variable "MM_ROOT" to "#{cwd}")
 
-  step %Q{I run `middleman build --verbose`}
+  step %(I run `middleman build --verbose`)
 end
 
 Given /^was successfully built$/ do
-  step %Q{the output should contain "Project built successfully."}
-  step %Q{the exit status should be 0}
-  step %Q{a directory named "build" should exist}
+  step %(the output should contain "Project built successfully.")
+  step %(the exit status should be 0)
+  step %(a directory named "build" should exist)
 end
 
 Given /^was not successfully built$/ do
-  step %Q{the output should not contain "Project built successfully."}
-  step %Q{the exit status should be 1}
-  step %Q{a directory named "build" should not exist}
+  step %(the output should not contain "Project built successfully.")
+  step %(the exit status should be 1)
+  step %(a directory named "build" should not exist)
 end
 
 Given /^a successfully built app at "([^\"]*)"$/ do |path|
-  step %Q{a built app at "#{path}"}
-  step %Q{was successfully built}
+  step %(a built app at "#{path}")
+  step %(was successfully built)
 end
 
 Given /^a built app at "([^\"]*)" with flags "([^\"]*)"$/ do |path, flags|
-  step %Q{a fixture app "#{path}"}
-  
+  step %(a fixture app "#{path}")
+
   cwd = File.expand_path(aruba.current_directory)
-  step %Q{I set the environment variable "MM_ROOT" to "#{cwd}"}
+  step %(I set the environment variable "MM_ROOT" to "#{cwd}")
 
-  step %Q{I run `middleman build #{flags}`}
+  step %(I run `middleman build #{flags}`)
 end
 
 Given /^a successfully built app at "([^\"]*)" with flags "([^\"]*)"$/ do |path, flags|
-  step %Q{a built app at "#{path}" with flags "#{flags}"}
-  step %Q{was successfully built}
+  step %(a built app at "#{path}" with flags "#{flags}")
+  step %(was successfully built)
 end
 
 Given /^I run the interactive middleman console$/ do
   cwd = File.expand_path(aruba.current_directory)
-  step %Q{I set the environment variable "MM_ROOT" to "#{cwd}"}
-  step %Q{I run `middleman console` interactively}
+  step %(I set the environment variable "MM_ROOT" to "#{cwd}")
+  step %(I run `middleman console` interactively)
 end
 
 Given /^I run the interactive middleman server$/ do
   cwd = File.expand_path(aruba.current_directory)
-  step %Q{I set the environment variable "MM_ROOT" to "#{cwd}"}
-  step %Q{I run `middleman server` interactively}
-end
+  step %(I set the environment variable "MM_ROOT" to "#{cwd}")
+  step %(I run `middleman server` interactively)
+end

data/lib/middleman-core/step_definitions/commandline_steps.rb

@@ -1,11 +1,11 @@
-When /^I stop (?:middleman|all commands) if the output( of the last command)? contains:$/ do |last_command, expected|
+When /^I stop (?:middleman|all commands) if the output( of the last command)? contains:$/ do |_last_command, expected|
   begin
     Timeout.timeout(aruba.config.exit_timeout) do
       loop do
-        fail "You need to start middleman interactively first." unless @interactive
+        raise 'You need to start middleman interactively first.' unless @interactive
 
-        if sanitize_text(@interactive.output) =~ Regexp.new(sanitize_text(expected))
-          all_commands.each { |p| p.terminate }
+        if sanitize_text(@interactive.output)&.match?(Regexp.new(sanitize_text(expected)))
+          all_commands.each(&:terminate)
           break
         end
 
@@ -80,7 +80,7 @@ end
 
 # Make sure each and every process is really dead
 After do
-  all_commands.each { |p| p.terminate }
+  all_commands.each(&:terminate)
 end
 
 Before '@ruby-2.1' do

data/lib/middleman-core/step_definitions/middleman_steps.rb

@@ -1,15 +1,14 @@
 require 'fileutils'
 
-Given /^app "([^\"]*)" is using config "([^\"]*)"$/ do |path, config_name|
-  target = File.join(PROJECT_ROOT_PATH, 'fixtures', path)
-  config_path = File.join(expand_path("."), "config-#{config_name}.rb")
-  config_dest = File.join(expand_path("."), 'config.rb')
+Given /^app "([^\"]*)" is using config "([^\"]*)"$/ do |_path, config_name|
+  config_path = File.join(expand_path('.'), "config-#{config_name}.rb")
+  config_dest = File.join(expand_path('.'), 'config.rb')
   FileUtils.cp(config_path, config_dest)
 end
 
 Given /^an empty app$/ do
-  step %Q{a directory named "empty_app"}
-  step %Q{I cd to "empty_app"}
+  step %(a directory named "empty_app")
+  step %(I cd to "empty_app")
   ENV['MM_ROOT'] = nil
 end
 
@@ -18,14 +17,14 @@ Given /^a fixture app "([^\"]*)"$/ do |path|
 
   # This step can be reentered from several places but we don't want
   # to keep re-copying and re-cd-ing into ever-deeper directories
-  next if File.basename(expand_path(".")) == path
+  next if File.basename(expand_path('.')) == path
 
-  step %Q{a directory named "#{path}"}
+  step %(a directory named "#{path}")
 
   target_path = File.join(PROJECT_ROOT_PATH, 'fixtures', path)
-  FileUtils.cp_r(target_path, expand_path("."))
+  FileUtils.cp_r(target_path, expand_path('.'))
 
-  step %Q{I cd to "#{path}"}
+  step %(I cd to "#{path}")
 end
 
 Then /^the file "([^\"]*)" has the contents$/ do |path, contents|
@@ -35,18 +34,18 @@ Then /^the file "([^\"]*)" has the contents$/ do |path, contents|
 end
 
 Then /^the file "([^\"]*)" is removed$/ do |path|
-  step %Q{I remove the file "#{path}"}
+  step %(I remove the file "#{path}")
 
   @server_inst.files.poll_once!
 end
 
 Given /^a modification time for a file named "([^\"]*)"$/ do |file|
-  target = File.join(expand_path("."), file)
+  target = File.join(expand_path('.'), file)
   @modification_times[target] = File.mtime(target)
 end
 
 Then /^the file "([^\"]*)" should not have been updated$/ do |file|
-  target = File.join(expand_path("."), file)
+  target = File.join(expand_path('.'), file)
   expect(File.mtime(target)).to eq(@modification_times[target])
 end
 
@@ -56,5 +55,5 @@ Then /^the file "([^"]*)" should contain '([^']*)'$/ do |file, partial_content|
 end
 
 And /the file "(.*)" should be gzipped/ do |file|
-  expect(File.binread(File.join(expand_path("."), file), 2)).to eq(['1F8B'].pack('H*'))
+  expect(File.binread(File.join(expand_path('.'), file), 2)).to eq(['1F8B'].pack('H*'))
 end

data/lib/middleman-core/step_definitions/server_steps.rb

@@ -10,9 +10,7 @@ end
 Given /^"([^\"]*)" feature is "([^\"]*)"$/ do |feature, state|
   @activation_commands ||= []
 
-  if state == 'enabled'
-    @activation_commands << lambda { activate(feature.to_sym) }
-  end
+  @activation_commands << -> { activate(feature.to_sym) } if state == 'enabled'
 end
 
 Given /^"([^\"]*)" feature is "enabled" with "([^\"]*)"$/ do |feature, options_str|
@@ -20,7 +18,7 @@ Given /^"([^\"]*)" feature is "enabled" with "([^\"]*)"$/ do |feature, options_s
 
   options = eval("{#{options_str}}")
 
-  @activation_commands << lambda { activate(feature.to_sym, options) }
+  @activation_commands << -> { activate(feature.to_sym, options) }
 end
 
 Given /^"([^\"]*)" is set to "([^\"]*)"$/ do |variable, value|
@@ -31,13 +29,13 @@ Given /^"([^\"]*)" is set to "([^\"]*)"$/ do |variable, value|
 end
 
 Given /^the Server is running$/ do
-  root_dir = File.expand_path(expand_path("."))
+  root_dir = File.expand_path(expand_path('.'))
 
-  if File.exists?(File.join(root_dir, 'source'))
-    ENV['MM_SOURCE'] = 'source'
-  else
-    ENV['MM_SOURCE'] = ''
-  end
+  ENV['MM_SOURCE'] = if File.exist?(File.join(root_dir, 'source'))
+                       'source'
+                     else
+                       ''
+                     end
 
   ENV['MM_ROOT'] = root_dir
 
@@ -54,7 +52,7 @@ Given /^the Server is running$/ do
 
     app.after_configuration_eval do
       activation_commands.each do |p|
-        app.config_context.instance_exec(&p)
+        instance_exec(&p)
       end
     end
   end
@@ -63,12 +61,12 @@ Given /^the Server is running$/ do
 end
 
 Given /^the Server is running at "([^\"]*)"$/ do |app_path|
-  step %Q{a fixture app "#{app_path}"}
-  step %Q{the Server is running}
+  step %(a fixture app "#{app_path}")
+  step %(the Server is running)
 end
 
 Given /^a template named "([^\"]*)" with:$/ do |name, string|
-  step %Q{a file named "source/#{name}" with:}, string
+  step %(a file named "source/#{name}" with:), string
 end
 
 When /^I go to "([^\"]*)"$/ do |url|
@@ -76,7 +74,7 @@ When /^I go to "([^\"]*)"$/ do |url|
 end
 
 Then /^going to "([^\"]*)" should not raise an exception$/ do |url|
-  expect{ visit(URI.encode(url).to_s) }.to_not raise_exception
+  expect { visit(URI.encode(url).to_s) }.to_not raise_exception
 end
 
 Then /^the content type should be "([^\"]*)"$/ do |expected|
@@ -120,5 +118,5 @@ Then /^the status code should be "([^\"]*)"$/ do |expected|
 end
 
 Then /^I should see "([^\"]*)" lines$/ do |lines|
-  expect(page.body.chomp.split($/).length).to eq lines.to_i
+  expect(page.body.chomp.split($INPUT_RECORD_SEPARATOR).length).to eq lines.to_i
 end

data/lib/middleman-core/template_context.rb

@@ -30,7 +30,7 @@ module Middleman
     # @param [Middleman::Application] app
     # @param [Hash] locs
     # @param [Hash] opts
-    def initialize(app, locs={}, opts={})
+    def initialize(app, locs = {}, opts = {})
       @app = app
       @locs = locs
       @opts = opts
@@ -42,7 +42,8 @@ module Middleman
     # @api private
     # @return [String] The old buffer.
     def save_buffer
-      @_out_buf, buf_was = '', @_out_buf
+      buf_was = @_out_buf
+      @_out_buf = ''
       buf_was
     end
 
@@ -100,7 +101,7 @@ module Middleman
     # @param [Proc] block A block will be evaluated to return internal contents.
     # @return [String]
     Contract Any, Or[Symbol, String], Hash => String, Maybe[Proc] => String
-    def render(_, name, options={}, &block)
+    def render(_, name, options = {}, &block)
       name = name.to_s
 
       partial_file = locate_partial(name, false) || locate_partial(name, true)
@@ -126,30 +127,35 @@ module Middleman
     # @param [String] partial_path
     # @return [String]
     Contract String, Maybe[Bool] => Maybe[IsA['Middleman::SourceFile']]
-    def locate_partial(partial_path, try_static=true)
+    def locate_partial(partial_path, try_static = true)
       partial_file = nil
       lookup_stack = []
       non_root     = partial_path.to_s.sub(/^\//, '')
       non_root_no_underscore = non_root.sub(/^_/, '').sub(/\/_/, '/')
 
-      if resource = current_resource
+      if current_resource
+        resource = current_resource
         current_dir  = resource.file_descriptor[:relative_path].dirname
         relative_dir = current_dir + Pathname(non_root)
         relative_dir_no_underscore = current_dir + Pathname(non_root_no_underscore)
-      end
 
-      if relative_dir
-        lookup_stack.push [relative_dir.to_s,
-                           { preferred_engine: resource.file_descriptor[:relative_path]
-                             .extname[1..-1].to_sym }]
+        if relative_dir
+          lookup_stack.push [relative_dir.to_s,
+                             { preferred_engine: resource.file_descriptor[:relative_path]
+                                                         .extname[1..-1].to_sym }]
+        end
       end
+
       lookup_stack.push [non_root]
+
       lookup_stack.push [non_root,
                          { try_static: try_static }]
+
       if relative_dir_no_underscore
         lookup_stack.push [relative_dir_no_underscore.to_s,
                            { try_static: try_static }]
       end
+
       lookup_stack.push [non_root_no_underscore,
                          { try_static: try_static }]
 
@@ -169,6 +175,7 @@ module Middleman
     # @return [Middleman::Sitemap::Resource]
     def current_resource
       return nil unless current_path
+
       sitemap.find_resource_by_destination_path(current_path)
     end
     alias current_page current_resource