Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
88c77f8ac6
|
|||
|
5aa382a4c9
|
+4
-6
@@ -300,8 +300,7 @@ 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));
|
||||
@@ -310,10 +309,9 @@ 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,7 +809,10 @@ 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