thrift 0.22.0 → 0.24.0

data/spec/json_protocol_spec.rb

@@ -1,4 +1,5 @@
 # encoding: UTF-8
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -21,7 +22,6 @@
 require 'spec_helper'
 
 describe 'JsonProtocol' do
-
   describe Thrift::JsonProtocol do
     before(:each) do
       @trans = Thrift::MemoryBufferTransport.new
@@ -221,25 +221,18 @@ describe 'JsonProtocol' do
       expect(@trans.read(@trans.available)).to eq("\"-Infinity\"")
     end
 
-    if RUBY_VERSION >= '1.9'
-      it 'should write string' do
-        @prot.write_string('this is a test string')
-        a = @trans.read(@trans.available)
-        expect(a).to eq('"this is a test string"'.force_encoding(Encoding::BINARY))
-        expect(a.encoding).to eq(Encoding::BINARY)
-      end
+    it 'should write string' do
+      @prot.write_string('this is a test string')
+      a = @trans.read(@trans.available)
+      expect(a).to eq('"this is a test string"'.b)
+      expect(a.encoding).to eq(Encoding::BINARY)
+    end
 
-      it 'should write string with unicode characters' do
-        @prot.write_string("this is a test string with unicode characters: \u20AC \u20AD")
-        a = @trans.read(@trans.available)
-        expect(a).to eq("\"this is a test string with unicode characters: \u20AC \u20AD\"".force_encoding(Encoding::BINARY))
-        expect(a.encoding).to eq(Encoding::BINARY)
-      end
-    else
-      it 'should write string' do
-        @prot.write_string('this is a test string')
-        expect(@trans.read(@trans.available)).to eq('"this is a test string"')
-      end
+    it 'should write string with unicode characters' do
+      @prot.write_string("this is a test string with unicode characters: \u20AC \u20AD")
+      a = @trans.read(@trans.available)
+      expect(a).to eq("\"this is a test string with unicode characters: \u20AC \u20AD\"".b)
+      expect(a.encoding).to eq(Encoding::BINARY)
     end
 
     it "should write binary" do
@@ -252,9 +245,14 @@ describe 'JsonProtocol' do
       expect(@trans.read(@trans.available)).to eq("\"AAECAwQFBgcICQoLDA0ODxAREhMUFRYXGBkaGxwdHh8gISIjJCUmJygpKissLS4vMDEyMzQ1Njc4OTo7PD0+P0BBQkNERUZHSElKS0xNTk9QUVJTVFVWV1hZWltcXV5fYGFiY2RlZmdoaWprbG1ub3BxcnN0dXZ3eHl6e3x9fn+AgYKDhIWGh4iJiouMjY6PkJGSk5SVlpeYmZqbnJ2en6ChoqOkpaanqKmqq6ytrq+wsbKztLW2t7i5uru8vb6/wMHCw8TFxsfIycrLzM3Oz9DR0tPU1dbX2Nna29zd3t/g4eLj5OXm5+jp6uvs7e7v8PHy8/T19vf4+fr7/P3+/w==\"")
     end
 
+    it "should write a uuid" do
+      @prot.write_uuid("00112233-4455-6677-8899-aabbccddeeff")
+      expect(@trans.read(@trans.available)).to eq("\"00112233-4455-6677-8899-aabbccddeeff\"")
+    end
+
     it "should get type name for type id" do
-      expect {@prot.get_type_name_for_type_id(Thrift::Types::STOP)}.to raise_error(NotImplementedError)
-      expect {@prot.get_type_name_for_type_id(Thrift::Types::VOID)}.to raise_error(NotImplementedError)
+      expect { @prot.get_type_name_for_type_id(Thrift::Types::STOP) }.to raise_error(NotImplementedError)
+      expect { @prot.get_type_name_for_type_id(Thrift::Types::VOID) }.to raise_error(NotImplementedError)
       expect(@prot.get_type_name_for_type_id(Thrift::Types::BOOL)).to eq("tf")
       expect(@prot.get_type_name_for_type_id(Thrift::Types::BYTE)).to eq("i8")
       expect(@prot.get_type_name_for_type_id(Thrift::Types::DOUBLE)).to eq("dbl")
@@ -269,7 +267,7 @@ describe 'JsonProtocol' do
     end
 
     it "should get type id for type name" do
-      expect {@prot.get_type_id_for_type_name("pp")}.to raise_error(NotImplementedError)
+      expect { @prot.get_type_id_for_type_name("pp") }.to raise_error(NotImplementedError)
       expect(@prot.get_type_id_for_type_name("tf")).to eq(Thrift::Types::BOOL)
       expect(@prot.get_type_id_for_type_name("i8")).to eq(Thrift::Types::BYTE)
       expect(@prot.get_type_id_for_type_name("dbl")).to eq(Thrift::Types::DOUBLE)
@@ -285,7 +283,7 @@ describe 'JsonProtocol' do
 
     it "should read json syntax char" do
       @trans.write('F')
-      expect {@prot.read_json_syntax_char('G')}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_json_syntax_char('G') }.to raise_error(Thrift::ProtocolException)
       @trans.write('H')
       @prot.read_json_syntax_char('H')
     end
@@ -321,7 +319,7 @@ describe 'JsonProtocol' do
 
     it "should read json string" do
       @trans.write("\"\\P")
-      expect {@prot.read_json_string(false)}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_json_string(false) }.to raise_error(Thrift::ProtocolException)
 
       @trans.write("\"this is a test string\"")
       expect(@prot.read_json_string).to eq("this is a test string")
@@ -358,7 +356,7 @@ describe 'JsonProtocol' do
 
     it "should read json integer" do
       @trans.write("1.45\"\"")
-      expect {@prot.read_json_integer}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_json_integer }.to raise_error(Thrift::ProtocolException)
       @prot.read_string
 
       @trans.write("1453T")
@@ -367,11 +365,11 @@ describe 'JsonProtocol' do
 
     it "should read json double" do
       @trans.write("1.45e3e01\"\"")
-      expect {@prot.read_json_double}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_json_double }.to raise_error(Thrift::ProtocolException)
       @prot.read_string
 
       @trans.write("\"1.453e01\"")
-      expect {@prot.read_json_double}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_json_double }.to raise_error(Thrift::ProtocolException)
 
       @trans.write("1.453e01\"\"")
       expect(@prot.read_json_double).to eq(14.53)
@@ -409,7 +407,7 @@ describe 'JsonProtocol' do
 
     it "should read_message_begin" do
       @trans.write("[2,")
-      expect {@prot.read_message_begin}.to raise_error(Thrift::ProtocolException)
+      expect { @prot.read_message_begin }.to raise_error(Thrift::ProtocolException)
 
       @trans.write("[1,\"name\",12,32\"\"")
       expect(@prot.read_message_begin).to eq(["name", 12, 32])
@@ -445,6 +443,13 @@ describe 'JsonProtocol' do
       expect(@prot.read_map_begin).to eq([12, 15, 2])
     end
 
+    it "should reject a negative map size" do
+      @trans.write("[\"rec\",\"lst\",-1,{")
+      expect { @prot.read_map_begin }.to raise_error(Thrift::ProtocolException, "Negative size") do |e|
+        expect(e.type).to eq(Thrift::ProtocolException::NEGATIVE_SIZE)
+      end
+    end
+
     it "should read map end" do
       @trans.write("}]")
       expect(@prot.read_map_end).to eq(nil)
@@ -455,6 +460,13 @@ describe 'JsonProtocol' do
       expect(@prot.read_list_begin).to eq([12, 2])
     end
 
+    it "should reject a negative list size" do
+      @trans.write("[\"rec\",-1\"\"")
+      expect { @prot.read_list_begin }.to raise_error(Thrift::ProtocolException, "Negative size") do |e|
+        expect(e.type).to eq(Thrift::ProtocolException::NEGATIVE_SIZE)
+      end
+    end
+
     it "should read list end" do
       @trans.write("]")
       expect(@prot.read_list_end).to eq(nil)
@@ -465,6 +477,13 @@ describe 'JsonProtocol' do
       expect(@prot.read_set_begin).to eq([12, 2])
     end
 
+    it "should reject a negative set size" do
+      @trans.write("[\"rec\",-1\"\"")
+      expect { @prot.read_set_begin }.to raise_error(Thrift::ProtocolException, "Negative size") do |e|
+        expect(e.type).to eq(Thrift::ProtocolException::NEGATIVE_SIZE)
+      end
+    end
+
     it "should read set end" do
       @trans.write("]")
       expect(@prot.read_set_end).to eq(nil)
@@ -504,25 +523,18 @@ describe 'JsonProtocol' do
       expect(@prot.read_double).to eq(12.23)
     end
 
-    if RUBY_VERSION >= '1.9'
-      it 'should read string' do
-        @trans.write('"this is a test string"'.force_encoding(Encoding::BINARY))
-        a = @prot.read_string
-        expect(a).to eq('this is a test string')
-        expect(a.encoding).to eq(Encoding::UTF_8)
-      end
+    it 'should read string' do
+      @trans.write('"this is a test string"'.b)
+      a = @prot.read_string
+      expect(a).to eq('this is a test string')
+      expect(a.encoding).to eq(Encoding::UTF_8)
+    end
 
-      it 'should read string with unicode characters' do
-        @trans.write('"this is a test string with unicode characters: \u20AC \u20AD"'.force_encoding(Encoding::BINARY))
-        a = @prot.read_string
-        expect(a).to eq("this is a test string with unicode characters: \u20AC \u20AD")
-        expect(a.encoding).to eq(Encoding::UTF_8)
-      end
-    else
-      it 'should read string' do
-        @trans.write('"this is a test string"')
-        expect(@prot.read_string).to eq('this is a test string')
-      end
+    it 'should read string with unicode characters' do
+      @trans.write('"this is a test string with unicode characters: \u20AC \u20AD"'.b)
+      a = @prot.read_string
+      expect(a).to eq("this is a test string with unicode characters: \u20AC \u20AD")
+      expect(a.encoding).to eq(Encoding::UTF_8)
     end
 
     it "should read binary" do
@@ -534,7 +546,17 @@ describe 'JsonProtocol' do
       @trans.write("\"AAECAwQFBgcICQoLDA0ODxAREhMUFRYXGBkaGxwdHh8gISIjJCUmJygpKissLS4vMDEyMzQ1Njc4OTo7PD0+P0BBQkNERUZHSElKS0xNTk9QUVJTVFVWV1hZWltcXV5fYGFiY2RlZmdoaWprbG1ub3BxcnN0dXZ3eHl6e3x9fn+AgYKDhIWGh4iJiouMjY6PkJGSk5SVlpeYmZqbnJ2en6ChoqOkpaanqKmqq6ytrq+wsbKztLW2t7i5uru8vb6/wMHCw8TFxsfIycrLzM3Oz9DR0tPU1dbX2Nna29zd3t/g4eLj5OXm5+jp6uvs7e7v8PHy8/T19vf4+fr7/P3+/w==\"")
       expect(@prot.read_binary.bytes.to_a).to eq((0...256).to_a)
     end
-  
+
+    it "should read a uuid" do
+      @trans.write("\"00112233-4455-6677-8899-aabbccddeeff\"")
+      expect(@prot.read_uuid).to eq("00112233-4455-6677-8899-aabbccddeeff")
+    end
+
+    it "should normalize uppercase uuid on read" do
+      @trans.write("\"00112233-4455-6677-8899-AABBCCDDEEFF\"")
+      expect(@prot.read_uuid).to eq("00112233-4455-6677-8899-aabbccddeeff")
+    end
+
     it "should provide a reasonable to_s" do
       expect(@prot.to_s).to eq("json(memory)")
     end

data/spec/multiplexed_processor_spec.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+require 'spec_helper'
+
+describe Thrift::MultiplexedProcessor do
+  before(:each) do
+    @processor = Thrift::MultiplexedProcessor.new
+    @iprot = double('MockInputProtocol')
+    @oprot = double('MockOutputProtocol')
+  end
+
+  it 'dispatches multiplexed calls to the registered service processor' do
+    actual_processor = double('ActualProcessor')
+    @processor.register_processor('ThriftTest', actual_processor)
+
+    expect(@iprot).to receive(:read_message_begin).and_return(['ThriftTest:testVoid', Thrift::MessageTypes::CALL, 1])
+    expect(actual_processor).to receive(:process) do |stored_protocol, oprot|
+      expect(stored_protocol.read_message_begin).to eq(['testVoid', Thrift::MessageTypes::CALL, 1])
+      expect(oprot).to eq(@oprot)
+      true
+    end
+
+    expect(@processor.process(@iprot, @oprot)).to eq(true)
+  end
+
+  it 'dispatches non-multiplexed calls to the default processor' do
+    default_processor = double('DefaultProcessor')
+    @processor.register_default(default_processor)
+
+    expect(@iprot).to receive(:read_message_begin).and_return(['testVoid', Thrift::MessageTypes::CALL, 2])
+    expect(default_processor).to receive(:process) do |stored_protocol, oprot|
+      expect(stored_protocol.read_message_begin).to eq(['testVoid', Thrift::MessageTypes::CALL, 2])
+      expect(oprot).to eq(@oprot)
+      true
+    end
+
+    expect(@processor.process(@iprot, @oprot)).to eq(true)
+  end
+
+  it 'raises for non-multiplexed calls when no default processor is registered' do
+    expect(@iprot).to receive(:read_message_begin).and_return(['testVoid', Thrift::MessageTypes::CALL, 3])
+
+    expect { @processor.process(@iprot, @oprot) }.to raise_error(
+      Thrift::Exception,
+      'Service name not found in message name: testVoid. Did you forget to use a Thrift::Protocol::MultiplexedProtocol in your client?'
+    )
+  end
+
+  it 'raises for unknown multiplexed service names' do
+    expect(@iprot).to receive(:read_message_begin).and_return(['Missing:testVoid', Thrift::MessageTypes::CALL, 4])
+
+    expect { @processor.process(@iprot, @oprot) }.to raise_error(
+      Thrift::Exception,
+      'Service name not found: Missing. Did you forget to call Thrift::MultiplexedProcessor#register_processor?'
+    )
+  end
+end

data/spec/namespaced_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -63,5 +64,4 @@ describe 'namespaced generation' do
   it "extended a service" do
     require "extended/extended_service"
   end
-
 end

data/spec/nonblocking_server_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -18,9 +19,9 @@
 #
 
 require 'spec_helper'
+require 'timeout'
 
 describe 'NonblockingServer' do
-
   class Handler
     def initialize
       @queue = Queue.new
@@ -76,7 +77,7 @@ describe 'NonblockingServer' do
       @transport.read(sz)
     end
 
-    def write(buf,sz=nil)
+    def write(buf, sz = nil)
       @transport.write(buf, sz)
     end
 
@@ -101,7 +102,7 @@ describe 'NonblockingServer' do
 
   describe Thrift::NonblockingServer do
     before(:each) do
-      @port = 43251
+      @port = available_port
       handler = Handler.new
       processor = SpecNamespace::NonblockingService::Processor.new(handler)
       queue = Queue.new
@@ -121,6 +122,7 @@ describe 'NonblockingServer' do
         end
       end
       queue.pop
+      wait_until_listening(@transport, @server_thread)
 
       @clients = []
       @catch_exceptions = false
@@ -128,9 +130,11 @@ describe 'NonblockingServer' do
 
     after(:each) do
       @clients.each { |client, trans| trans.close }
-      # @server.shutdown(1)
-      @server_thread.kill
-      @transport.close
+      @server.shutdown(1, false) if @server
+      @server_thread.join(2) if @server_thread
+      @server_thread.kill if @server_thread && @server_thread.alive?
+      @server_thread.join(2) if @server_thread
+      @transport.close if @transport
     end
 
     def setup_client(queue = nil)
@@ -166,7 +170,7 @@ describe 'NonblockingServer' do
               break
             end
           end
-          @clients.each { |c,t| t.close and break if c == client } #close the transport
+          @clients.each { |c, t| t.close and break if c == client } # close the transport
         rescue => e
           raise e unless @catch_exceptions
         end
@@ -245,7 +249,7 @@ describe 'NonblockingServer' do
     it "should kill active messages when they don't expire while shutting down" do
       result = Queue.new
       client = setup_client_thread(result)
-      client << [:sleep, 10]
+      client << [:sleep, 10.0]
       sleep 0.1 # start processing the client's message
       @server.shutdown(1)
       @catch_exceptions = true
@@ -260,4 +264,166 @@ describe 'NonblockingServer' do
       expect(@server_thread.join(2)).not_to be_nil
     end
   end
+
+  describe Thrift::NonblockingServer::IOManager do
+    def build_io_manager
+      logger = Logger.new(IO::NULL)
+      logger.level = Logger::FATAL
+      Thrift::NonblockingServer::IOManager.new(
+        double('processor'),
+        double('server_transport'),
+        Thrift::BaseTransportFactory.new,
+        Thrift::BinaryProtocolFactory.new,
+        1,
+        logger
+      )
+    end
+
+    it "closes tracked connections and signal pipes during forced cleanup" do
+      io_manager = build_io_manager
+      connection = double('connection', :close => nil)
+      pipe_a = double('pipe_a', :closed? => false, :close => nil)
+      pipe_b = double('pipe_b', :closed? => false, :close => nil)
+
+      io_manager.instance_variable_set(:@connections, [connection])
+      io_manager.instance_variable_set(:@buffers, { connection => 'frame' })
+      io_manager.instance_variable_set(:@signal_pipes, [pipe_a, pipe_b])
+      io_manager.instance_variable_set(:@worker_threads, [])
+
+      io_manager.ensure_closed
+
+      expect(connection).to have_received(:close)
+      expect(pipe_a).to have_received(:close)
+      expect(pipe_b).to have_received(:close)
+      expect(io_manager.instance_variable_get(:@connections)).to be_empty
+      expect(io_manager.instance_variable_get(:@buffers)).to be_empty
+    end
+
+    it "continues closing remaining signal pipes when one close raises" do
+      io_manager = build_io_manager
+      pipe_a = double('pipe_a', :closed? => false)
+      pipe_b = double('pipe_b', :closed? => false, :close => nil)
+
+      allow(pipe_a).to receive(:close).and_raise(IOError)
+
+      io_manager.instance_variable_set(:@signal_pipes, [pipe_a, pipe_b])
+      io_manager.instance_variable_set(:@worker_threads, [])
+
+      io_manager.send(:close_signal_pipes)
+
+      expect(pipe_a).to have_received(:close)
+      expect(pipe_b).to have_received(:close)
+    end
+
+    it "drops removed connections from bookkeeping" do
+      io_manager = build_io_manager
+      connection = double('connection', :close => nil)
+
+      io_manager.instance_variable_set(:@connections, [connection])
+      io_manager.instance_variable_set(:@buffers, { connection => 'frame' })
+
+      io_manager.send(:remove_connection, connection)
+
+      expect(io_manager.instance_variable_get(:@connections)).to be_empty
+      expect(io_manager.instance_variable_get(:@buffers)).to be_empty
+    end
+  end
+
+  describe "#{Thrift::NonblockingServer} with TLS transport" do
+    before(:each) do
+      @port = available_port
+      handler = Handler.new
+      processor = SpecNamespace::NonblockingService::Processor.new(handler)
+      @transport = Thrift::SSLServerSocket.new('localhost', @port, create_server_ssl_context)
+      transport_factory = Thrift::FramedTransportFactory.new
+      logger = Logger.new(STDERR)
+      logger.level = Logger::WARN
+      @server = Thrift::NonblockingServer.new(processor, @transport, transport_factory, nil, 5, logger)
+      handler.server = @server
+
+      @server_thread = Thread.new(Thread.current) do |master_thread|
+        begin
+          @server.serve
+        rescue => e
+          master_thread.raise e
+        end
+      end
+
+      @clients = []
+      wait_until_listening(@transport, @server_thread)
+    end
+
+    after(:each) do
+      @clients.each(&:close)
+      @server.shutdown if @server
+      @server_thread.join(2) if @server_thread
+      @transport.close if @transport
+    end
+
+    it "should handle requests over TLS" do
+      expect(@server_thread).to be_alive
+
+      client = setup_tls_client
+      expect(client.greeting(true)).to eq(SpecNamespace::Hello.new)
+
+      @server.shutdown
+      expect(@server_thread.join(2)).to be_an_instance_of(Thread)
+    end
+
+    def setup_tls_client
+      transport = Thrift::FramedTransport.new(
+        Thrift::SSLSocket.new('localhost', @port, nil, create_client_ssl_context)
+      )
+      protocol = Thrift::BinaryProtocol.new(transport)
+      client = SpecNamespace::NonblockingService::Client.new(protocol)
+      transport.open
+      @clients << transport
+      client
+    end
+
+    def ssl_keys_dir
+      File.expand_path('../../../test/keys', __dir__)
+    end
+
+    def create_server_ssl_context
+      OpenSSL::SSL::SSLContext.new.tap do |ctx|
+        ctx.verify_mode = OpenSSL::SSL::VERIFY_PEER
+        if ctx.respond_to?(:min_version=) && OpenSSL::SSL.const_defined?(:TLS1_2_VERSION)
+          ctx.min_version = OpenSSL::SSL::TLS1_2_VERSION
+        end
+        ctx.ca_file = File.join(ssl_keys_dir, 'CA.pem')
+        ctx.cert = OpenSSL::X509::Certificate.new(File.read(File.join(ssl_keys_dir, 'server.crt')))
+        ctx.cert_store = OpenSSL::X509::Store.new
+        ctx.cert_store.add_file(File.join(ssl_keys_dir, 'client.pem'))
+        ctx.key = OpenSSL::PKey::RSA.new(File.read(File.join(ssl_keys_dir, 'server.key')))
+      end
+    end
+
+    def create_client_ssl_context
+      OpenSSL::SSL::SSLContext.new.tap do |ctx|
+        ctx.verify_mode = OpenSSL::SSL::VERIFY_PEER
+        if ctx.respond_to?(:min_version=) && OpenSSL::SSL.const_defined?(:TLS1_2_VERSION)
+          ctx.min_version = OpenSSL::SSL::TLS1_2_VERSION
+        end
+        ctx.ca_file = File.join(ssl_keys_dir, 'CA.pem')
+        ctx.cert = OpenSSL::X509::Certificate.new(File.read(File.join(ssl_keys_dir, 'client.crt')))
+        ctx.cert_store = OpenSSL::X509::Store.new
+        ctx.cert_store.add_file(File.join(ssl_keys_dir, 'server.pem'))
+        ctx.key = OpenSSL::PKey::RSA.new(File.read(File.join(ssl_keys_dir, 'client.key')))
+      end
+    end
+  end
+
+  def wait_until_listening(server_transport, server_thread)
+    Timeout.timeout(2) do
+      until server_transport.handle
+        raise "Server thread exited unexpectedly" unless server_thread.alive?
+        sleep 0.01
+      end
+    end
+  end
+
+  def available_port
+    TCPServer.open('localhost', 0) { |server| server.addr[1] }
+  end
 end

data/spec/processor_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -20,7 +21,6 @@
 require 'spec_helper'
 
 describe 'Processor' do
-
   class ProcessorSpec
     include Thrift::Processor
   end