ymdp 0.1.6 → 0.1.7

data/spec/compiler_spec.rb

@@ -34,16 +34,22 @@ describe "Compiler" do
     end
   end
   
+  describe "log" do
+    it "should return output formatted with time" do
+      @compiler.log("Hello").should match(/Hello$/)
+    end
+  end
+  
   describe "process" do
     before(:each) do
       YMDP::ApplicationView.stub!(:supported_languages).and_return([])
-      
-      @view_template = mock('view_template').as_null_object
-      YMDP::Compiler::Template::View.stub!(:new).and_return(@view_template)
-      @js_template = mock('js_template').as_null_object
-      YMDP::Compiler::Template::JavaScript.stub!(:new).and_return(@js_template)
-      @yrb_template = mock('yrb_template').as_null_object
-      YMDP::Compiler::Template::YRB.stub!(:new).and_return(@yrb_template)
+
+      # @view_template = mock('view_template').as_null_object
+      # YMDP::Compiler::Template::View.stub!(:new).and_return(@view_template)
+      # @js_template = mock('js_template').as_null_object
+      # YMDP::Compiler::Template::JavaScript.stub!(:new).and_return(@js_template)
+      # @yrb_template = mock('yrb_template').as_null_object
+      # YMDP::Compiler::Template::YRB.stub!(:new).and_return(@yrb_template)
       
       Dir.stub!(:[]).and_return([])
     end
@@ -52,9 +58,36 @@ describe "Compiler" do
       File.should_receive(:exists?).with("#{@base_path}/servers/#{@domain}")
       @compiler.process_all
     end
+    
+    describe "copy_images" do
+      it "should remove images folder" do
+        FileUtils.should_receive(:rm_rf).with("./app/servers/#{@domain}/assets/images")
+        @compiler.process_all
+      end
+      
+      it "should create images folder" do
+        FileUtils.should_receive(:mkdir_p).with("./app/servers/#{@domain}/assets")
+        @compiler.process_all
+      end
+      
+      it "should copy images from app to server" do
+        FileUtils.should_receive(:cp_r).with("./app/assets/images/", "./app/servers/#{@domain}/assets")
+        @compiler.process_all
+      end
+      
+      it "should log in verbose mode" do
+        @compiler = YMDP::Compiler::Base.new(@domain, @git_hash, :base_path => @base_path, :server => "staging", :verbose => true)
+        $stdout.should_receive(:puts).with(/Moving images/)
+        @compiler.process_all
+      end
+    end
 
     describe "translations" do
       before(:each) do
+
+        @yrb_template = mock('yrb_template').as_null_object
+        YMDP::Compiler::Template::YRB.stub!(:new).and_return(@yrb_template)
+        
         @supported_languages = ["en-US", "de-DE"]
         YMDP::ApplicationView.stub!(:supported_languages).and_return(@supported_languages)
       end
@@ -102,6 +135,9 @@ describe "Compiler" do
         before(:each) do
           @files = ["./app/views/view.html.haml"]
           Dir.stub!(:[]).with("./app/views/**/*").and_return(@files)
+
+          @view_template = mock('view_template').as_null_object
+          YMDP::Compiler::Template::View.stub!(:new).and_return(@view_template)
         end
     
         it "should run on all views in the path" do
@@ -125,6 +161,9 @@ describe "Compiler" do
         before(:each) do
           @files = ["./app/views/view.html.erb"]
           Dir.stub!(:[]).with("./app/views/**/*").and_return(@files)
+
+          @view_template = mock('view_template').as_null_object
+          YMDP::Compiler::Template::View.stub!(:new).and_return(@view_template)
         end
     
         it "should run on all views in the path" do
@@ -146,6 +185,9 @@ describe "Compiler" do
     end
     
     it "should run on all views in the path" do
+      @js_template = mock('js_template').as_null_object
+      YMDP::Compiler::Template::JavaScript.stub!(:new).and_return(@js_template)
+
       files = ["./app/assets/javascripts/view.js"]
       Dir.should_receive(:[]).with("./app/assets/**/*").and_return(files)
       @compiler.process_all

data/spec/compiler_template_spec.rb

@@ -3,23 +3,99 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 require 'compiler/base'
 require 'compiler/template'
 
-
-describe "Compiler" do
+describe "Template" do
   before(:each) do
+    stub_screen_io
+    
     reset_constant(:YMDP_ENV, "build")
-    reset_constant(:CONFIG, mock('config').as_null_object)
 
     Application.stub!(:current_view=)
     
-    stub_io
+    @domain = "staging"
+    @filename = "filename.html.haml"
+    @git_hash = "asjkhfasjkfhjk"
+    @message = "This is a commit message."
+    
+    stub_yrb_configuration
+    
+    stub_config
   end
   
-  describe "view" do
+  describe "javascript" do
+    before(:each) do
+      stub_file_utils
+      
+      @filename = "filename.pres"
+      
+      @params = {
+        :verbose => false,
+        :domain => @domain,
+        :file => @filename,
+        :git_hash => @git_hash,
+        :message => @message
+      }
+      
+      @js_template = YMDP::Compiler::Template::YRB.new(@params)
+    end
+    
+    it "should instantiate" do
+      @js_template.should_not be_nil
+    end
+    
+    describe "build" do
+      before(:each) do
+        @file = mock('file', :write => true)
+        File.stub!(:open).with(/filename.json/, "w").and_return(@file)
+        F.stub!(:save_to_file)
+      end
+    
+      it "should save json" do
+        File.stub!(:read).with("filename.pres").and_return("KEY=value")
+        @js_template.build.should == "{\"KEY\":\"value\"}"
+      end
+      
+      it "should skip lines that aren't keys" do
+        File.stub!(:read).with("filename.pres").and_return("KEY=value\nnot a key\n")
+        @js_template.build.should == "{\"KEY\":\"value\"}"
+      end
+      
+      it "should raise error for duplicate keys" do
+        File.stub!(:read).with("filename.pres").and_return("KEY=value\nKEY=value\n")
+        lambda {
+          @js_template.build
+        }.should raise_error("Duplicate key error")
+      end
+      
+      describe "methods" do
+        before(:each) do
+          @file = mock('file', :write => true)
+          File.stub!(:open).with(/filename.json/, "w").and_return(@file)
+          F.stub!(:save_to_file)
+          File.stub!(:read).with("filename.pres").and_return("KEY=value")
+        end
+        
+        it "should return json" do
+          @js_template.to_json.should == "{\"KEY\":\"value\"}"
+        end
+        
+        it "should return yaml" do
+          @js_template.to_yaml.should == "--- \nkey: value\n"
+        end
+        
+        it "should validate" do
+          YMDP::Validator::JSON.should_receive(:validate)
+          @js_template.validate
+        end
+      end
+    end
+  end
+
+  
+  describe "javascript" do
     before(:each) do
-      @domain = "staging"
-      @filename = "filename.html.haml"
-      @git_hash = "asjkhfasjkfhjk"
-      @message = "This is a commit message."
+      stub_file_utils
+      
+      @filename = "filename.js"
       
       @params = {
         :verbose => false,
@@ -28,12 +104,76 @@ describe "Compiler" do
         :git_hash => @git_hash,
         :message => @message
       }
-      stub_yrb_configuration
       
+      @js_template = YMDP::Compiler::Template::JavaScript.new(@params)
+    end
+    
+    it "should instantiate" do
+      @js_template.should_not be_nil
+    end
+    
+    it "should build" do
+      File.stub!(:read).with("filename.js").and_return("unprocessed")
+      
+      # process the contents of filename.html.erb through ERB
+      #
+      @erb = mock('erb', :result => "processed template output")
+      ERB.should_receive(:new).with("unprocessed", anything, anything).and_return(@erb)
+      
+      F.stub!(:save_to_file).with("processed template output")
+      
+      @js_template.build.should == "processed template output"
+    end
+    
+    it "should compress" do
+      @config.stub!(:compress_embedded_js?).and_return(true)
+      
+      # process the contents of filename.html.erb through ERB
+      #
+      @erb = mock('erb', :result => "processed template output")
+      ERB.should_receive(:new).with("unprocessed", anything, anything).and_return(@erb)      
+      
+      YMDP::Compressor::JavaScript.stub!(:compress).and_return("compressed template")
+      File.stub!(:read).with("filename.js").and_return("unprocessed")
+      
+      F.stub!(:save_to_file)
+      
+      @js_template.build.should == "compressed template"
+    end
+  end
+  
+  describe "view" do
+    before(:each) do
+      @params = {
+        :verbose => false,
+        :domain => @domain,
+        :file => @filename,
+        :git_hash => @git_hash,
+        :message => @message
+      }      
       @view_template = YMDP::Compiler::Template::View.new(@params)
     end
     
     describe "instantiation" do
+      it "should raise an error if called on base" do
+        lambda {
+          @view_template = YMDP::Compiler::Template::Base.new(@params)
+        }.should raise_error("Define in child")
+      end
+      
+      it "should raise an error if process_template is called from base" do
+        class Thing < YMDP::Compiler::Template::Base
+          def base_filename(filename)
+            filename
+          end
+        end
+        
+        @thing = Thing.new(@params)
+        lambda {
+          @thing.process_template("template")
+        }.should raise_error("Define in child")
+      end
+      
       it "should instantiate" do
         @view_template.should_not be_nil
       end
@@ -65,11 +205,103 @@ describe "Compiler" do
       end
     end
     
+    describe "build" do
+      before(:each) do
+        @processed_haml = "processed haml"
+        stub_haml_class
+        
+        YMDP::Validator::HTML.stub!(:validate)
+      end
+      
+      it "should not build if it's a partial" do
+        @view_template = YMDP::Compiler::Template::View.new(@params.merge({:file => "_partial.html.haml"}))
+        @view_template.build.should be_nil
+      end
+      
+      it "should get Haml when no layout exists" do
+        stub_file_utils
+        
+        # File.exists? will return false, so no layouts exist
+        
+        # read the contents of filename.html.haml
+        #
+        File.stub!(:read).with("filename.html.haml").and_return("unprocessed")
+        
+        # process the contents of filename.html.haml through Haml::Engine
+        #
+        @haml = mock('haml', :render => "processed_template output")
+        Haml::Engine.stub!(:new).with("unprocessed", :filename=>"filename.html.haml").and_return(@haml)
+        
+        @view_template.build.should == "processed_template output"
+      end
+      
+      it "should get ERB when no layout exists" do
+        stub_file_utils
+        
+        # File.exists? will return false, so no layouts exist
+        
+        # read the contents of filename.html.haml
+        #
+        File.stub!(:read).with("filename.html.erb").and_return("unprocessed")
+        
+        # process the contents of filename.html.erb through ERB
+        #
+        @erb = mock('erb', :result => "processed_template output")
+        ERB.should_receive(:new).with("unprocessed", anything, anything).and_return(@erb)
+        
+        @view_template = YMDP::Compiler::Template::View.new(@params.merge({:file => "filename.html.erb"}))
+        @view_template.build.should == "processed_template output"
+      end
+      
+      describe "haml template exists" do
+        it "should return processed layout" do
+          stub_file_utils
+          
+          File.stub!(:exists?).with(/application.html.haml/).and_return(true)
+          
+          # read the contents of application.html.haml
+          #
+          File.stub!(:read).with(/application.html.haml/).and_return("unprocessed layout") 
+          
+          # process the contents of application.html.haml through Haml::Engine
+          #
+          @layout_haml = mock('layout_haml', :render => "processed layout")
+          Haml::Engine.stub!(:new).with("unprocessed layout", :filename=>"./base_path//app/views/layouts/application.html.haml").and_return(@layout_haml)
+          
+          # read the contents of filename.html.haml
+          #
+          File.stub!(:read).with("filename.html.haml").and_return("unprocessed")
+          
+          # process the contents of filename.html.haml through Haml::Engine
+          #
+          @haml = mock('haml', :render => "processed template output")
+          Haml::Engine.stub!(:new).with("unprocessed", :filename=>"filename.html.haml").and_return(@haml)
+          
+          @view_template.build.should == "processed layout"        
+        end
+      end
+    end
+    
     describe "validation" do
       it "should raise an error without a server" do
         lambda {
           @view_template = YMDP::Compiler::Template::View.new(@params.merge({:domain => "nowhere"}))
-        }.should raise_error
+        }.should raise_error("Server settings are required.")
+      end
+      
+      it "should raise an error without a server" do
+        @invalid_servers = {
+          "staging" => {
+            "application_id" => "abcdefg_1"
+          }
+        }
+        YMDP::Base.configure do |config|
+          config.servers = @invalid_servers
+        end
+        
+        lambda {
+          @view_template = YMDP::Compiler::Template::View.new(@params)
+        }.should raise_error("Server name does not exist in server settings.")
       end
     end
   end

data/spec/compressor_spec.rb

@@ -0,0 +1,123 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "Compressor" do
+  before(:each) do
+    stub_io
+  end
+  
+  describe "Base" do
+    it "should use the compressed file if it already exists" do
+      File.stub(:exists?).and_return(true)
+      @file = "compressed file"
+      File.stub(:read).and_return(@file)
+      YMDP::Compressor::Base.compress("file.js", "type" => "js").should == @file
+    end
+    
+    describe "generate a compressed file if one doesn't exist" do
+      before(:each) do
+        File.stub!(:exists?).and_return(false, true)
+      end
+      
+      it "should log what it's doing" do
+        $stdout.should_receive(:print).with(/file.js  compressing . . ./)
+        YMDP::Compressor::Base.compress("file.js", "type" => "js")
+      end
+      
+      it "should run the compressor" do
+        F.should_receive(:execute).with(/yuicompressor/, :return => true).and_return("")
+        YMDP::Compressor::Base.compress("file.js", "type" => "js")
+      end
+      
+      describe "options" do
+        it "should set nomunge" do
+          F.should_receive(:execute).with(/nomunge/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.js", "type" => "js", "obfuscate" => true)
+        end
+      
+        it "should not set nomunge" do
+          F.stub!(:execute).with(/yuicompressor/, :return => true).and_return("")
+          F.should_not_receive(:execute).with(/nomunge/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.js", "type" => "js", "obfuscate" => false)
+        end
+      
+        it "should set verbose" do
+          F.should_receive(:execute).with(/verbose/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.js", "type" => "js", "verbose" => true)
+        end
+      
+        it "should not set verbose" do
+          F.stub!(:execute).with(/yuicompressor/, :return => true).and_return("")
+          F.should_not_receive(:execute).with(/verbose/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.js", "type" => "js", "verbose" => false)
+        end
+      
+        it "should set preserve-semi on javascript" do
+          F.should_receive(:execute).with(/preserve-semi/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.js", "type" => "js")
+        end
+      
+        it "should not set preserve-semi on css" do
+          F.stub!(:execute).with(/yuicompressor/, :return => true).and_return("")
+          F.should_not_receive(:execute).with(/preserve-semi/, :return => true).and_return("")
+          YMDP::Compressor::Base.compress("file.css", "type" => "css")
+        end
+      end
+      
+      describe "on errors" do
+        before(:each) do
+          F.stub!(:execute).with(/yuicompressor/, :return => true).and_return("[ERROR] 12:13: Too much fruzzlegump")
+        end
+        
+        it "should raise an exception" do
+          lambda {
+            YMDP::Compressor::Base.compress("file.js", "type" => "js")
+          }.should raise_error(/JavaScript errors/)
+        end
+        
+        it "should growl" do
+          @g.should_receive(:notify)
+          lambda {
+            YMDP::Compressor::Base.compress("file.js", "type" => "js")
+          }.should raise_error(/JavaScript errors/)
+        end
+        
+        it "should show the source code" do
+          F.should_receive(:get_line_from_file).with("file.js", 12).and_return("")
+          lambda {
+            YMDP::Compressor::Base.compress("file.js", "type" => "js")
+          }.should raise_error(/JavaScript errors/)
+        end
+      end
+      
+      it "should report OK" do
+        $stdout.should_receive(:puts).with("OK")
+        YMDP::Compressor::Base.compress("file.js", "type" => "js")
+      end
+      
+      it "should raise an error if the compressed file doesn't exist" do
+        File.stub!(:exists?).and_return(false)
+        lambda {
+          YMDP::Compressor::Base.compress("file.js", "type" => "js")
+        }.should raise_error(/File does not exist/)
+      end
+    end
+  end
+  
+  describe "JavaScript" do
+    it "should call Base with type js" do
+      File.stub(:exists?).and_return(true)
+      @file = "compressed file"
+      File.stub(:read).and_return(@file)
+      YMDP::Compressor::JavaScript.compress("file.js").should == @file
+    end
+  end
+  
+  describe "Stylesheet" do
+    it "should call Base with type css" do
+      File.stub(:exists?).and_return(true)
+      @file = "compressed file"
+      File.stub(:read).and_return(@file)
+      YMDP::Compressor::Stylesheet.compress("file.css").should == @file
+    end
+  end
+end