strelka 0.0.1.pre129 → 0.0.1.pre148

data/spec/strelka/app/sessions_spec.rb

@@ -0,0 +1,109 @@
+#!/usr/bin/env ruby
+
+BEGIN {
+	require 'pathname'
+	basedir = Pathname.new( __FILE__ ).dirname.parent.parent.parent
+	$LOAD_PATH.unshift( basedir ) unless $LOAD_PATH.include?( basedir )
+}
+
+require 'rspec'
+
+require 'spec/lib/helpers'
+
+require 'strelka'
+require 'strelka/app/plugins'
+require 'strelka/app/sessions'
+
+require 'strelka/behavior/plugin'
+
+
+#####################################################################
+###	C O N T E X T S
+#####################################################################
+
+describe Strelka::App::Sessions do
+
+	before( :all ) do
+		setup_logging( :fatal )
+		@request_factory = Mongrel2::RequestFactory.new( route: '' )
+	end
+
+	after( :all ) do
+		reset_logging()
+	end
+
+
+	it_should_behave_like( "A Strelka::App Plugin" )
+
+
+	it "has an associated session class" do
+		Strelka::App::Sessions.session_class.should be_a( Class )
+		Strelka::App::Sessions.session_class.should < Strelka::Session
+	end
+
+	it "is can be configured to use a different session class" do
+		# First, hook the anonymous class up to the 'testing' name using the PluginFactory API
+		test_session_class = Class.new( Strelka::Session )
+		Strelka::Session.derivatives[ 'testing' ] = test_session_class
+
+		# Now configuring it to use the 'testing' session type should set it to use the 
+		# anonymous class
+		Strelka::App::Sessions.configure( :session_class => 'testing' )
+
+		Strelka::App::Sessions.session_class.should == test_session_class
+	end
+
+
+	describe "an including App" do
+
+		before( :each ) do
+			@app = Class.new( Strelka::App ) do
+				Strelka.log.info "Anonymous App class: %p" % [ self ]
+				Strelka.log.info "ID constant is: %p" % [ const_defined?(:ID) ? const_get(:ID) : "(not defined)" ]
+
+				self::ID = 'monkeyshines'
+				# Guard against ID leakage
+				unless const_defined?( :ID, false )
+					raise ScriptError, "Awwww, damn. Apparently ID got defined in a parent class?"
+				end
+				plugin :sessions
+
+				def initialize( appid='params-test', sspec=TEST_SEND_SPEC, rspec=TEST_RECV_SPEC )
+					super
+				end
+
+				def handle( req )
+					req.session[ :test ] = 'session data'
+					return req.response
+				end
+			end
+		end
+
+		after( :each ) do
+			@app = nil
+
+			# Guard against ID leaking up into the base class
+			if Strelka::App.const_defined?( :ID )
+				raise ScriptError, "Awww crap. Somehow ID got defined in Strelka::App"
+			end
+		end
+
+
+		it "has a default session key that's the same as its appid" do
+			@app.session_namespace.should == @app.default_appid
+		end
+
+		it "can set its session namespace to something else" do
+			@app.class_eval do
+				session_namespace :findizzle
+			end
+
+			@app.session_namespace.should == :findizzle
+		end
+
+
+	end
+
+
+end
+

data/spec/strelka/app/templating_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'

data/spec/strelka/app_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'
@@ -29,7 +31,8 @@ describe Strelka::App do
 		Mongrel2::Config.init_database
 
 		# Skip loading the 'strelka' gem, which probably doesn't exist in the right version
-		strelkaspec = make_gemspec( 'strelka', '0.0.1', false )
+		# in the dev environment
+		strelkaspec = make_gemspec( 'strelka', Strelka::VERSION, false )
 		loaded_specs = Gem.instance_variable_get( :@loaded_specs )
 		loaded_specs['strelka'] = strelkaspec
 
@@ -97,6 +100,8 @@ describe Strelka::App do
 		rabbit_path  = specs[:rabbit_new].full_gem_path
 		giraffe_path = specs[:giraffe].full_gem_path
 
+		Dir.should_receive( :glob ).with( Pathname('data/strelka/{apps,handlers}/**/*') ).
+			and_return( [] )
 		Dir.should_receive( :glob ).with( "#{giraffe_path}/data/giraffe/{apps,handlers}/**/*" ).
 			and_return([ "#{giraffe_path}/data/giraffe/apps/app" ])
 		Dir.should_receive( :glob ).with( "#{rabbit_path}/data/rabbit/{apps,handlers}/**/*" ).
@@ -120,6 +125,8 @@ describe Strelka::App do
 		gemspec = make_gemspec( 'blood-orgy', '0.0.3' )
 		Gem::Specification.should_receive( :each ).and_yield( gemspec ).at_least( :once )
 
+		Dir.should_receive( :glob ).with( Pathname('data/strelka/{apps,handlers}/**/*') ).
+			and_return( [] )
 		Dir.should_receive( :glob ).with( "#{gemspec.full_gem_path}/data/blood-orgy/{apps,handlers}/**/*" ).
 			and_return([ "#{gemspec.full_gem_path}/data/blood-orgy/apps/kurzweil" ])
 
@@ -140,6 +147,8 @@ describe Strelka::App do
 		gemspec = make_gemspec( 'blood-orgy', '0.0.3' )
 		Gem::Specification.should_receive( :each ).and_yield( gemspec ).at_least( :once )
 
+		Dir.should_receive( :glob ).with( Pathname('data/strelka/{apps,handlers}/**/*') ).
+			and_return( [] )
 		Dir.should_receive( :glob ).with( "#{gemspec.full_gem_path}/data/blood-orgy/{apps,handlers}/**/*" ).
 			and_return([ "#{gemspec.full_gem_path}/data/blood-orgy/apps/kurzweil" ])
 

data/spec/strelka/cookie_spec.rb

@@ -0,0 +1,194 @@
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
+
+BEGIN {
+	require 'pathname'
+	basedir = Pathname.new( __FILE__ ).dirname.parent.parent
+
+	$LOAD_PATH.unshift( basedir ) unless $LOAD_PATH.include?( basedir )
+}
+
+require 'uri'
+require 'rspec'
+require 'spec/lib/helpers'
+require 'strelka/cookie'
+
+
+#####################################################################
+###	C O N T E X T S
+#####################################################################
+
+describe Strelka::Cookie do
+
+	before( :all ) do
+		setup_logging( :fatal )
+	end
+
+	after( :all ) do
+		reset_logging()
+	end
+
+	it "parses a 'nil' Cookie header field as an empty Hash" do
+		Strelka::Cookie.parse( nil ).should == {}
+	end
+
+	it "parses an empty string Cookie header field as an empty Hash" do
+		Strelka::Cookie.parse( '' ).should == {}
+	end
+
+	it "parses a cookie header field with a single value as a cookie with a single value" do
+		result = Strelka::Cookie.parse( 'a=b' )
+
+		result.should be_a( Hash )
+		result.should have(1).member
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b']
+	end
+
+	it "parses a cookie header field with a cookie with multiple values as a cookie with multiple values" do
+		result = Strelka::Cookie.parse( 'a=b&c' )
+
+		result.should be_a( Hash )
+		result.should have(1).member
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b', 'c']
+	end
+
+	it "parses a cookie header field with multiple cookies and multiple values correctly" do
+		result = Strelka::Cookie.parse( 'a=b&c; f=o&o' )
+
+		result.should be_a( Hash )
+		result.should have(2).members
+
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b', 'c']
+
+		result['f'].should be_a( Strelka::Cookie )
+		result['f'].name.should == 'f'
+		result['f'].value.should == 'o'
+		result['f'].values.should == ['o', 'o']
+	end
+
+	it "parses a cookie header field with an empty value as a cookie with a nil value" do
+		result = Strelka::Cookie.parse( 'a=' )
+
+		result.should have( 1 ).member
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should be_nil()
+		result['a'].values.should == []
+	end
+
+	it "parses a cookie header field with a version as a cookie with a version" do
+		result = Strelka::Cookie.parse( %{$Version=1; a="b"} )
+
+		result.should be_a( Hash )
+		result.should have( 1 ).member
+
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b']
+		result['a'].version.should == 1
+	end
+
+	it "parses a cookie header field with a path as a cookie with a path" do
+		result = Strelka::Cookie.parse( %{a=b; $Path=/Strelka} )
+
+		result.should be_a( Hash )
+		result.should have( 1 ).member
+
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b']
+
+		result['a'].path.should == "/Strelka"
+	end
+
+	it "parses a cookie header field with a domain as a cookie with a domain" do
+		result = Strelka::Cookie.parse( %{a=b; $domain=rubycrafters.com} )
+
+		result.should be_a( Hash )
+		result.should have( 1 ).member
+
+		result['a'].should be_a( Strelka::Cookie )
+		result['a'].name.should == 'a'
+		result['a'].value.should == 'b'
+		result['a'].values.should == ['b']
+
+		result['a'].domain.should == '.rubycrafters.com'
+	end
+
+	it "doesn't raise an error if asked to parse an invalid cookie header" do
+		result = Strelka::Cookie.parse( "{a}=foo" )
+		result.should == {}
+	end
+
+
+	context "instance" do
+
+		before( :each ) do
+			@cookie = Strelka::Cookie.new( :by_rickirac, '9917eb' )
+		end
+
+		it "stringifies as a valid header value" do
+			@cookie.to_s.should == 'by_rickirac=9917eb'
+		end
+
+		it "still stringifies correctly with two values" do
+			@cookie.values += ['brer lapin']
+			@cookie.to_s.should == "by_rickirac=9917eb&brer+lapin"
+		end
+
+		it "stringifies with a version number if its version is set to something other than 0" do
+			@cookie.version = 1
+			@cookie.to_s.should == %{by_rickirac=9917eb; Version=1}
+		end
+
+		it "stringifies with a domain if one is set" do
+			@cookie.domain = '.example.com'
+			@cookie.to_s.should == %{by_rickirac=9917eb; Domain=.example.com}
+		end
+
+		it "stringifies with a dot prepended to the domain if the set doesn't have one" do
+			@cookie.domain = 'example.com'
+			@cookie.to_s.should == %{by_rickirac=9917eb; Domain=.example.com}
+		end
+
+		it "stringifies correctly even if one of its values contains a semicolon" do
+			@cookie.values += [%{"modern technology"; ain't it a paradox?}]
+			@cookie.to_s.should ==
+				"by_rickirac=9917eb&%22modern+technology%22%3B+ain%27t+it+a+paradox%3F"
+		end
+
+		it "stringifies with an expires date if one is set" do
+			@cookie.expires = Time.at( 1331761184 )
+			@cookie.to_s.should == 'by_rickirac=9917eb; Expires=Wed, 14-Mar-2012 21:39:44 GMT'
+		end
+
+		it "hashes the same as another cookie with the same name, regardless of value" do
+			@cookie.hash.should == Strelka::Cookie.new('by_rickirac', 'something_else').hash
+		end
+
+
+		it "sets its expiration time to a time in the past if it's told to expire" do
+			@cookie.expire!
+			@cookie.expires.should < Time.now
+		end
+
+		it "uses the hash of its name as its hash value" do
+			@cookie.hash.should == @cookie.name.to_s.hash
+		end
+	end
+
+
+end
+

data/spec/strelka/cookieset_spec.rb

@@ -0,0 +1,159 @@
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
+
+BEGIN {
+	require 'pathname'
+	basedir = Pathname.new( __FILE__ ).dirname.parent.parent
+
+	$LOAD_PATH.unshift( basedir ) unless $LOAD_PATH.include?( basedir )
+}
+
+require 'uri'
+require 'rspec'
+require 'spec/lib/helpers'
+require 'strelka/cookie'
+require 'strelka/cookieset'
+
+
+#####################################################################
+###	C O N T E X T S
+#####################################################################
+
+describe Strelka::CookieSet do
+
+	before( :each ) do
+		@cookieset = Strelka::CookieSet.new
+	end
+
+
+	it "is able to enummerate over each cookie in the set" do
+		pants_cookie = Strelka::Cookie.new( 'pants', 'baggy' )
+		shirt_cookie = Strelka::Cookie.new( 'shirt', 'pirate' )
+		@cookieset << shirt_cookie << pants_cookie
+
+		cookies = []
+		@cookieset.each do |cookie|
+			cookies << cookie
+		end
+
+		cookies.length.should == 2
+		cookies.should include( pants_cookie )
+		cookies.should include( shirt_cookie )
+	end
+
+	it "is able to add a cookie referenced symbolically" do
+		pants_cookie = Strelka::Cookie.new( 'pants', 'denim' )
+		@cookieset[:pants] = pants_cookie
+		@cookieset['pants'].should == pants_cookie
+	end
+
+
+	it "autos-create a cookie for a non-cookie passed to the index setter" do
+		lambda { @cookieset['bar'] = 'badgerbadgerbadgerbadger' }.should_not raise_error()
+
+		@cookieset['bar'].should be_an_instance_of( Strelka::Cookie )
+		@cookieset['bar'].value.should == 'badgerbadgerbadgerbadger'
+	end
+
+	it "raises an exception if the name of a cookie being set doesn't agree with the key it being set with" do
+		pants_cookie = Strelka::Cookie.new( 'pants', 'corduroy' )
+		lambda { @cookieset['shirt'] = pants_cookie }.should raise_error( ArgumentError )
+	end
+
+	it "implements Enumerable" do
+		Enumerable.instance_methods( false ).each do |meth|
+			@cookieset.should respond_to( meth )
+		end
+	end
+
+	it "is able to set a cookie's value symbolically to something other than a String" do
+		@cookieset[:wof] = Digest::MD5.hexdigest( Time.now.to_s )
+	end
+
+	it "is able to set a cookie with a Symbol key" do
+		@cookieset[:wof] = Strelka::Cookie.new( :wof, "something" )
+	end
+
+
+	describe "created with an Array of cookies" do
+		it "should flatten the array" do
+			cookie_array = []
+			cookie_array << Strelka::Cookie.new( 'foo', 'bar' )
+			cookie_array << [Strelka::Cookie.new( 'shmoop', 'torgo!' )]
+
+			cookieset = nil
+			lambda {cookieset = Strelka::CookieSet.new(cookie_array)}.should_not raise_error()
+			cookieset.length.should == 2
+		end
+	end
+
+
+	describe "with a 'foo' cookie" do
+		before(:each) do
+			@cookie = Strelka::Cookie.new( 'foo', 'bar' )
+			@cookieset = Strelka::CookieSet.new( @cookie )
+		end
+
+		it "contains only one cookie" do
+			@cookieset.length.should == 1
+		end
+
+		it "is able to return the 'foo' Strelka::Cookie via its index operator" do
+			@cookieset[ 'foo' ].should == @cookie
+		end
+
+
+		it "is able to return the 'foo' Strelka::Cookie via its symbolic name" do
+			@cookieset[ :foo ].should == @cookie
+		end
+
+		it "knows if it includes a cookie named 'foo'" do
+			@cookieset.should include( 'foo' )
+		end
+
+		it "knows if it includes a cookie referenced by :foo" do
+			@cookieset.should include( :foo )
+		end
+
+		it "knows that it doesn't contain a cookie named 'lollypop'" do
+			@cookieset.should_not include( 'lollypop' )
+		end
+
+		it "knows that it includes the 'foo' cookie object" do
+			@cookieset.should include( @cookie )
+		end
+
+
+		it "adds a cookie to the set if it has a different name" do
+			new_cookie = Strelka::Cookie.new( 'bar', 'foo' )
+			@cookieset << new_cookie
+
+			@cookieset.length.should == 2
+			@cookieset.should include( new_cookie )
+		end
+
+
+		it "replaces any existing same-named cookie added via appending" do
+			new_cookie = Strelka::Cookie.new( 'foo', 'giant scallops of doom' )
+			@cookieset << new_cookie
+
+			@cookieset.length.should == 1
+			@cookieset.should include( new_cookie )
+			@cookieset['foo'].should equal( new_cookie )
+		end
+
+		it "replaces any existing same-named cookie set via the index operator" do
+			new_cookie = Strelka::Cookie.new( 'foo', 'giant scallops of doom' )
+			@cookieset[:foo] = new_cookie
+
+			@cookieset.length.should == 1
+			@cookieset.should include( new_cookie )
+			@cookieset['foo'].should equal( new_cookie )
+		end
+
+	end
+
+end
+
+# vim: set nosta noet ts=4 sw=4:

data/spec/strelka/exceptions_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'

data/spec/strelka/httprequest/acceptparams_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'

data/spec/strelka/httprequest/negotiation_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'

data/spec/strelka/httprequest/session_spec.rb

@@ -0,0 +1,43 @@
+# -*- rspec -*-
+# vim: set nosta noet ts=4 sw=4:
+
+BEGIN {
+	require 'pathname'
+	basedir = Pathname.new( __FILE__ ).dirname.parent.parent.parent
+
+	libdir = basedir + "lib"
+
+	$LOAD_PATH.unshift( basedir ) unless $LOAD_PATH.include?( basedir )
+	$LOAD_PATH.unshift( libdir ) unless $LOAD_PATH.include?( libdir )
+}
+
+require 'rspec'
+
+require 'spec/lib/helpers'
+
+require 'strelka'
+require 'strelka/httprequest/session'
+
+
+#####################################################################
+###	C O N T E X T S
+#####################################################################
+
+describe Strelka::HTTPRequest::Session do
+
+	before( :all ) do
+		setup_logging( :fatal )
+		@request_factory = Mongrel2::RequestFactory.new( route: '/service/user' )
+	end
+
+	after( :all ) do
+		reset_logging()
+	end
+
+	before( :each ) do
+		@req = @request_factory.get( '/service/user/estark' )
+		@req.extend( described_class )
+	end
+
+
+end

data/spec/strelka/httprequest_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'
@@ -11,6 +13,7 @@ require 'uri'
 require 'rspec'
 require 'spec/lib/helpers'
 require 'strelka/httprequest'
+require 'strelka/cookie'
 
 
 #####################################################################
@@ -240,5 +243,29 @@ describe Strelka::HTTPRequest do
 
 	end
 
+
+	describe "cookie support" do
+
+		before( :each ) do
+			@req = @request_factory.get( '/directory/userinfo/ged' )
+		end
+
+
+		it "parses a single cookie into a cookieset with the cookie in it" do
+			@req.header.cookie = 'foom=chuckUfarly'
+			@req.cookies.should have( 1 ).member
+			@req.cookies['foom'].value.should == 'chuckUfarly'
+		end
+
+		it "parses multiple cookies into a cookieset with multiple cookies in it" do
+			@req.header.cookie = 'foom=chuckUfarly; glarn=hotchinfalcheck'
+
+			@req.cookies.should have( 2 ).members
+			@req.cookies['foom'].value.should == 'chuckUfarly'
+			@req.cookies['glarn'].value.should == 'hotchinfalcheck'
+		end
+
+	end
+
 end
 

data/spec/strelka/httpresponse/negotiation_spec.rb

@@ -433,7 +433,7 @@ describe Strelka::HTTPResponse::Negotiation do
 			@res.should_not have_acceptable_encoding()
 		end
 
-		it "knows that it is not acceptable if it doesn't have any explicit content codings " + 
+		it "knows that it is not acceptable if it doesn't have any explicit content codings " +
 		   "and 'identity' is explicitly not accepted in its originating request" do
 			@req.headers.accept_encoding = 'gzip;q=1.0, identity; q=0'
 			@res.encodings.clear
@@ -441,7 +441,7 @@ describe Strelka::HTTPResponse::Negotiation do
 			@res.should_not have_acceptable_encoding()
 		end
 
-		it "knows that it is not acceptable if it doesn't have any explicit content codings " + 
+		it "knows that it is not acceptable if it doesn't have any explicit content codings " +
 		   "and 'identity' is explicitly not accepted in its originating request" do
 			@req.headers.accept_encoding = 'gzip;q=1.0, identity; q=0'
 			@res.encodings.clear
@@ -449,7 +449,7 @@ describe Strelka::HTTPResponse::Negotiation do
 			@res.should_not have_acceptable_encoding()
 		end
 
-		it "knows that it is not acceptable if it doesn't have any explicit content codings, " + 
+		it "knows that it is not acceptable if it doesn't have any explicit content codings, " +
 		   "the wildcard content-coding is disallowed, and 'identity' is not explicitly accepted" do
 			@req.headers.accept_encoding = 'gzip;q=1.0, *;q=0'
 			@res.encodings.clear

data/spec/strelka/httpresponse_spec.rb

@@ -110,5 +110,18 @@ describe Strelka::HTTPResponse do
 		@res.header_data.should =~ /content-language: en, sv-chef\s*$/i
 	end
 
+
+	it "allows cookies to be set via a Hash-like interface" do
+		@res.cookies[:foom] = 'chuckUfarly'
+		@res.header_data.should =~ /set-cookie: foom=chuckufarly/i
+	end
+
+	it "allows cookies to appended" do
+		@res.cookies << Strelka::Cookie.new( 'session', '64a3a92eb7403a8199301e03e8b83810' )
+		@res.cookies << Strelka::Cookie.new( 'cn', '18', :expires => '+1d' )
+		@res.header_data.should =~ /set-cookie: session=64a3a92eb7403a8199301e03e8b83810/i
+		@res.header_data.should =~ /set-cookie: cn=18; expires=/i
+	end
+
 end
 

data/spec/strelka/logging_spec.rb

@@ -1,4 +1,6 @@
-#!/usr/bin/env ruby
+# -*- ruby -*-
+# vim: set nosta noet ts=4 sw=4:
+# encoding: utf-8
 
 BEGIN {
 	require 'pathname'