blob: c4038330aefcc6a8b6d89d02060859aa9e528984 [file] [log] [blame]
David Reissea2cba82009-03-30 21:35:00 +00001#
2# Licensed to the Apache Software Foundation (ASF) under one
3# or more contributor license agreements. See the NOTICE file
4# distributed with this work for additional information
5# regarding copyright ownership. The ASF licenses this file
6# to you under the Apache License, Version 2.0 (the
7# "License"); you may not use this file except in compliance
8# with the License. You may obtain a copy of the License at
9#
10# http://www.apache.org/licenses/LICENSE-2.0
11#
12# Unless required by applicable law or agreed to in writing,
13# software distributed under the License is distributed on an
14# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15# KIND, either express or implied. See the License for the
16# specific language governing permissions and limitations
17# under the License.
18#
Jake Farrella87810f2012-09-28 01:59:04 +000019require 'spec_helper'
Kevin Clarkccc86582008-06-18 01:09:00 +000020
Jake Farrella87810f2012-09-28 01:59:04 +000021describe 'Server' do
Jake Farrella87810f2012-09-28 01:59:04 +000022 describe Thrift::BaseServer do
James E. King III9aaf2952018-03-20 15:06:08 -040023 before(:each) do
James E. King III27247072018-03-22 20:50:23 -040024 @processor = double("Processor")
25 @serverTrans = double("ServerTransport")
26 @trans = double("BaseTransport")
27 @prot = double("BaseProtocol")
James E. King III9aaf2952018-03-20 15:06:08 -040028 @server = described_class.new(@processor, @serverTrans, @trans, @prot)
Kevin Clarkccc86582008-06-18 01:09:00 +000029 end
30
James E. King III9aaf2952018-03-20 15:06:08 -040031 it "should default to BaseTransportFactory and BinaryProtocolFactory when not specified" do
James E. King III27247072018-03-22 20:50:23 -040032 @server = Thrift::BaseServer.new(double("Processor"), double("BaseServerTransport"))
33 expect(@server.instance_variable_get(:'@transport_factory')).to be_an_instance_of(Thrift::BaseTransportFactory)
34 expect(@server.instance_variable_get(:'@protocol_factory')).to be_an_instance_of(Thrift::BinaryProtocolFactory)
James E. King III9aaf2952018-03-20 15:06:08 -040035 end
36
37 it "should not serve" do
Dmytro Shteflyuk3b0ab4d2026-03-11 17:46:48 -040038 expect { @server.serve() }.to raise_error(NotImplementedError)
James E. King III9aaf2952018-03-20 15:06:08 -040039 end
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -040040
James E. King III9aaf2952018-03-20 15:06:08 -040041 it "should provide a reasonable to_s" do
James E. King III27247072018-03-22 20:50:23 -040042 expect(@serverTrans).to receive(:to_s).once.and_return("serverTrans")
43 expect(@trans).to receive(:to_s).once.and_return("trans")
44 expect(@prot).to receive(:to_s).once.and_return("prot")
45 expect(@server.to_s).to eq("server(prot(trans(serverTrans)))")
James E. King III9aaf2952018-03-20 15:06:08 -040046 end
Kevin Clarkccc86582008-06-18 01:09:00 +000047 end
48
Jake Farrella87810f2012-09-28 01:59:04 +000049 describe Thrift::SimpleServer do
Kevin Clarkccc86582008-06-18 01:09:00 +000050 before(:each) do
James E. King III27247072018-03-22 20:50:23 -040051 @processor = double("Processor")
52 @serverTrans = double("ServerTransport")
53 @trans = double("BaseTransport")
54 @prot = double("BaseProtocol")
55 @client = double("Client")
Jake Farrella87810f2012-09-28 01:59:04 +000056 @server = described_class.new(@processor, @serverTrans, @trans, @prot)
Kevin Clarkccc86582008-06-18 01:09:00 +000057 end
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -040058
James E. King III9aaf2952018-03-20 15:06:08 -040059 it "should provide a reasonable to_s" do
James E. King III27247072018-03-22 20:50:23 -040060 expect(@serverTrans).to receive(:to_s).once.and_return("serverTrans")
61 expect(@trans).to receive(:to_s).once.and_return("trans")
62 expect(@prot).to receive(:to_s).once.and_return("prot")
63 expect(@server.to_s).to eq("simple(server(prot(trans(serverTrans))))")
James E. King III9aaf2952018-03-20 15:06:08 -040064 end
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -040065
Kevin Clarkccc86582008-06-18 01:09:00 +000066 it "should serve in the main thread" do
James E. King III27247072018-03-22 20:50:23 -040067 expect(@serverTrans).to receive(:listen).ordered
68 expect(@serverTrans).to receive(:accept).exactly(3).times.and_return(@client)
69 expect(@trans).to receive(:get_transport).exactly(3).times.with(@client).and_return(@trans)
70 expect(@prot).to receive(:get_protocol).exactly(3).times.with(@trans).and_return(@prot)
Kevin Clarkccc86582008-06-18 01:09:00 +000071 x = 0
James E. King III27247072018-03-22 20:50:23 -040072 expect(@processor).to receive(:process).exactly(3).times.with(@prot, @prot) do
Kevin Clarkccc86582008-06-18 01:09:00 +000073 case (x += 1)
Bryan Duxburye3ab50d2009-03-25 21:06:53 +000074 when 1 then raise Thrift::TransportException
75 when 2 then raise Thrift::ProtocolException
76 when 3 then throw :stop
Kevin Clarkccc86582008-06-18 01:09:00 +000077 end
78 end
James E. King III27247072018-03-22 20:50:23 -040079 expect(@trans).to receive(:close).exactly(3).times
80 expect(@serverTrans).to receive(:close).ordered
81 expect { @server.serve }.to throw_symbol(:stop)
Kevin Clarkccc86582008-06-18 01:09:00 +000082 end
83 end
84
Jake Farrella87810f2012-09-28 01:59:04 +000085 describe Thrift::ThreadedServer do
86 before(:each) do
James E. King III27247072018-03-22 20:50:23 -040087 @processor = double("Processor")
88 @serverTrans = double("ServerTransport")
89 @trans = double("BaseTransport")
90 @prot = double("BaseProtocol")
91 @client = double("Client")
Jake Farrella87810f2012-09-28 01:59:04 +000092 @server = described_class.new(@processor, @serverTrans, @trans, @prot)
Kevin Clarkccc86582008-06-18 01:09:00 +000093 end
94
James E. King III9aaf2952018-03-20 15:06:08 -040095 it "should provide a reasonable to_s" do
James E. King III27247072018-03-22 20:50:23 -040096 expect(@serverTrans).to receive(:to_s).once.and_return("serverTrans")
97 expect(@trans).to receive(:to_s).once.and_return("trans")
98 expect(@prot).to receive(:to_s).once.and_return("prot")
99 expect(@server.to_s).to eq("threaded(server(prot(trans(serverTrans))))")
James E. King III9aaf2952018-03-20 15:06:08 -0400100 end
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -0400101
Kevin Clarkccc86582008-06-18 01:09:00 +0000102 it "should serve using threads" do
James E. King III27247072018-03-22 20:50:23 -0400103 expect(@serverTrans).to receive(:listen).ordered
104 expect(@serverTrans).to receive(:accept).exactly(3).times.and_return(@client)
105 expect(@trans).to receive(:get_transport).exactly(3).times.with(@client).and_return(@trans)
106 expect(@prot).to receive(:get_protocol).exactly(3).times.with(@trans).and_return(@prot)
107 expect(Thread).to receive(:new).with(@prot, @trans).exactly(3).times.and_yield(@prot, @trans)
Kevin Clarkccc86582008-06-18 01:09:00 +0000108 x = 0
James E. King III27247072018-03-22 20:50:23 -0400109 expect(@processor).to receive(:process).exactly(3).times.with(@prot, @prot) do
Kevin Clarkccc86582008-06-18 01:09:00 +0000110 case (x += 1)
Bryan Duxburye3ab50d2009-03-25 21:06:53 +0000111 when 1 then raise Thrift::TransportException
112 when 2 then raise Thrift::ProtocolException
113 when 3 then throw :stop
Kevin Clarkccc86582008-06-18 01:09:00 +0000114 end
115 end
James E. King III27247072018-03-22 20:50:23 -0400116 expect(@trans).to receive(:close).exactly(3).times
117 expect(@serverTrans).to receive(:close).ordered
118 expect { @server.serve }.to throw_symbol(:stop)
Kevin Clarkccc86582008-06-18 01:09:00 +0000119 end
120 end
121
Jake Farrella87810f2012-09-28 01:59:04 +0000122 describe Thrift::ThreadPoolServer do
123 before(:each) do
James E. King III27247072018-03-22 20:50:23 -0400124 @processor = double("Processor")
125 @server_trans = double("ServerTransport")
126 @trans = double("BaseTransport")
127 @prot = double("BaseProtocol")
128 @client = double("Client")
Jake Farrell96be0072012-10-06 00:26:28 +0000129 @server = described_class.new(@processor, @server_trans, @trans, @prot)
James E. King III9aaf2952018-03-20 15:06:08 -0400130 sleep(0.15)
Kevin Clarkccc86582008-06-18 01:09:00 +0000131 end
132
James E. King III9aaf2952018-03-20 15:06:08 -0400133 it "should provide a reasonable to_s" do
James E. King III27247072018-03-22 20:50:23 -0400134 expect(@server_trans).to receive(:to_s).once.and_return("server_trans")
135 expect(@trans).to receive(:to_s).once.and_return("trans")
136 expect(@prot).to receive(:to_s).once.and_return("prot")
137 expect(@server.to_s).to eq("threadpool(server(prot(trans(server_trans))))")
James E. King III9aaf2952018-03-20 15:06:08 -0400138 end
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -0400139
Kevin Clarkccc86582008-06-18 01:09:00 +0000140 it "should serve inside a thread" do
Jake Farrell96be0072012-10-06 00:26:28 +0000141 exception_q = @server.instance_variable_get(:@exception_q)
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -0400142 expect_any_instance_of(described_class).to receive(:serve) do
Jake Farrell96be0072012-10-06 00:26:28 +0000143 exception_q.push(StandardError.new('ERROR'))
144 end
145 expect { @server.rescuable_serve }.to(raise_error('ERROR'))
James E. King IIIb92ca5f2018-06-09 23:02:12 -0400146 sleep(0.15)
Kevin Clarkccc86582008-06-18 01:09:00 +0000147 end
148
149 it "should avoid running the server twice when retrying rescuable_serve" do
Jake Farrell96be0072012-10-06 00:26:28 +0000150 exception_q = @server.instance_variable_get(:@exception_q)
Dmytro Shteflyukf5c80a42026-03-08 19:09:43 -0400151 expect_any_instance_of(described_class).to receive(:serve) do
Jake Farrell96be0072012-10-06 00:26:28 +0000152 exception_q.push(StandardError.new('ERROR1'))
153 exception_q.push(StandardError.new('ERROR2'))
154 end
155 expect { @server.rescuable_serve }.to(raise_error('ERROR1'))
156 expect { @server.rescuable_serve }.to(raise_error('ERROR2'))
Kevin Clarkccc86582008-06-18 01:09:00 +0000157 end
158
159 it "should serve using a thread pool" do
James E. King III27247072018-03-22 20:50:23 -0400160 thread_q = double("SizedQueue")
161 exception_q = double("Queue")
Jake Farrell96be0072012-10-06 00:26:28 +0000162 @server.instance_variable_set(:@thread_q, thread_q)
163 @server.instance_variable_set(:@exception_q, exception_q)
James E. King III27247072018-03-22 20:50:23 -0400164 expect(@server_trans).to receive(:listen).ordered
165 expect(thread_q).to receive(:push).with(:token)
166 expect(thread_q).to receive(:pop)
167 expect(Thread).to receive(:new).and_yield
168 expect(@server_trans).to receive(:accept).exactly(3).times.and_return(@client)
169 expect(@trans).to receive(:get_transport).exactly(3).times.and_return(@trans)
170 expect(@prot).to receive(:get_protocol).exactly(3).times.and_return(@prot)
Kevin Clarkccc86582008-06-18 01:09:00 +0000171 x = 0
172 error = RuntimeError.new("Stopped")
James E. King III27247072018-03-22 20:50:23 -0400173 expect(@processor).to receive(:process).exactly(3).times.with(@prot, @prot) do
Kevin Clarkccc86582008-06-18 01:09:00 +0000174 case (x += 1)
Bryan Duxburye3ab50d2009-03-25 21:06:53 +0000175 when 1 then raise Thrift::TransportException
176 when 2 then raise Thrift::ProtocolException
177 when 3 then raise error
Kevin Clarkccc86582008-06-18 01:09:00 +0000178 end
179 end
James E. King III27247072018-03-22 20:50:23 -0400180 expect(@trans).to receive(:close).exactly(3).times
181 expect(exception_q).to receive(:push).with(error).and_throw(:stop)
182 expect(@server_trans).to receive(:close)
Jake Farrell96be0072012-10-06 00:26:28 +0000183 expect { @server.serve }.to(throw_symbol(:stop))
Kevin Clarkccc86582008-06-18 01:09:00 +0000184 end
185 end
186end