diff --git a/tests/contest/contest/src/tests/fd_control/mod.rs b/tests/contest/contest/src/tests/fd_control/mod.rs
index a64d14b37..3d76fe105 100644
--- a/tests/contest/contest/src/tests/fd_control/mod.rs
+++ b/tests/contest/contest/src/tests/fd_control/mod.rs
@@ -102,6 +102,7 @@ pub fn get_fd_control_test() -> TestGroup {
     );
     let test_closes_fd = Test::new("closes_fd", Box::new(closes_fd_test));
     let test_pass_single_fd = Test::new("pass_single_fd", Box::new(pass_single_fd_test));
+    // adding separately as one is conditional test and others are normal
     test_group.add(vec![Box::new(test_only_stdio)]);
     test_group.add(vec![
         Box::new(test_closes_fd),
diff --git a/tests/contest/contest/src/tests/pidfile/pidfile_test.rs b/tests/contest/contest/src/tests/pidfile/pidfile_test.rs
index 2c3d9525d..08362a034 100644
--- a/tests/contest/contest/src/tests/pidfile/pidfile_test.rs
+++ b/tests/contest/contest/src/tests/pidfile/pidfile_test.rs
@@ -16,9 +16,6 @@ fn cleanup(id: &Uuid, bundle: &tempfile::TempDir) {
     delete_container(&str_id, bundle).unwrap().wait().unwrap();
 }
 
-// here we have to manually create and manage the container
-// as the test_inside_container does not provide a way to set the pid file argument
-// TODO: this comment is now out of date, the test just needs updating
 fn test_pidfile() -> TestResult {
     // create id for the container and pidfile
     let container_id = generate_uuid();