@@ -177,7 +177,6 @@ impl Default for TestState {
#[derive(Serialize, Default, Clone)]
pub struct TestResult {
pub state: TestState,
- #[serde(serialize_with = "TestResult::serialize_target_url")]
pub target_url: Option<String>,
pub description: Option<String>,
#[serde(serialize_with = "TestResult::serialize_context")]
@@ -185,15 +184,6 @@ pub struct TestResult {
}
impl TestResult {
- fn serialize_target_url<S>(target_url: &Option<String>, ser: S)
- -> Result<S::Ok, S::Error> where S: Serializer {
- if target_url.is_none() {
- serde::Serialize::serialize(&Some("http://no.url".to_string()), ser)
- } else {
- serde::Serialize::serialize(target_url, ser)
- }
- }
-
fn serialize_context<S>(context: &Option<String>, ser: S)
-> Result<S::Ok, S::Error> where S: Serializer {
if context.is_none() {
At some point, Patchwork didn't like it when we submitted a Check without a target_url attached, so we sent "no.url" as a default. That appears to have changed, so get rid of the dummy URL. Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com> --- applies on top of current patchwork series, completely untested v1->v2: - remember to change branches before writing patch... --- src/patchwork.rs | 10 ---------- 1 file changed, 10 deletions(-)