Re: [PATCH v2 0/3] MR1070: Add more tests for \Device\NamedPipe, \Device\NamedPipe\, and exotic pipe names

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/ntdll/tests/pipe.c:
> -
>       "Expected STATUS_OBJECT_NAME_NOT_FOUND opening %s, got %#lx\n",
> -
>       debugstr_wn(no_open_name.Buffer, no_open_name.Length / sizeof(WCHAR)), status);
> -
>    ok(client == NULL, "expected NULL handle, got %p\n", client);
> - }
> -
> - NtClose(pipe);
>
> +}
> -
>
> +static void test_exotic_pipe_names(void)
> +{
> - static const struct pipe_name_test_modifier modifiers[] = {
> -
>    { OBJ_CASE_INSENSITIVE },
> -
>    { OBJ_CASE_INSENSITIVE | OBJ_OPENIF },
> -
>    { 0 },
> -
>    { OBJ_OPENIF },
> - };

Why test attributes? The behaviour doesn't vary, and it's not obvious to me why OBJ_OPENIF or OBJ_CASE_INSENSITIVE would even potentially change behaviour either.