biryani 0.0.1 → 0.0.3

data/spec/data_buffer_spec.rb

@@ -6,125 +6,125 @@ RSpec.describe DataBuffer do
       DataBuffer.new
     end
     let(:send_window1) do
-      Window.new
+      Window.new(65_535)
     end
     let(:streams_ctx1) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
       streams_ctx
     end
     it 'should take' do
-      expect(data_buffer1.take!(send_window1, streams_ctx1).length).to eq 0
-      expect(send_window1.length).to eq 2**16 - 1
-      expect(streams_ctx1[1].send_window.length).to eq 2**16 - 1
-      expect(streams_ctx1[2].send_window.length).to eq 2**16 - 1
+      expect(data_buffer1.take!(send_window1, streams_ctx1, 16_384).length).to eq 0
+      expect(send_window1.length).to eq 65_535
+      expect(streams_ctx1[1].send_window.length).to eq 65_535
+      expect(streams_ctx1[2].send_window.length).to eq 65_535
     end
 
     let(:data_buffer2) do
       data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 1, 'one', nil)
-      data_buffer << Frame::Data.new(false, 2, 'two', nil)
+      data_buffer.store(1, 'one')
+      data_buffer.store(2, 'two')
       data_buffer
     end
     let(:send_window2) do
-      Window.new
+      Window.new(65_535)
     end
     let(:streams_ctx2) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
       streams_ctx
     end
     it 'should take' do
-      datas = data_buffer2.take!(send_window2, streams_ctx2)
+      datas = data_buffer2.take!(send_window2, streams_ctx2, 16_384)
       expect(datas.length).to eq 2
       expect(datas.map(&:stream_id)).to eq [1, 2]
       expect(datas.map(&:data)).to eq %w[one two]
-      expect(send_window2.length).to eq 2**16 - 7
-      expect(streams_ctx2[1].send_window.length).to eq 2**16 - 4
-      expect(streams_ctx2[2].send_window.length).to eq 2**16 - 4
+      expect(send_window2.length).to eq 65_535 - 6
+      expect(streams_ctx2[1].send_window.length).to eq 65_535 - 3
+      expect(streams_ctx2[2].send_window.length).to eq 65_535 - 3
     end
 
     let(:data_buffer3) do
       data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 2, 'two', nil)
+      data_buffer.store(2, 'two')
       data_buffer
     end
     let(:send_window3) do
-      Window.new
+      Window.new(65_535)
     end
     let(:streams_ctx3) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
       streams_ctx
     end
     it 'should take' do
-      datas = data_buffer3.take!(send_window3, streams_ctx3)
+      datas = data_buffer3.take!(send_window3, streams_ctx3, 16_384)
       expect(datas.length).to eq 1
       expect(datas.first.stream_id).to eq 2
       expect(datas.first.data).to eq 'two'
-      expect(send_window3.length).to eq 2**16 - 4
-      expect(streams_ctx3[1].send_window.length).to eq 2**16 - 1
-      expect(streams_ctx3[2].send_window.length).to eq 2**16 - 4
+      expect(send_window3.length).to eq 65_535 - 3
+      expect(streams_ctx3[1].send_window.length).to eq 65_535
+      expect(streams_ctx3[2].send_window.length).to eq 65_535 - 3
     end
 
     let(:data_buffer4) do
       data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 1, 'one', nil)
-      data_buffer << Frame::Data.new(false, 2, 'two', nil)
+      data_buffer.store(1, 'one')
+      data_buffer.store(2, 'two')
       data_buffer
     end
     let(:send_window4) do
-      Window.new
+      Window.new(65_535)
     end
     let(:streams_ctx4) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx[1].send_window.consume!(2**16 - 1)
-      streams_ctx.new_context(2, do_nothing_proc)
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx[1].send_window.consume!(65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
       streams_ctx
     end
     it 'should take' do
-      datas = data_buffer4.take!(send_window4, streams_ctx4)
+      datas = data_buffer4.take!(send_window4, streams_ctx4, 16_384)
       expect(datas.length).to eq 1
       expect(datas.first.stream_id).to eq 2
       expect(datas.first.data).to eq 'two'
-      expect(send_window4.length).to eq 2**16 - 4
+      expect(send_window4.length).to eq 65_535 - 3
       expect(streams_ctx4[1].send_window.length).to eq 0
-      expect(streams_ctx4[2].send_window.length).to eq 2**16 - 4
+      expect(streams_ctx4[2].send_window.length).to eq 65_535 - 3
     end
 
     let(:data_buffer5) do
       data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 1, 'one', nil)
-      data_buffer << Frame::Data.new(false, 2, 'two', nil)
+      data_buffer.store(1, 'one')
+      data_buffer.store(2, 'two')
       data_buffer
     end
     let(:send_window5) do
-      send_window = Window.new
-      send_window.consume!(2**16 - 1)
+      send_window = Window.new(65_535)
+      send_window.consume!(65_535)
       send_window
     end
     let(:streams_ctx5) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
       streams_ctx
     end
     it 'should take' do
-      expect(data_buffer5.take!(send_window5, streams_ctx5).length).to eq 0
+      expect(data_buffer5.take!(send_window5, streams_ctx5, 16_384).length).to eq 0
       expect(send_window5.length).to eq 0
-      expect(streams_ctx5[1].send_window.length).to eq 2**16 - 1
-      expect(streams_ctx5[2].send_window.length).to eq 2**16 - 1
+      expect(streams_ctx5[1].send_window.length).to eq 65_535
+      expect(streams_ctx5[2].send_window.length).to eq 65_535
     end
   end
 
   context 'has?' do
     let(:data_buffer6) do
       data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 1, 'one', nil)
+      data_buffer.store(1, 'one')
       data_buffer
     end
     it 'should take' do

data/spec/hpack/decoder_spec.rb

@@ -9,8 +9,8 @@ RSpec.describe HPACK::Decoder do
     it 'should decode' do
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.3.1
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8286 8441 0f77 7777 2e65 7861 6d70 6c65
-   2e63 6f6d
+  8286 8441 0f77 7777 2e65 7861 6d70 6c65
+  2e63 6f6d
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -20,7 +20,7 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.3.2
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8286 84be 5808 6e6f 2d63 6163 6865
+  8286 84be 5808 6e6f 2d63 6163 6865
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -31,8 +31,8 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.3.3
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8287 85bf 400a 6375 7374 6f6d 2d6b 6579
-   0c63 7573 746f 6d2d 7661 6c75 65
+  8287 85bf 400a 6375 7374 6f6d 2d6b 6579
+  0c63 7573 746f 6d2d 7661 6c75 65
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -46,8 +46,8 @@ HEXDUMP
     it 'should decode' do
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.4.1
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8286 8441 8cf1 e3c2 e5f2 3a6b a0ab 90f4
-   ff
+  8286 8441 8cf1 e3c2 e5f2 3a6b a0ab 90f4
+  ff
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -57,7 +57,7 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.4.2
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8286 84be 5886 a8eb 1064 9cbf
+  8286 84be 5886 a8eb 1064 9cbf
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -68,8 +68,8 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.4.3
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   8287 85bf 4088 25a8 49e9 5ba9 7d7f 8925
-   a849 e95b b8e8 b4bf
+  8287 85bf 4088 25a8 49e9 5ba9 7d7f 8925
+  a849 e95b b8e8 b4bf
 HEXDUMP
                            )).to eq [
                              [':method', 'GET'],
@@ -83,11 +83,11 @@ HEXDUMP
     it 'should decode' do
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.5.1
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   4803 3330 3258 0770 7269 7661 7465 611d
-   4d6f 6e2c 2032 3120 4f63 7420 3230 3133
-   2032 303a 3133 3a32 3120 474d 546e 1768
-   7474 7073 3a2f 2f77 7777 2e65 7861 6d70
-   6c65 2e63 6f6d
+  4803 3330 3258 0770 7269 7661 7465 611d
+  4d6f 6e2c 2032 3120 4f63 7420 3230 3133
+  2032 303a 3133 3a32 3120 474d 546e 1768
+  7474 7073 3a2f 2f77 7777 2e65 7861 6d70
+  6c65 2e63 6f6d
 HEXDUMP
                            )).to eq [
                              [':status', '302'],
@@ -97,7 +97,7 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.5.2
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   4803 3330 37c1 c0bf
+  4803 3330 37c1 c0bf
 HEXDUMP
                            )).to eq [
                              [':status', '307'],
@@ -107,13 +107,13 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.5.3
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   88c1 611d 4d6f 6e2c 2032 3120 4f63 7420
-   3230 3133 2032 303a 3133 3a32 3220 474d
-   54c0 5a04 677a 6970 7738 666f 6f3d 4153
-   444a 4b48 514b 425a 584f 5157 454f 5049
-   5541 5851 5745 4f49 553b 206d 6178 2d61
-   6765 3d33 3630 303b 2076 6572 7369 6f6e
-   3d31
+  88c1 611d 4d6f 6e2c 2032 3120 4f63 7420
+  3230 3133 2032 303a 3133 3a32 3220 474d
+  54c0 5a04 677a 6970 7738 666f 6f3d 4153
+  444a 4b48 514b 425a 584f 5157 454f 5049
+  5541 5851 5745 4f49 553b 206d 6178 2d61
+  6765 3d33 3630 303b 2076 6572 7369 6f6e
+  3d31
 HEXDUMP
                            )).to eq [
                              [':status', '200'],
@@ -128,10 +128,10 @@ HEXDUMP
     it 'should decode' do
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.6.1
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   4882 6402 5885 aec3 771a 4b61 96d0 7abe
-   9410 54d4 44a8 2005 9504 0b81 66e0 82a6
-   2d1b ff6e 919d 29ad 1718 63c7 8f0b 97c8
-   e9ae 82ae 43d3
+  4882 6402 5885 aec3 771a 4b61 96d0 7abe
+  9410 54d4 44a8 2005 9504 0b81 66e0 82a6
+  2d1b ff6e 919d 29ad 1718 63c7 8f0b 97c8
+  e9ae 82ae 43d3
 HEXDUMP
                            )).to eq [
                              [':status', '302'],
@@ -141,7 +141,7 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.6.2
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   4883 640e ffc1 c0bf
+  4883 640e ffc1 c0bf
 HEXDUMP
                            )).to eq [
                              [':status', '307'],
@@ -151,11 +151,11 @@ HEXDUMP
                            ]
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.6.3
       expect(decoder.decode([<<HEXDUMP.split.join].pack('H*')
-   88c1 6196 d07a be94 1054 d444 a820 0595
-   040b 8166 e084 a62d 1bff c05a 839b d9ab
-   77ad 94e7 821d d7f2 e6c7 b335 dfdf cd5b
-   3960 d5af 2708 7f36 72c1 ab27 0fb5 291f
-   9587 3160 65c0 03ed 4ee5 b106 3d50 07
+  88c1 6196 d07a be94 1054 d444 a820 0595
+  040b 8166 e084 a62d 1bff c05a 839b d9ab
+  77ad 94e7 821d d7f2 e6c7 b335 dfdf cd5b
+  3960 d5af 2708 7f36 72c1 ab27 0fb5 291f
+  9587 3160 65c0 03ed 4ee5 b106 3d50 07
 HEXDUMP
                            )).to eq [
                              [':status', '200'],

data/spec/hpack/encoder_spec.rb

@@ -14,10 +14,10 @@ RSpec.describe HPACK::Encoder do
                               ['date', 'Mon, 21 Oct 2013 20:13:21 GMT'],
                               ['location', 'https://www.example.com']
                             ])).to eq [<<HEXDUMP.split.join].pack('H*')
-   4882 6402 5885 aec3 771a 4b61 96d0 7abe
-   9410 54d4 44a8 2005 9504 0b81 66e0 82a6
-   2d1b ff6e 919d 29ad 1718 63c7 8f0b 97c8
-   e9ae 82ae 43d3
+  4882 6402 5885 aec3 771a 4b61 96d0 7abe
+  9410 54d4 44a8 2005 9504 0b81 66e0 82a6
+  2d1b ff6e 919d 29ad 1718 63c7 8f0b 97c8
+  e9ae 82ae 43d3
 HEXDUMP
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.6.2
       expect(encoder.encode([
@@ -26,7 +26,7 @@ HEXDUMP
                               ['date', 'Mon, 21 Oct 2013 20:13:21 GMT'],
                               ['location', 'https://www.example.com']
                             ])).to eq [<<HEXDUMP.split.join].pack('H*')
-   4883 640e ffc1 c0bf
+  4883 640e ffc1 c0bf
 HEXDUMP
       # https://datatracker.ietf.org/doc/html/rfc7541#appendix-C.6.3
       expect(encoder.encode([
@@ -37,11 +37,11 @@ HEXDUMP
                               ['content-encoding', 'gzip'],
                               ['set-cookie', 'foo=ASDJKHQKBZXOQWEOPIUAXQWEOIU; max-age=3600; version=1']
                             ])).to eq [<<HEXDUMP.split.join].pack('H*')
-   88c1 6196 d07a be94 1054 d444 a820 0595
-   040b 8166 e084 a62d 1bff c05a 839b d9ab
-   77ad 94e7 821d d7f2 e6c7 b335 dfdf cd5b
-   3960 d5af 2708 7f36 72c1 ab27 0fb5 291f
-   9587 3160 65c0 03ed 4ee5 b106 3d50 07
+  88c1 6196 d07a be94 1054 d444 a820 0595
+  040b 8166 e084 a62d 1bff c05a 839b d9ab
+  77ad 94e7 821d d7f2 e6c7 b335 dfdf cd5b
+  3960 d5af 2708 7f36 72c1 ab27 0fb5 291f
+  9587 3160 65c0 03ed 4ee5 b106 3d50 07
 HEXDUMP
     end
   end

data/spec/hpack/fields_spec.rb

@@ -7,9 +7,9 @@ RSpec.describe HPACK::Fields do
     end
     it 'should decode' do
       expect(HPACK::Fields.decode([<<HEXDUMP.split.join].pack('H*'),
-   8286 8441 8a08 9d5c 0b81 70dc 79e7
-   9e40 8721 eaa8 a449 8f57 88ea 52d6
-   b0e8 3772 ff
+  8286 8441 8a08 9d5c 0b81 70dc 79e7
+  9e40 8721 eaa8 a449 8f57 88ea 52d6
+  b0e8 3772 ff
 HEXDUMP
                                   dynamic_table)).to eq [[':method', 'GET'], [':scheme', 'http'], [':path', '/'], [':authority', '127.0.0.1:8888'], ['connection', 'keep-alive']]
     end

data/spec/streams_context_spec.rb

@@ -0,0 +1,79 @@
+require_relative 'spec_helper'
+
+RSpec.describe StreamsContext do
+  context 'close_all' do
+    let(:streams_ctx) do
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
+      streams_ctx
+    end
+    it 'should close' do
+      streams_ctx.close_all
+      expect { streams_ctx[1].tx << nil }.to raise_error Ractor::ClosedError
+      expect { streams_ctx[2].tx << nil }.to raise_error Ractor::ClosedError
+    end
+  end
+
+  context 'close_all' do
+    let(:streams_ctx) do
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
+      streams_ctx
+    end
+    it 'should close' do
+      streams_ctx.close_all
+      expect { streams_ctx[1].tx << nil }.to raise_error Ractor::ClosedError
+      expect { streams_ctx[2].tx << nil }.to raise_error Ractor::ClosedError
+    end
+  end
+
+  context 'remove_closed' do
+    let(:streams_ctx1) do
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
+      streams_ctx
+    end
+    let(:data_buffer1) do
+      DataBuffer.new
+    end
+    it 'should remove' do
+      streams_ctx1.remove_closed(data_buffer1)
+      expect(streams_ctx1.length).to eq 2
+    end
+
+    let(:streams_ctx2) do
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
+      streams_ctx[2].state.close
+      streams_ctx
+    end
+    let(:data_buffer2) do
+      DataBuffer.new
+    end
+    it 'should remove' do
+      streams_ctx2.remove_closed(data_buffer2)
+      expect(streams_ctx2.length).to eq 2 # remain stream_id
+    end
+
+    let(:streams_ctx3) do
+      streams_ctx = StreamsContext.new(do_nothing_proc)
+      streams_ctx.new_context(1, 65_535, 65_535)
+      streams_ctx.new_context(2, 65_535, 65_535)
+      streams_ctx[2].state.close
+      streams_ctx
+    end
+    let(:data_buffer3) do
+      data_buffer = DataBuffer.new
+      data_buffer.store(2, 'two')
+      data_buffer
+    end
+    it 'should remove' do
+      streams_ctx3.remove_closed(data_buffer3)
+      expect(streams_ctx3.length).to eq 2
+    end
+  end
+end

data/spec/utils_spec.rb

@@ -0,0 +1,41 @@
+require_relative 'spec_helper'
+
+RSpec.describe Biryani do
+  let(:data) do
+    Frame::Data.new(false, 2, 'Hello, world!', 'Howdy!')
+  end
+  let(:connection_error) do
+    ConnectionError.new(ErrorCode::NO_ERROR, 'debug')
+  end
+  let(:stream_error) do
+    StreamError.new(ErrorCode::NO_ERROR, 1, 'debug')
+  end
+
+  context 'err?' do
+    it 'should be not error' do
+      expect(Biryani.err?(data)).to eq false
+    end
+
+    it 'should be error' do
+      expect(Biryani.err?(connection_error)).to be true
+    end
+
+    it 'should be error' do
+      expect(Biryani.err?(stream_error)).to be true
+    end
+  end
+
+  context 'unwrap' do
+    it 'should unwrap' do
+      expect(Biryani.unwrap(data, 0x01)).to eq data
+    end
+
+    it 'should unwrap' do
+      expect(Biryani.unwrap(connection_error, 0x01)).to be_kind_of Frame::Goaway
+    end
+
+    it 'should unwrap' do
+      expect(Biryani.unwrap(stream_error, 0x01)).to be_kind_of Frame::RstStream
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: biryani
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.3
 platform: ruby
 authors:
 - thekuwayama
@@ -75,19 +75,19 @@ files:
 - lib/biryani/stream.rb
 - lib/biryani/stream_error.rb
 - lib/biryani/streams_context.rb
+- lib/biryani/utils.rb
 - lib/biryani/version.rb
 - lib/biryani/window.rb
-- spec/connection/close_all_streams_spec.rb
 - spec/connection/handle_connection_window_update_spec.rb
+- spec/connection/handle_data_spec.rb
+- spec/connection/handle_headers_spec.rb
 - spec/connection/handle_ping_spec.rb
 - spec/connection/handle_rst_stream_spec.rb
 - spec/connection/handle_settings_spec.rb
 - spec/connection/handle_stream_window_update_spec.rb
 - spec/connection/read_http2_magic_spec.rb
-- spec/connection/remove_closed_streams_spec.rb
 - spec/connection/send_spec.rb
 - spec/connection/transition_state_send_spec.rb
-- spec/connection/unwrap_spec.rb
 - spec/data_buffer_spec.rb
 - spec/frame/continuation_spec.rb
 - spec/frame/data_spec.rb
@@ -109,6 +109,8 @@ files:
 - spec/hpack/string_spec.rb
 - spec/http_request_builder_spec.rb
 - spec/spec_helper.rb
+- spec/streams_context_spec.rb
+- spec/utils_spec.rb
 homepage: https://github.com/thekuwayama/biryani
 licenses:
 - MIT
@@ -131,17 +133,16 @@ rubygems_version: 4.0.3
 specification_version: 4
 summary: An HTTP/2 server implemented using Ruby Ractor
 test_files:
-- spec/connection/close_all_streams_spec.rb
 - spec/connection/handle_connection_window_update_spec.rb
+- spec/connection/handle_data_spec.rb
+- spec/connection/handle_headers_spec.rb
 - spec/connection/handle_ping_spec.rb
 - spec/connection/handle_rst_stream_spec.rb
 - spec/connection/handle_settings_spec.rb
 - spec/connection/handle_stream_window_update_spec.rb
 - spec/connection/read_http2_magic_spec.rb
-- spec/connection/remove_closed_streams_spec.rb
 - spec/connection/send_spec.rb
 - spec/connection/transition_state_send_spec.rb
-- spec/connection/unwrap_spec.rb
 - spec/data_buffer_spec.rb
 - spec/frame/continuation_spec.rb
 - spec/frame/data_spec.rb
@@ -163,3 +164,5 @@ test_files:
 - spec/hpack/string_spec.rb
 - spec/http_request_builder_spec.rb
 - spec/spec_helper.rb
+- spec/streams_context_spec.rb
+- spec/utils_spec.rb

data/spec/connection/close_all_streams_spec.rb

@@ -1,17 +0,0 @@
-require_relative '../spec_helper'
-
-RSpec.describe Connection do
-  context 'close_all_streams' do
-    let(:streams_ctx) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
-      streams_ctx
-    end
-    it 'should close' do
-      Connection.close_all_streams(streams_ctx)
-      expect { streams_ctx[1].tx << nil }.to raise_error Ractor::ClosedError
-      expect { streams_ctx[2].tx << nil }.to raise_error Ractor::ClosedError
-    end
-  end
-end

data/spec/connection/remove_closed_streams_spec.rb

@@ -1,51 +0,0 @@
-require_relative '../spec_helper'
-
-RSpec.describe Connection do
-  context 'remove_closed_streams' do
-    let(:streams_ctx1) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
-      streams_ctx
-    end
-    let(:data_buffer1) do
-      DataBuffer.new
-    end
-    it 'should close' do
-      Connection.remove_closed_streams(streams_ctx1, data_buffer1)
-      expect(streams_ctx1.length).to eq 2
-    end
-
-    let(:streams_ctx2) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
-      streams_ctx[2].state.close
-      streams_ctx
-    end
-    let(:data_buffer2) do
-      DataBuffer.new
-    end
-    it 'should close' do
-      Connection.remove_closed_streams(streams_ctx2, data_buffer2)
-      expect(streams_ctx2.length).to eq 2 # remain stream_id
-    end
-
-    let(:streams_ctx3) do
-      streams_ctx = StreamsContext.new
-      streams_ctx.new_context(1, do_nothing_proc)
-      streams_ctx.new_context(2, do_nothing_proc)
-      streams_ctx[2].state.close
-      streams_ctx
-    end
-    let(:data_buffer3) do
-      data_buffer = DataBuffer.new
-      data_buffer << Frame::Data.new(false, 2, 'two', nil)
-      data_buffer
-    end
-    it 'should close' do
-      Connection.remove_closed_streams(streams_ctx3, data_buffer3)
-      expect(streams_ctx3.length).to eq 2
-    end
-  end
-end

data/spec/connection/unwrap_spec.rb

@@ -1,28 +0,0 @@
-require_relative '../spec_helper'
-
-RSpec.describe Connection do
-  context 'unwrap' do
-    let(:data) do
-      Frame::Data.new(false, 2, 'Hello, world!', 'Howdy!')
-    end
-    it 'should ensure' do
-      expect(Connection.unwrap(data, 0x01)).to eq data
-    end
-
-    let(:connection_error) do
-      ConnectionError.new(ErrorCode::NO_ERROR, 'debug')
-    end
-    it 'should ensure' do
-      frame = Connection.unwrap(connection_error, 0x01)
-      expect(frame).to be_kind_of Frame::Goaway
-    end
-
-    let(:stream_error) do
-      StreamError.new(ErrorCode::NO_ERROR, 0x01, 'debug')
-    end
-    it 'should ensure' do
-      frame = Connection.unwrap(stream_error, 0x01)
-      expect(frame).to be_kind_of Frame::RstStream
-    end
-  end
-end