guard-livereload 2.4.0 → 2.5.0

data/spec/lib/guard/livereload/template_spec.rb

@@ -0,0 +1,67 @@
+require 'guard/compat/test/template'
+
+RSpec.describe Guard::LiveReload do
+  describe 'template' do
+    subject { Guard::Compat::Test::Template.new(described_class) }
+
+    context 'Rails' do
+      context 'with view template files' do
+        it 'reloads full page' do
+          expect(subject.changed('app/views/foo.haml')).to eq(%w(app/views/foo.haml))
+          expect(subject.changed('app/views/foo.html.erb')).to eq(%w(app/views/foo.html.erb))
+        end
+      end
+
+      context 'when static files changes' do
+        it 'reloads just the assets' do
+          expect(subject.changed('public/foo.html')).to eq(%w(public/foo.html))
+          expect(subject.changed('public/foo.css')).to eq(%w(public/foo.css))
+          expect(subject.changed('public/foo.js')).to eq(%w(public/foo.js))
+          expect(subject.changed('public/foo.png')).to eq(%w(public/foo.png))
+          expect(subject.changed('public/foo.gif')).to eq(%w(public/foo.gif))
+          expect(subject.changed('public/foo.jpg')).to eq(%w(public/foo.jpg))
+          expect(subject.changed('public/foo.jpeg')).to eq(%w(public/foo.jpeg))
+        end
+      end
+
+      context 'with old Rails asset pipeline naming' do
+        it 'reloads' do
+          expect(subject.changed('app/assets/css/foo.css')).to eq(%w(/assets/foo.css))
+          expect(subject.changed('app/assets/css/foo.css.scss')).to eq(%w(/assets/foo.css))
+          expect(subject.changed('app/assets/css/foo.css.scss.erb')).to eq(%w(/assets/foo.css))
+
+          expect(subject.changed('app/assets/css/foo.js')).to eq(%w(/assets/foo.js))
+          expect(subject.changed('app/assets/css/foo.html')).to eq(%w(/assets/foo.html))
+          expect(subject.changed('app/assets/css/foo.png')).to eq(%w(/assets/foo.png))
+          expect(subject.changed('app/assets/css/foo.jpg')).to eq(%w(/assets/foo.jpg))
+        end
+
+        it 'handles edge cases' do
+          expect(subject.changed('app/assets/css/foo.css.')).to eq([])
+          expect(subject.changed('app/assets/css/foo.cssx')).to eq([])
+          expect(subject.changed('app/assets/css/foo.cssx.scss')).to eq([])
+          expect(subject.changed('app/assets/css/foo.bar.css')).to eq([])
+        end
+      end
+
+      context 'with new Rails asset pipeline naming' do
+        it 'reloads' do
+          expect(subject.changed('app/assets/css/foo.scss')).to eq(%w(/assets/foo.css))
+          expect(subject.changed('app/assets/css/foo.sass')).to eq(%w(/assets/foo.css))
+          expect(subject.changed('app/assets/js/foo.coffee')).to eq(%w(/assets/foo.js))
+
+          expect(subject.changed('app/assets/css/foo.sass.erb')).to eq(%w(/assets/foo.css))
+        end
+
+        it 'handles edge cases' do
+          expect(subject.changed('app/assets/css/foo_sass')).to eq([])
+          expect(subject.changed('app/assets/css/foo.sass.')).to eq([])
+          expect(subject.changed('app/assets/css/foo..sass')).to eq([])
+          expect(subject.changed('app/assets/css/foo.sassx')).to eq([])
+          expect(subject.changed('app/assets/css/foo.cssx.scss')).to eq([])
+          expect(subject.changed('app/assets/css/foo.bar.css')).to eq([])
+        end
+      end
+    end
+  end
+end

data/spec/lib/guard/livereload_spec.rb

@@ -1,12 +1,28 @@
-require 'spec_helper'
+require 'guard/compat/test/helper'
 
-describe Guard::LiveReload do
+RSpec.describe Guard::LiveReload do
   let(:plugin) { Guard::LiveReload.new }
   let(:reactor) { double(Guard::LiveReload::Reactor) }
-  before { allow(plugin).to receive(:reactor) { reactor } }
 
-  describe "#initialize" do
-    describe ":host option" do
+  let(:tmp_path) { '/tmp/livereload-123' }
+  let(:snippet) { instance_double(Guard::LiveReload::Snippet, path: tmp_path) }
+
+  before do
+    allow(File).to receive(:expand_path) do |*args|
+      fail "stub called for File.expand_path(#{args.map(&:inspect) * ','})"
+    end
+
+    allow(File).to receive(:expand_path).
+      with('../../../js/livereload.js.erb', anything).
+      and_return('/foo/js/livereload.js.erb')
+
+    allow(Guard::LiveReload::Snippet).to receive(:new).and_return(snippet)
+
+    allow(plugin).to receive(:reactor) { reactor }
+  end
+
+  describe '#initialize' do
+    describe ':host option' do
       it "is '0.0.0.0' by default" do
         plugin = Guard::LiveReload.new
         expect(plugin.options[:host]).to eq '0.0.0.0'
@@ -18,7 +34,7 @@ describe Guard::LiveReload do
       end
     end
 
-    describe ":port option" do
+    describe ':port option' do
       it "is '35729' by default" do
         plugin = Guard::LiveReload.new
         expect(plugin.options[:port]).to eq '35729'
@@ -30,83 +46,119 @@ describe Guard::LiveReload do
       end
     end
 
-    describe ":apply_css_live option" do
-      it "is true by default" do
+    describe ':apply_css_live option' do
+      it 'is true by default' do
         plugin = Guard::LiveReload.new
         expect(plugin.options[:apply_css_live]).to be_truthy
       end
 
-      it "can be set to false" do
+      it 'can be set to false' do
         plugin = Guard::LiveReload.new(apply_css_live: false)
         expect(plugin.options[:apply_css_live]).to be_falsey
       end
     end
 
-    describe ":override_url option" do
-      it "is false by default" do
+    describe ':override_url option' do
+      it 'is false by default' do
         plugin = Guard::LiveReload.new
         expect(plugin.options[:override_url]).to be_falsey
       end
 
-      it "can be set to false" do
+      it 'can be set to false' do
         plugin = Guard::LiveReload.new(override_url: true)
         expect(plugin.options[:override_url]).to be_truthy
       end
     end
 
-    describe ":grace_period option" do
-      it "is 0 by default" do
+    describe ':grace_period option' do
+      it 'is 0 by default' do
         plugin = Guard::LiveReload.new
         expect(plugin.options[:grace_period]).to eq 0
       end
 
-      it "can be set to 0.5" do
+      it 'can be set to 0.5' do
         plugin = Guard::LiveReload.new(grace_period: 0.5)
         expect(plugin.options[:grace_period]).to eq 0.5
       end
     end
+
+    describe ':js_template option' do
+      subject { described_class.new(*args) }
+
+      context 'when no value is provided' do
+        let(:args) { [] }
+
+        it 'is set to full path to default JS' do
+          expect(subject.options[:js_template]).to eq '/foo/js/livereload.js.erb'
+        end
+
+        it 'evalutes the default snippet' do
+          expect(Guard::LiveReload::Snippet).to receive(:new).
+            with('/foo/js/livereload.js.erb', anything).and_return(snippet)
+          subject
+        end
+      end
+
+      context 'with a custom path' do
+        let(:args) { [js_template: 'foo/bar.js.erb'] }
+
+        it 'is set to the given JS' do
+          expect(subject.options[:js_template]).to eq 'foo/bar.js.erb'
+        end
+
+        it 'evalutes the provided snippet' do
+          expect(Guard::LiveReload::Snippet).to receive(:new).
+            with('foo/bar.js.erb', anything).and_return(snippet)
+          subject
+        end
+      end
+    end
   end
 
-  describe "#start" do
-    it "creates reactor with default options" do
+  describe '#start' do
+    it 'creates reactor with default options' do
       plugin = Guard::LiveReload.new
       expect(Guard::LiveReload::Reactor).to receive(:new).with(
         host:           '0.0.0.0',
         port:           '35729',
         apply_css_live: true,
         override_url:   false,
-        grace_period:  0
+        grace_period:  0,
+        js_template: '/foo/js/livereload.js.erb',
+        livereload_js_path: '/tmp/livereload-123'
       )
       plugin.start
     end
 
-    it "creates reactor with given options" do
+    it 'creates reactor with given options' do
       plugin = Guard::LiveReload.new(host: '127.3.3.1', port: '12345', apply_css_live: false, override_url: true, grace_period: 1)
       expect(Guard::LiveReload::Reactor).to receive(:new).with(
         host:           '127.3.3.1',
         port:           '12345',
         apply_css_live: false,
         override_url:   true,
-        grace_period:   1
+        grace_period:   1,
+        js_template: '/foo/js/livereload.js.erb',
+        livereload_js_path: '/tmp/livereload-123'
       )
       plugin.start
     end
   end
 
-  describe "#stop" do
-    it "stops the reactor" do
+  describe '#stop' do
+    it 'stops the reactor' do
       expect(reactor).to receive(:stop)
       plugin.stop
     end
   end
 
-  describe "#run_on_modifications" do
-    it "reloads browser" do
+  describe '#run_on_modifications' do
+    it 'reloads browser' do
       expect(reactor).to receive(:reload_browser).with(['foo'])
       plugin.run_on_modifications(['foo'])
     end
 
-    it "can wait 0.5 seconds before reloading the browser" do
+    it 'can wait 0.5 seconds before reloading the browser' do
       expect(reactor).to receive(:reload_browser).with(['foo'])
       expect(plugin).to receive(:sleep).with(0.5)
       plugin.options[:grace_period] = 0.5

data/spec/spec_helper.rb

@@ -3,10 +3,33 @@ Coveralls.wear!
 
 require 'rspec'
 require 'guard/livereload'
-ENV["GUARD_ENV"] = 'test'
 
 RSpec.configure do |config|
-  config.color = true
-  config.filter_run focus: true
+
+  def ci?
+    ENV['CI'] == 'true'
+  end
+
+  config.expect_with :rspec do |expectations|
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  config.mock_with :rspec do |mocks|
+    mocks.verify_partial_doubles = true
+  end
+
+  config.filter_run focus: !ci?
   config.run_all_when_everything_filtered = true
+
+  config.disable_monkey_patching!
+
+  config.warnings = true
+
+  config.default_formatter = 'doc' if config.files_to_run.one?
+
+  # config.profile_examples = 10
+
+  config.order = :random
+
+  Kernel.srand config.seed
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: guard-livereload
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.5.0
 platform: ruby
 authors:
 - Thibaud Guillaume-Gentil
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-12 00:00:00.000000000 Z
+date: 2015-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.8'
+- !ruby/object:Gem::Dependency
+  name: guard-compat
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: em-websocket
   requirement: !ruby/object:Gem::Requirement
@@ -66,34 +80,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.5
-- !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: rspec
-  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'
 description: Guard::LiveReload automatically reloads your browser when 'view' files
   are modified.
 email: thibaud@thibaud.me
@@ -102,6 +88,13 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".hound.yml"
+- ".hound/defaults.yml"
+- ".hound/overrides.yml"
+- ".rspec"
+- ".rubocop.yml"
+- ".rubocop_merged_for_hound.yml"
+- ".rubocop_todo.yml"
 - ".travis.yml"
 - Gemfile
 - Guardfile
@@ -109,13 +102,16 @@ files:
 - README.md
 - Rakefile
 - guard-livereload.gemspec
-- js/livereload.js
+- js/livereload.js.erb
 - lib/guard/livereload.rb
 - lib/guard/livereload/reactor.rb
+- lib/guard/livereload/snippet.rb
 - lib/guard/livereload/templates/Guardfile
 - lib/guard/livereload/version.rb
 - lib/guard/livereload/websocket.rb
 - spec/lib/guard/livereload/reactor_spec.rb
+- spec/lib/guard/livereload/snippet_spec.rb
+- spec/lib/guard/livereload/template_spec.rb
 - spec/lib/guard/livereload_spec.rb
 - spec/spec_helper.rb
 homepage: https://rubygems.org/gems/guard-livereload
@@ -138,11 +134,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Guard plugin for livereload
 test_files:
 - spec/lib/guard/livereload/reactor_spec.rb
+- spec/lib/guard/livereload/snippet_spec.rb
+- spec/lib/guard/livereload/template_spec.rb
 - spec/lib/guard/livereload_spec.rb
 - spec/spec_helper.rb