ruby_fs 1.0.5 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 32b68c933f40664fb0b884507a542b4d10914e91
-  data.tar.gz: e7e1821584741db70d7040deeadd2017f15bc1f3
+  metadata.gz: 793eaf957a7ea614c7f4ee1f12c1f4abe171274e
+  data.tar.gz: 39dbc95b03ebe95e8985aafa7fcdb206fa2929f8
 SHA512:
-  metadata.gz: 4a1ddf66fe30a97596f9bf9f83fcde3909e69aeb3fb05480e21781eb0207af4c811ef345c8768b7b1d9ff1da8e05a8b88d963f3fea7475d34e8bc559fe729bc8
-  data.tar.gz: 9f138ddc9b654c4e9f7188ace388920413fceefa08461ec959cf76f055b096027139ebdd41677d7b4c3f28895fedd1833207eb655da45309aec20e2a619cb8d1
+  metadata.gz: 7aa7b7c3f0783108b296a7b3fd0d36d9baf3d1b10dbaf206a5e3b8f7db5a09e0d839170bfdca9c65a1cf326bfabe6dc17c0bcd12c833bedfdbe4ed0038b51964
+  data.tar.gz: 4b5cf980549c77d98bb91417242c2ee8f3198f35d3537d213b4b21ca166d6905801ae2f686f6e92bf4edf41036cfb9e77f756f86fc407b867853c642c0f1d9a2

data/.travis.yml

@@ -2,6 +2,7 @@ language: ruby
 rvm:
   - 1.9.2
   - 1.9.3
+  - 2.0.0
   - rbx-19mode
   - jruby-19mode
   - ruby-head

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # develop
 
+# 1.1.0
+  * Feature: Allow specifying an event mask (other than 'ALL') when connecting
+  * CS: Minor performance improvements from removing repeated regex compilation
+
 # 1.0.5
   * Bugfix: Allow sending large app arguments. Application arguments (headers in general) are limited to 2048 bytes. The work-around is to send them in the body of the message with a content-length header.
   * CS: Avoid Celluloid deprecation warnings

data/lib/ruby_fs/event.rb

@@ -3,7 +3,7 @@ require 'ruby_fs/response'
 module RubyFS
   class Event < Response
     #
-    # @return [String[ The name of the event
+    # @return [String] The name of the event
     def event_name
       content[:event_name]
     end

data/lib/ruby_fs/lexer.rb

@@ -1,8 +1,7 @@
 module RubyFS
   class Lexer
     ContentLengthPattern  = /Content-length:\s*(\d+)/i
-    MaxLineLength         = 16 * 1024
-    MaxBinaryLength       = 32 * 1024 * 1024
+    HeaderFormatPattern   = /\A([^\s:]+)\s*:\s*/
 
     def initialize(callback)
       @callback = callback
@@ -137,9 +136,9 @@ module RubyFS
     def headers_2_hash(headers)
       {}.tap do |hash|
         headers.each do |h|
-          if /\A([^\s:]+)\s*:\s*/ =~ h
+          if HeaderFormatPattern =~ h
             tail = $'.dup
-            hash[$1.downcase.gsub(/-/, "_").intern] = tail
+            hash[$1.downcase.gsub('-', "_").intern] = tail
           end
         end
       end

data/lib/ruby_fs/stream.rb

@@ -168,7 +168,7 @@ module RubyFS
         @command_callbacks.pop.call CommandReply.new(headers, (content == '' ? nil : content))
       when 'auth/request'
         command "auth #{@secret}" do
-          async.command "event json ALL" if @events
+          async.command "event json #{event_mask}" if @events
         end
       when 'text/disconnect-notice'
         terminate
@@ -181,7 +181,7 @@ module RubyFS
       json = JSON.parse content
       {}.tap do |hash|
         json.each_pair do |k, v|
-          hash[k.downcase.gsub(/-/,"_").intern] = v
+          hash[k.downcase.gsub('-',"_").intern] = v
         end
       end
     end
@@ -190,5 +190,11 @@ module RubyFS
       @state = :started
       fire_event Connected.new
     end
+
+    def event_mask
+      return 'ALL' if @events == true
+
+      Array(@events).join(' ')
+    end
   end
 end

data/lib/ruby_fs/version.rb

@@ -1,3 +1,3 @@
 module RubyFS
-  VERSION = "1.0.5"
+  VERSION = "1.1.0"
 end

data/spec/ruby_fs/stream_spec.rb

@@ -301,7 +301,7 @@ Reply-Text: +OK accepted
     end
 
     context 'with events turned off' do
-      it 'does not the event mask after authenticating' do
+      it 'does not set the event mask after authenticating' do
         mocked_server(1, lambda { |server| server.send_data "Content-Type: auth/request\n\n" }) do |val, server|
           val.should == "auth ClueCon\n\n"
           server.send_data %Q(
@@ -318,6 +318,68 @@ Reply-Text: +OK accepted
       end
     end
 
+    context 'with an event mask fully specified as a string' do
+      let(:events) { 'CODEC CHANNEL_STATE' }
+
+      it 'sets the event mask after authenticating' do
+        mocked_server(2, lambda { |server| server.send_data "Content-Type: auth/request\n\n" }) do |val, server|
+          case @sequence
+          when 1
+            val.should == "auth ClueCon\n\n"
+            server.send_data %Q(
+Content-Type: command/reply
+Reply-Text: +OK accepted
+
+)
+          when 2
+            val.should == "event json CODEC CHANNEL_STATE\n\n"
+            server.send_data %Q(
+Content-Type: command/reply
+Reply-Text: +OK accepted
+
+)
+          end
+          @sequence += 1
+        end
+
+        client_messages.should be == [
+          Stream::Connected.new,
+          Stream::Disconnected.new
+        ]
+      end
+    end
+
+    context 'with an event mask fully specified as an array' do
+      let(:events) { ['CODEC', 'CHANNEL_STATE'] }
+
+      it 'sets the event mask after authenticating' do
+        mocked_server(2, lambda { |server| server.send_data "Content-Type: auth/request\n\n" }) do |val, server|
+          case @sequence
+          when 1
+            val.should == "auth ClueCon\n\n"
+            server.send_data %Q(
+Content-Type: command/reply
+Reply-Text: +OK accepted
+
+)
+          when 2
+            val.should == "event json CODEC CHANNEL_STATE\n\n"
+            server.send_data %Q(
+Content-Type: command/reply
+Reply-Text: +OK accepted
+
+)
+          end
+          @sequence += 1
+        end
+
+        client_messages.should be == [
+          Stream::Connected.new,
+          Stream::Disconnected.new
+        ]
+      end
+    end
+
     context 'when receiving a disconnect notice' do
       it 'puts itself in the stopped state and fires a disconnected event' do
         expect_connected_event

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_fs
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.1.0
 platform: ruby
 authors:
 - Ben Langfeld
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-23 00:00:00.000000000 Z
+date: 2013-04-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: celluloid-io