browserify-rails 0.4.1 → 0.4.2

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    MzQ3ZTNjN2UxMGQ3NzE2MzBkNGMzOTExNDZhN2NmMTY0ZDM0ZTQ2OQ==
-  data.tar.gz: !binary |-
-    MWI0YWIwN2I0YzQxZmZlMGI2MGU1NzE2OWQzMjk2YjkyNTExOGNmNg==
+SHA1:
+  metadata.gz: c8fc489be291bbeb57acb5004f86574a3254fe2f
+  data.tar.gz: 3dac86e18f58cf52c91dbe1a8166ff59b5cdf70b
 SHA512:
-  metadata.gz: !binary |-
-    MjE2Y2FmOGY1ZTg3ODIyZGY4YTQyYWNhNzI3YWQ1NzAyNTAyODZiZjIyZGJm
-    NGRjNGY5N2Q4YWFiZjJiYWQ2MWFiODYxOThjNjkyMDZlYzAxYmVhYjcxZTRj
-    YmExZGUxMjM1MzJkYTYwYTJjYzcyM2Y1ZTA1OWVmODM4NWE1OTc=
-  data.tar.gz: !binary |-
-    M2E2OWM5NTVlZDY0YWVmZjkyODU1NTRjMWFjZDg1YjllNjhkNTc1ZTViYWJi
-    MTM3MjhkYjUyYjhlZmIxMGE1ZmNlNjQ5ODhlODg0NzBjYjQyNjA5M2E2N2Qw
-    YjYxZWIwYjg2ZTA4ZjMyNDI0MTkzNjE0MmE3ZDIzNjYwZTA5OTQ=
+  metadata.gz: 923b205738755acb59616ddea742f1cc57f554d3c02c664bd6db905e61b0eeb9ab8f324b193b8963cb56676b93f0542c68cf8fa9e48fa0b3f960cd7cc9514c5f
+  data.tar.gz: defd05b16d32bfd53b3b3831cdd65dd506cdf41f44e6f735bf65dc061c5af18ed76fc1d104f070888c4402b7206c6aebe7e3ad1889ea4b5fe6e31b2423697e00

data/lib/browserify-rails/browserify_processor.rb

@@ -35,7 +35,7 @@ module BrowserifyRails
     end
 
     def should_browserify?
-      in_path? && commonjs_module?
+      in_path? && !browserified? && commonjs_module?
     end
 
     # Is this file in any of the configured paths?
@@ -45,6 +45,11 @@ module BrowserifyRails
       end
     end
 
+    # Is this file already packaged for the browser?
+    def browserified?
+      data.to_s.include?("define.amd") || data.to_s.include?("_dereq_")
+    end
+
     # Is this a commonjs module?
     #
     # Be here as strict as possible, so that non-commonjs files are not

data/lib/browserify-rails/version.rb

@@ -1,3 +1,3 @@
 module BrowserifyRails
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

data/test/compilation_test.rb

@@ -21,6 +21,7 @@ class BrowserifyTest < ActionController::IntegrationTest
     copy_example_file "secondary.js.example"
     copy_example_file "a_huge_library.js.example"
     copy_example_file "some_folder/answer.js.example"
+    copy_example_file "browserified.js.example"
   end
 
   test "asset pipeline should serve application.js" do
@@ -107,6 +108,11 @@ class BrowserifyTest < ActionController::IntegrationTest
     assert_equal false, @response.body.include?("Error: Cannot find module 'some_folder/answer'")
   end
 
+  test "skips files that are already browserified" do
+    get "/assets/browserified.js"
+    assert_equal fixture("browserified.out.js"), @response.body.strip
+  end
+
   test "uses config/browserify.yml to mark a module as globally available via --require" do
     expected_output = fixture("main.out.js")
 

data/test/dummy/app/assets/javascripts/browserified.js.example

@@ -0,0 +1,6 @@
+!function(e){if("object"==typeof exports)module.exports=e();else if("function"==typeof define&&define.amd)define(e);else{var o;"undefined"!=typeof window?o=window:"undefined"!=typeof global?o=global:"undefined"!=typeof self&&(o=self),o.Foo=e()}}(function(){var define,module,exports;return (function e(t,n,r){function s(o,u){if(!n[o]){if(!t[o]){var a=typeof require=="function"&&require;if(!u&&a)return a(o,!0);if(i)return i(o,!0);throw new Error("Cannot find module '"+o+"'")}var f=n[o]={exports:{}};t[o][0].call(f.exports,function(e){var n=t[o][1][e];return s(n?n:e)},f,f.exports,e,t,n,r)}return n[o].exports}var i=typeof require=="function"&&require;for(var o=0;o<r.length;o++)s(r[o]);return s})({1:[function(_dereq_,module,exports){
+module.exports = true;
+
+},{}]},{},[1])
+(1)
+});

data/test/fixtures/browserified.out.js

@@ -0,0 +1,6 @@
+!function(e){if("object"==typeof exports)module.exports=e();else if("function"==typeof define&&define.amd)define(e);else{var o;"undefined"!=typeof window?o=window:"undefined"!=typeof global?o=global:"undefined"!=typeof self&&(o=self),o.Foo=e()}}(function(){var define,module,exports;return (function e(t,n,r){function s(o,u){if(!n[o]){if(!t[o]){var a=typeof require=="function"&&require;if(!u&&a)return a(o,!0);if(i)return i(o,!0);throw new Error("Cannot find module '"+o+"'")}var f=n[o]={exports:{}};t[o][0].call(f.exports,function(e){var n=t[o][1][e];return s(n?n:e)},f,f.exports,e,t,n,r)}return n[o].exports}var i=typeof require=="function"&&require;for(var o=0;o<r.length;o++)s(r[o]);return s})({1:[function(_dereq_,module,exports){
+module.exports = true;
+
+},{}]},{},[1])
+(1)
+});

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: browserify-rails
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Henry Hsu, Cymen Vig
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-15 00:00:00.000000000 Z
+date: 2014-09-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sprockets
@@ -42,14 +42,14 @@ dependencies:
   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
@@ -70,42 +70,42 @@ dependencies:
   name: coffee-rails
   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: mocha
   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: pry
   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: Browserify + Rails = CommonJS Heaven
@@ -137,6 +137,7 @@ files:
 - test/dummy/app/assets/javascripts/app_main.js.example
 - test/dummy/app/assets/javascripts/app_secondary.js.example
 - test/dummy/app/assets/javascripts/application.js.example
+- test/dummy/app/assets/javascripts/browserified.js.example
 - test/dummy/app/assets/javascripts/coffee.js.coffee.example
 - test/dummy/app/assets/javascripts/foo.js.example
 - test/dummy/app/assets/javascripts/main.js.example
@@ -181,6 +182,7 @@ files:
 - test/fixtures/application.changed.out.js
 - test/fixtures/application.foo_changed.out.js
 - test/fixtures/application.out.js
+- test/fixtures/browserified.out.js
 - test/fixtures/empty_module.js
 - test/fixtures/foo.out.js
 - test/fixtures/main.out.js
@@ -198,12 +200,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
@@ -211,7 +213,7 @@ rubyforge_project:
 rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
-summary: ! 'Get the best of both worlds: Browserify + Rails = CommonJS Heaven'
+summary: 'Get the best of both worlds: Browserify + Rails = CommonJS Heaven'
 test_files:
 - test/browserify_processor_test.rb
 - test/compilation_test.rb
@@ -221,6 +223,7 @@ test_files:
 - test/dummy/app/assets/javascripts/app_main.js.example
 - test/dummy/app/assets/javascripts/app_secondary.js.example
 - test/dummy/app/assets/javascripts/application.js.example
+- test/dummy/app/assets/javascripts/browserified.js.example
 - test/dummy/app/assets/javascripts/coffee.js.coffee.example
 - test/dummy/app/assets/javascripts/foo.js.example
 - test/dummy/app/assets/javascripts/main.js.example
@@ -265,6 +268,7 @@ test_files:
 - test/fixtures/application.changed.out.js
 - test/fixtures/application.foo_changed.out.js
 - test/fixtures/application.out.js
+- test/fixtures/browserified.out.js
 - test/fixtures/empty_module.js
 - test/fixtures/foo.out.js
 - test/fixtures/main.out.js