opal-sprockets 0.4.9.1.0.3.7 → 1.0.0

data/spec/sprockets/server_spec.rb

@@ -20,60 +20,130 @@ describe Opal::Sprockets::Server do
   end
 
   it 'serves assets with complex sprockets requires' do
-    asset = app.sprockets['complex_sprockets']
+    asset = app.sprockets['complex_sprockets', accept: "application/javascript"]
     expect(asset).to be_truthy
 
-    assets = asset.to_a.map do |sub_asset|
-      sub_asset.logical_path.gsub(/\.self\.js$/, '.js')
+    assets = asset.metadata[:included].map do |sub_asset|
+      URI.parse(sub_asset).path.split(%r'/fixtures/|/stdlib/').last
     end
 
     %w[
-      base64.js
-      no_requires.js
-      jst_file.js
-      required_tree_test/required_file1.js
-      required_tree_test/required_file2.js
+      base64.rb
+      no_requires.rb
+      jst_file.jst.ejs
+      required_tree_test/required_file1.rb
+      required_tree_test/required_file2.rb
       file_with_directives.js
     ].each do |logical_path|
       expect(assets).to include(logical_path)
     end
   end
 
+  it 'recompiles the asset when its dependencies change' do
+    does_include = proc do |it, what, what_not=[], what_else=nil, what_else_not=nil|
+      expect(it).to include what_else if what_else
+      expect(it).not_to include what_else_not if what_else_not
+      what.each { |i| expect(it).to include %{modules["required_tree_test/required_file#{i}"]} }
+      what_not.each { |i| expect(it).not_to include %{modules["required_tree_test/required_file#{i}"]} }
+    end
+
+    ["default", "debug"].each do |pipeline|
+      asset = app.sprockets['require_tree_test', pipeline: pipeline, accept: "application/javascript"]
+      expect(asset).to be_truthy
+      does_include.(asset.to_s, [1,2], [3], nil, "UNIQUESTRING")
+      get "/assets/require_tree_test.#{pipeline}.js"
+      does_include.(last_response.body, [1,2], [3], nil, "UNIQUESTRING")
+
+      sleep 1 # Make sure to modify mtime
+
+      File.write(__dir__+"/../fixtures/required_tree_test/required_file2.rb", "p 'UNIQUESTRING1'")
+
+      asset = app.sprockets['require_tree_test', pipeline: pipeline, accept: "application/javascript"]
+      expect(asset).to be_truthy
+      does_include.(asset.to_s, [1,2], [3], "UNIQUESTRING1")
+      get "/assets/require_tree_test.#{pipeline}.js"
+      does_include.(last_response.body, [1,2], [3], "UNIQUESTRING1")
+
+      sleep 1 # Make sure to modify mtime
+
+      File.write(__dir__+"/../fixtures/required_tree_test/required_file2.rb", "p 'UNIQUESTRING2'")
+
+      asset = app.sprockets['require_tree_test', pipeline: pipeline, accept: "application/javascript"]
+      expect(asset).to be_truthy
+      does_include.(asset.to_s, [1,2], [3], "UNIQUESTRING2")
+      get "/assets/require_tree_test.#{pipeline}.js"
+      does_include.(last_response.body, [1,2], [3], "UNIQUESTRING2")
+
+      sleep 1 # Make sure to modify mtime
+
+      File.write(__dir__+"/../fixtures/required_tree_test/required_file2.rb", "p 'UNIQUESTRING3'")
+      File.write(__dir__+"/../fixtures/required_tree_test/required_file3.rb", "p 3")
+
+      asset = app.sprockets['require_tree_test', pipeline: pipeline, accept: "application/javascript"]
+      expect(asset).to be_truthy
+      does_include.(asset.to_s, [1,2,3], [], "UNIQUESTRING3")
+      get "/assets/require_tree_test.#{pipeline}.js"
+      does_include.(last_response.body, [1,2], [], "UNIQUESTRING3") # fails with 3 - it doesn't get new files
+
+      sleep 1 # Make sure to modify mtime
+    ensure
+      File.write(__dir__+"/../fixtures/required_tree_test/required_file2.rb", "p 2\n")
+      File.unlink(__dir__+"/../fixtures/required_tree_test/required_file3.rb") rescue nil
+    end
+  end
+
   describe 'source maps' do
-    RSpec::Matchers.define :include_inline_source_map do
+    RSpec::Matchers.define :include_source_map do
       match do |actual_response|
         actual_response.ok? &&
-        actual_response.body.lines.last.start_with?('//# sourceMappingURL=data:application/json;base64,')
+        actual_response.body.lines.last.start_with?('//# sourceMappingURL=')
       end
     end
 
-    def extract_inline_map(response)
+    def extract_map(path, response)
       last_line   = response.body.lines.last
-      b64_encoded = last_line.split('//# sourceMappingURL=data:application/json;base64,', 2)[1]
-      json_string = Base64.decode64(b64_encoded)
+      if last_line.start_with? "//# sourceMappingURL=data:application/json;base64,"
+        b64_encoded = last_line.split('//# sourceMappingURL=data:application/json;base64,', 2)[1]
+        json_string = Base64.decode64(b64_encoded)
+      else
+        map_file = last_line.split('//# sourceMappingURL=', 2)[1].chomp
+        map_file = relative_path(path, map_file)
+        map_file = get map_file
+        json_string = map_file.body
+      end
       JSON.parse(json_string, symbolize_names: true)
     end
 
+    def relative_path(path, file)
+      URI.join("http://example.com/", path, file).path
+    end
+
     it 'serves map for a top level file' do
-      get '/assets/opal_file.js'
-      expect(last_response).to include_inline_source_map
+      path = '/assets/opal_file.debug.js'
+      get path
+      expect(last_response).to include_source_map
 
-      map = extract_inline_map(last_response)
+      map = extract_map(path, last_response)
 
-      expect(map[:sources]).to eq(['opal_file.rb']).or eq(['opal_file'])
-      expect(map[:sourcesContent]).to eq(["require 'opal'\nputs 'hi from opal!'\n"])
+      expect(map[:file]).to eq('opal_file.rb')
+      expect(map[:sections].last[:map][:sources]).to eq(['opal_file.source.rb'])
+
+      expect(get(relative_path(path, map[:sections].last[:map][:sources][0])).body.chomp)
+        .to eq("require 'opal'\nputs 'hi from opal!'")
     end
 
     it 'serves map for a subfolder file' do
-      get '/assets/source_map/subfolder/other_file.self.js'
-      expect(last_response).to include_inline_source_map
+      path = '/assets/source_map/subfolder/other_file.debug.js'
+      get path
+      expect(last_response).to include_source_map
+
+      map = extract_map(path, last_response)
 
-      map = extract_inline_map(last_response)
+      expect(map[:file]).to eq('source_map/subfolder/other_file.rb')
+      expect(map[:sections].first[:map][:sources]).to eq(['other_file.source.rb'])
 
-      expect(map[:sources])
-        .to eq(['source_map/subfolder/other_file.rb'])
-        .or eq(['source_map/subfolder/other_file'])
-      expect(map[:sourcesContent]).to eq(["puts 'other!'\n"])
+      expect(get(relative_path(path, map[:sections][0][:map][:sources][0])).body.chomp)
+        .to eq("puts 'other!'")
     end
   end
 end

data/spec/tilt/opal_spec.rb

@@ -5,7 +5,7 @@ describe Opal::TiltTemplate do
   %w[rb js.rb opal js.opal].each do |ext|
     let(:ext) { ext }
 
-    describe %Q{with extension ".#{ext}"} do
+    describe %{with extension ".#{ext}"} do
       it "is registered for '.#{ext}' files" do
         expect(Tilt["test.#{ext}"]).to be <= described_class
       end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: opal-sprockets
 version: !ruby/object:Gem::Version
-  version: 0.4.9.1.0.3.7
+  version: 1.0.0
 platform: ruby
 authors:
 - Elia Schito
 - Adam Beynon
 autorequire:
-bindir: bin
+bindir: exe
 cert_chain: []
-date: 2020-09-10 00:00:00.000000000 Z
+date: 2021-02-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sprockets
@@ -17,28 +17,34 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: opal
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: tilt
   requirement: !ruby/object:Gem::Requirement
@@ -109,19 +115,51 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Sprockets support for Opal.
+- !ruby/object:Gem::Dependency
+  name: ejs
+  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:
 email: elia@schito.me
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/build.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - README.md
 - Rakefile
+- bin/console
+- bin/rake
+- bin/rspec
+- bin/setup
 - example/Gemfile
 - example/Gemfile.lock
 - example/app/application.rb
@@ -129,17 +167,20 @@ files:
 - example/config.ru
 - example/index.html.erb
 - lib/opal-sprockets.rb
+- lib/opal/environment.rb
+- lib/opal/processor.rb
 - lib/opal/sprockets.rb
+- lib/opal/sprockets/assets_helper.rb
 - lib/opal/sprockets/environment.rb
 - lib/opal/sprockets/erb.rb
-- lib/opal/sprockets/path_reader.rb
+- lib/opal/sprockets/mime_types.rb
 - lib/opal/sprockets/processor.rb
 - lib/opal/sprockets/server.rb
 - lib/opal/sprockets/version.rb
 - opal-sprockets.gemspec
 - spec/fixtures/complex_sprockets.js.rb.erb
 - spec/fixtures/file_with_directives.js
-- spec/fixtures/jst_file.js.jst
+- spec/fixtures/jst_file.jst.ejs
 - spec/fixtures/no_requires.rb
 - spec/fixtures/opal_file.rb
 - spec/fixtures/require_tree_test.rb
@@ -150,18 +191,19 @@ files:
 - spec/fixtures/source_map/subfolder/other_file.rb
 - spec/fixtures/sprockets_file.js.rb
 - spec/fixtures/sprockets_require_tree_test.rb
-- spec/shared/path_finder_shared.rb
-- spec/shared/path_reader_shared.rb
 - spec/spec_helper.rb
 - spec/sprockets/erb_spec.rb
-- spec/sprockets/path_reader_spec.rb
 - spec/sprockets/processor_spec.rb
 - spec/sprockets/server_spec.rb
 - spec/sprockets_spec.rb
 - spec/tilt/opal_spec.rb
 homepage: https://github.com/opal/opal-sprockets#readme
-licenses: []
-metadata: {}
+licenses:
+- MIT
+metadata:
+  homepage_uri: https://github.com/opal/opal-sprockets#readme
+  source_code_uri: https://github.com/opal/opal-sprockets
+  changelog_uri: https://github.com/opal/opal-sprockets/blob/master/CHANGELOG.md
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -170,21 +212,24 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
+  - - "<"
+    - !ruby/object:Gem::Version
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: Sprockets support for Opal.
 test_files:
 - spec/fixtures/complex_sprockets.js.rb.erb
 - spec/fixtures/file_with_directives.js
-- spec/fixtures/jst_file.js.jst
+- spec/fixtures/jst_file.jst.ejs
 - spec/fixtures/no_requires.rb
 - spec/fixtures/opal_file.rb
 - spec/fixtures/require_tree_test.rb
@@ -195,11 +240,8 @@ test_files:
 - spec/fixtures/source_map/subfolder/other_file.rb
 - spec/fixtures/sprockets_file.js.rb
 - spec/fixtures/sprockets_require_tree_test.rb
-- spec/shared/path_finder_shared.rb
-- spec/shared/path_reader_shared.rb
 - spec/spec_helper.rb
 - spec/sprockets/erb_spec.rb
-- spec/sprockets/path_reader_spec.rb
 - spec/sprockets/processor_spec.rb
 - spec/sprockets/server_spec.rb
 - spec/sprockets_spec.rb

data/.travis.yml

@@ -1,21 +0,0 @@
-language: ruby
-
-branches:
-  only:
-  - master
-  - /^.*-stable$/
-  - /^.*ci-check$/
-
-matrix:
-  include:
-    - rvm: 2.3
-      env: OPAL_VERSION='~> 1.0.0'
-    - rvm: 2.4
-      env: OPAL_VERSION='~> 1.0.0'
-    - rvm: 2.5
-      env: OPAL_VERSION='~> 1.0.0'
-    - rvm: 2.6
-      env: OPAL_VERSION='~> 1.0.0'
-
-notifications:
-  irc: "irc.freenode.org#opal"

data/lib/opal/sprockets/path_reader.rb

@@ -1,36 +0,0 @@
-module Opal
-  module Sprockets
-
-    class PathReader
-      def initialize(env, context)
-        @env ||= env
-        @context ||= context
-      end
-
-      def read path
-        if path.end_with? '.js'
-          context.depend_on_asset(path)
-          env[path, bundle: true].to_s
-        else
-          context.depend_on(path)
-          File.open(expand(path), 'rb:UTF-8'){|f| f.read}
-        end
-      rescue ::Sprockets::FileNotFound
-        nil
-      end
-
-      def expand path
-        env.resolve(path) or
-          # Sprockets 3 just returns nil for unknown paths
-          raise ::Sprockets::FileNotFound, path.inspect
-      end
-
-      def paths
-        env.paths
-      end
-
-      attr_reader :env, :context
-    end
-
-  end
-end

data/spec/shared/path_finder_shared.rb

@@ -1,19 +0,0 @@
-require 'spec_helper'
-
-shared_examples :path_finder do
-  # @param path
-  # the path we want to read
-  #
-  # @param full_path
-  # the expanded path that should be found
-  #
-  it 'returns the full path if the path exists' do
-    expect(path_finder.find(path)).to eq(full_path)
-  end
-
-  it 'returns nil if the path is missing' do
-    expect(path_finder.find('unexpected-path')).to eq(nil)
-  end
-end
-
-

data/spec/shared/path_reader_shared.rb

@@ -1,31 +0,0 @@
-require 'spec_helper'
-
-# Below the helpers expected from a spec that
-# includes these shared examples:
-#
-# @object [PathReader] path_reader the object under test
-# @method [String]     path        the path we want to read
-# @method [String]     contents    the contents we expect to be read
-#
-shared_examples :path_reader do
-  describe '#paths' do
-    it 'is an Enumberable' do
-      expect(path_reader.paths).to be_an(Enumerable)
-    end
-
-    it 'includes Opal.paths' do
-      paths = path_reader.paths.to_a
-      Opal.paths.each { |path| expect(paths).to include(path) }
-    end
-  end
-
-  describe '#read' do
-    it 'responds to #path' do
-      expect(path_reader.read(path)).to eq(contents)
-    end
-
-    it 'returns nil if the file is missing' do
-      expect(path_reader.read('unexpected-path!')).to be_nil
-    end
-  end
-end