Skip to content

Commit 6330c42

Browse files
committed
Address some of @jvazquez-r7's spec feedback
1 parent 5b1b7c2 commit 6330c42

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

spec/lib/rex/proto/steam/message_spec.rb

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3,45 +3,45 @@
33
require 'rex/proto/steam/message'
44

55
describe Rex::Proto::Steam do
6-
subject do
6+
subject(:steam) do
77
mod = Module.new
88
mod.extend described_class
99
mod
1010
end
1111

1212
describe '#encode_message' do
13-
it 'should properly encode messages' do
14-
message = subject.encode_message('T', 'Test')
13+
it 'properly encodes messages' do
14+
message = steam.encode_message('T', 'Test')
1515
expect(message).to eq("\xFF\xFF\xFF\xFF\x54Test")
1616
end
1717
end
1818

1919
describe '#decode_message' do
20-
it 'should not decode overly short messages' do
21-
expect(subject.decode_message('foo')).to eq(nil)
20+
it 'does not decode overly short messages' do
21+
expect(steam.decode_message('foo')).to eq(nil)
2222
end
2323

24-
it 'should not decode unknown messages' do
25-
expect(subject.decode_message("\xFF\xFF\xFF\x01blahblahblah")).to eq(nil)
24+
it 'does not decode unknown messages' do
25+
expect(steam.decode_message("\xFF\xFF\xFF\x01blahblahblah")).to eq(nil)
2626
end
2727

28-
it 'should properly decode valid messages' do
29-
header, type, message = subject.decode_message("\xFF\xFF\xFF\xFF\x54Test")
28+
it 'properly decodes valid messages' do
29+
header, type, message = steam.decode_message("\xFF\xFF\xFF\xFF\x54Test")
3030
expect(header).to eq(Rex::Proto::Steam::UNFRAGMENTED_HEADER)
3131
expect(type).to eq(0x54)
3232
expect(message).to eq('Test')
3333
end
3434
end
3535

3636
describe '#a2s_info_decode' do
37-
it 'should extract a2s_info fields properly' do
37+
it 'extracts a2s_info fields properly' do
3838
expected_info = {
3939
version: 17, name: "-=THE BATTLEGROUNDS *HARDCORE*=-", map: "aoc_battleground",
4040
folder: "ageofchivalry", game_name: "Age of Chivalry", game_id: 17510,
4141
players: "22/32", bots: 0, game_version: "1.0.0.6", type: "Dedicated",
4242
environment: "Linux", visibility: "public", VAC: "secured"
4343
}
44-
actual_info = subject.a2s_info_decode(IO.read(File.join(File.dirname(__FILE__), 'steam_info.bin')))
44+
actual_info = steam.a2s_info_decode(IO.read(File.join(File.dirname(__FILE__), 'steam_info.bin')))
4545
expect(actual_info).to eq(expected_info)
4646
end
4747
end

0 commit comments

Comments
 (0)