[trafficserver] branch master updated: Removes disabled test; case is covered elsewhere

[am...@apache.org]

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

amc 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 b47de39  Removes disabled test; case is covered elsewhere
b47de39 is described below

commit b47de3921185cae38ff0f6b6fc60a6cf60a731e2
Author: Derek Dagit <de...@yahoo-inc.com>
AuthorDate: Tue Jun 13 14:37:47 2017 +0000

    Removes disabled test; case is covered elsewhere
    
    This test was disabled. Its case is now covered by http204_response.test.py.
---
 .../body_factory/custom_response.test.py           | 63 ----------------------
 1 file changed, 63 deletions(-)

diff --git a/tests/gold_tests/body_factory/custom_response.test.py b/tests/gold_tests/body_factory/custom_response.test.py
deleted file mode 100644
index 42c22c6..0000000
--- a/tests/gold_tests/body_factory/custom_response.test.py
+++ /dev/null
@@ -1,63 +0,0 @@
-'''
-'''
-#  Licensed to the Apache Software Foundation (ASF) under one
-#  or more contributor license agreements.  See the NOTICE file
-#  distributed with this work for additional information
-#  regarding copyright ownership.  The ASF licenses this file
-#  to you under the Apache License, Version 2.0 (the
-#  "License"); you may not use this file except in compliance
-#  with the License.  You may obtain a copy of the License at
-#
-#      http://www.apache.org/licenses/LICENSE-2.0
-#
-#  Unless required by applicable law or agreed to in writing, software
-#  distributed under the License is distributed on an "AS IS" BASIS,
-#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-#  See the License for the specific language governing permissions and
-#  limitations under the License.
-
-import os
-
-Test.Summary = '''
-Tests for custom reponse body
-'''
-
-# this test currently fails and it should not
-Test.SkipIf(Condition.true("This test fails at the moment as is turned off"))
-Test.SkipUnless(Condition.HasProgram("curl", "Curl need to be installed on system for this test to work"))
-
-ts = Test.MakeATSProcess("ts")
-ts.Disk.records_config.update({
-    'proxy.config.body_factory.enable_customizations': 3,  # enable domain specific body factory
-})
-ts.Disk.remap_config.AddLine(
-    'map / http://www.linkedin.com/ @action=deny'
-)
-
-
-domain_directory = ['www.linkedin.com', '127.0.0.1', 'www.foobar.net']
-body_factory_dir=ts.Variables.BODY_FACTORY_TEMPLATE_DIR
-# for each domain
-set = False
-for directory_item in domain_directory:
-    # write out a files with some content for Traffic server for given domain
-    ts.Disk.File(os.path.join(body_factory_dir, directory_item, "access#denied")).\
-        WriteOn("{0} 44 Not 89 found".format(directory_item))
-
-    ts.Disk.File(os.path.join(body_factory_dir, directory_item, ".body_factory_info")).\
-        WriteOn("")
-    # make a test run for a given domain
-    tr = Test.AddTestRun("Test domain {0}".format(directory_item))
-    if not set:
-        # Start the ATS process for first test run
-        tr.Processes.Default.StartBefore(Test.Processes.ts)
-        set = True
-        tr.StillRunningAfter = ts
-    else:
-        # test that ats is still running before and after
-        tr.StillRunningBefore = ts
-        tr.StillRunningAfter = ts
-
-    tr.Processes.Default.Command = "curl --proxy 127.0.0.1:{1} {0}".format(directory_item, ts.Variables.port)
-    tr.Processes.Default.ReturnCode = 0
-    tr.Streams.All = Testers.ContainsExpression("{0} Not found".format(directory_item), "should contain custom data")

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].