Re: Tentative tests.

On Tue, May 23, 2017 at 5:14 PM, Anne van Kesteren <annevk@annevk.nl> wrote:

> On Tue, May 23, 2017 at 3:10 PM, Mike West <mkwst@google.com> wrote:
> > 1.  Put the tests in a new file with a `-tentative.*` suffix (e.g.
> > `//content-security-policy/script-src/new-test-tentative.sub.html`).
>
> This doesn't mix with -manual. Not sure if that's a problem, but
> dot-delimiting might also be okay?
>

Sure. `.tentative.*` is fine by me.


> > 2.  Put the tests in a `tentative` subdirectory of an existing suite
> (e.g.
> > `//content-security-policy/tentative/new-script-src-test.sub.html`).
>
> We don't assign meaning to directories elsewhere other than informally
> "resources" / "support". At least, as far as I know. I think I'd
> prefer just the one convention.


I like the directory simply for grouping. If there's a new feature with a
lot of tests, the suffixes won't sort together as a collected bunch of
tests that folks who haven't hopped on board with the feature should be
ignoring.

But if I'm the only one who likes that idea, I'm happy to drop it in favor
of the simplicity of a single approach.

-mike

Received on Tuesday, 23 May 2017 15:40:54 UTC