thrift 0.9.2.0 → 0.15.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +7 -0
- data/ext/binary_protocol_accelerated.c +12 -12
- data/ext/compact_protocol.c +1 -0
- data/ext/struct.c +14 -1
- data/ext/thrift_native.c +17 -0
- data/lib/thrift/multiplexed_processor.rb +76 -0
- data/lib/thrift/processor.rb +24 -6
- data/lib/thrift/protocol/base_protocol.rb +11 -3
- data/lib/thrift/protocol/binary_protocol.rb +8 -1
- data/lib/thrift/protocol/binary_protocol_accelerated.rb +8 -0
- data/lib/thrift/protocol/compact_protocol.rb +8 -0
- data/lib/thrift/protocol/json_protocol.rb +21 -4
- data/lib/thrift/protocol/multiplexed_protocol.rb +44 -0
- data/lib/thrift/protocol/protocol_decorator.rb +194 -0
- data/lib/thrift/server/base_server.rb +8 -2
- data/lib/thrift/server/simple_server.rb +5 -1
- data/lib/thrift/server/thread_pool_server.rb +5 -1
- data/lib/thrift/server/threaded_server.rb +5 -1
- data/lib/thrift/transport/base_server_transport.rb +1 -1
- data/lib/thrift/transport/base_transport.rb +8 -0
- data/lib/thrift/transport/buffered_transport.rb +9 -1
- data/lib/thrift/transport/framed_transport.rb +9 -1
- data/lib/thrift/transport/http_client_transport.rb +7 -0
- data/lib/thrift/transport/io_stream_transport.rb +4 -1
- data/lib/thrift/transport/memory_buffer_transport.rb +4 -0
- data/lib/thrift/transport/server_socket.rb +6 -1
- data/lib/thrift/transport/socket.rb +21 -17
- data/lib/thrift/transport/ssl_server_socket.rb +41 -0
- data/lib/thrift/transport/ssl_socket.rb +51 -0
- data/lib/thrift/transport/unix_server_socket.rb +5 -1
- data/lib/thrift/transport/unix_socket.rb +5 -1
- data/lib/thrift/union.rb +3 -6
- data/lib/thrift.rb +8 -4
- data/spec/BaseService.thrift +27 -0
- data/spec/ExtendedService.thrift +25 -0
- data/spec/base_protocol_spec.rb +79 -71
- data/spec/base_transport_spec.rb +155 -117
- data/spec/binary_protocol_accelerated_spec.rb +6 -2
- data/spec/binary_protocol_spec.rb +16 -8
- data/spec/binary_protocol_spec_shared.rb +75 -72
- data/spec/bytes_spec.rb +38 -38
- data/spec/client_spec.rb +41 -42
- data/spec/compact_protocol_spec.rb +32 -17
- data/spec/exception_spec.rb +54 -54
- data/spec/flat_spec.rb +62 -0
- data/spec/http_client_spec.rb +74 -33
- data/spec/json_protocol_spec.rb +170 -131
- data/spec/namespaced_spec.rb +10 -5
- data/spec/nonblocking_server_spec.rb +16 -16
- data/spec/processor_spec.rb +26 -26
- data/spec/serializer_spec.rb +20 -20
- data/spec/server_socket_spec.rb +27 -22
- data/spec/server_spec.rb +91 -51
- data/spec/socket_spec.rb +23 -16
- data/spec/socket_spec_shared.rb +31 -31
- data/spec/spec_helper.rb +9 -1
- data/spec/ssl_server_socket_spec.rb +34 -0
- data/spec/ssl_socket_spec.rb +78 -0
- data/spec/struct_nested_containers_spec.rb +24 -24
- data/spec/struct_spec.rb +120 -120
- data/spec/thin_http_server_spec.rb +18 -18
- data/spec/types_spec.rb +56 -53
- data/spec/union_spec.rb +51 -40
- data/spec/unix_socket_spec.rb +43 -34
- metadata +205 -143
data/spec/base_protocol_spec.rb
CHANGED
@@ -22,41 +22,46 @@ require 'spec_helper'
|
|
22
22
|
describe 'BaseProtocol' do
|
23
23
|
|
24
24
|
before(:each) do
|
25
|
-
@trans =
|
25
|
+
@trans = double("MockTransport")
|
26
26
|
@prot = Thrift::BaseProtocol.new(@trans)
|
27
27
|
end
|
28
28
|
|
29
29
|
describe Thrift::BaseProtocol do
|
30
30
|
# most of the methods are stubs, so we can ignore them
|
31
31
|
|
32
|
+
it "should provide a reasonable to_s" do
|
33
|
+
expect(@trans).to receive(:to_s).once.and_return("trans")
|
34
|
+
expect(@prot.to_s).to eq("trans")
|
35
|
+
end
|
36
|
+
|
32
37
|
it "should make trans accessible" do
|
33
|
-
@prot.trans.
|
38
|
+
expect(@prot.trans).to eql(@trans)
|
34
39
|
end
|
35
40
|
|
36
41
|
it 'should write out a field nicely (deprecated write_field signature)' do
|
37
|
-
@prot.
|
38
|
-
@prot.
|
39
|
-
@prot.
|
42
|
+
expect(@prot).to receive(:write_field_begin).with('field', 'type', 'fid').ordered
|
43
|
+
expect(@prot).to receive(:write_type).with({:name => 'field', :type => 'type'}, 'value').ordered
|
44
|
+
expect(@prot).to receive(:write_field_end).ordered
|
40
45
|
@prot.write_field('field', 'type', 'fid', 'value')
|
41
46
|
end
|
42
47
|
|
43
48
|
it 'should write out a field nicely' do
|
44
|
-
@prot.
|
45
|
-
@prot.
|
46
|
-
@prot.
|
49
|
+
expect(@prot).to receive(:write_field_begin).with('field', 'type', 'fid').ordered
|
50
|
+
expect(@prot).to receive(:write_type).with({:name => 'field', :type => 'type', :binary => false}, 'value').ordered
|
51
|
+
expect(@prot).to receive(:write_field_end).ordered
|
47
52
|
@prot.write_field({:name => 'field', :type => 'type', :binary => false}, 'fid', 'value')
|
48
53
|
end
|
49
54
|
|
50
55
|
it 'should write out the different types (deprecated write_type signature)' do
|
51
|
-
@prot.
|
52
|
-
@prot.
|
53
|
-
@prot.
|
54
|
-
@prot.
|
55
|
-
@prot.
|
56
|
-
@prot.
|
57
|
-
@prot.
|
58
|
-
struct =
|
59
|
-
struct.
|
56
|
+
expect(@prot).to receive(:write_bool).with('bool').ordered
|
57
|
+
expect(@prot).to receive(:write_byte).with('byte').ordered
|
58
|
+
expect(@prot).to receive(:write_double).with('double').ordered
|
59
|
+
expect(@prot).to receive(:write_i16).with('i16').ordered
|
60
|
+
expect(@prot).to receive(:write_i32).with('i32').ordered
|
61
|
+
expect(@prot).to receive(:write_i64).with('i64').ordered
|
62
|
+
expect(@prot).to receive(:write_string).with('string').ordered
|
63
|
+
struct = double('Struct')
|
64
|
+
expect(struct).to receive(:write).with(@prot).ordered
|
60
65
|
@prot.write_type(Thrift::Types::BOOL, 'bool')
|
61
66
|
@prot.write_type(Thrift::Types::BYTE, 'byte')
|
62
67
|
@prot.write_type(Thrift::Types::DOUBLE, 'double')
|
@@ -72,16 +77,16 @@ describe 'BaseProtocol' do
|
|
72
77
|
end
|
73
78
|
|
74
79
|
it 'should write out the different types' do
|
75
|
-
@prot.
|
76
|
-
@prot.
|
77
|
-
@prot.
|
78
|
-
@prot.
|
79
|
-
@prot.
|
80
|
-
@prot.
|
81
|
-
@prot.
|
82
|
-
@prot.
|
83
|
-
struct =
|
84
|
-
struct.
|
80
|
+
expect(@prot).to receive(:write_bool).with('bool').ordered
|
81
|
+
expect(@prot).to receive(:write_byte).with('byte').ordered
|
82
|
+
expect(@prot).to receive(:write_double).with('double').ordered
|
83
|
+
expect(@prot).to receive(:write_i16).with('i16').ordered
|
84
|
+
expect(@prot).to receive(:write_i32).with('i32').ordered
|
85
|
+
expect(@prot).to receive(:write_i64).with('i64').ordered
|
86
|
+
expect(@prot).to receive(:write_string).with('string').ordered
|
87
|
+
expect(@prot).to receive(:write_binary).with('binary').ordered
|
88
|
+
struct = double('Struct')
|
89
|
+
expect(struct).to receive(:write).with(@prot).ordered
|
85
90
|
@prot.write_type({:type => Thrift::Types::BOOL}, 'bool')
|
86
91
|
@prot.write_type({:type => Thrift::Types::BYTE}, 'byte')
|
87
92
|
@prot.write_type({:type => Thrift::Types::DOUBLE}, 'double')
|
@@ -98,13 +103,13 @@ describe 'BaseProtocol' do
|
|
98
103
|
end
|
99
104
|
|
100
105
|
it 'should read the different types (deprecated read_type signature)' do
|
101
|
-
@prot.
|
102
|
-
@prot.
|
103
|
-
@prot.
|
104
|
-
@prot.
|
105
|
-
@prot.
|
106
|
-
@prot.
|
107
|
-
@prot.
|
106
|
+
expect(@prot).to receive(:read_bool).ordered
|
107
|
+
expect(@prot).to receive(:read_byte).ordered
|
108
|
+
expect(@prot).to receive(:read_i16).ordered
|
109
|
+
expect(@prot).to receive(:read_i32).ordered
|
110
|
+
expect(@prot).to receive(:read_i64).ordered
|
111
|
+
expect(@prot).to receive(:read_double).ordered
|
112
|
+
expect(@prot).to receive(:read_string).ordered
|
108
113
|
@prot.read_type(Thrift::Types::BOOL)
|
109
114
|
@prot.read_type(Thrift::Types::BYTE)
|
110
115
|
@prot.read_type(Thrift::Types::I16)
|
@@ -120,14 +125,14 @@ describe 'BaseProtocol' do
|
|
120
125
|
end
|
121
126
|
|
122
127
|
it 'should read the different types' do
|
123
|
-
@prot.
|
124
|
-
@prot.
|
125
|
-
@prot.
|
126
|
-
@prot.
|
127
|
-
@prot.
|
128
|
-
@prot.
|
129
|
-
@prot.
|
130
|
-
@prot.
|
128
|
+
expect(@prot).to receive(:read_bool).ordered
|
129
|
+
expect(@prot).to receive(:read_byte).ordered
|
130
|
+
expect(@prot).to receive(:read_i16).ordered
|
131
|
+
expect(@prot).to receive(:read_i32).ordered
|
132
|
+
expect(@prot).to receive(:read_i64).ordered
|
133
|
+
expect(@prot).to receive(:read_double).ordered
|
134
|
+
expect(@prot).to receive(:read_string).ordered
|
135
|
+
expect(@prot).to receive(:read_binary).ordered
|
131
136
|
@prot.read_type({:type => Thrift::Types::BOOL})
|
132
137
|
@prot.read_type({:type => Thrift::Types::BYTE})
|
133
138
|
@prot.read_type({:type => Thrift::Types::I16})
|
@@ -144,13 +149,13 @@ describe 'BaseProtocol' do
|
|
144
149
|
end
|
145
150
|
|
146
151
|
it "should skip the basic types" do
|
147
|
-
@prot.
|
148
|
-
@prot.
|
149
|
-
@prot.
|
150
|
-
@prot.
|
151
|
-
@prot.
|
152
|
-
@prot.
|
153
|
-
@prot.
|
152
|
+
expect(@prot).to receive(:read_bool).ordered
|
153
|
+
expect(@prot).to receive(:read_byte).ordered
|
154
|
+
expect(@prot).to receive(:read_i16).ordered
|
155
|
+
expect(@prot).to receive(:read_i32).ordered
|
156
|
+
expect(@prot).to receive(:read_i64).ordered
|
157
|
+
expect(@prot).to receive(:read_double).ordered
|
158
|
+
expect(@prot).to receive(:read_string).ordered
|
154
159
|
@prot.skip(Thrift::Types::BOOL)
|
155
160
|
@prot.skip(Thrift::Types::BYTE)
|
156
161
|
@prot.skip(Thrift::Types::I16)
|
@@ -158,52 +163,51 @@ describe 'BaseProtocol' do
|
|
158
163
|
@prot.skip(Thrift::Types::I64)
|
159
164
|
@prot.skip(Thrift::Types::DOUBLE)
|
160
165
|
@prot.skip(Thrift::Types::STRING)
|
161
|
-
@prot.skip(Thrift::Types::STOP) # should do absolutely nothing
|
162
166
|
end
|
163
167
|
|
164
168
|
it "should skip structs" do
|
165
169
|
real_skip = @prot.method(:skip)
|
166
|
-
@prot.
|
167
|
-
@prot.
|
170
|
+
expect(@prot).to receive(:read_struct_begin).ordered
|
171
|
+
expect(@prot).to receive(:read_field_begin).exactly(4).times.and_return(
|
168
172
|
['field 1', Thrift::Types::STRING, 1],
|
169
173
|
['field 2', Thrift::Types::I32, 2],
|
170
174
|
['field 3', Thrift::Types::MAP, 3],
|
171
175
|
[nil, Thrift::Types::STOP, 0]
|
172
176
|
)
|
173
|
-
@prot.
|
174
|
-
@prot.
|
175
|
-
@prot.
|
176
|
-
@prot.
|
177
|
+
expect(@prot).to receive(:read_field_end).exactly(3).times
|
178
|
+
expect(@prot).to receive(:read_string).exactly(3).times
|
179
|
+
expect(@prot).to receive(:read_i32).ordered
|
180
|
+
expect(@prot).to receive(:read_map_begin).ordered.and_return([Thrift::Types::STRING, Thrift::Types::STRING, 1])
|
177
181
|
# @prot.should_receive(:read_string).exactly(2).times
|
178
|
-
@prot.
|
179
|
-
@prot.
|
182
|
+
expect(@prot).to receive(:read_map_end).ordered
|
183
|
+
expect(@prot).to receive(:read_struct_end).ordered
|
180
184
|
real_skip.call(Thrift::Types::STRUCT)
|
181
185
|
end
|
182
186
|
|
183
187
|
it "should skip maps" do
|
184
188
|
real_skip = @prot.method(:skip)
|
185
|
-
@prot.
|
186
|
-
@prot.
|
187
|
-
@prot.
|
188
|
-
@prot.
|
189
|
-
@prot.
|
190
|
-
@prot.
|
189
|
+
expect(@prot).to receive(:read_map_begin).ordered.and_return([Thrift::Types::STRING, Thrift::Types::STRUCT, 1])
|
190
|
+
expect(@prot).to receive(:read_string).ordered
|
191
|
+
expect(@prot).to receive(:read_struct_begin).ordered.and_return(["some_struct"])
|
192
|
+
expect(@prot).to receive(:read_field_begin).ordered.and_return([nil, Thrift::Types::STOP, nil]);
|
193
|
+
expect(@prot).to receive(:read_struct_end).ordered
|
194
|
+
expect(@prot).to receive(:read_map_end).ordered
|
191
195
|
real_skip.call(Thrift::Types::MAP)
|
192
196
|
end
|
193
197
|
|
194
198
|
it "should skip sets" do
|
195
199
|
real_skip = @prot.method(:skip)
|
196
|
-
@prot.
|
197
|
-
@prot.
|
198
|
-
@prot.
|
200
|
+
expect(@prot).to receive(:read_set_begin).ordered.and_return([Thrift::Types::I64, 9])
|
201
|
+
expect(@prot).to receive(:read_i64).ordered.exactly(9).times
|
202
|
+
expect(@prot).to receive(:read_set_end)
|
199
203
|
real_skip.call(Thrift::Types::SET)
|
200
204
|
end
|
201
205
|
|
202
206
|
it "should skip lists" do
|
203
207
|
real_skip = @prot.method(:skip)
|
204
|
-
@prot.
|
205
|
-
@prot.
|
206
|
-
@prot.
|
208
|
+
expect(@prot).to receive(:read_list_begin).ordered.and_return([Thrift::Types::DOUBLE, 11])
|
209
|
+
expect(@prot).to receive(:read_double).ordered.exactly(11).times
|
210
|
+
expect(@prot).to receive(:read_list_end)
|
207
211
|
real_skip.call(Thrift::Types::LIST)
|
208
212
|
end
|
209
213
|
end
|
@@ -211,7 +215,11 @@ describe 'BaseProtocol' do
|
|
211
215
|
describe Thrift::BaseProtocolFactory do
|
212
216
|
it "should raise NotImplementedError" do
|
213
217
|
# returning nil since Protocol is just an abstract class
|
214
|
-
|
218
|
+
expect {Thrift::BaseProtocolFactory.new.get_protocol(double("MockTransport"))}.to raise_error(NotImplementedError)
|
219
|
+
end
|
220
|
+
|
221
|
+
it "should provide a reasonable to_s" do
|
222
|
+
expect(Thrift::BaseProtocolFactory.new.to_s).to eq("base")
|
215
223
|
end
|
216
224
|
end
|
217
225
|
end
|