diff --git a/command_test.go b/command_test.go
index ccee031d..6e483a3e 100644
--- a/command_test.go
+++ b/command_test.go
@@ -1164,12 +1164,13 @@ func TestPersistentHooks(t *testing.T) {
 		t.Errorf("Unexpected error: %v", err)
 	}
 
-	// TODO: This test fails, but should not.
-	// Related to https://github.com/spf13/cobra/issues/252.
-	//
-	// if parentPersPreArgs != "one two" {
-	// 	t.Errorf("Expected parentPersPreArgs %q, got %q", "one two", parentPersPreArgs)
-	// }
+	// TODO: currently PersistenPreRun* defined in parent does not
+	// run if the matchin child subcommand has PersistenPreRun.
+	// If the behavior changes (https://github.com/spf13/cobra/issues/252)
+	// this test must be fixed.
+	if parentPersPreArgs != "" {
+		t.Errorf("Expected blank parentPersPreArgs, got %q", parentPersPreArgs)
+	}
 	if parentPreArgs != "" {
 		t.Errorf("Expected blank parentPreArgs, got %q", parentPreArgs)
 	}
@@ -1179,12 +1180,13 @@ func TestPersistentHooks(t *testing.T) {
 	if parentPostArgs != "" {
 		t.Errorf("Expected blank parentPostArgs, got %q", parentPostArgs)
 	}
-	// TODO: This test fails, but should not.
-	// Related to https://github.com/spf13/cobra/issues/252.
-	//
-	// if parentPersPostArgs != "one two" {
-	// 	t.Errorf("Expected parentPersPostArgs %q, got %q", "one two", parentPersPostArgs)
-	// }
+	// TODO: currently PersistenPostRun* defined in parent does not
+	// run if the matchin child subcommand has PersistenPostRun.
+	// If the behavior changes (https://github.com/spf13/cobra/issues/252)
+	// this test must be fixed.
+	if parentPersPostArgs != "" {
+		t.Errorf("Expected blank parentPersPostArgs, got %q", parentPersPostArgs)
+	}
 
 	if childPersPreArgs != "one two" {
 		t.Errorf("Expected childPersPreArgs %q, got %q", "one two", childPersPreArgs)