test: Clean up test skipping

Author: eschnett

test/test_io_shared.jl

@@ -48,12 +48,8 @@ MPI.File.sync(fh)
 MPI.File.write_ordered(fh, fill(Int64(rank), rank+1))
 MPI.Barrier(comm)
 MPI.File.sync(fh)
-if Sys.isapple()
-    # TODO: this has to be fixed: https://github.com/JuliaParallel/MPI.jl/issues/555
-    @test_skip MPI.File.get_position_shared(fh) == sum(1:sz)
-else
-    @test MPI.File.get_position_shared(fh) == sum(1:sz)
-end
+# TODO: this has to be fixed: https://github.com/JuliaParallel/MPI.jl/issues/555
+@test MPI.File.get_position_shared(fh) == sum(1:sz) skip=Sys.isapple()
 
 MPI.File.seek_shared(fh, 0)
 @test MPI.File.get_position_shared(fh) == 0
@@ -68,12 +64,8 @@ MPI.File.read_ordered!(fh, buf)
 MPI.Barrier(comm)
 MPI.File.sync(fh)
 MPI.Barrier(comm)
-#TODO if Sys.iswindows()
-#TODO     # TODO: this has to be fixed: https://github.com/JuliaParallel/MPI.jl/issues/555
-#TODO     @test_skip MPI.File.get_position_shared(fh) == sum(1:sz)
-#TODO else
-    @test MPI.File.get_position_shared(fh) == sum(1:sz)
-#TODO end
+# TODO: this has to be fixed: https://github.com/JuliaParallel/MPI.jl/issues/555
+@test MPI.File.get_position_shared(fh) == sum(1:sz) skip=Sys.iswindows()
 
 MPI.File.set_view!(fh, 0, MPI.Datatype(UInt8), MPI.Datatype(UInt8))
 MPI.Barrier(comm)