2fe966a Adds different error for second message — radicle-ci-broker

Radicle CI broker

Adds different error for second message

Michalis Zampetakis committed 2 years ago

commit 2fe966abf39936111e99217e8918597c25c0f562
parent d8a1035da0996f1cc98511072cc47c99ed65087f

1 file changed +7 -1

modified src/adapter.rs

@@ -113,7 +113,7 @@ impl Adapter {

                     run.set_result(result);
                     status.push_run(run.clone());
                 }
                 _ => return Err(AdapterError::NotTriggered(resp)),
                 _ => return Err(AdapterError::NotFinished(resp)),
             }
         }

@@ -166,6 +166,10 @@ pub enum AdapterError {

     #[error("adapter's first message is not 'triggered', but {0:?}")]
     NotTriggered(Response),
     /// Second message is not `Response::Finished`
     #[error("adapter's second message is not 'finished', but {0:?}")]
     NotFinished(Response),
     /// Too many messages from adapter.
     #[error("adapter sent too many messages: first extra is {0:#?}")]
     TooMany(Response),

@@ -179,8 +183,10 @@ mod test {

     use radicle::git::Oid;
     use radicle::prelude::RepoId;
     use serde_json::json;
     use super::{Adapter, Run, StatusPage};
     use crate::msg::RunId;
     use crate::{
         adapter::AdapterError,
         msg::{MessageError, Response, RunResult},