amiel-sprockets 1.0.4 → 1.0.5

data/test/test_secretary.rb

@@ -0,0 +1,83 @@
+require "test_helper"
+
+class SecretaryTest < Test::Unit::TestCase
+  def test_load_locations_are_not_expanded_when_expand_paths_is_false
+    secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH)
+    secretary.add_load_location("src/**/", :expand_paths => false)
+    
+    assert_equal [File.join(FIXTURES_PATH, "src/**"), FIXTURES_PATH], 
+                 secretary.environment.load_path.map { |pathname| pathname.absolute_location }
+  end
+  
+  def test_load_locations_are_expanded_when_expand_paths_is_true
+    secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH)
+    secretary.add_load_location("src/**/", :expand_paths => true)
+    
+    assert_equal [File.join(FIXTURES_PATH, "src", "foo"), File.join(FIXTURES_PATH, "src"), FIXTURES_PATH],
+                 secretary.environment.load_path.map { |pathname| pathname.absolute_location }
+  end
+  
+  def test_source_files_are_not_expanded_when_expand_paths_is_false
+    secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH)
+    assert_raises(Sprockets::LoadError) do
+      secretary.add_source_file("src/f*.js", :expand_paths => false)
+    end
+  end
+  
+  def test_source_files_are_expanded_when_expand_paths_is_true
+    secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH)
+    secretary.add_source_file("src/f*.js", :expand_paths => true)
+    
+    assert_equal [File.join(FIXTURES_PATH, "src", "foo.js")],
+                 secretary.preprocessor.source_files.map { |source_file| source_file.pathname.absolute_location }
+  end
+  
+  def test_install_assets_into_empty_directory
+    with_temporary_directory do |temp|
+      secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH, :asset_root => temp)
+      secretary.add_source_file("src/script_with_assets.js")
+
+      assert_equal [], Dir[File.join(temp, "**", "*")]
+      secretary.install_assets
+      assert_equal paths_relative_to(temp, 
+        "images", "images/script_with_assets", "images/script_with_assets/one.png", 
+        "images/script_with_assets/two.png", "stylesheets", "stylesheets/script_with_assets.css"),
+        Dir[File.join(temp, "**", "*")].sort
+    end
+  end
+  
+  def test_install_assets_into_nonexistent_directory
+    with_temporary_directory do |temp|
+      temp = File.join(temp, "assets")
+      secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH, :asset_root => temp)
+      secretary.add_source_file("src/script_with_assets.js")
+
+      assert_equal [], Dir[File.join(temp, "**", "*")]
+      secretary.install_assets
+      assert_equal paths_relative_to(temp, 
+        "images", "images/script_with_assets", "images/script_with_assets/one.png", 
+        "images/script_with_assets/two.png", "stylesheets", "stylesheets/script_with_assets.css"),
+        Dir[File.join(temp, "**", "*")].sort
+    end
+  end
+  
+  def test_install_assets_into_subdirectories_that_already_exist
+    with_temporary_directory do |temp|
+      secretary = Sprockets::Secretary.new(:root => FIXTURES_PATH, :asset_root => temp)
+      secretary.add_source_file("src/script_with_assets.js")
+
+      FileUtils.mkdir_p(File.join(temp, "images", "script_with_assets"))
+      assert_equal paths_relative_to(temp, "images", "images/script_with_assets"), Dir[File.join(temp, "**", "*")]
+      secretary.install_assets
+      assert_equal paths_relative_to(temp, 
+        "images", "images/script_with_assets", "images/script_with_assets/one.png", 
+        "images/script_with_assets/two.png", "stylesheets", "stylesheets/script_with_assets.css"),
+        Dir[File.join(temp, "**", "*")].sort
+    end
+  end
+  
+  protected
+    def paths_relative_to(root, *paths)
+      paths.map { |path| File.join(root, path) }
+    end
+end

data/test/test_source_file.rb

@@ -0,0 +1,34 @@
+require "test_helper"
+require "enumerator"
+Enumerator = Enumerable::Enumerator unless defined?(Enumerator) # for 1.9.1 compatibility
+
+class SourceFileTest < Test::Unit::TestCase
+  def setup
+    @environment = environment_for_fixtures
+  end
+  
+  def test_each_source_line
+    source_file_lines = Enumerator.new(source_file("src/foo/bar.js"), :each_source_line).to_a
+    assert_equal content_of_fixture("src/foo/bar.js"), source_file_lines.map { |line| line.line }.join
+    assert_equal 4, source_file_lines.length
+  end
+  
+  def test_find_should_return_pathname_for_file_relative_to_the_current_pathname
+    assert_absolute_location_ends_with "test/fixtures/src/foo/bar.js", source_file("src/foo/foo.js").find("bar.js")
+  end
+  
+  def test_find_should_return_nil_for_nonexistent_file
+    assert_nil source_file("src/foo/foo.js").find("nonexistent.js")
+  end
+  
+  def test_equality_of_source_files
+    assert_equal source_file("src/foo/foo.js"), source_file("src/foo/foo.js")
+    assert_equal source_file("src/foo/foo.js"), source_file("src/foo/../foo/foo.js")
+    assert_not_equal source_file("src/foo/foo.js"), source_file("src/foo.js")
+    assert_not_equal source_file("src/foo/foo.js"), source_file("src/foo/bar.js")
+  end
+  
+  def test_mtime_should_return_now_if_file_does_not_exist
+    assert source_file("src/foo/nonexistent.js").mtime.instance_of?(Time)
+  end
+end

data/test/test_source_line.rb

@@ -0,0 +1,89 @@
+require "test_helper"
+
+class SourceLineTest < Test::Unit::TestCase
+  def test_line_that_begins_with_double_slash_should_be_a_comment
+    assert source_line("//").comment?
+    assert source_line("//test").comment?
+    assert source_line("//= require").comment?
+    assert source_line("//= require <foo>").comment?
+    assert source_line(" //").comment?
+    assert source_line("\t//").comment?
+  end
+
+  def test_line_that_begins_a_multiline_comment
+    assert source_line(" /*").begins_multiline_comment?
+    assert source_line(" /**").begins_multiline_comment?
+  end
+
+  def test_line_that_begins_a_pdoc_comment
+    assert !source_line(" /*").begins_pdoc_comment?
+    assert source_line(" /**").begins_pdoc_comment?
+  end
+
+  def test_line_that_ends_a_multiline_comment
+    assert source_line(" */").ends_multiline_comment?
+    assert source_line(" **/").ends_multiline_comment?
+  end
+
+  def test_line_that_ends_a_pdoc_comment
+    assert !source_line(" */").ends_pdoc_comment?
+    assert source_line(" **/").ends_pdoc_comment?
+  end
+
+  def test_line_that_contains_but_does_not_begin_with_double_slash_should_not_be_a_comment
+    assert !source_line("f //").comment?
+    assert !source_line("f //= require <foo>").comment?
+  end
+
+  def test_comment_should_be_extracted_from_comment_lines
+    assert_equal "test", source_line("//test").comment
+    assert_equal " test", source_line("// test").comment
+    assert_equal nil, source_line("f //test").comment
+  end
+  
+  def test_line_that_contains_require_comment_should_be_a_require
+    assert source_line("//= require <foo>").require?
+    assert !source_line("//= require<foo>").require?
+    assert source_line("//= require \"foo\"").require?
+    assert !source_line("//= require <foo> f").require?
+  end
+  
+  def test_require_should_be_extracted_from_require_lines
+    assert_nil source_line("//= require").require
+    assert_equal "<foo>", source_line("//= require <foo>").require
+    assert_equal "<foo>", source_line("//= require   <foo> ").require
+    assert_equal "\"foo\"", source_line("//= require \"foo\"").require
+  end
+
+  def test_line_that_contains_a_provide_comment_should_be_a_provide
+    assert source_line("//= provide \"../assets\"").provide?
+    assert !source_line("//= provide").provide?
+    assert !source_line("//= provide <../assets>").provide?
+  end
+  
+  def test_provide_should_be_extracted_from_provide_lines
+    assert_nil source_line("//= provide").provide
+    assert_equal "../assets", source_line("//= provide \"../assets\"").provide
+  end
+
+  def test_inspect_should_include_source_file_location_and_line_number
+    environment = environment_for_fixtures
+    pathname    = Sprockets::Pathname.new(environment, "/a/b/c.js")
+    source_file = Sprockets::SourceFile.new(environment, pathname)
+    assert_equal "line 25 of #{File.expand_path("/a/b/c.js")}", source_line("hello", source_file, 25).inspect
+  end
+  
+  def test_interpolation_of_constants
+    assert_equal %(var VERSION = "1.0";\n), source_line('var VERSION = "<%= VERSION %>";').to_s("VERSION" => "1.0")
+  end
+  
+  def test_interpolation_of_missing_constant_raises_undefined_constant_error
+    assert_raises(Sprockets::UndefinedConstantError) do
+      source_line('<%= NONEXISTENT %>').to_s("VERSION" => "1.0")
+    end
+  end
+  
+  def test_to_s_should_strip_trailing_whitespace_before_adding_line_ending
+    assert_equal "hello();\n", source_line("hello();     \t  \r\n").to_s({})
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: amiel-sprockets
 version: !ruby/object:Gem::Version 
-  version: 1.0.4
+  version: 1.0.5
 platform: ruby
 authors: 
 - Sam Stephenson
@@ -22,8 +22,54 @@ extensions: []
 
 extra_rdoc_files: []
 
-files: []
-
+files: 
+- Rakefile
+- bin/sprocketize
+- lib/sprockets
+- lib/sprockets/concatenation.rb
+- lib/sprockets/environment.rb
+- lib/sprockets/error.rb
+- lib/sprockets/pathname.rb
+- lib/sprockets/preprocessor.rb
+- lib/sprockets/secretary.rb
+- lib/sprockets/source_file.rb
+- lib/sprockets/source_line.rb
+- lib/sprockets/version.rb
+- lib/sprockets.rb
+- test/fixtures
+- test/fixtures/assets
+- test/fixtures/assets/images
+- test/fixtures/assets/images/script_with_assets
+- test/fixtures/assets/images/script_with_assets/one.png
+- test/fixtures/assets/images/script_with_assets/two.png
+- test/fixtures/assets/stylesheets
+- test/fixtures/assets/stylesheets/script_with_assets.css
+- test/fixtures/constants.yml
+- test/fixtures/double_slash_comments_that_are_not_requires_should_be_ignored_when_strip_comments_is_false.js
+- test/fixtures/double_slash_comments_that_are_not_requires_should_be_removed_by_default.js
+- test/fixtures/multiline_comments_should_be_removed_by_default.js
+- test/fixtures/requiring_a_file_after_it_has_already_been_required_should_do_nothing.js
+- test/fixtures/requiring_a_file_that_does_not_exist_should_raise_an_error.js
+- test/fixtures/requiring_a_file_with_css_style_comments_should_replace_the_require_comment_with_file_contents.css
+- test/fixtures/requiring_a_single_file_should_replace_the_require_comment_with_the_file_contents.js
+- test/fixtures/requiring_the_current_file_should_do_nothing.js
+- test/fixtures/src
+- test/fixtures/src/constants.yml
+- test/fixtures/src/foo
+- test/fixtures/src/foo/bar.js
+- test/fixtures/src/foo/foo.js
+- test/fixtures/src/foo.css
+- test/fixtures/src/foo.js
+- test/fixtures/src/script_with_assets.js
+- test/test_concatenation.rb
+- test/test_environment.rb
+- test/test_helper.rb
+- test/test_pathname.rb
+- test/test_preprocessor.rb
+- test/test_secretary.rb
+- test/test_source_file.rb
+- test/test_source_line.rb
+- ext/nph-sprockets.cgi
 has_rdoc: false
 homepage: http://getsprockets.org/
 post_install_message: 
@@ -45,7 +91,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   version: 
 requirements: []
 
-rubyforge_project: 
+rubyforge_project: sprockets
 rubygems_version: 1.2.0
 signing_key: 
 specification_version: 2