test: fix vacuous is_err() assertions in add_pr_comment and update_pr

src/safe_outputs/add_pr_comment.rs

@@ -551,8 +551,9 @@ mod tests {
             line: None,
             status: "active".to_string(),
         };
-        let result: Result<AddPrCommentResult, _> = params.try_into();
-        assert!(result.is_err());
+        let err: Result<AddPrCommentResult, _> = params.try_into();
+        let err = err.unwrap_err().to_string();
+        assert!(err.contains("pull_request_id must be positive"), "got: {err}");
     }
 
     #[test]
@@ -566,8 +567,9 @@ mod tests {
             line: None,
             status: "active".to_string(),
         };
-        let result: Result<AddPrCommentResult, _> = params.try_into();
-        assert!(result.is_err());
+        let err: Result<AddPrCommentResult, _> = params.try_into();
+        let err = err.unwrap_err().to_string();
+        assert!(err.contains("content must be at least 10 characters"), "got: {err}");
     }
 
     #[test]
@@ -581,8 +583,9 @@ mod tests {
             line: None,
             status: "active".to_string(),
         };
-        let result: Result<AddPrCommentResult, _> = params.try_into();
-        assert!(result.is_err());
+        let err: Result<AddPrCommentResult, _> = params.try_into();
+        let err = err.unwrap_err().to_string();
+        assert!(err.contains("pipeline command"), "got: {err}");
     }
 
     #[test]
@@ -626,8 +629,9 @@ mod tests {
             line: Some(10),
             status: "active".to_string(),
         };
-        let result: Result<AddPrCommentResult, _> = params.try_into();
-        assert!(result.is_err());
+        let err: Result<AddPrCommentResult, _> = params.try_into();
+        let err = err.unwrap_err().to_string();
+        assert!(err.contains("line requires file_path"), "got: {err}");
     }
 
     #[test]

src/safe_outputs/update_pr.rs

@@ -1059,8 +1059,9 @@ mod tests {
             vote: None,
             description: None,
         };
-        let result: Result<UpdatePrResult, _> = params.try_into();
-        assert!(result.is_err());
+        let err: Result<UpdatePrResult, _> = params.try_into();
+        let err = err.unwrap_err().to_string();
+        assert!(err.contains("operation must be one of"), "got: {err}");
     }
 
     #[test]