httpimagestore 0.5.0 → 1.0.0

data/spec/configuration_output_spec.rb

@@ -0,0 +1,338 @@
+require_relative 'spec_helper'
+require_relative 'support/cuba_response_env'
+
+require 'httpimagestore/configuration'
+Configuration::Scope.logger = Logger.new('/dev/null')
+
+require 'httpimagestore/configuration/output'
+require 'httpimagestore/configuration/file'
+MemoryLimit.logger = Logger.new('/dev/null')
+
+describe Configuration do
+	let :state do
+		Configuration::RequestState.new('abc')
+	end
+
+	let :env do
+		CubaResponseEnv.new
+	end
+
+	describe Configuration::OutputImage do
+		subject do
+			Configuration.read(<<-EOF)
+			put "test" {
+				output_image "input"
+			}
+			EOF
+		end
+
+		before :each do
+			subject.handlers[0].image_sources[0].realize(state)
+		end
+
+		it 'should provide given image' do
+			subject.handlers[0].output.should be_a Configuration::OutputImage
+			subject.handlers[0].output.realize(state)
+
+			env.instance_eval &state.output_callback
+			env.res.status.should == 200
+			env.res.data.should == 'abc'
+		end
+
+		it 'should use default content type if not defined on image' do
+			subject.handlers[0].output.realize(state)
+
+			env.instance_eval &state.output_callback
+			env.res['Content-Type'].should == 'application/octet-stream'
+		end
+
+		it 'should use image mime type if available' do
+			state.images['input'].mime_type = 'image/jpeg'
+
+			subject.handlers[0].output.realize(state)
+
+			env.instance_eval &state.output_callback
+			env.res['Content-Type'].should == 'image/jpeg'
+		end
+
+		describe 'Cache-Control header support' do
+			subject do
+				Configuration.read(<<-EOF)
+				put "test" {
+					output_image "input" cache-control="public, max-age=999, s-maxage=666"
+				}
+				EOF
+			end
+
+			it 'should allow setting Cache-Control header' do
+				subject.handlers[0].output.realize(state)
+
+				env.instance_eval &state.output_callback
+				env.res['Cache-Control'].should == 'public, max-age=999, s-maxage=666'
+			end
+		end
+	end
+
+	describe 'output store paths and URLs' do
+		let :in_file do
+			Pathname.new("/tmp/test.in")
+		end
+
+		let :out_file do
+			Pathname.new("/tmp/test.out")
+		end
+
+		let :out2_file do
+			Pathname.new("/tmp/test.out2")
+		end
+
+		before :each do
+			in_file.open('w'){|io| io.write('abc')}
+			out_file.unlink if out_file.exist?
+			out2_file.unlink if out2_file.exist?
+		end
+
+		after :each do
+			out_file.unlink if out_file.exist?
+			out2_file.unlink if out2_file.exist?
+			in_file.unlink
+		end
+
+		describe Configuration::OutputStorePath do
+			it 'should provide file store path' do
+				subject = Configuration.read(<<-EOF)
+				path {
+					"out"	"test.out"
+				}
+
+				post "single" {
+					store_file "input" root="/tmp" path="out"
+
+					output_store_path "input"
+				}
+				EOF
+
+				subject.handlers[0].image_sources[0].realize(state)
+				subject.handlers[0].stores[0].realize(state)
+				subject.handlers[0].output.realize(state)
+
+				env.instance_eval &state.output_callback
+				env.res['Content-Type'].should == 'text/plain'
+				env.res.data.should == "test.out\r\n"
+			end
+
+			it 'should provide multiple file store paths' do
+				subject = Configuration.read(<<-EOF)
+				path {
+					"in"	"test.in"
+					"out"	"test.out"
+					"out2"	"test.out2"
+				}
+
+				post "multi" {
+					source_file "original" root="/tmp" path="in"
+
+					store_file "input" root="/tmp" path="out"
+					store_file "original" root="/tmp" path="out2"
+
+					output_store_path {
+						"input"
+						"original"
+					}
+				}
+				EOF
+
+				subject.handlers[0].image_sources[0].realize(state)
+				subject.handlers[0].image_sources[1].realize(state)
+				subject.handlers[0].stores[0].realize(state)
+				subject.handlers[0].stores[1].realize(state)
+				subject.handlers[0].output.realize(state)
+
+				env.instance_eval &state.output_callback
+				env.res['Content-Type'].should == 'text/plain'
+				env.res.data.should == "test.out\r\ntest.out2\r\n"
+			end
+
+			describe 'conditional inclusion support' do
+				let :state do
+					Configuration::RequestState.new('abc', list: 'input,image2')
+				end
+
+				subject do
+					Configuration.read(<<-'EOF')
+					path {
+						"in"		"test.in"
+						"out1"	"test.out1"
+						"out2"	"test.out2"
+						"out3"	"test.out3"
+					}
+
+					post "multi" {
+						source_file "image1" root="/tmp" path="in"
+						source_file "image2" root="/tmp" path="in"
+
+						store_file "input" root="/tmp" path="out1"
+						store_file "image1" root="/tmp" path="out2"
+						store_file "image2" root="/tmp" path="out3"
+
+						output_store_path {
+							"input"		if-image-name-on="#{list}"
+							"image1"	if-image-name-on="#{list}"
+							"image2"	if-image-name-on="#{list}"
+						}
+					}
+					EOF
+				end
+
+				it 'should output store path only for images that names match if-image-name-on list' do
+					subject.handlers[0].image_sources[0].realize(state)
+					subject.handlers[0].image_sources[1].realize(state)
+					subject.handlers[0].image_sources[2].realize(state)
+					subject.handlers[0].stores[0].realize(state)
+					subject.handlers[0].stores[1].realize(state)
+					subject.handlers[0].stores[2].realize(state)
+					subject.handlers[0].output.realize(state)
+
+					env.instance_eval &state.output_callback
+					env.res['Content-Type'].should == 'text/plain'
+					env.res.data.should == "test.out1\r\ntest.out3\r\n"
+				end
+			end
+
+			describe 'error handling' do
+				it 'should raise StorePathNotSetForImage for output of not stored image' do
+					subject = Configuration.read(<<-EOF)
+					post "single" {
+						output_store_path "input"
+					}
+					EOF
+
+					subject.handlers[0].image_sources[0].realize(state)
+
+					expect {
+						subject.handlers[0].output.realize(state)
+					}.to raise_error Configuration::StorePathNotSetForImage, %{store path not set for image 'input'}
+				end
+			end
+		end
+
+		describe Configuration::OutputStoreURL do
+			it 'should provide file store URL' do
+				subject = Configuration.read(<<-EOF)
+				path {
+					"out"	"test.out"
+				}
+
+				post "single" {
+					store_file "input" root="/tmp" path="out"
+
+					output_store_url "input"
+				}
+				EOF
+
+				subject.handlers[0].image_sources[0].realize(state)
+				subject.handlers[0].stores[0].realize(state)
+				subject.handlers[0].output.realize(state)
+
+				env.instance_eval &state.output_callback
+				env.res['Content-Type'].should == 'text/uri-list'
+				env.res.data.should == "file://test.out\r\n"
+			end
+
+			it 'should provide multiple file store URLs' do
+				subject = Configuration.read(<<-EOF)
+				path {
+					"in"	"test.in"
+					"out"	"test.out"
+					"out2"	"test.out2"
+				}
+
+				post "multi" {
+					source_file "original" root="/tmp" path="in"
+
+					store_file "input" root="/tmp" path="out"
+					store_file "original" root="/tmp" path="out2"
+
+					output_store_url {
+						"input"
+						"original"
+					}
+				}
+				EOF
+
+				subject.handlers[0].image_sources[0].realize(state)
+				subject.handlers[0].image_sources[1].realize(state)
+				subject.handlers[0].stores[0].realize(state)
+				subject.handlers[0].stores[1].realize(state)
+				subject.handlers[0].output.realize(state)
+
+				env.instance_eval &state.output_callback
+				env.res['Content-Type'].should == 'text/uri-list'
+				env.res.data.should == "file://test.out\r\nfile://test.out2\r\n"
+			end
+			
+			describe 'conditional inclusion support' do
+				let :state do
+					Configuration::RequestState.new('abc', list: 'input,image2')
+				end
+
+				subject do
+					Configuration.read(<<-'EOF')
+					path {
+						"in"		"test.in"
+						"out1"	"test.out1"
+						"out2"	"test.out2"
+						"out3"	"test.out3"
+					}
+
+					post "multi" {
+						source_file "image1" root="/tmp" path="in"
+						source_file "image2" root="/tmp" path="in"
+
+						store_file "input" root="/tmp" path="out1"
+						store_file "image1" root="/tmp" path="out2"
+						store_file "image2" root="/tmp" path="out3"
+
+						output_store_url {
+							"input"		if-image-name-on="#{list}"
+							"image1"	if-image-name-on="#{list}"
+							"image2"	if-image-name-on="#{list}"
+						}
+					}
+					EOF
+				end
+
+				it 'should output store url only for images that names match if-image-name-on list' do
+					subject.handlers[0].image_sources[0].realize(state)
+					subject.handlers[0].image_sources[1].realize(state)
+					subject.handlers[0].image_sources[2].realize(state)
+					subject.handlers[0].stores[0].realize(state)
+					subject.handlers[0].stores[1].realize(state)
+					subject.handlers[0].stores[2].realize(state)
+					subject.handlers[0].output.realize(state)
+
+					env.instance_eval &state.output_callback
+					env.res['Content-Type'].should == 'text/uri-list'
+					env.res.data.should == "file://test.out1\r\nfile://test.out3\r\n"
+				end
+			end
+
+			describe 'error handling' do
+				it 'should raise StoreURLNotSetForImage for output of not stored image' do
+					subject = Configuration.read(<<-EOF)
+					post "single" {
+						output_store_url "input"
+					}
+					EOF
+
+					subject.handlers[0].image_sources[0].realize(state)
+
+					expect {
+						subject.handlers[0].output.realize(state)
+					}.to raise_error Configuration::StoreURLNotSetForImage, %{store URL not set for image 'input'}
+				end
+			end
+		end
+	end
+end
+

data/spec/configuration_path_spec.rb

@@ -0,0 +1,92 @@
+require_relative 'spec_helper'
+require 'httpimagestore/configuration'
+Configuration::Scope.logger = Logger.new('/dev/null')
+
+require 'httpimagestore/configuration/path'
+
+describe Configuration do
+	describe 'path rendering' do
+		it 'should load paths form single line and multi line declarations and render spec templates' do
+			subject = Configuration.read(<<-'EOF')
+			path "uri"						"#{path}"
+			path "hash"						"#{digest}.#{extension}"
+			path {
+				"hash-name"					"#{digest}/#{imagename}.#{extension}"
+				"structured"				"#{dirname}/#{digest}/#{basename}.#{extension}"
+				"structured-name"		"#{dirname}/#{digest}/#{basename}-#{imagename}.#{extension}"
+			}
+			EOF
+
+			subject.paths['uri'].render(path: 'test/abc.jpg').should == 'test/abc.jpg'
+			subject.paths['hash'].render(path: 'test/abc.jpg', body: 'hello').should == '2cf24dba5fb0a30e.jpg'
+			subject.paths['hash-name'].render(path: 'test/abc.jpg', body: 'hello', imagename: 'xbrna').should == '2cf24dba5fb0a30e/xbrna.jpg'
+			subject.paths['structured'].render(path: 'test/abc.jpg', body: 'hello').should == 'test/2cf24dba5fb0a30e/abc.jpg'
+			subject.paths['structured-name'].render(path: 'test/abc.jpg', body: 'hello', imagename: 'xbrna').should == 'test/2cf24dba5fb0a30e/abc-xbrna.jpg'
+		end
+
+		describe 'error handling' do
+			it 'should raise NoValueError on missing path name' do
+				expect {
+					Configuration.read(<<-'EOF')
+						path
+					EOF
+				}.to raise_error Configuration::NoValueError, %{syntax error while parsing 'path': expected path name}
+			end
+
+			it 'should raise NoValueError on missing path template' do
+				expect {
+					Configuration.read(<<-'EOF')
+						path {
+							"blah"
+						}
+					EOF
+				}.to raise_error Configuration::NoValueError, %{syntax error while parsing '"blah"': expected path template}
+			end
+
+			it 'should raise PathNotDefinedError if path lookup fails' do
+				subject = Configuration.read('')
+
+				expect {
+					subject.paths['blah']
+				}.to raise_error Configuration::PathNotDefinedError, "path 'blah' not defined"
+			end
+
+			it 'should raise NoValueForPathTemplatePlaceholerError if locals value is not found' do
+				subject = Configuration.read(<<-'EOF')
+				path {
+					"test"								"#{abc}#{xyz}"
+				}
+				EOF
+
+				expect {
+					subject.paths['test'].render
+				}.to raise_error Configuration::NoValueForPathTemplatePlaceholerError, %q{cannot generate path 'test' from template '#{abc}#{xyz}': no value for '#{abc}'} 
+			end
+
+			it 'should raise NoValueForPathTemplatePlaceholerError if path value is not found' do
+				subject = Configuration.read(<<-'EOF')
+				path {
+					"test"								"#{dirname}#{basename}"
+				}
+				EOF
+
+				expect {
+					subject.paths['test'].render
+				}.to raise_error Configuration::NoMetaValueForPathTemplatePlaceholerError, %q{cannot generate path 'test' from template '#{dirname}#{basename}': need 'path' to generate value for '#{dirname}'}
+			end
+
+			it 'should raise NoValueForPathTemplatePlaceholerError if body value is not found' do
+				subject = Configuration.read(<<-'EOF')
+				path {
+					"test"								"#{digest}"
+				}
+				EOF
+
+				expect {
+					subject.paths['test'].render(path: '')
+				}.to raise_error Configuration::NoMetaValueForPathTemplatePlaceholerError, %q{cannot generate path 'test' from template '#{digest}': need 'body' to generate value for '#{digest}'}
+			end
+		end
+	end
+end
+