fix: issue with marking non child tests as skipped

snaps/skip.go

@@ -58,8 +58,11 @@ func testSkipped(testID, runOnly string) bool {
 		return true
 	}
 
+	// testID form: Test.*/runName - 1
+	testName := strings.Split(testID, " - ")[0]
+
 	for _, name := range skippedTests.values {
-		if strings.HasPrefix(testID, name) {
+		if testName == name || strings.HasPrefix(testName, name+"/") {
 			return true
 		}
 	}

snaps/skip_test.go

@@ -172,6 +172,28 @@ func TestSkip(t *testing.T) {
 			},
 		)
 
+		t.Run("should not mark tests skipped if not not a child", func(t *testing.T) {
+			t.Cleanup(func() {
+				skippedTests = newSyncSlice()
+			})
+
+			runOnly := ""
+			mockT := MockTestingT{
+				mockSkipNow: func() {},
+				mockHelper:  func() {},
+				mockName: func() string {
+					return "Test"
+				},
+			}
+			// This is for populating skippedTests.values and following the normal flow
+			SkipNow(mockT)
+
+			Equal(t, true, testSkipped("Test", runOnly))
+			Equal(t, true, testSkipped("Test/chid", runOnly))
+			Equal(t, false, testSkipped("TestMock", runOnly))
+			Equal(t, false, testSkipped("TestMock/child", runOnly))
+		})
+
 		t.Run("should use regex match for runOnly", func(t *testing.T) {
 			Equal(t, false, testSkipped("MyTest - 1", "Test"))
 			Equal(t, true, testSkipped("MyTest - 1", "^Test"))