jsus 0.2.7 → 0.3.0

data/spec/data/ComplexDependencies/Output/package.js

@@ -0,0 +1,70 @@
+/*
+---
+
+script: Input.js
+
+description: Generic Input
+
+license: MIT-style license
+
+authors:
+- Yaroslaff Fedin
+
+provides: [Input]
+
+...
+*/
+/*
+---
+
+script: Core.js
+
+description: Mootools fake core
+
+license: MIT-style license
+
+authors:
+- Valerio Proietti
+
+provides: [Core]
+
+...
+*/
+/*
+---
+
+script: Color.js
+
+description: A library to work with colors
+
+license: MIT-style license
+
+authors:
+- Valerio Proietti
+
+requires:
+  - Mootools/Core
+
+provides: [Color]
+
+...
+*/
+/*
+---
+
+script: Input.Color.js
+
+description: Cool colorpicker for everyone to enjoy
+
+license: MIT-style license
+
+authors:
+- Yaroslaff Fedin
+
+requires:
+- Color
+
+provides: [Input.Color]
+
+...
+*/

data/spec/data/ComplexDependencies/Output/scripts.json

@@ -0,0 +1,14 @@
+{
+  "Package": {
+    "requires": [
+
+    ],
+    "desc": "Jsus package with external dependency",
+    "provides": [
+      "Input",
+      "Color",
+      "Input.Color",
+      "Mootools/Core"
+    ]
+  }
+}

data/spec/data/ComplexDependencies/Output/tree.json

@@ -0,0 +1,33 @@
+{
+  "Widget": {
+    "Input": {
+      "Input.Color": {
+        "provides": [
+          "Input.Color"
+        ],
+        "requires": [
+          "Color"
+        ],
+        "desc": "Cool colorpicker for everyone to enjoy"
+      },
+      "provides": [
+        "Input"
+      ],
+      "requires": [
+
+      ],
+      "desc": "Generic Input"
+    }
+  },
+  "Library": {
+    "Color": {
+      "provides": [
+        "Color"
+      ],
+      "requires": [
+        "Mootools/Core"
+      ],
+      "desc": "A library to work with colors"
+    }
+  }
+}

data/spec/data/ComplexDependencies/Source/Library/Color.js

@@ -0,0 +1,19 @@
+/*
+---
+
+script: Color.js
+
+description: A library to work with colors
+
+license: MIT-style license
+
+authors:
+- Valerio Proietti
+
+requires:
+  - Mootools/Core
+
+provides: [Color]
+
+...
+*/

data/spec/data/ComplexDependencies/Source/Widget/Input/Input.Color.js

@@ -0,0 +1,20 @@
+/*
+---
+
+script: Input.Color.js
+
+description: Cool colorpicker for everyone to enjoy
+
+license: MIT-style license
+
+authors:
+- Yaroslaff Fedin
+
+requires:
+- Color
+- Input
+
+provides: [Input.Color]
+
+...
+*/

data/spec/data/ComplexDependencies/Source/Widget/Input.js

@@ -0,0 +1,16 @@
+/*
+---
+
+script: Input.js
+
+description: Generic Input
+
+license: MIT-style license
+
+authors:
+- Yaroslaff Fedin
+
+provides: [Input]
+
+...
+*/

data/spec/data/ComplexDependencies/package.yml

@@ -0,0 +1,10 @@
+name: Package
+filename: package.js
+web: http://github.com/markiz/jsus
+description: Jsus package with external dependency
+license: Public Domain, http://unlicense.org/UNLICENSE
+authors: Mark Abramov (http://github.com/markiz)
+sources:
+  - "Source/Library/Color.js"
+  - "Source/Widget/Input/Input.Color.js"
+  - "Source/Widget/Input.js"

data/spec/data/mooforge_quirky_source.js

@@ -0,0 +1,20 @@
+/*
+---
+
+script: mooforge_quirky_source.js
+
+description: plugin support file
+
+license: UNLICENSE
+
+authors:
+- Mark Abramov
+
+provides:
+  - Support
+
+requires:
+  - mootools_core/1.3.0: Core
+
+...
+*/

data/spec/data/unicode_source.js

@@ -0,0 +1,23 @@
+/*
+---
+
+script: Array.Extras.js
+
+name: Array.Extras
+
+description: Extends the Array native object to include useful methods to work with arrays.
+
+license: MIT-style license
+
+authors:
+  - Christoph Pojer
+  - Sebastian Markbåge
+
+requires:
+  - Core/Array
+  - MooTools.More
+
+provides: [Array.Extras]
+
+...
+*/

data/spec/data/unicode_source_with_bom.js

@@ -0,0 +1,23 @@
+/*
+---
+
+script: Array.Extras.js
+
+name: Array.Extras
+
+description: Extends the Array native object to include useful methods to work with arrays.
+
+license: MIT-style license
+
+authors:
+  - Christoph Pojer
+  - Sebastian Markbåge
+
+requires:
+  - Core/Array
+  - MooTools.More
+
+provides: [Array.Extras]
+
+...
+*/

data/spec/jsus/middleware_spec.rb

@@ -0,0 +1,216 @@
+require 'spec_helper'
+require 'sinatra/base'
+require 'rack/test'
+
+describe Jsus::Middleware do
+  include Rack::Test::Methods
+  def new_server
+    Sinatra.new do
+      use Jsus::Middleware
+      set :port, 4567
+      set :raise_errors, true
+      set :show_exceptions, false
+      get("/") { "Ping" }
+    end
+  end
+
+  def suppress_output
+    old_stdout, old_stderr = $stdout, $stderr
+    $stdout, $stderr = IO.new('/dev/null'), IO.new('/dev/null')
+    yield
+    $stdout, $stderr = old_stdout, old_stderr
+  end
+
+  before(:all) do
+    @server = new_server
+    @server_thread = Thread.new { suppress_output { @server.run! } }
+  end
+
+  after(:all) { @server_thread.kill }
+
+  let(:app) { Rack::Lint.new(@server) }
+
+  it "should not override non-jsus calls" do
+    get("/").body.should == "Ping"
+  end
+
+  describe "settings" do
+    it "should have some defaults" do
+      described_class.settings.should be_a(Hash)
+    end
+
+    it "should merge settings, instead of full overriding on .settings=" do
+      described_class.settings = {:hello => :world}
+      described_class.settings = {:people => :worthless}
+      described_class.settings.should include({:hello => :world, :people => :worthless})
+    end
+  end
+
+  context "for basic package with an external dependency" do
+    let(:packages_dir) { File.expand_path("spec/data/ComplexDependencies") }
+    before(:each) { described_class.settings = {:packages_dir => packages_dir} }
+
+    describe "/javascripts/jsus/require/Package.js" do
+      let(:path) { "/javascripts/jsus/require/Package.js" }
+
+      it "should be successful" do
+        get(path).should be_successful
+      end
+
+      it "should respond with type text/javascript" do
+        get(path).content_type.should == "text/javascript"
+      end
+
+      it "should respond with generated package content" do
+        get(path).body.should include("script: Input.js")
+        get(path).body.should include("script: Color.js")
+        get(path).body.should include("script: Input.Color.js")
+      end
+
+      it "should include dependencies by default" do
+        get(path).body.should include("script: Core.js")
+      end
+
+      it "should preserve correct order" do
+        body = get(path).body
+        body.index("script: Core.js").should < body.index("script: Color.js")
+        body.index("script: Color.js").should < body.index("script: Input.Color.js")
+        body.index("script: Input.js").should < body.index("script: Input.Color.js")
+      end
+    end
+
+    describe "using ~Tags" do
+      let(:path) { "/javascripts/jsus/require/Package~Package:Color.js" }
+
+      it "should be successful" do
+        get(path).should be_successful
+      end
+
+      it "should respond with type text/javascript" do
+        get(path).content_type.should == "text/javascript"
+      end
+
+      it "should respond with generated content" do
+        get(path).body.should include("script: Input.Color.js")
+      end
+
+      it "should not include dependencies mentioned in exclude directive" do
+        get(path).body.should_not include("script: Color.js")
+        get(path).body.should_not include("script: Core.js")
+      end
+
+      it "should not fail when used with non-existent tag" do
+        result = get("/javascripts/jsus/require/Package~Package:Colorful.js")
+        result.should be_successful
+        result.body.should include("script: Input.Color.js")
+      end
+
+      it "should be chainable" do
+        result = get("/javascripts/jsus/require/Package~Mootools:Core~Package:Input.js")
+        result.should be_successful
+        result.body.should include("script: Input.Color.js")
+        result.body.should include("script: Color.js")
+        result.body.should_not include("script: Input.js")
+        result.body.should_not include("script: Core.js")
+      end
+    end
+
+    describe "using +Tags" do
+      let(:path) { "/javascripts/jsus/require/Package:Color+Package:Input.js" }
+
+      it "should be successful" do
+        get(path).should be_successful
+      end
+
+      it "should respond with type text/javascript" do
+        get(path).content_type.should == "text/javascript"
+      end
+
+      it "should respond with generated content" do
+        get(path).body.should include("script: Color.js")
+        get(path).body.should include("script: Input.js")
+      end
+
+      it "should not include files not mentioned in include directive" do
+        get(path).body.should_not include("script: Input.Color.js")
+      end
+
+      it "should not fail when used with non-existent tag" do
+        result = get("/javascripts/jsus/require/Package:Color+Package:Colorful.js")
+        result.should be_successful
+        result.body.should include("script: Color.js")
+      end
+
+      it "should be chainable" do
+        result = get("/javascripts/jsus/require/Package:Color+Package:Input~Mootools:Core.js")
+        result.should be_successful
+        result.body.should include("script: Input.js")
+        result.body.should include("script: Color.js")
+        result.body.should_not include("script: Input.Color.js")
+        result.body.should_not include("script: Core.js")
+      end
+    end
+
+    describe "/javascripts/jsus/require/Package:Input.Color.js" do
+      let(:path) { "/javascripts/jsus/require/Package:Input.Color.js" }
+
+      it "should be successful" do
+        get(path).should be_successful
+      end
+
+      it "should respond with type text/javascript" do
+        get(path).content_type.should == "text/javascript"
+      end
+
+      it "should respond with generated content" do
+        get(path).body.should include("script: Input.Color.js")
+      end
+
+      it "should include dependencies by default" do
+        get(path).body.should include("script: Color.js")
+        get(path).body.should include("script: Core.js")
+      end
+
+      it "should preserve correct order" do
+        body = get(path).body
+        body.index("script: Color.js").should < body.index("script: Input.Color.js")
+        body.index("script: Color.js").should > body.index("script: Core.js")
+      end
+    end
+  end
+
+  describe "for invalid paths" do
+    describe "when package is non-existent" do
+      let(:path) { "/javascripts/jsus/require/randomshit.js" }
+      it "should return 404" do
+        get(path).should be_not_found
+      end
+    end
+
+    describe "when required file is non-existent" do
+      let(:path) { "/javascripts/jsus/require/nonexistent:randomshit.js" }
+      it "should return 404" do
+        get(path).should be_not_found
+      end
+    end
+
+    describe "when given random gibberish" do
+      let(:path) { "/javascripts/jsus/require/+++---asda~~:sda_s+__+-dr928213dasasda=d%20%32%13__=_-=--/asa/sd/.asd13/.js" }
+      it "should return 404" do
+        get(path).should be_not_found
+      end
+    end
+  end
+
+  describe "caching" do
+    let(:cache_path) { "spec/tmp" }
+    before(:each) { Jsus::Middleware.settings = {:cache => true, :cache_path => cache_path} }
+    after(:each) { FileUtils.rm_rf(cache_path) }
+    let(:path) { "/javascripts/jsus/require/Package.js" }
+    it "should save output of requests to files" do
+      result = get(path).body
+      File.exists?("#{cache_path}/Package.js").should be_true
+      File.read("#{cache_path}/Package.js").should == result
+    end
+  end
+end

data/spec/jsus/package_spec.rb

@@ -31,7 +31,7 @@ describe Jsus::Package do
         end
 
         it "should set up outside dependencies" do
-          subject.dependencies_names.should == ['core/Class']
+          subject.dependencies_names.should == ['Core/Class']
         end
 
         it "should set directory field" do
@@ -61,12 +61,12 @@ describe Jsus::Package do
       required_files = Dir["#{input_dir}/**/*.js"].map {|f| IO.read(f) }
       required_files.each {|f| compiled_content.should include(f)}
     end
-    
+
     context "when given nil" do
       it "should not raise errors" do
         lambda { subject.compile(nil) }.should_not raise_error
       end
-      
+
       it "should return a string with compiled content" do
         compiled_content = subject.compile(nil)
         required_files = Dir["#{input_dir}/**/*.js"].map {|f| IO.read(f) }
@@ -84,7 +84,7 @@ describe Jsus::Package do
       info["provides"].should have_exactly(4).items
       info["provides"].should include("Color", "Widget", "Input", "Input.Color")
     end
-    
+
     context "when external dependencies are included" do
       let(:lib_dir) { "spec/data/ChainDependencies/app/javascripts" }
       let(:pool) { Jsus::Pool.new(lib_dir) }
@@ -122,7 +122,7 @@ describe Jsus::Package do
       tree["Widget"]["Input"]["Input.Color"]["requires"].should include("Input", "Color")
       tree["Widget"]["Input"]["Input.Color"]["provides"].should == ["Input.Color"]
     end
-    
+
     it "should allow different filenames" do
       subject.generate_tree(output_dir, "structure.json")
       File.exists?("#{output_dir}/structure.json").should be_true
@@ -130,7 +130,7 @@ describe Jsus::Package do
       tree["Library"]["Color"]["provides"].should == ["Color"]
     end
   end
-  
+
   describe "#required_files" do
    it "should list required files in correct order" do
      required_files = subject.required_files
@@ -174,4 +174,10 @@ describe Jsus::Package do
     end
 
   end
+
+  describe "#filename" do
+    it "should convert package name to snake_case" do
+      subject.filename.should == "orwik.js"
+    end
+  end
 end

data/spec/jsus/source_file_spec.rb

@@ -1,3 +1,4 @@
+# encoding: utf-8
 require 'spec_helper'
 
 describe Jsus::SourceFile do
@@ -34,12 +35,12 @@ describe Jsus::SourceFile do
           lambda { Jsus::SourceFile.from_file('spec/data/non-existant-file.js') }.should raise_error
         end
       end
-      
+
       context "when some error happens" do
         it "should raise error" do
           YAML.stub!(:load) { raise "Could not parse the file!" }
           lambda { subject }.should raise_error(RuntimeError, /spec\/data\/test_source_one\.js/)
-        end        
+        end
       end
     end
 
@@ -56,6 +57,28 @@ describe Jsus::SourceFile do
       source.header.should            == subject.header
     end
 
+    it "should not break on unicode files" do
+      source = nil
+      lambda { source = Jsus::SourceFile.from_file("spec/data/unicode_source.js") }.should_not raise_error
+      source.header["authors"][1].should == "Sebastian Markbåge"
+    end
+
+    it "should recognize and ignore BOM marker" do
+      source = nil
+      lambda { source = Jsus::SourceFile.from_file("spec/data/unicode_source_with_bom.js")  }.should_not raise_error
+      source.header["authors"][1].should == "Sebastian Markbåge"
+    end
+
+    if defined?(Encoding) && Encoding.respond_to?(:default_external)
+      it "should not break on unicode files even when external encoding is set to non-utf" do
+        old_external = Encoding.default_external
+        Encoding.default_external = 'us-ascii'
+        source = nil
+        lambda { source = Jsus::SourceFile.from_file("spec/data/unicode_source.js") }.should_not raise_error
+        source.header["authors"][1].should == "Sebastian Markbåge"
+        Encoding.default_external = old_external
+      end
+    end
   end
 
   context "when no package set, " do
@@ -79,13 +102,13 @@ describe Jsus::SourceFile do
         subject.dependencies_names(:short => true).should == ["Class"]
       end
     end
-    
+
     describe "#external_dependencies" do
       it "should be empty" do
         subject.external_dependencies.should be_empty
       end
     end
-    
+
     describe "#external_dependencies_names" do
       it "should be empty" do
         subject.external_dependencies_names.should be_empty
@@ -126,18 +149,18 @@ describe Jsus::SourceFile do
         subject.dependencies_names(:short => true).should include("Class", "Mash/Mash")
       end
     end
-    
+
     describe "#external_dependencies" do
-      it "should include external dependencies" do 
+      it "should include external dependencies" do
         subject.should have_exactly(1).external_dependencies
         subject.external_dependencies.should include(Jsus::Tag["Mash/Mash"])
       end
-      
+
       it "should not include internal dependencies" do
         subject.external_dependencies.should_not include(Jsus::Tag["Orwik/Class"])
       end
     end
-    
+
     describe "#external_dependencies_names" do
       it "should include names of external dependencies" do
         subject.external_dependencies_names.should include("Mash/Mash")
@@ -147,7 +170,7 @@ describe Jsus::SourceFile do
         subject.external_dependencies_names.should_not include("Orwik/Class")
       end
     end
-    
+
   end
 
   context "when pool is not set, " do
@@ -259,18 +282,23 @@ describe Jsus::SourceFile do
       end
     end
   end
-  
+
+  it "should allow quirky mooforge dependencies syntax" do
+    subject = described_class.from_file("spec/data/mooforge_quirky_source.js")
+    subject.dependencies_names.should == ["MootoolsCore/Core"]
+  end
+
   describe "#==, eql, hash" do
     it "should return true for source files pointing to the same physical file" do
       subject.should == described_class.from_file(subject.filename)
       subject.should eql(described_class.from_file(subject.filename))
       subject.hash.should == described_class.from_file(subject.filename).hash
     end
-    
+
     it "should return false for source files pointing to different physical files" do
       subject.should_not == described_class.from_file("spec/data/Extensions/app/javascripts/Orwik/Extensions/Class.js")
       subject.should_not eql(described_class.from_file("spec/data/Extensions/app/javascripts/Orwik/Extensions/Class.js"))
       subject.hash.should_not == described_class.from_file("spec/data/Extensions/app/javascripts/Orwik/Extensions/Class.js").hash
-    end    
-  end  
-end
+    end
+  end
+end

data/spec/jsus/tag_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Jsus::Tag do
   subject { Jsus::Tag.new("Wtf") }
 
-  context "initialization" do  
+  context "initialization" do
     it "should set given name" do
       Jsus::Tag.new("Wtf").name.should == "Wtf"
     end
@@ -45,6 +45,12 @@ describe Jsus::Tag do
     it "should return a given tag if given a tag" do
       Jsus::Tag.new(subject).should == subject
     end
+
+    it "should translate mooforge styled names into jsus-styled names" do
+      Jsus::Tag.new("mootools_core/Wtf").should      == Jsus::Tag.new("MootoolsCore/Wtf")
+      Jsus::Tag.new("Effects.Fx/Hello.World").should == Jsus::Tag.new("EffectsFx/Hello.World")
+      Jsus::Tag.new("effects.fx/Wtf").should         == Jsus::Tag.new("EffectsFx/Wtf")
+    end
   end
 
   describe "#name" do
@@ -106,7 +112,7 @@ describe Jsus::Tag do
     it "should return true when tag is empty" do
       Jsus::Tag.new("").should be_empty
     end
-    
+
     it "should return false when tag is not empty" do
       Jsus::Tag.new("Core/Mash").should_not be_empty
     end