[trafficserver] branch master updated: Fix and reenable url_sig Au test.

[sh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

shinrich pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new ef24634  Fix and reenable url_sig Au test.
ef24634 is described below

commit ef246343dea2862a9191e89c71281c6f8072ba11
Author: Walter Karas <wk...@verizonmedia.com>
AuthorDate: Thu Jun 20 11:34:24 2019 -0500

    Fix and reenable url_sig Au test.
---
 tests/gold_tests/pluginTest/url_sig/url_sig.gold    | 9 ---------
 tests/gold_tests/pluginTest/url_sig/url_sig.test.py | 3 +--
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/tests/gold_tests/pluginTest/url_sig/url_sig.gold b/tests/gold_tests/pluginTest/url_sig/url_sig.gold
index 35f62da..256898b 100644
--- a/tests/gold_tests/pluginTest/url_sig/url_sig.gold
+++ b/tests/gold_tests/pluginTest/url_sig/url_sig.gold
@@ -1,21 +1,12 @@
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 403 Forbidden
-Authorization Denied*   Trying 127.0.0.1...
 < HTTP/1.1 200 OK
 < HTTP/1.1 200 OK
 < HTTP/1.1 200 OK
diff --git a/tests/gold_tests/pluginTest/url_sig/url_sig.test.py b/tests/gold_tests/pluginTest/url_sig/url_sig.test.py
index 3b83837..2f65cb3 100644
--- a/tests/gold_tests/pluginTest/url_sig/url_sig.test.py
+++ b/tests/gold_tests/pluginTest/url_sig/url_sig.test.py
@@ -25,7 +25,6 @@ Test url_sig plugin
 '''
 
 Test.ContinueOnFail = True
-Test.SkipIf(Condition.true("Test is temporarily turned off, to be fixed according to an incompatible plugin API change (PR #4964)"))
 
 # Skip if plugins not present.
 Test.SkipUnless(Condition.PluginExists('url_sig.so'))
@@ -100,7 +99,7 @@ ts.Disk.remap_config.AddLine(
 
 # Validation failure tests.
 
-LogTee = " 2>&1 | tee -a {}/url_sig_long.log".format(Test.RunDirectory)
+LogTee = " 2>&1 | grep '^<' | tee -a {}/url_sig_long.log".format(Test.RunDirectory)
 
 # Bad client / MD5 / P=101 / URL pristine / URL altered.
 #