web_resource_bundler 0.0.13

data/spec/web_resource_bundler/content_management/block_parser_spec.rb

@@ -0,0 +1,100 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../spec_helper"))
+module WebResourceBundler
+  describe BlockParser do
+
+    before(:each) do
+      @file_manager = FileManager.new(settings.resource_dir, settings.cache_dir)
+      @block_parser = BlockParser.new
+    end
+
+    #conditional comment spec
+    it "matches with different conditional comments" do
+      tests = ['<!--[if lte IE 7]>...<![endif]-->','<!--[if !(IE 7)]>...<![endif]-->',
+        '<!--[if IE]>...<![endif]-->']
+      tests.each do |test|
+        BlockParser::CONDITIONAL_BLOCK_PATTERN.should match(test)
+      end
+    end
+
+    #links pattern spec
+    it "matches with css or js link" do
+      tests = ["\<link href='\/cache\/style.css' \/\>","\<script src='\/cache\/script.js' \/script\>"]
+      tests.each do |link|
+        BlockParser::LINK_PATTERN.should match(link)
+      end
+    end
+
+    describe "#remove_links" do
+      it "deletes all links to resources (js, css) from block" do
+        block = @sample_block_helper.construct_links_block(styles, scripts)
+        block += @sample_block_helper.sample_inline_block
+        block += @sample_block_helper.construct_links_block(styles, scripts)
+        block += @sample_block_helper.sample_inline_block
+        @block_parser.remove_links(block).should == @sample_block_helper.sample_inline_block + @sample_block_helper.sample_inline_block
+      end
+
+      it "doesn't delete links to non js or css resource, like favicon for example" do
+        text = "<link href='1.css' /><link rel='shortcut icon' href='/favicon.ico' />"
+        @block_parser.remove_links(text).should == "<link rel='shortcut icon' href='/favicon.ico' />"
+      end
+
+      it "doesn't touch remote resources" do
+        text = "<link href='http://google.com/1.css' type='text/css' rel='stylesheet' />"
+        @block_parser.remove_links(text).include?(text).should be_true
+      end
+    end
+
+    describe "#parse" do
+
+      it "returns BlockData object" do
+        @block_parser.parse("").is_a?(BlockData).should be_true
+      end
+
+      
+
+      it "returns empty BlockData when block is empty" do
+        data = @block_parser.parse("")
+        data.files.should be_empty
+      end
+
+      def compare_block_datas(a,b)
+        a.files.size.should == b.files.size
+        (a.files.map {|f| f.path} - b.files.map{|f| f.path}).should be_empty
+        a.child_blocks.size.should == b.child_blocks.size
+        a.condition.should == b.condition
+      end
+
+      it "return BlockData with all content and child inline blocks" do
+        block_data = @block_parser.parse(@sample_block_helper.sample_block)
+        compare_block_datas(block_data, @sample_block_helper.sample_block_data)
+        compare_block_datas(block_data.child_blocks[0], @sample_block_helper.sample_block_data.child_blocks[0])
+      end
+
+    end
+
+    describe "#find_files" do
+    
+      it "returns array of css and js files linked in block" do
+        result = @block_parser.find_files(@sample_block_helper.construct_links_block(styles, scripts))
+        files = styles + scripts
+        result.each do |f|
+          files.include?(f.path).should be_true
+        end
+      end
+
+      it "recognize only css and js files" do
+        block = "<link href='/rss.atom' /> <link href='valid.css' />"
+        result = @block_parser.find_files(block)
+        result.first.path.should == 'valid.css'
+      end
+
+      it "recognize files only on disk, not full urls" do
+        block = "<link href='http://glogle.com/web.css' /> <link href='valid.css' />"
+        result = @block_parser.find_files(block)
+        result.first.path.should == 'valid.css'
+      end
+
+    end
+
+  end
+end

data/spec/web_resource_bundler/content_management/css_url_rewriter_spec.rb

@@ -0,0 +1,27 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../spec_helper"))
+describe WebResourceBundler::CssUrlRewriter do
+  describe "#rewrite_relative_path" do
+    it "returns absolute path using url and css file path" do
+      tests = {
+        "../image.gif" => "/styles/image.gif",
+        "./image.gif" => "/styles/skin/image.gif",
+        "../../image.gif" => "/image.gif",
+        "../.././1.gif" => '/1.gif',
+        '.././../styles/./../styles/3.jpg' => '/styles/3.jpg'
+      }
+      css_file_path = "/styles/skin/1.css"
+      tests.each do |key, value|
+        CssUrlRewriter.rewrite_relative_path(css_file_path, key).should == value 
+      end
+    end
+  end
+  describe "#rewrite_content_urls" do
+    it "rewrites all urls in css block" do
+      css = "abracada: url\t('../image.gif'); \n backaground-image: url(\"../../image.gif\");background: url(./i/backgrounds/menu.png) repeat-x 0 100%; }"
+      result = "abracada: url('\/styles\/image.gif'); \n backaground-image: url('\/image.gif');background: url('/styles/skin/i/backgrounds/menu.png') repeat-x 0 100%; }"
+      CssUrlRewriter.rewrite_content_urls!("/styles/skin/1.css", css).should == result
+    end
+  end
+
+end
+

data/spec/web_resource_bundler/content_management/resource_file_spec.rb

@@ -0,0 +1,37 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../spec_helper"))
+describe WebResourceBundler::ResourceFile do
+  describe "#new_css_file" do
+    it "creates new resource file of css type" do
+      f = WebResourceBundler::ResourceFile.new_css_file('a', 'b')
+      f.path.should == 'a'
+      f.content.should == 'b'
+      f.type.should == WebResourceBundler::ResourceFileType::CSS
+    end
+  end
+  describe "#new_js_file" do
+    it "creates new resource file of css type" do
+      f = WebResourceBundler::ResourceFile.new_js_file('a', 'b')
+      f.path.should == 'a'
+      f.content.should == 'b'
+      f.type.should == WebResourceBundler::ResourceFileType::JS
+    end
+  end
+  describe "#new_mhtml_file" do
+    it "creates new resource file of mhtml type" do
+      f = WebResourceBundler::ResourceFile.new_mhtml_file('a', 'b')
+      f.path.should == 'a'
+      f.content.should == 'b'
+      f.type.should == WebResourceBundler::ResourceFileType::MHTML
+    end
+  end
+  describe "#clone" do
+    it "creates full clone of resource file object" do
+      f = WebResourceBundler::ResourceFile.new_css_file('a', 'b')
+      clon = f.clone
+      f.object_id.should_not == clon.object_id
+      f.path.object_id.should_not == clon.path.object_id
+      f.content.object_id.should_not == clon.content.object_id
+    end
+  end
+end
+

data/spec/web_resource_bundler/file_manager_spec.rb

@@ -0,0 +1,60 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../spec_helper"))
+describe WebResourceBundler::FileManager do
+  before(:each) do
+    @settings = settings
+  end
+
+  def create_stub_file(name)
+    File.open(File.join(@settings.resource_dir, name), "w") do |file|
+      file.print "hi there"
+    end
+  end
+
+  before(:each) do
+    temp_dir = File.join(@settings.resource_dir, 'temp')
+    Dir.mkdir(temp_dir) unless File.exist?(temp_dir)
+    @bundle_url = 'temp/bundle.dat'
+    @bundle_path = File.join(@settings.resource_dir, @bundle_url)
+    create_stub_file(@bundle_url)
+    @manager = FileManager.new(@settings.resource_dir, @settings.cache_dir)
+  end
+
+  after(:each) do
+    FileUtils.rm_rf(File.join(@settings.resource_dir, 'temp'))
+  end
+
+  describe "#create_cache_dir" do
+    it "creates cache dir if it doesn't exists" do
+      dir_path = File.join(@settings.resource_dir, @settings.cache_dir)
+      FileUtils.rm_rf(dir_path)
+      @manager.create_cache_dir
+      File.exist?(dir_path).should == true
+    end
+  end
+
+  describe "#get_content" do
+    it "reads file and returns its content" do
+      @manager.get_content(@bundle_url).should == 'hi there'
+    end
+  end
+
+  describe "#full_path" do
+
+    it "returns full path using file url" do
+      @manager.full_path(@bundle_url).should == @bundle_path 
+    end
+
+  end
+
+  describe "#exist?" do
+
+    it "returns false when no such file in resource dir" do
+      @manager.exist?("non_existent_file.data").should == false
+    end
+
+    it "return true when file exists in resource dir" do
+      @manager.exist?(@bundle_url).should == true
+    end
+  end
+
+end

data/spec/web_resource_bundler/filters/bundle_filter/filter_spec.rb

@@ -0,0 +1,40 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../../spec_helper"))
+describe WebResourceBundler::Filters::BundleFilter::Filter do
+  before(:each) do
+    clean_cache_dir
+    @settings = settings
+    @bundle_settings = bundle_settings
+    @filter = Filters::BundleFilter::Filter.new(@bundle_settings, FileManager.new(@settings.resource_dir, @settings.cache_dir))
+    @block_data = @sample_block_helper.sample_block_data
+    css_type = ResourceFileType::CSS
+    js_type = ResourceFileType::JS
+    items = [@block_data.styles.map {|f| f.path}.sort] + @bundle_settings[:md5_additional_data]
+    @css_md5_value = Digest::MD5.hexdigest(items.flatten.join('|'))
+    @css_bundle_file = File.join(@settings.cache_dir, [css_type[:name] + '_' + @css_md5_value, 'en', css_type[:ext]].join('.'))
+    items = [@block_data.scripts.map {|f| f.path}.sort]  + @bundle_settings[:md5_additional_data]
+    js_md5_value = Digest::MD5.hexdigest(items.flatten.join('|'))
+    @js_bundle_file = File.join(@settings.cache_dir, [js_type[:name] + '_' + js_md5_value, 'en', js_type[:ext]].join('.'))
+  end
+
+  describe "#apply" do
+    it "bundles each block_data resources in single file" do
+      @filter.apply!(@block_data)
+      @block_data.styles.first.path.should == @css_bundle_file
+      @block_data.scripts.first.path.should == @js_bundle_file
+    end
+  end
+
+  describe "#get_md5" do
+    it "returns md5 from sorted filepaths and another additional data" do
+      @filter.get_md5(@block_data.styles).should == @css_md5_value
+    end
+  end
+
+  describe "#bundle_filepath" do
+    it "returns filename of bundle constructed from passed files" do
+      @filter.bundle_filepath(WebResourceBundler::ResourceFileType::CSS, @block_data.styles).should == @css_bundle_file 
+    end
+  end
+
+end
+

data/spec/web_resource_bundler/filters/bundle_filter/resource_packager_spec.rb

@@ -0,0 +1,41 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../../spec_helper"))
+require 'digest/md5'
+describe WebResourceBundler::Filters::BundleFilter::ResourcePackager do
+  before(:each) do
+    @settings = settings
+    file_manager = FileManager.new(@settings.resource_dir, @settings.cache_dir)
+    @file_packager = Filters::BundleFilter::ResourcePackager.new(@settings, file_manager)
+    @file_paths = styles.map do |url|
+      File.join(@settings.resource_dir, url)
+    end
+  end
+
+  describe "#extract_imported_files!" do
+    it "returns array of imported css files" do
+      content = "@import 'import/first.css';\n@import 'import/second.css';"
+      imported_files = @file_packager.extract_imported_files!(content, 'styles/base.css')
+      content.should == "\n"
+      imported_files.should == ['styles/import/first.css', 'styles/import/second.css']
+    end
+  end
+
+  describe "#bundle_files" do
+    it "throws ResourceNotFoundError exception if one of imported files not found" do
+      #creating file with content with imported unexistent files
+      files = [WebResourceBundler::ResourceFile.new_css_file('styles/base.css', "@import 'import/first.css';\n@import 'import/second.css';")]
+      lambda { @file_packager.bundle_files(files) }.should raise_error(Exceptions::ResourceNotFoundError) 
+    end
+    
+    it 'should bundle files in original order' do
+      files = [
+        WebResourceBundler::ResourceFile.new_js_file('files/jquery.js', 'JQUERY_FILE'),
+        WebResourceBundler::ResourceFile.new_js_file('files/jquery.carousel.js','JQUERY_CAROUSEL_FILE')
+      ]
+      result = @file_packager.bundle_files(files)
+      (result =~ /JQUERY_FILE/).should < (result =~ /JQUERY_CAROUSEL_FILE/)
+    end
+    
+  end
+
+end
+

data/spec/web_resource_bundler/filters/cdn_filter_spec.rb

@@ -0,0 +1,76 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../spec_helper"))
+describe WebResourceBundler::Filters::CdnFilter do
+  before(:each) do
+    @settings = settings
+    @cdn_settings = cdn_settings
+    @cdn_settings[:http_hosts] = ['http://boogle.com']
+    @cdn_settings[:https_hosts] = ['http://froogle.com']
+    @settings[:cdn_filter][:http_hosts] = @cdn_settings[:http_hosts] 
+    @settings[:cdn_filter][:https_hosts] = @cdn_settings[:https_hosts]
+    @file_manager = FileManager.new(@settings.resource_dir, @settings.cache_dir) 
+    @filter = Filters::CdnFilter::Filter.new(@cdn_settings, @file_manager)
+  end
+
+  describe "#host_for_image" do
+    it "returns host for image using its hash" do
+     @cdn_settings[:http_hosts] << 'http://froogle.com'
+     url = '/images/1.gif'
+     @filter.host_for_image(url).should == @cdn_settings[:http_hosts][url.hash % @cdn_settings[:http_hosts].size]
+    end
+
+    it "returns https host if request was https" do
+      @cdn_settings[:protocol] = 'https'
+      @filter = Filters::CdnFilter::Filter.new(@cdn_settings, @file_manager)
+      url = '/images/1.gif'
+      @filter.host_for_image(url).should == @cdn_settings[:https_hosts][url.hash % @cdn_settings[:https_hosts].size]
+    end
+  end
+
+  describe "#new_filename" do
+    it "adds cdn_ prefix to original file name" do
+      path = 'styles/1.css'
+      @filter.new_filepath(path).should == 'cache/cdn_1.css'
+    end
+  end
+
+  describe "#rewrite_content_urls!" do
+    before(:each) do
+      @file_path = '/styles/skin/1.css'
+    end
+
+    it "adds hosts to image urls" do
+      content = "background: url('../images/1.png');"
+      @filter.rewrite_content_urls!(@file_path, content)
+      content.should == "background: url('http://boogle.com/styles/images/1.png');"
+    end
+
+    it "doesn't add hosts for images encoded in base64" do
+      content = "background:url('data:image/png;base64,iVBORw0KGg); *background:url(mhtml:http://domain.com/cache/base64_ie_style_9648c01be7e50284958eb07877c70e03.en.css!rails5804) no-repeat 0 100%;"
+      clon = content.dup
+      @filter.rewrite_content_urls!(@file_path, content.dup)
+      content.should == clon 
+    end
+
+    it "binds image to one particular host" do
+      @cdn_settings[:http_hosts] << 'http://froogle.com'
+      @filter = Filters::CdnFilter::Filter.new(@cdn_settings, @file_manager)
+      content = "background: url('../images/1.png');background-image: url('../images/1.png');" 
+      host = @cdn_settings[:http_hosts]['/styles/images/1.png'.hash % @cdn_settings[:http_hosts].size]
+      url = "#{host}/styles/images/1.png"
+      @filter.rewrite_content_urls!(@file_path, content)
+      content.should == "background: url('#{url}');background-image: url('#{url}');"
+    end
+  end
+
+  describe "#apply" do
+    it "rewrites urls properly in all css file of given block_data" do
+      file = WebResourceBundler::ResourceFile.new_css_file('/temp.css', "background: url('./images/1.png');background-image: url('./images/1.png');")
+      block_data = BlockData.new
+      block_data.files = [file]
+      @filter.apply!(block_data)
+      block_data.files.first.path.should == File.join(@settings.cache_dir, 'cdn_temp.css')
+      block_data.files.first.content.should == "background: url('http://boogle.com/images/1.png');background-image: url('http://boogle.com/images/1.png');"
+    end
+  end
+
+end

data/spec/web_resource_bundler/filters/image_encode_filter/css_generator_spec.rb

@@ -0,0 +1,104 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "../../../spec_helper"))
+module WebResourceBundler::Filters::ImageEncodeFilter
+  describe CssGenerator do
+    before(:each) do
+      @settings = Settings.new base64_settings
+      @generator = CssGenerator.new(@settings, FileManager.new(@settings.resource_dir, @settings.cache_dir))
+    end
+    
+    describe "#pattern" do
+      it "should match with correct original tags" do
+        correct_values = ["#{CssGenerator::TAGS[0]}:url('temp/image.png');",
+          "  #{CssGenerator::TAGS[1]}\t  :\n  url('temp/image.png') ;",
+          "background:url('temp/1.png') repeat 0 0;"]
+        correct_values.each do |v|
+          v.should match(CssGenerator::PATTERN)	
+        end
+      end
+      
+      it "should not match with incorrect tags" do
+        incorrect_values = ["handy_dandy_tag:url('goody/truly.png')","background-image:urlec('asd/asdf.gif');", 
+        "background-imageis:url('asdf/asdf.jpg');}"]
+        incorrect_values.each do |v|
+          v.should_not match(CssGenerator::PATTERN)
+        end
+      end
+    end
+    
+    describe "#encode_images_basic" do
+
+      before(:each) do
+        @content = "background-image: url('images/logo.jpg'); background: url('images/logo.jpg');"
+        @images = @generator.encode_images_basic!(@content) do |image_data, tag|
+          tag + image_data.extension
+        end
+      end
+
+      it "substitute each image tag (image should exist and has proper size) with result of a yield" do
+        @content.should == "background-image: jpg; background: jpg;"
+      end
+
+      it "returns hash of images found and with proper size" do
+        @images.size.should == 1
+        @images['images/logo.jpg'].should be_an_instance_of(ImageData)
+      end
+
+    end
+
+    context "no images in css file" do
+      before(:each) do
+        @path = 'path'
+        @content = 'margin: 10px;'
+        @file = WebResourceBundler::ResourceFile.new_css_file(@path, @content)
+      end
+      describe "#encode_images!" do
+        it "content isn't changed if no images found" do
+          images = @generator.encode_images!(@file.content)
+          @file.content.should == @content
+          images.should be_empty
+        end
+              end
+      describe "#encode_images_for_ie" do
+        it "returns original content if no images found" do
+          @generator.encode_images_for_ie!(@file.content, 'cache/1.mhtml')
+          @file.content.should == @content
+        end
+        
+      end
+    end
+    context "css files has images" do
+      before(:each) do
+        @path = 'style.css' 
+        @content = "background: #eeeeee url('images/logo.jpg') repeat-x 0 100%;" 
+        @file = WebResourceBundler::ResourceFile.new_css_file(@path, @content)
+      end
+      describe "#encode_images" do
+        it "return images hash" do
+          images = @generator.encode_images!(@content)
+          images.size.should == 1
+        end
+
+        it "modifies content with encoded images" do
+          @generator.encode_images!(@file.content)
+          @file.content.include?("background: #eeeeee url('data:image").should be_true
+          @file.content.include?("repeat-x 0 100%").should be_true
+        end
+      end
+      describe "#encode_images_for_ie" do
+        it "changes urls to mhtml link" do
+          @generator.encode_images_for_ie!(@content, 'cache/1.mhtml')
+          @content.include?("mhtml:#{@settings.protocol}://#{@settings.domain}/cache/1.mhtml!").should be_true
+          @content.include?("background: #eeeeee url(mhtml:").should be_true
+          @content.include?("repeat-x 0 100%").should be_true
+        end
+      end
+    end
+    
+    describe "#construct_mhtml_link" do
+      it "should create link without public folder" do
+        @generator.construct_mhtml_link("temp.css").should == "http://#{@settings.domain}/temp.css"
+      end
+    end
+  end
+end
+