ymdp 0.1.6 → 0.1.7

data/spec/configuration_spec.rb

@@ -1,13 +1,41 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "Configuration" do
+  before(:each) do
+    stub_screen_io
+    reset_constant(:YMDP_ENV, "test")
+  end
+  
   describe "Config" do
     before(:each) do
+      # using strings for some boolean values to avoid false positives
+      
       @config_yml = {
         "config" => {
           "username" => "captmal",
-          "password" => "iluvinara",
-          "growl" => true
+          "password" => "inara",
+          "growl" => 'growl',
+          "verbose" => 'verbose',
+          "compress" => {
+            "embedded_js" => "compress_embedded_js",
+            "js_assets" => "compress_js_assets",
+            "css" => "compress_css",
+            "obfuscate" => "obfuscate"
+          },
+          "validate" => {
+            "embedded_js" => {
+              "test" => "validate_embedded_js"
+            },
+            "js_assets" => {
+              "test" => "validate_js_assets"
+            },
+            "json_assets" => {
+              "test" => "validate_json_assets"
+            },
+            "html" => {
+              "test" => "validate_html"
+            }
+          }
         }
       }
       YAML.stub!(:load_file).and_return(@config_yml)
@@ -22,10 +50,60 @@ describe "Configuration" do
       @config.password.should == @config_yml["config"]["password"]
     end
     
-    it "should return growl" do
+    it "should return growl?" do
       @config.growl?.should == @config_yml["config"]["growl"]
     end
     
+    it "should return verbose?" do
+      @config.verbose?.should == @config_yml["config"]["verbose"]
+    end
+    
+    describe "compress settings" do
+    
+      it "should return compress_embedded_js?" do
+        @config.compress_embedded_js?.should == @config_yml["config"]["compress"]["embedded_js"]
+      end
+    
+      it "should return compress_js_assets?" do
+        @config.compress_js_assets?.should == @config_yml["config"]["compress"]["js_assets"]
+      end
+    
+      it "should return compress_css?" do
+        @config.compress_css?.should == @config_yml["config"]["compress"]["css"]
+      end
+    
+      it "should return obfuscate?" do
+        @config.obfuscate?.should == @config_yml["config"]["compress"]["obfuscate"]
+      end
+    end
+    
+    describe "validate settings" do
+      it "should return validate_embedded_js?" do
+        @config.validate_embedded_js?.should == @config_yml["config"]["validate"]["embedded_js"][YMDP_ENV]
+      end
+    
+      it "should return validate_js_assets?" do
+        @config.validate_js_assets?.should == @config_yml["config"]["validate"]["js_assets"][YMDP_ENV]
+      end
+    
+      it "should return validate_json_assets?" do
+        @config.validate_json_assets?.should == @config_yml["config"]["validate"]["json_assets"][YMDP_ENV]
+      end
+    
+      it "should return validate_html?" do
+        @config.validate_html?.should == @config_yml["config"]["validate"]["html"][YMDP_ENV]
+      end
+    end
+    
+    describe "each" do
+      it "should go through each of the values" do
+        $stdout.should_receive(:puts).exactly(@config_yml["config"].keys.length).times
+        @config.each do |key, value|
+          $stdout.puts key
+        end
+      end
+    end
+    
     describe "[]" do
       it "should return username" do
         @config["username"].should == @config_yml["config"]["username"]
@@ -35,5 +113,40 @@ describe "Configuration" do
         @config["password"].should == @config_yml["config"]["password"]
       end
     end
+    
+    describe "file not found" do
+      before(:each) do
+        File.stub!(:exists?).and_return(false)
+      end
+      
+      it "should raise an error" do
+        lambda {
+          @config = YMDP::Configuration::Config.new
+        }.should raise_error(/File not found/)
+      end
+      
+      it "should output to the screen" do
+        $stdout.should_receive(:puts).with(/with the following command/)
+        lambda {
+          @config = YMDP::Configuration::Config.new
+        }.should raise_error(/File not found/)        
+      end
+    end
+  end
+  
+  describe "Server" do
+    before(:each) do
+      @servers_yml = {
+        "servers" => {
+          "server" => "staging"
+        }
+      }
+      YAML.stub!(:load_file).and_return(@servers_yml)
+      @servers = YMDP::Configuration::Servers.new
+    end
+    
+    it "should return servers hash" do
+      @servers.servers.should == @servers_yml["servers"]
+    end
   end
 end

data/spec/file_spec.rb

@@ -0,0 +1,103 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "F" do
+  before(:each) do
+    stub_file_io
+  end
+  
+  describe "concat_files" do
+    before(:each) do
+      Dir.stub!(:[]).with("./dir/*").and_return(["./dir/first.html", "./dir/second.html"])
+    end
+    
+    it "should get the directory" do
+      Dir.should_receive(:[]).with("./dir/*").and_return([])
+      F.concat_files("./dir/*", "destination")
+    end
+    
+    it "should read each file in the directory" do
+      File.should_receive(:read).and_return("first", "second")
+      F.concat_files("./dir/*", "destination")
+    end
+    
+    it "should write the contents of each file in the directory" do
+      File.stub!(:read).and_return("first", "second")
+      @file.should_receive(:puts).with("first")
+      @file.should_receive(:puts).with("second")
+      F.concat_files("./dir/*", "destination")
+    end
+    
+    it "should write to the destination file" do
+      File.should_receive(:open).with("destination", "a").and_yield(@file)
+      F.concat_files("./dir/*", "destination")
+    end
+  end
+  
+  describe "save_to_file" do
+    before(:each) do
+      @output = "output"
+      @destination_path = "@destination"
+    end
+    
+    it "should check the existence of destination_path" do
+      File.should_receive(:exists?).with(@destination_path)
+      F.save_to_file(@output, @destination_path)
+    end
+    
+    it "should return false if it exists" do
+      File.stub!(:exists?).with(@destination_path).and_return(true)
+      F.save_to_file(@output, @destination_path).should == false
+    end
+    
+    it "should open the destination path" do
+      File.should_receive(:open).with(@destination_path, "w").and_return(@file)
+      F.save_to_file(@output, @destination_path)
+    end
+    
+    it "should write to the destination file" do
+      @file.should_receive(:write).with(@output)
+      F.save_to_file(@output, @destination_path)
+    end
+    
+    it "should return true" do
+      F.save_to_file(@output, @destination_path).should be_true
+    end
+  end
+  
+  describe "get_line_from_file" do
+    before(:each) do
+      @path = "path"
+      @lines = ["first\n", "second\n", "third\n", "fourth\n", "fifth\n", "sixth\n", "seventh\n"]
+      File.stub!(:readlines).with(@path).and_return(@lines)
+    end
+    
+    it "should read lines from file" do
+      File.should_receive(:readlines).and_return(@lines)
+      F.get_line_from_file(@path, 7)
+    end
+    
+    it "should get the previous three lines" do
+      F.get_line_from_file(@path, 7).should == "\nfifth\nsixth\nseventh\n\n"
+    end
+  end
+  
+  # I'm not sure how to test the ` method because it calls system calls that don't seem
+  # reachable.
+  #
+  describe "execute" do
+    it "should call system" do
+      Kernel.should_receive(:system).with("ls")
+      F.execute("ls")
+    end
+    
+    it "should call system" do
+      Kernel.should_receive(:system).with("ls")
+      F.execute("ls", :reutrn => false)
+    end
+    
+    it "should not call system" do
+      Kernel.should_not_receive(:system)
+      F.execute("ls", :return => true)
+    end
+  end
+end

data/spec/git_helper_spec.rb

@@ -0,0 +1,53 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+require 'compiler/git_helper'
+
+describe "GitHelper" do
+  before(:each) do
+    stub_screen_io
+    
+    @repo = mock('repo')
+    Repo.stub!(:new).and_return(@repo)
+    
+    F.stub!(:execute).with("git status", :return => true).and_return("# On branch git-helper")
+    @commit = mock('commit', :id => "abcdefg")
+    @commits = mock('commits', :first => @commit)
+    @repo.stub!(:commits).with("git-helper").and_return(@commits)
+    
+    @git = YMDP::GitHelper.new
+  end
+  
+  it "should instantiate" do
+    @git.should_not be_nil
+  end
+  
+  it "should get hash" do
+    @git.get_hash("branch").should == "abcdefg"
+  end
+  
+  it "should get current branch" do
+    @git.get_current_branch.should == "git-helper"
+  end
+  
+  describe "commit" do
+    before(:each) do
+      @repo.stub!(:add)
+      F.stub!(:execute).with(/git commit/, :return => true).and_return("Committed")
+    end
+    
+    it "should add current directory" do
+      @repo.should_receive(:add).with(".")
+      @git.do_commit("message")
+    end
+    
+    it "should commit with message" do
+      F.should_receive(:execute).with("git commit -am \"message\"", :return => true).and_return("Committed")
+      @git.do_commit("message")
+    end
+    
+    it "should output results" do
+      $stdout.should_receive(:puts).with("Committed")
+      @git.do_commit("message")
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -6,7 +6,11 @@ CONFIG_PATH = "#{BASE_PATH}/config"
 SERVERS_PATH = "#{BASE_PATH}/servers"
 TMP_PATH = "#{BASE_PATH}/tmp"
 
+YMDP_ENV = "test"
+
 require 'ymdp'
+require 'g'
+require 'active_support'
 require 'spec'
 require 'spec/autorun'
 

data/spec/stubs.rb

@@ -3,6 +3,7 @@ def stub_io
   stub_file_io
   stub_file_utils
   stub_yaml
+  stub_growl
 end
 
 def stub_screen_io
@@ -13,9 +14,14 @@ end
 def stub_file_io(unprocessed_file="")
   @file ||= mock('file').as_null_object
   @file.stub!(:read).and_return(unprocessed_file)
-    
+  @file.stub!(:write)
+  @file.stub!(:puts)
+  
+  File.stub!(:new).and_return(@file)
   File.stub!(:exists?).and_return(false)
   File.stub!(:open).and_yield(@file) 
+  File.stub!(:read).and_return(unprocessed_file)
+  File.stub!(:readlines).and_return(["first\n", "second\n"])
 end
 
 def stub_file_utils
@@ -24,6 +30,10 @@ def stub_file_utils
   FileUtils.stub!(:cp_r)
   FileUtils.stub!(:mkdir_p)
   F.stub!(:concat_files)
+  F.stub!(:get_line_from_file).and_return("")
+  F.stub!(:save_to_file)
+  F.stub!(:save_to_tmp_file)
+  F.stub!(:execute).and_return("")
 end
 
 def stub_yaml(output_hash={})
@@ -36,9 +46,24 @@ def stub_erb(processed_file="")
   ERB.stub!(:new).and_return(@erb)
 end
 
+def stub_haml_class
+  eval %(
+  module Haml
+    class Engine
+    end
+  end
+  )
+end
+
+def stub_haml(processed_file)
+  @haml = mock('haml').as_null_object
+  @haml.stub!(:render).and_return(processed_file)
+  Haml::Engine.stub!(:new).and_return(@haml)
+end
+
 def stub_git_helper
   @git_helper = mock('git_helper').as_null_object
-  GitHelper.stub!(:new).and_return(@git_helper)
+  YMDP::GitHelper.stub!(:new).and_return(@git_helper)
 end
 
 def stub_timer
@@ -47,7 +72,22 @@ def stub_timer
   Timer.stub!(:new).and_return(@timer)
 end
 
+def stub_growl
+  @g = Object.new
+  Growl.stub(:new).and_return(@g)
+  @g.stub(:notify).as_null_object  
+end
+
 def reset_constant(constant, value)
   Object.send(:remove_const, constant)
   Object.const_set(constant, value)
+end
+
+def stub_config
+  @config = mock('config')
+  @config.stub!(:[]).with("doctype").and_return("HTML 4.0 Transitional")
+  @config.stub!(:validate_html?).and_return(false)
+  @config.stub!(:compress_embedded_js?).and_return(false)
+  @config.stub!(:verbose?).and_return(false)
+  reset_constant(:CONFIG, @config)
 end

data/spec/translator_spec.rb

@@ -3,4 +3,320 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 require 'translator/base'
 
 describe "Translator" do
-end
+  before(:each) do
+    stub_timer
+    stub_screen_io
+  end
+  
+  describe "Yaml" do
+    before(:each) do
+      YAML.stub!(:load_file).and_return({"hi" => "what"})
+      @original_translations = ["application_en-US.yml", "sidebar_en-US.yml"]
+      Dir.stub!(:[]).with(/app\/assets\/yrb\/en-US\/\*\.yml/).and_return(@original_translations)
+    end
+    
+    describe "class methods" do
+      it "should output a message for each file" do
+        File.stub!(:readlines).and_return([""])
+        @original_translations.each do |file|
+          $stdout.should_receive(:puts).with(/Processing /)
+        end
+        YMDP::Translator::Yaml.translate
+      end
+      
+      it "should instantiate a new template" do
+        @original_translations.each do |file|
+          @yaml = mock('yaml', :copy => true)
+          YMDP::Translator::Yaml.should_receive(:new).with(file).and_return(@yaml)
+        end
+        YMDP::Translator::Yaml.translate
+      end
+      
+      it "should copy the new template" do
+        @original_translations.each do |file|
+          @yaml = mock('yaml', :copy => true)
+          @yaml.should_receive(:copy)
+          YMDP::Translator::Yaml.stub!(:new).with(file).and_return(@yaml)
+        end
+        YMDP::Translator::Yaml.translate
+      end
+    end
+    
+    describe "instance methods" do
+      before(:each) do
+        @path = "path_en-US.yml"
+        @yml = YMDP::Translator::Yaml.new(@path)
+      end
+      
+      it "should instantiate" do
+        @yml.should_not be_nil
+      end
+      
+      it "should copy keys" do
+        File.stub!(:readlines).with("path_en-US.yml").and_return(["key: value\n"])
+        # 
+        @file = mock('file', :puts => true)
+        @file.should_receive(:puts).with(/key: translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::Yaml::LOCALES.size - 1
+        
+        YMDP::Translator::Yaml::LOCALES.each do |lang, code|
+          unless lang == "en-US"
+            Dir.should_receive(:[]).with(/app\/assets\/yrb\/#{lang}\/\*\.yml/).and_return(["keys_#{lang}.yml"])
+          end
+        end
+        
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yml.copy
+      end
+      
+      it "should not copy comments" do
+        File.stub!(:readlines).with("path_en-US.yml").and_return(["# comment\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_not_receive(:puts).with(/key: translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::Yaml::LOCALES.size - 1
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yml.copy
+      end
+      
+      it "should not copy empty keys" do
+        File.stub!(:readlines).with("path_en-US.yml").and_return(["KEY=\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_not_receive(:puts).with(/key: translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::Yaml::LOCALES.size - 1
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yml.copy
+      end
+      
+      it "should copy keys with variables" do
+        File.stub!(:readlines).with("path_en-US.yml").and_return(["key: value {0}\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_receive(:puts).with(/key: translated value {0}/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::Yaml::LOCALES.size - 1
+        
+        YMDP::Translator::Yaml::LOCALES.each do |lang, code|
+          unless lang == "en-US"
+            Dir.should_receive(:[]).with(/app\/assets\/yrb\/#{lang}\/\*\.yml/).and_return(["keys_#{lang}.yml"])
+          end
+        end
+        
+        Translate.stub!(:t).with("value [0]", "ENGLISH", anything).exactly(size).and_return("translated value [0]")
+        
+        @yml.copy
+      end
+    end
+  end
+  
+  describe "YRB" do
+    before(:each) do
+      @original_translations = ["application_en-US.pres", "sidebar_en-US.pres"]
+      Dir.stub!(:[]).with(/app\/assets\/yrb\/en-US\/\*\.pres/).and_return(@original_translations)
+    end
+    
+    describe "class methods" do
+      it "should output a message for each file" do
+        File.stub!(:readlines).and_return([""])
+        @original_translations.each do |file|
+          $stdout.should_receive(:puts).with(/Processing /)
+        end
+        YMDP::Translator::YRB.translate
+      end
+      
+      it "should instantiate a new template" do
+        @original_translations.each do |file|
+          @yrb = mock('yrb', :copy => true)
+          YMDP::Translator::YRB.should_receive(:new).with(file).and_return(@yrb)
+        end
+        YMDP::Translator::YRB.translate
+      end
+      
+      it "should copy the new template" do
+        @original_translations.each do |file|
+          @yrb = mock('yrb', :copy => true)
+          @yrb.should_receive(:copy)
+          YMDP::Translator::YRB.stub!(:new).with(file).and_return(@yrb)
+        end
+        YMDP::Translator::YRB.translate
+      end
+    end
+    
+    describe "instance methods" do
+      before(:each) do
+        @path = "path_en-US.pres"
+        @yrb = YMDP::Translator::YRB.new(@path)
+      end
+      
+      it "should instantiate" do
+        @yrb.should_not be_nil
+      end
+      
+      it "should copy keys" do
+        File.stub!(:readlines).with("path_en-US.pres").and_return(["KEY=value\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_receive(:puts).with(/KEY=translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::YRB::LOCALES.size - 1
+        
+        @template = mock('template', :to_hash => {"key" => "value"})
+        YMDP::Compiler::Template::YRB.stub!(:new).and_return(@template)
+        
+        YMDP::Translator::YRB::LOCALES.each do |lang, code|
+          unless lang == "en-US"
+            Dir.should_receive(:[]).with(/app\/assets\/yrb\/#{lang}\/\*\.pres/).and_return(["keys_#{lang}.pres"])
+          end
+        end
+        
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yrb.copy
+      end
+      
+      it "should not copy comments" do
+        File.stub!(:readlines).with("path_en-US.pres").and_return(["# comment\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_not_receive(:puts).with(/KEY=translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::YRB::LOCALES.size - 1
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yrb.copy
+      end
+      
+      it "should not copy empty keys" do
+        File.stub!(:readlines).with("path_en-US.pres").and_return(["KEY=\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_not_receive(:puts).with(/KEY=translated value/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::YRB::LOCALES.size - 1
+        Translate.stub!(:t).with("value", "ENGLISH", anything).exactly(size).and_return("translated value")
+        
+        @yrb.copy
+      end
+      
+      it "should copy keys with variables" do
+        File.stub!(:readlines).with("path_en-US.pres").and_return(["KEY=value {0}\n"])
+        
+        @file = mock('file', :puts => true)
+        @file.should_receive(:puts).with(/KEY=translated value {0}/)
+        
+        File.stub!(:open).with(anything, "a").and_yield(@file)
+        
+        size = YMDP::Translator::YRB::LOCALES.size - 1
+        
+        @template = mock('template', :to_hash => {"key" => "value"})
+        YMDP::Compiler::Template::YRB.stub!(:new).and_return(@template)
+        
+        YMDP::Translator::YRB::LOCALES.each do |lang, code|
+          unless lang == "en-US"
+            Dir.should_receive(:[]).with(/app\/assets\/yrb\/#{lang}\/\*\.pres/).and_return(["keys_#{lang}.pres"])
+          end
+        end
+        
+        Translate.stub!(:t).with("value [0]", "ENGLISH", anything).exactly(size).and_return("translated value [0]")
+        
+        @yrb.copy
+      end
+    end
+  end
+  
+  describe "Base" do
+    describe "class methods" do
+      it "should raise error on original_translations" do
+        lambda {
+          YMDP::Translator::Base.original_translations
+        }.should raise_error("Define in child")
+      end
+    
+      it "should raise error on all_source_files" do
+        lambda {
+          YMDP::Translator::Base.all_source_files
+        }.should raise_error("Define in child")
+      end
+    
+      it "should raise error on template" do
+        lambda {
+          YMDP::Translator::Base.template
+        }.should raise_error("Define in child")
+      end
+    
+      it "should raise error on translate" do
+        lambda {
+          YMDP::Translator::Base.translate
+        }.should raise_error("Define in child")
+      end
+    end
+  
+    describe "instance methods" do
+      it "should raise an error on instantiation" do
+        lambda {
+          @base = YMDP::Translator::Base.new("path")
+        }.should raise_error("Define in child")
+      end
+    end
+
+    describe "raise in child" do
+      before(:each) do
+        class YMDPTranslatorBase < YMDP::Translator::Base
+          def language(path)
+            "en-US"
+          end
+      
+          def base_filename(path)
+            path
+          end
+        end
+        @thing = YMDPTranslatorBase.new("path")
+      end
+
+      it "should raise an error calling parse_template" do
+        lambda {
+          @thing.parse_template("path")
+        }.should raise_error("Define in child")
+      end
+
+      it "should raise an error calling format" do
+        lambda {
+          @thing.format("a", "b")
+        }.should raise_error("Define in child")
+      end
+
+      it "should raise an error calling key_and_value_from_line" do
+        lambda {
+          @thing.key_and_value_from_line("line")
+        }.should raise_error("Define in child")
+      end
+
+      it "should raise an error calling comment?" do
+        lambda {
+          @thing.comment?("line")
+        }.should raise_error("Define in child")
+      end
+    end
+  end
+end