diff options
author | Steve Kemp <steve@steve.org.uk> | 2015-07-29 13:06:01 +0300 |
---|---|---|
committer | Steve Kemp <steve@steve.org.uk> | 2015-07-29 13:06:01 +0300 |
commit | b698db43443e32b8c71b19a23f4265a22e343985 (patch) | |
tree | 7199389ab50bb7d43e32fead7e2292086a320042 | |
parent | 487c229014a5d6234b01d163f1b449511725a861 (diff) |
Loosen teh grammar on tcp-tests.
In the past we needed to write:
must run tcp on 3306.
Now we can add the "port" to match the rest of the tests:
must run tcp on port 3306.
-rw-r--r-- | lib/custodian/protocoltest/tcp.rb | 7 | ||||
-rwxr-xr-x | t/test-custodian-testfactory.rb | 2 |
2 files changed, 7 insertions, 2 deletions
diff --git a/lib/custodian/protocoltest/tcp.rb b/lib/custodian/protocoltest/tcp.rb index 684452b..c50c144 100644 --- a/lib/custodian/protocoltest/tcp.rb +++ b/lib/custodian/protocoltest/tcp.rb @@ -68,10 +68,13 @@ module Custodian # # Save the port # + @port = nil + if line =~ /on\s+([0-9]+)/ @port = $1.dup - else - @port = nil + end + if line =~ /on\s+port\s+([0-9]+)/ + @port = $1.dup end # diff --git a/t/test-custodian-testfactory.rb b/t/test-custodian-testfactory.rb index 63cf6c1..f864544 100755 --- a/t/test-custodian-testfactory.rb +++ b/t/test-custodian-testfactory.rb @@ -98,6 +98,8 @@ class TestTestFactory < Test::Unit::TestCase "foo must run rEDIs otherwise 'alert'" => '6379', "foo must run rdp otherwise 'alert'" => '3389', "foo must run RDP otherwise 'alert'" => '3389', + "foo must run tcp on 22 otherwise 'alert'" => '22', + "foo must run tcp on port 22 otherwise 'alert'" => '22', "foo must run mysql on 33 otherwise 'alert'" => '33', } |