bane 0.1.1 → 0.2.0

data/lib/bane/behaviors.rb

@@ -58,6 +58,17 @@ module Bane
       include ForEachLine
     end
 
+    # Sends a newline character as the only response 
+    class NewlineResponse < BasicBehavior
+      def serve(io, options)
+        io.write "\n"
+      end
+    end
+
+    class NewlineResponseForEachLine < NewlineResponse
+      include ForEachLine
+    end
+
     # Sends a random response.
     class RandomResponse < BasicBehavior
       def serve(io, options)
@@ -75,7 +86,7 @@ module Bane
       include ForEachLine
     end
 
-    # Sends a fixed response chacter-by-character, pausing in between each character.
+    # Sends a fixed response character-by-character, pausing between each character.
     #
     # Options:
     #  - message: The response to send. Default: "Hello, world!"

data/test/bane/behaviors_test.rb

@@ -6,21 +6,6 @@ class BehaviorsTest < Test::Unit::TestCase
 
   include Bane::Behaviors
 
-  class FakeConnection < StringIO
-    def will_send(query)
-      @query = query
-    end
-
-    def gets
-      @query
-      @read_query = true
-    end
-
-    def read_query?
-      @read_query
-    end
-  end
-
   def setup
     @fake_connection = FakeConnection.new
   end
@@ -31,8 +16,14 @@ class BehaviorsTest < Test::Unit::TestCase
     assert_equal "Test Message", response
   end
 
+  def test_newline_response_sends_only_a_newline_character
+    query_server(create(NewlineResponse))
+
+    assert_equal "\n", response
+  end
+
   def test_deluge_response_sends_one_million_bytes_by_default
-    query_server(create DelugeResponse)
+    query_server(create(DelugeResponse))
 
     assert_response_length 1_000_000
   end
@@ -44,13 +35,13 @@ class BehaviorsTest < Test::Unit::TestCase
   end
 
   def test_close_immediately_sends_no_response
-    query_server(create CloseImmediately)
+    query_server(create(CloseImmediately))
 
     assert_empty_response()
   end
 
   def test_never_respond_never_sends_a_response
-    server = create NeverRespond
+    server = create(NeverRespond)
 
     assert_raise Timeout::Error do
       Timeout::timeout(3) { query_server(server) }
@@ -59,7 +50,7 @@ class BehaviorsTest < Test::Unit::TestCase
   end
 
   def test_close_after_pause_sleeps_30_seconds_by_default_and_sends_nothing
-    server = create CloseAfterPause
+    server = create(CloseAfterPause)
     server.expects(:sleep).with(30)
     
     query_server(server)
@@ -68,13 +59,13 @@ class BehaviorsTest < Test::Unit::TestCase
   end
 
   def test_close_after_pause_accepts_duration_parameter
-    server = create CloseAfterPause
+    server = create(CloseAfterPause)
 
     within(2) { query_server(server, :duration => 1) }
   end
 
   def test_slow_response_sends_a_message_slowly
-    server = create SlowResponse
+    server = create(SlowResponse)
     message = "Hi!"
     delay = 0.5
     max_delay = (message.length + 1) * delay
@@ -85,7 +76,7 @@ class BehaviorsTest < Test::Unit::TestCase
   end
 
   def test_random_response_sends_a_nonempty_response
-    query_server(create RandomResponse)
+    query_server(create(RandomResponse))
 
     assert (!response.empty?), "Should have served a nonempty response"
   end
@@ -96,7 +87,7 @@ class BehaviorsTest < Test::Unit::TestCase
     server = create(HttpRefuseAllCredentials)
     query_server(server)
 
-    assert @fake_connection.read_query?, "Should have read the HTTP query before sending response"
+    assert @fake_connection.read_all_queries?, "Should have read the HTTP query before sending response"
     assert_match /HTTP\/1.1 401 Unauthorized/, response, 'Should have responded with the 401 response code'
   end
 
@@ -104,6 +95,17 @@ class BehaviorsTest < Test::Unit::TestCase
     assert_equal "SlowResponse", Bane::Behaviors::SlowResponse.simple_name
   end
 
+  def test_for_each_line_reads_a_line_before_responding
+    server = create(Bane::Behaviors::FixedResponseForEachLine)
+
+    @fake_connection.will_send "irrelevant\n"
+
+    query_server(server, {:message => "Dynamic"})
+    assert_equal "Dynamic", response
+
+    assert @fake_connection.read_all_queries?
+  end
+
   private
 
   def create(server_class)
@@ -134,4 +136,4 @@ class BehaviorsTest < Test::Unit::TestCase
     end
   end
 
-end
+end

data/test/bane/fake_connection_test.rb

@@ -0,0 +1,34 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class FakeConnectionTest < Test::Unit::TestCase
+
+  def setup
+    @fake_connection = FakeConnection.new
+  end
+
+  def test_fake_connection_returns_nil_if_no_commands_to_read
+    assert_nil @fake_connection.gets
+  end
+
+  def test_fake_connection_reports_when_one_command_set_and_read
+    @fake_connection.will_send("Command #1")
+
+    @fake_connection.gets
+
+    assert @fake_connection.read_all_queries?, "Should have read all queries"
+  end
+
+  def test_fake_connection_reports_when_all_commands_read
+    @fake_connection.will_send("Command #1")
+    @fake_connection.will_send("Command #2")
+
+    @fake_connection.gets
+
+    assert !@fake_connection.read_all_queries?, "Did not read all queries yet"
+
+    @fake_connection.gets
+
+    assert @fake_connection.read_all_queries?, "Should have read all queries"
+  end
+
+end

data/test/test_helper.rb

@@ -7,3 +7,24 @@ require 'bane'
 
 
 IRRELEVANT_PORT = 4001
+
+class FakeConnection < StringIO
+
+  def will_send(query)
+    commands.push query
+  end
+
+  def gets
+    commands.pop
+  end
+
+  def read_all_queries?
+    commands.empty?
+  end
+
+  private
+
+  def commands
+    @commands ||= []
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: bane
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: false
   segments: 
   - 0
-  - 1
-  - 1
-  version: 0.1.1
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
 authors: 
 - Daniel Wellman
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-28 00:00:00 -04:00
+date: 2010-07-02 00:00:00 -04:00
 default_executable: bane
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -64,6 +64,7 @@ files:
 - test/bane/configuration_parser_test.rb
 - test/bane/configuration_test.rb
 - test/bane/delegating_gserver_test.rb
+- test/bane/fake_connection_test.rb
 - test/bane/integration_test.rb
 - test/bane/launcher_test.rb
 - test/bane/naive_http_response_test.rb
@@ -111,6 +112,7 @@ test_files:
 - test/bane/configuration_parser_test.rb
 - test/bane/configuration_test.rb
 - test/bane/delegating_gserver_test.rb
+- test/bane/fake_connection_test.rb
 - test/bane/integration_test.rb
 - test/bane/launcher_test.rb
 - test/bane/naive_http_response_test.rb