Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
77eab11588
|
|||
|
6628e776e9
|
+6
-4
@@ -300,7 +300,8 @@ mod tests {
|
||||
/// Test mock new_revision() returns configured error
|
||||
#[tokio::test]
|
||||
async fn mock_new_revision_returns_error() {
|
||||
let mock = MockJjExecutor::new().with_new_revision_response(Err(Error::RepositoryLocked));
|
||||
let mock =
|
||||
MockJjExecutor::new().with_new_revision_response(Err(Error::RepositoryLocked));
|
||||
let result = mock.new_revision("@").await;
|
||||
assert!(result.is_err());
|
||||
assert!(matches!(result.unwrap_err(), Error::RepositoryLocked));
|
||||
@@ -309,9 +310,10 @@ mod tests {
|
||||
/// Test mock new_revision() records revset even on error
|
||||
#[tokio::test]
|
||||
async fn mock_new_revision_records_revset_on_error() {
|
||||
let mock = MockJjExecutor::new().with_new_revision_response(Err(Error::JjOperation {
|
||||
context: "failed".to_string(),
|
||||
}));
|
||||
let mock =
|
||||
MockJjExecutor::new().with_new_revision_response(Err(Error::JjOperation {
|
||||
context: "failed".to_string(),
|
||||
}));
|
||||
let result = mock.new_revision("abc").await;
|
||||
assert!(result.is_err());
|
||||
let calls = mock.new_revision_calls();
|
||||
|
||||
@@ -783,8 +783,8 @@ mod tests {
|
||||
/// Test workflow new_revision() propagates executor errors
|
||||
#[tokio::test]
|
||||
async fn workflow_new_revision_propagates_error() {
|
||||
let mock_executor =
|
||||
MockJjExecutor::new().with_new_revision_response(Err(Error::RepositoryLocked));
|
||||
let mock_executor = MockJjExecutor::new()
|
||||
.with_new_revision_response(Err(Error::RepositoryLocked));
|
||||
let workflow = CommitWorkflow::new(mock_executor);
|
||||
|
||||
let result = workflow.new_revision("@").await;
|
||||
@@ -809,10 +809,7 @@ mod tests {
|
||||
let workflow = CommitWorkflow::with_prompts(mock_executor, mock_prompts);
|
||||
|
||||
workflow.run_for_revset("@").await.expect("describe failed");
|
||||
workflow
|
||||
.new_revision("@")
|
||||
.await
|
||||
.expect("new_revision failed");
|
||||
workflow.new_revision("@").await.expect("new_revision failed");
|
||||
|
||||
let messages = workflow.executor.describe_messages();
|
||||
assert_eq!(messages.len(), 1);
|
||||
|
||||
Reference in New Issue
Block a user