[PATCH 2/2] fsutil: Add support for creating hardlinks

[Arkadiusz Hiler]

On Tue, Aug 04, 2020 at 10:25:48AM -0500, Zebediah Figura wrote:
> > +static void test_hardlink(void)
> > +{
> > +    DWORD rc;
> > +    HANDLE hfile;
> > +    BY_HANDLE_FILE_INFORMATION info;
> > +
> > +    hfile = CreateFileA("file", GENERIC_WRITE, 0, NULL, CREATE_ALWAYS,
> > +                        FILE_ATTRIBUTE_NORMAL, NULL);
> > +    ok(hfile != INVALID_HANDLE_VALUE, "failed to create a file\n");
> > +
> > +    if (hfile == INVALID_HANDLE_VALUE)
> > +    {
> > +        skip("skipping hardlink tests\n");
> > +        return;
> > +    }
> 
> I'm personally of the opinion that these kinds of checks aren't
> particularly useful—if a test fails, that's already a problem and needs
> to be addressed.
<SNIP>
> > +START_TEST(fsutil)
> > +{
> > +    char tmpdir[MAX_PATH];
> > +
> > +    GetTempPathA(MAX_PATH, tmpdir);
> > +    SetCurrentDirectoryA(tmpdir);
> > +    trace("%s\n", tmpdir);
> 
> Is this trace() message doing anything particularly useful?

Those two were borrowed from other tests. I was checking how xcopy and
find are tested while looking for inspiration.

One can argue that if we haven't managed to create the source file
everything past that is just noise, but I also see the point of making
such failures more colorful and noticable :-)

As of that trace - I thought it may be useful for debugging to see where
the files are located for manual inspection upon failure.

Anyway, I went with all of your suggestion, as I am still developing my
own taste here.

-- 
Cheers,
Arek