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',      } | 
