strelka 0.4.0 → 0.5.0.pre.393

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 58ca9bc4fbcc4953f0b9fb9b15407b726b0a9512
-  data.tar.gz: 946729ae577f2cd5496b3a14542094abb35ec907
+  metadata.gz: 1d2cffaf505aa41496b0ce345534e70d6f4b45a0
+  data.tar.gz: c922b2aa4deafe3472b9be7d2d960237dc2e2b11
 SHA512:
-  metadata.gz: 429cab795c50159c7fdad07e17ecf0da7dbf6d43ef6e5798752d942a371dd592292815393c89563a0f29da887943b576fc7580a3d7ac1b06c091bae56d66bd99
-  data.tar.gz: 27c80311955501589a68d377e90c4c532f3acd3a074c648a4d5ea9eefc9246a6eb8669793c07583146bb9ef3cdd0e689440b8912dd37f82ac703f4a34d5a7b64
+  metadata.gz: 295a62542aae175dd4c70d425fc70f43d314412db557a01989aa63b015931cdf20cad5793fefd4e15a237f49d0f50a17c95b72b5905f321a260520ba4c13e06f
+  data.tar.gz: 590d6dfab3b2a652e916b9e78ad497440b5e5de314abbd12b5c289fd917d43a1c9e4b212ec55fdc2d030693ef312ea44d8ce0b530c140ee5c714241388a0ae0d

checksums.yaml.gz.sig

@@ -1 +1 @@
-���~&U�ϱ�ʺ��� ��ؿ�G��!$)e���'F��n���z_a���a�4��'��v@ھ��$��ev�\�a:��#'�����j+��*�A��j�M�!�z1ע8�xs�
�ѷILR���\~_'�/�m��1hr<i����S���pfj	�	��'Z���yu�����:ϟe���h�q	��@t
�ԭc���y���hQ��gq���i�v���t)��TK��%~E ��勐>l�-e�
+Q�]O$S�S�������U��pMAx�q�T6��b?g7, ���C�����:�;|vN��ϤY�-�DΚ˴����[h���ŝ�t��؎���D���æ�}�٧л��^[h�o���ަ�5鑉gU��ַӢ�O�AOe���2�N���8�����L�dB����2��Ry~�����1�с�L�'%A��B�:�@00!Cu���4��YQ�b[J-��3g�r�+�<DUb��

data/Rakefile

@@ -27,7 +27,7 @@ hoespec = Hoe.spec 'strelka' do
 	self.dependency 'inversion',       '~> 0.12'
 	self.dependency 'loggability',     '~> 0.5'
 	self.dependency 'mongrel2',        '~> 0.36'
-	self.dependency 'pluggability',    '~> 0.0'
+	self.dependency 'pluggability',    '~> 0.2'
 	self.dependency 'sysexits',        '~> 1.1'
 	self.dependency 'trollop',         '~> 2.0'
 	self.dependency 'uuidtools',       '~> 2.1'

data/lib/strelka/app/negotiation.rb

@@ -21,21 +21,7 @@ require 'strelka/httpresponse/negotiation'
 #
 #       plugins :routing, :negotiation
 #
-#       add_content_type :tnetstring, 'text/x-tnetstring' do |response|
-#           tnetstr = nil
-#           begin
-#               response.body.rewind
-#               tnetstr = TNetString.dump( response.body )
-#           rescue => err
-#               self.log.error "%p while transforming entity body to a TNetString: %s" %
-#                   [ err.class, err.message ]
-#               return false
-#           else
-#               response.body = tnetstr
-#               response.content_type = 'text/x-tnetstring'
-#               return true
-#           end
-#       end
+#       add_content_type :tnetstring, 'text/x-tnetstring', TNetstring.method( :dump )
 #
 #   end # class UserService
 #
@@ -50,31 +36,16 @@ module Strelka::App::Negotiation
 	# Class methods to add to classes with content-negotiation.
 	module ClassMethods # :nodoc:
 
-		# Content-type tranform registry, keyed by name
-		@content_type_transforms = {}
-		attr_reader :content_type_transforms
-
-		# Content-type transform names, keyed by mimetype
-		@transform_names = {}
-		attr_reader :transform_names
-
-
-		### Extension callback -- add instance variables to extending objects.
-		def inherited( subclass )
-			super
-			subclass.instance_variable_set( :@content_type_transforms, @content_type_transforms.dup )
-			subclass.instance_variable_set( :@transform_names, @transform_names.dup )
-		end
-
-
 		### Define a new media-type associated with the specified +name+ and +mimetype+. Responses
-		### whose requests accept content of the given +mimetype+ will pass their response to the
-		### specified +transform_block+, which should re-write the response's entity body if it can
-		### transform it to its mimetype. If it successfully does so, it should return +true+, else
-		### the next-best mimetype's transform will be called, etc.
-		def add_content_type( name, mimetype, &transform_block )
-			self.transform_names[ mimetype ] = name
-			self.content_type_transforms[ name ] = transform_block
+		### whose requests accept content of the given +mimetype+ will #call the
+		### specified +callback+ with the current response body, and should return the
+		### transformed body. If the conversion failed, the callback can either raise an
+		### exception or return +nil+, either of which will continue on to any remaining
+		### transforms. If no +callback+ is given, the method's block will be used.
+		def add_content_type( name, mimetype, callback=nil ) # :yield: response.body
+			callback ||= Proc.new # Use the block
+			Strelka::HTTPResponse::Negotiation.mimetype_map[ name.to_sym ] = mimetype
+			Strelka::HTTPResponse::Negotiation.stringifiers[ mimetype ] = callback
 		end
 
 	end # module ClassMethods

data/lib/strelka/httpresponse/negotiation.rb

@@ -8,6 +8,7 @@ require 'yajl'
 
 require 'strelka/constants'
 require 'strelka/exceptions'
+require 'strelka/mixins'
 require 'strelka/httpresponse' unless defined?( Strelka::HTTPResponse )
 
 
@@ -23,11 +24,12 @@ require 'strelka/httpresponse' unless defined?( Strelka::HTTPResponse )
 # is acceptable according to its request's `Accept*` headers.
 #
 module Strelka::HTTPResponse::Negotiation
+	extend Strelka::MethodUtilities
 	include Strelka::Constants
 
 	# TODO: Perhaps replace this with something like this:
 	#   Mongrel2::Config::Mimetype.to_hash( :extension => :mimetype )
-	BUILTIN_MIMETYPES = {
+	BUILTIN_MIMETYPE_MAP = {
 		:html => 'text/html',
 		:text => 'text/plain',
 
@@ -43,10 +45,10 @@ module Strelka::HTTPResponse::Negotiation
 		:atom => 'application/atom+xml',
 	}
 
-	# A collection of stringifier callbacks, keyed by mimetype. If an object other
-	# than a String is returned by a content callback, and an entry for the callback's
-	# mimetype exists in this Hash, it will be #call()ed to stringify the object.
-	STRINGIFIERS = {
+	# A collection of default stringifier callbacks, keyed by mimetype. If an entry
+	# for the content-negotiation callback's mimetype exists in this Hash, it will
+	# be #call()ed on the callback's return value to stringify the body.
+	BUILTIN_STRINGIFIERS = {
 		'application/x-yaml' => YAML.method( :dump ),
 		'application/json'   => Yajl.method( :dump ),
 		'text/plain'         => Proc.new {|obj| obj.to_s },
@@ -62,6 +64,18 @@ module Strelka::HTTPResponse::Negotiation
 	]
 
 
+	##
+	# The Hash of symbolic mediatypes of the form:
+	#   { <name (Symbol)> => <mimetype> }
+	singleton_attr_reader :mimetype_map
+	@mimetype_map = BUILTIN_MIMETYPE_MAP.dup
+
+	##
+	# The Hash of stringification callbacks, keyed by mimetype.
+	singleton_attr_reader :stringifiers
+	@stringifiers = BUILTIN_STRINGIFIERS.dup
+
+
 	### Add some instance variables for negotiation.
 	def initialize( * )
 		@mediatype_callbacks = {}
@@ -267,7 +281,11 @@ module Strelka::HTTPResponse::Negotiation
 	### to HTTP::OK.
 	def for( *mediatypes, &callback )
 		mediatypes.each do |mimetype|
-			mimetype = BUILTIN_MIMETYPES[ mimetype ] if mimetype.is_a?( Symbol )
+			if mimetype.is_a?( Symbol )
+				mimetype = Strelka::HTTPResponse::Negotiation.mimetype_map[ mimetype ] or
+					raise "No known mimetype mapped to %p" % [ mimetype ]
+			end
+
 			self.mediatype_callbacks[ mimetype ] = callback
 		end
 
@@ -307,17 +325,19 @@ module Strelka::HTTPResponse::Negotiation
 	### until one returns a true-ish value, which becomes the new entity
 	### body. If the body object is not a String,
 	def transform_content_type
+		self.log.debug "Applying content-type transforms (if any)"
 		return if self.mediatype_callbacks.empty?
 
-		self.log.debug "Applying content-type transforms (if any)"
+		self.log.debug "  transform callbacks registered: %p" % [ self.mediatype_callbacks ]
 		self.better_mediatypes.each do |mediatype|
 			callbacks = self.mediatype_callbacks.find_all do |mimetype, _|
 				mediatype =~ mimetype
 			end
 
 			if callbacks.empty?
-				self.log.debug "  no transforms for %s" % [ mediatype ]
+				self.log.debug "    no transforms for %s" % [ mediatype ]
 			else
+				self.log.debug "    %d transform/s for %s" % [ callbacks.length, mediatype ]
 				callbacks.each do |mimetype, callback|
 					return if self.try_content_type_callback( mimetype, callback )
 				end
@@ -334,9 +354,13 @@ module Strelka::HTTPResponse::Negotiation
 
 		new_body = callback.call( mimetype ) or return false
 
-		self.log.debug "  successfully transformed: %p! Setting up response." % [ new_body ]
-		new_body = STRINGIFIERS[ mimetype ].call( new_body ) if
-			STRINGIFIERS.key?( mimetype ) && !new_body.is_a?( String )
+		self.log.debug "  successfully transformed: %p! Setting up response." % [ new_body.class ]
+		stringifiers = Strelka::HTTPResponse::Negotiation.stringifiers
+		if stringifiers.key?( mimetype )
+			new_body = stringifiers[ mimetype ].call( new_body )
+		else
+			self.log.debug "    no stringifier registered for %p" % [ mimetype ]
+		end
 
 		self.body = new_body
 		self.content_type = mimetype.dup # :TODO: Why is this frozen?
@@ -512,7 +536,7 @@ module Strelka::HTTPResponse::Negotiation
 
 				# Don't close the FD when this IO goes out of scope
 				oldbody = self.body
-				oldbody.auto_close = false
+				oldbody.autoclose = false
 
 				# Re-open the same file descriptor, but transcoding to the wanted encoding
 				self.body = IO.for_fd( oldbody.fileno, internal_encoding: enc )

data/lib/strelka.rb

@@ -24,7 +24,7 @@ module Strelka
 	log_as :strelka
 
 	# Library version constant
-	VERSION = '0.4.0'
+	VERSION = '0.5.0'
 
 	# Version-control revision constant
 	REVISION = %q$Revision: 49ae27e4588e $

data/spec/strelka/app/negotiation_spec.rb

@@ -46,6 +46,19 @@ describe Strelka::App::Negotiation do
 				def initialize( appid='conneg-test', sspec=TEST_SEND_SPEC, rspec=TEST_RECV_SPEC )
 					super
 				end
+
+				add_content_type :tnetstring, 'text/x-tnetstring', TNetstring.method(:dump)
+
+				def handle_request( req )
+					super do
+						res = req.response
+						res.for( :tnetstring ) {[ 'an', {'array' => 'of stuff'} ]}
+						res.for( :html ) do
+							"<html><head><title>Yep</title></head><body>Yeah!</body></html>"
+						end
+						res
+					end
+				end
 			end
 		end
 
@@ -54,18 +67,6 @@ describe Strelka::App::Negotiation do
 		end
 
 
-		it "has its config inherited by subclasses" do
-			@app.add_content_type :text, 'text/plain' do
-				"It's all text now, baby"
-			end
-			subclass = Class.new( @app )
-
-			subclass.transform_names.should == @app.transform_names
-			subclass.transform_names.should_not equal( @app.transform_names )
-			subclass.content_type_transforms.should == @app.content_type_transforms
-			subclass.content_type_transforms.should_not equal( @app.content_type_transforms )
-		end
-
 		it "gets requests that have been extended with content-negotiation" do
 			req = @request_factory.get( '/service/user/estark' )
 			@app.new.handle( req )
@@ -80,6 +81,13 @@ describe Strelka::App::Negotiation do
 				should include( Strelka::HTTPResponse::Negotiation )
 		end
 
+		it "adds custom content-type transforms to outgoing responses" do
+			req = @request_factory.get( '/service/user/astark', :accept => 'text/x-tnetstring' )
+			res = @app.new.handle( req )
+			res.content_type.should == 'text/x-tnetstring'
+			res.body.read.should == '28:2:an,19:5:array,8:of stuff,}]'
+		end
+
 	end
 
 

data/spec/strelka/httpresponse/negotiation_spec.rb

@@ -63,11 +63,11 @@ describe Strelka::HTTPResponse::Negotiation do
 		it "can provide blocks for bodies of several different mediatypes" do
 			@req.headers.accept = 'application/x-yaml, application/json; q=0.7, text/xml; q=0.2'
 
-			@res.for( 'application/json' ) { %{["a JSON dump"]} }
-			@res.for( 'application/x-yaml' ) { "---\na: YAML dump\n\n" }
+			@res.for( 'application/json' ) { ["a JSON dump"] }
+			@res.for( 'application/x-yaml' ) { { 'a' => "YAML dump" } }
 
 			@res.negotiated_body.rewind
-			@res.negotiated_body.read.should == "---\na: YAML dump\n\n"
+			@res.negotiated_body.read.should == "---\na: YAML dump\n"
 			@res.content_type.should == "application/x-yaml"
 			@res.header_data.should =~ /accept(?!-)/i
 		end
@@ -100,6 +100,12 @@ describe Strelka::HTTPResponse::Negotiation do
 			@res.header_data.should =~ /accept(?!-)/i
 		end
 
+		it "raises an exception if given a block for an unknown symbolic mediatype" do
+			expect {
+				@res.for( :yaquil ) {}
+			}.to raise_error( StandardError, /no known mimetype/i )
+		end
+
 	end
 
 
@@ -132,7 +138,7 @@ describe Strelka::HTTPResponse::Negotiation do
 				@res.body = File.open( __FILE__, 'r:iso-8859-5' )
 				@res.content_type = 'text/plain'
 
-				@res.negotiated_body.encoding.should == Encoding::KOI8_R
+				@res.negotiated_body.read.encoding.should == Encoding::KOI8_R
 				@res.header_data.should =~ /accept-charset(?!-)/i
 			end
 		end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: strelka
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0.pre.393
 platform: ruby
 authors:
 - Michael Granger
@@ -30,7 +30,7 @@ cert_chain:
   6mKCwjpegytE0oifXfF8k75A9105cBnNiMZOe1tXiqYc/exCgWvbggurzDOcRkZu
   /YSusaiDXHKU2O3Akc3htA==
   -----END CERTIFICATE-----
-date: 2013-03-26 00:00:00.000000000 Z
+date: 2013-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: configurability
@@ -122,14 +122,14 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.0'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: sysexits
   requirement: !ruby/object:Gem::Requirement