fix: from py obj impl for framework and outcome

Author: joseph-sentry

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "test_results_parser"
-version = "0.5.4"  # 0.5.4 release is a copy of 0.5.1 and diverges from main
+version = "0.5.4"            # 0.5.4 release is a copy of 0.5.1 and diverges from main
 edition = "2021"
 
 [lib]
@@ -11,7 +11,7 @@ crate-type = ["cdylib", "rlib"]
 anyhow = "1.0.94"
 base16ct = { version = "0.2.0", features = ["std"] }
 indexmap = "2.6.0"
-pyo3 = { version = "0.24.1", features = ["abi3-py310", "anyhow"] }
+pyo3 = { version = "0.24.1", features = ["abi3-py39", "anyhow"] }
 quick-xml = "0.37.1"
 regex = "1.11.1"
 serde = { version = "1.0.215", features = ["derive"] }

src/testrun.rs

@@ -1,3 +1,5 @@
+use std::borrow::Cow;
+
 use pyo3::prelude::*;
 use pyo3::types::PyString;
 use pyo3::{PyAny, PyResult};
@@ -59,8 +61,8 @@ impl<'py> IntoPyObject<'py> for Outcome {
 
 impl<'py> FromPyObject<'py> for Outcome {
     fn extract_bound(ob: &Bound<'py, PyAny>) -> PyResult<Self> {
-        let s = ob.extract::<&str>()?;
-        match s {
+        let s: String = ob.extract()?;
+        match s.as_str() {
             "pass" => Ok(Outcome::Pass),
             "failure" => Ok(Outcome::Failure),
             "skip" => Ok(Outcome::Skip),
@@ -98,14 +100,14 @@ impl<'py> IntoPyObject<'py> for Framework {
 
 impl<'py> FromPyObject<'py> for Framework {
     fn extract_bound(ob: &Bound<'py, PyAny>) -> PyResult<Self> {
-        let s = ob.extract::<&str>()?;
-        match s {
+        let s: String = ob.extract()?;
+        match s.as_str() {
             "Pytest" => Ok(Framework::Pytest),
             "Vitest" => Ok(Framework::Vitest),
             "Jest" => Ok(Framework::Jest),
             "PHPUnit" => Ok(Framework::PHPUnit),
             _ => Err(PyErr::new::<pyo3::exceptions::PyValueError, _>(format!(
-                "Invalid outcome: {}",
+                "Invalid framework: {}",
                 s
             ))),
         }