trafficserver-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alan Carroll <>
Subject Re: Location for test plugins
Date Thu, 08 Jun 2017 21:25:14 GMT
I agree with Jason.

On Thursday, June 8, 2017, 3:44:53 PM CDT, Jason Kenny <>

given some time reflex on this.
I think I rather have the directory for plugins-for-testing to be under:
and the location for the test for a given plugin to be under
tests/plugins/<plugin-name>or tests/gold_tests/plugins 

The main point is the plugins-for-testing should be under the location where other code exists
for supporting testing... which is currently in tests/tools

On ‎Friday‎, ‎June‎ ‎2‎, ‎2017‎ ‎03‎:‎19‎:‎52‎ ‎PM‎ ‎CDT,
Jason Kenny <> wrote:

I am fine with tests/plugins.
We just need to make sure the code is documented to say what it tests. Some cases will be
one-off plugins while other will be reusable

On Friday, June 2, 2017, 3:17:52 PM CDT, Steven R. Feltner <> wrote:

+1 for tests/plugins.  It should cut down on the confusion of someone trying to use it as
a ‘real’ plugin versus putting it in the plugins/* directory structure.

On 6/2/17, 2:32 PM, "Leif Hedstrom" <> wrote:

    +1 for tests/plugins as well.
    > On Jun 2, 2017, at 11:30 AM, Bryan Call <> wrote:
    > +1 for tests/plugins
    > -Bryan
    >> On Jun 2, 2017, at 9:50 AM, Alan Carroll <>
    >> PR 2042 has a special plugin in it which is used only for performing tests.
This is just the first of something that is going to occur repeatedly in the future. These
plugins are not written to be general purpose but will be effectively custom built specifically
for the test. We should decide now how we want that structured. Some options are
    >> * Put them in example.* Put them in tests/plugins* Put them in the directory
containing the test.
    >> My personal preference is the second. I think these plugins are unlikely to
be good example plugins and they certainly aren't production ready. On the other hand, having
a single place to look for test plugins is in my view useful and better than scattering them
in different sub directories under tests.

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message