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