[libcxx] [test] Split the file_time_type synopsis test

[mstorsjo]

Summary

Split the resolution check to a separate test, which is marked as unsupported on windows.

On windows (both with MS STL and libstdc++), the file time has 100 ns resolution; the standard doesn't mandate a specific resolution.

[ldionne]

libcxx/test/std/input.output/filesystems/fs.filesystem.synopsis/file_time_type.pass.cpp

35

I would rather you mark the whole test as // UNSUPPORTED on Windows. Or split this part of the test into another test, and mark that as unsupported.

I'm trying to get rid of platform specific #ifs in the test suite when possible. It gives a better overview of what is and isn't supported when running the test suite on various platforms.

[mstorsjo]

Split the test into two, instead of adding ifdefs.

[ldionne]

LGTM with the suggested change.

libcxx/test/std/input.output/filesystems/fs.filesystem.synopsis/file_time_type_resolution.pass.cpp

24 ↗

(On Diff #298718)

Can you rename to file_time_type_resolution.compile.pass.cpp and just use:

using namespace fs;
using Dur = file_time_type::duration;
using Period = Dur::period;
ASSERT_SAME_TYPE(Period, std::nano);

No need for main or test_time_point_resolution_and_range().

libcxx/test/std/input.output/filesystems/fs.filesystem.synopsis/file_time_type.pass.cpp

void test_trivial_clock() {
   using Clock = file_time_type::clock;
   ASSERT_NOEXCEPT(Clock::now());
   ASSERT_SAME_TYPE(decltype(Clock::now()), file_time_type);
   ASSERT_SAME_TYPE(Clock::time_point, file_time_type);
   volatile auto* odr_use = &Clock::is_steady;
   ((void)odr_use);
 }
 
+#ifndef _WIN32
 void test_time_point_resolution_and_range() {
   using namespace fs;
   using Dur = file_time_type::duration;
   using Period = Dur::period;
   ASSERT_SAME_TYPE(Period, std::nano);
 }
+#endif
 
 int main(int, char**) {
   test_trivial_clock();
+#ifndef _WIN32
   test_time_point_resolution_and_range();
+#endif
 
   return 0;
 }