refactor: use Skip() for skipped tests

NBISweden · Nov 28, 2023 · c631842 · c631842
1 parent 78d0d11
commit c631842
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/upload/upload_test.go b/upload/upload_test.go
@@ -426,8 +426,8 @@ func (suite *TestSuite) TestRecursiveToDifferentTarget() {
 
 func (suite *TestSuite) TestUploadInvalidCharacters() {
 	// Filenames with :\?* can not be created on windows
-	if runtime.GOOS == "windows" {
-		return
+	if runtime.GOOS != "windows" {
+		suite.T().Skip("Skipping. Cannot create filenames with invalid characters on windows")
 	}
 
 	// Create a fake s3 backend