update the result_tests.rs to be compatible with new Spi API

eeeebbbbrrrr · eeeebbbbrrrr · commit bae392485ad1 · 2022-12-19T21:18:19.000-05:00
diff --git a/pgx-tests/src/tests/result_tests.rs b/pgx-tests/src/tests/result_tests.rs
@@ -48,12 +48,12 @@ mod tests {
 
     #[pg_test]
     fn test_return_some_optional_result() {
-        assert_eq!(Some(42), Spi::get_one::<i32>("SELECT tests.return_some_optional_result()"));
+        assert_eq!(Ok(Some(42)), Spi::get_one::<i32>("SELECT tests.return_some_optional_result()"));
     }
 
     #[pg_test]
     fn test_return_none_optional_result() {
-        assert_eq!(None, Spi::get_one::<i32>("SELECT tests.return_none_optional_result()"));
+        assert_eq!(Ok(None), Spi::get_one::<i32>("SELECT tests.return_none_optional_result()"));
     }
 
     #[pg_test(error = "error!")]
@@ -72,7 +72,7 @@ mod tests {
     }
 
     #[pg_test(error = "got proper sql errorcode")]
-    fn test_proper_sql_errcode() -> Option<i32> {
+    fn test_proper_sql_errcode() -> Result<Option<i32>, pgx::spi::Error> {
         PgTryBuilder::new(|| Spi::get_one::<i32>("SELECT tests.return_eyre_result_error()"))
             .catch_when(PgSqlErrorCode::ERRCODE_DATA_EXCEPTION, |_| {
                 panic!("got proper sql errorcode")

Original file line number	Diff line number	Diff line change
`@@ -48,12 +48,12 @@ mod tests {`
`48`	`48`
`49`	`49`	`#[pg_test]`
`50`	`50`	`fn test_return_some_optional_result() {`
`51`		`- assert_eq!(Some(42), Spi::get_one::<i32>("SELECT tests.return_some_optional_result()"));`
	`51`	`+ assert_eq!(Ok(Some(42)), Spi::get_one::<i32>("SELECT tests.return_some_optional_result()"));`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`#[pg_test]`
`55`	`55`	`fn test_return_none_optional_result() {`
`56`		`- assert_eq!(None, Spi::get_one::<i32>("SELECT tests.return_none_optional_result()"));`
	`56`	`+ assert_eq!(Ok(None), Spi::get_one::<i32>("SELECT tests.return_none_optional_result()"));`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`#[pg_test(error = "error!")]`
`@@ -72,7 +72,7 @@ mod tests {`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`#[pg_test(error = "got proper sql errorcode")]`
`75`		`- fn test_proper_sql_errcode() -> Option<i32> {`
	`75`	`+ fn test_proper_sql_errcode() -> Result<Option<i32>, pgx::spi::Error> {`
`76`	`76`	`PgTryBuilder::new(\|\| Spi::get_one::<i32>("SELECT tests.return_eyre_result_error()"))`
`77`	`77`	`.catch_when(PgSqlErrorCode::ERRCODE_DATA_EXCEPTION, \|_\| {`
`78`	`78`	`panic!("got proper sql errorcode")`