David Reiss | ea2cba8 | 2009-03-30 21:35:00 +0000 | [diff] [blame] | 1 | # |
| 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 | # |
| 19 | |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 20 | require 'spec_helper' |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 21 | |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 22 | describe 'BaseProtocol' do |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 23 | |
| 24 | before(:each) do |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 25 | @trans = mock("MockTransport") |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 26 | @prot = Thrift::BaseProtocol.new(@trans) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 27 | end |
| 28 | |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 29 | describe Thrift::BaseProtocol do |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 30 | # most of the methods are stubs, so we can ignore them |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 31 | |
James E. King III | 9aaf295 | 2018-03-20 15:06:08 -0400 | [diff] [blame^] | 32 | it "should provide a reasonable to_s" do |
| 33 | @trans.should_receive(:to_s).once.and_return("trans") |
| 34 | @prot.to_s.should == "trans" |
| 35 | end |
| 36 | |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 37 | it "should make trans accessible" do |
| 38 | @prot.trans.should eql(@trans) |
| 39 | end |
| 40 | |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 41 | it 'should write out a field nicely (deprecated write_field signature)' do |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 42 | @prot.should_receive(:write_field_begin).with('field', 'type', 'fid').ordered |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 43 | @prot.should_receive(:write_type).with({:name => 'field', :type => 'type'}, 'value').ordered |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 44 | @prot.should_receive(:write_field_end).ordered |
| 45 | @prot.write_field('field', 'type', 'fid', 'value') |
| 46 | end |
| 47 | |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 48 | it 'should write out a field nicely' do |
| 49 | @prot.should_receive(:write_field_begin).with('field', 'type', 'fid').ordered |
| 50 | @prot.should_receive(:write_type).with({:name => 'field', :type => 'type', :binary => false}, 'value').ordered |
| 51 | @prot.should_receive(:write_field_end).ordered |
| 52 | @prot.write_field({:name => 'field', :type => 'type', :binary => false}, 'fid', 'value') |
| 53 | end |
| 54 | |
| 55 | it 'should write out the different types (deprecated write_type signature)' do |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 56 | @prot.should_receive(:write_bool).with('bool').ordered |
| 57 | @prot.should_receive(:write_byte).with('byte').ordered |
| 58 | @prot.should_receive(:write_double).with('double').ordered |
| 59 | @prot.should_receive(:write_i16).with('i16').ordered |
| 60 | @prot.should_receive(:write_i32).with('i32').ordered |
| 61 | @prot.should_receive(:write_i64).with('i64').ordered |
| 62 | @prot.should_receive(:write_string).with('string').ordered |
| 63 | struct = mock('Struct') |
| 64 | struct.should_receive(:write).with(@prot).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 65 | @prot.write_type(Thrift::Types::BOOL, 'bool') |
| 66 | @prot.write_type(Thrift::Types::BYTE, 'byte') |
| 67 | @prot.write_type(Thrift::Types::DOUBLE, 'double') |
| 68 | @prot.write_type(Thrift::Types::I16, 'i16') |
| 69 | @prot.write_type(Thrift::Types::I32, 'i32') |
| 70 | @prot.write_type(Thrift::Types::I64, 'i64') |
| 71 | @prot.write_type(Thrift::Types::STRING, 'string') |
| 72 | @prot.write_type(Thrift::Types::STRUCT, struct) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 73 | # all other types are not implemented |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 74 | [Thrift::Types::STOP, Thrift::Types::VOID, Thrift::Types::MAP, Thrift::Types::SET, Thrift::Types::LIST].each do |type| |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 75 | expect { @prot.write_type(type, type.to_s) }.to raise_error(NotImplementedError) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 76 | end |
| 77 | end |
| 78 | |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 79 | it 'should write out the different types' do |
| 80 | @prot.should_receive(:write_bool).with('bool').ordered |
| 81 | @prot.should_receive(:write_byte).with('byte').ordered |
| 82 | @prot.should_receive(:write_double).with('double').ordered |
| 83 | @prot.should_receive(:write_i16).with('i16').ordered |
| 84 | @prot.should_receive(:write_i32).with('i32').ordered |
| 85 | @prot.should_receive(:write_i64).with('i64').ordered |
| 86 | @prot.should_receive(:write_string).with('string').ordered |
| 87 | @prot.should_receive(:write_binary).with('binary').ordered |
| 88 | struct = mock('Struct') |
| 89 | struct.should_receive(:write).with(@prot).ordered |
| 90 | @prot.write_type({:type => Thrift::Types::BOOL}, 'bool') |
| 91 | @prot.write_type({:type => Thrift::Types::BYTE}, 'byte') |
| 92 | @prot.write_type({:type => Thrift::Types::DOUBLE}, 'double') |
| 93 | @prot.write_type({:type => Thrift::Types::I16}, 'i16') |
| 94 | @prot.write_type({:type => Thrift::Types::I32}, 'i32') |
| 95 | @prot.write_type({:type => Thrift::Types::I64}, 'i64') |
| 96 | @prot.write_type({:type => Thrift::Types::STRING}, 'string') |
| 97 | @prot.write_type({:type => Thrift::Types::STRING, :binary => true}, 'binary') |
| 98 | @prot.write_type({:type => Thrift::Types::STRUCT}, struct) |
| 99 | # all other types are not implemented |
| 100 | [Thrift::Types::STOP, Thrift::Types::VOID, Thrift::Types::MAP, Thrift::Types::SET, Thrift::Types::LIST].each do |type| |
| 101 | expect { @prot.write_type({:type => type}, type.to_s) }.to raise_error(NotImplementedError) |
| 102 | end |
| 103 | end |
| 104 | |
| 105 | it 'should read the different types (deprecated read_type signature)' do |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 106 | @prot.should_receive(:read_bool).ordered |
| 107 | @prot.should_receive(:read_byte).ordered |
| 108 | @prot.should_receive(:read_i16).ordered |
| 109 | @prot.should_receive(:read_i32).ordered |
| 110 | @prot.should_receive(:read_i64).ordered |
| 111 | @prot.should_receive(:read_double).ordered |
| 112 | @prot.should_receive(:read_string).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 113 | @prot.read_type(Thrift::Types::BOOL) |
| 114 | @prot.read_type(Thrift::Types::BYTE) |
| 115 | @prot.read_type(Thrift::Types::I16) |
| 116 | @prot.read_type(Thrift::Types::I32) |
| 117 | @prot.read_type(Thrift::Types::I64) |
| 118 | @prot.read_type(Thrift::Types::DOUBLE) |
| 119 | @prot.read_type(Thrift::Types::STRING) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 120 | # all other types are not implemented |
Roger Meier | 19dbbef | 2012-12-27 01:24:20 +0100 | [diff] [blame] | 121 | [Thrift::Types::STOP, Thrift::Types::VOID, Thrift::Types::MAP, |
| 122 | Thrift::Types::SET, Thrift::Types::LIST, Thrift::Types::STRUCT].each do |type| |
| 123 | expect { @prot.read_type(type) }.to raise_error(NotImplementedError) |
| 124 | end |
| 125 | end |
| 126 | |
| 127 | it 'should read the different types' do |
| 128 | @prot.should_receive(:read_bool).ordered |
| 129 | @prot.should_receive(:read_byte).ordered |
| 130 | @prot.should_receive(:read_i16).ordered |
| 131 | @prot.should_receive(:read_i32).ordered |
| 132 | @prot.should_receive(:read_i64).ordered |
| 133 | @prot.should_receive(:read_double).ordered |
| 134 | @prot.should_receive(:read_string).ordered |
| 135 | @prot.should_receive(:read_binary).ordered |
| 136 | @prot.read_type({:type => Thrift::Types::BOOL}) |
| 137 | @prot.read_type({:type => Thrift::Types::BYTE}) |
| 138 | @prot.read_type({:type => Thrift::Types::I16}) |
| 139 | @prot.read_type({:type => Thrift::Types::I32}) |
| 140 | @prot.read_type({:type => Thrift::Types::I64}) |
| 141 | @prot.read_type({:type => Thrift::Types::DOUBLE}) |
| 142 | @prot.read_type({:type => Thrift::Types::STRING}) |
| 143 | @prot.read_type({:type => Thrift::Types::STRING, :binary => true}) |
| 144 | # all other types are not implemented |
| 145 | [Thrift::Types::STOP, Thrift::Types::VOID, Thrift::Types::MAP, |
| 146 | Thrift::Types::SET, Thrift::Types::LIST, Thrift::Types::STRUCT].each do |type| |
| 147 | expect { @prot.read_type({:type => type}) }.to raise_error(NotImplementedError) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 148 | end |
| 149 | end |
| 150 | |
| 151 | it "should skip the basic types" do |
| 152 | @prot.should_receive(:read_bool).ordered |
| 153 | @prot.should_receive(:read_byte).ordered |
| 154 | @prot.should_receive(:read_i16).ordered |
| 155 | @prot.should_receive(:read_i32).ordered |
| 156 | @prot.should_receive(:read_i64).ordered |
| 157 | @prot.should_receive(:read_double).ordered |
| 158 | @prot.should_receive(:read_string).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 159 | @prot.skip(Thrift::Types::BOOL) |
| 160 | @prot.skip(Thrift::Types::BYTE) |
| 161 | @prot.skip(Thrift::Types::I16) |
| 162 | @prot.skip(Thrift::Types::I32) |
| 163 | @prot.skip(Thrift::Types::I64) |
| 164 | @prot.skip(Thrift::Types::DOUBLE) |
| 165 | @prot.skip(Thrift::Types::STRING) |
| 166 | @prot.skip(Thrift::Types::STOP) # should do absolutely nothing |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 167 | end |
| 168 | |
| 169 | it "should skip structs" do |
| 170 | real_skip = @prot.method(:skip) |
| 171 | @prot.should_receive(:read_struct_begin).ordered |
| 172 | @prot.should_receive(:read_field_begin).exactly(4).times.and_return( |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 173 | ['field 1', Thrift::Types::STRING, 1], |
| 174 | ['field 2', Thrift::Types::I32, 2], |
| 175 | ['field 3', Thrift::Types::MAP, 3], |
| 176 | [nil, Thrift::Types::STOP, 0] |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 177 | ) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 178 | @prot.should_receive(:read_field_end).exactly(3).times |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 179 | @prot.should_receive(:read_string).exactly(3).times |
| 180 | @prot.should_receive(:read_i32).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 181 | @prot.should_receive(:read_map_begin).ordered.and_return([Thrift::Types::STRING, Thrift::Types::STRING, 1]) |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 182 | # @prot.should_receive(:read_string).exactly(2).times |
| 183 | @prot.should_receive(:read_map_end).ordered |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 184 | @prot.should_receive(:read_struct_end).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 185 | real_skip.call(Thrift::Types::STRUCT) |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 186 | end |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 187 | |
| 188 | it "should skip maps" do |
| 189 | real_skip = @prot.method(:skip) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 190 | @prot.should_receive(:read_map_begin).ordered.and_return([Thrift::Types::STRING, Thrift::Types::STRUCT, 1]) |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 191 | @prot.should_receive(:read_string).ordered |
| 192 | @prot.should_receive(:read_struct_begin).ordered.and_return(["some_struct"]) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 193 | @prot.should_receive(:read_field_begin).ordered.and_return([nil, Thrift::Types::STOP, nil]); |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 194 | @prot.should_receive(:read_struct_end).ordered |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 195 | @prot.should_receive(:read_map_end).ordered |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 196 | real_skip.call(Thrift::Types::MAP) |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 197 | end |
| 198 | |
| 199 | it "should skip sets" do |
| 200 | real_skip = @prot.method(:skip) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 201 | @prot.should_receive(:read_set_begin).ordered.and_return([Thrift::Types::I64, 9]) |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 202 | @prot.should_receive(:read_i64).ordered.exactly(9).times |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 203 | @prot.should_receive(:read_set_end) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 204 | real_skip.call(Thrift::Types::SET) |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 205 | end |
| 206 | |
| 207 | it "should skip lists" do |
| 208 | real_skip = @prot.method(:skip) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 209 | @prot.should_receive(:read_list_begin).ordered.and_return([Thrift::Types::DOUBLE, 11]) |
Bryan Duxbury | c016628 | 2009-02-02 00:48:17 +0000 | [diff] [blame] | 210 | @prot.should_receive(:read_double).ordered.exactly(11).times |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 211 | @prot.should_receive(:read_list_end) |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 212 | real_skip.call(Thrift::Types::LIST) |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 213 | end |
| 214 | end |
| 215 | |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 216 | describe Thrift::BaseProtocolFactory do |
Bryan Duxbury | e8ae5d3 | 2009-03-24 05:23:52 +0000 | [diff] [blame] | 217 | it "should raise NotImplementedError" do |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 218 | # returning nil since Protocol is just an abstract class |
Jake Farrell | a87810f | 2012-09-28 01:59:04 +0000 | [diff] [blame] | 219 | lambda {Thrift::BaseProtocolFactory.new.get_protocol(mock("MockTransport"))}.should raise_error(NotImplementedError) |
Kevin Clark | 9db1c2e | 2008-06-18 01:06:42 +0000 | [diff] [blame] | 220 | end |
James E. King III | 9aaf295 | 2018-03-20 15:06:08 -0400 | [diff] [blame^] | 221 | |
| 222 | it "should provide a reasonable to_s" do |
| 223 | Thrift::BaseProtocolFactory.new.to_s.should == "base" |
| 224 | end |
Kevin Clark | dc39973 | 2008-06-18 01:06:15 +0000 | [diff] [blame] | 225 | end |
| 226 | end |