use std::sync::{
Mutex,
atomic::{AtomicUsize, Ordering},
};
use clap::Parser;
use secrecy::SecretString;
use serde_json::json;
use time::{Duration, OffsetDateTime};
use wiremock::{
Mock, MockServer, ResponseTemplate,
matchers::{body_json, method, path},
};
use super::CardCommand;
use crate::{
cli::Cli,
client::WekanClientFactory,
command_result::{CardDeleteMode, CardSubmittedField, CommandSuccess, DestructiveOperation},
commands::RootCommand,
credentials::{
CredentialCreateOutcome, CredentialDeleteOutcome, CredentialError, CredentialMutation,
CredentialRecord, CredentialStore, CredentialTarget,
},
error::{AppError, ErrorCode},
input::{ConfirmationProvider, FakeConfirmationProvider},
};
struct FakeCredentialStore {
record: Mutex<Option<CredentialRecord>>,
replacement_on_lock: Mutex<Option<CredentialRecord>>,
lock_count: AtomicUsize,
}
impl FakeCredentialStore {
fn authenticated(server: &MockServer) -> Self {
Self {
record: Mutex::new(Some(CredentialRecord::new(
format!("{}/", server.uri()),
"user-1".to_owned(),
SecretString::from("card-token".to_owned()),
OffsetDateTime::now_utc() + Duration::hours(1),
))),
replacement_on_lock: Mutex::new(None),
lock_count: AtomicUsize::new(0),
}
}
fn replace_on_next_lock(&self, replacement: CredentialRecord) {
*self.replacement_on_lock.lock().unwrap() = Some(replacement);
}
}
impl CredentialStore for FakeCredentialStore {
fn check_available(&self, _target: &CredentialTarget) -> Result<(), CredentialError> {
Ok(())
}
fn exists(&self, _target: &CredentialTarget) -> Result<bool, CredentialError> {
Ok(self.record.lock().unwrap().is_some())
}
fn load(
&self,
_target: &CredentialTarget,
) -> Result<Option<CredentialRecord>, CredentialError> {
Ok(self.record.lock().unwrap().clone())
}
fn create(
&self,
_target: &CredentialTarget,
_record: &CredentialRecord,
) -> Result<CredentialCreateOutcome, CredentialError> {
panic!("card commands must not store credentials")
}
fn delete(&self, _target: &CredentialTarget) -> Result<bool, CredentialError> {
panic!("card commands must not delete credentials")
}
fn delete_if_matches(
&self,
_target: &CredentialTarget,
_expected: &CredentialRecord,
) -> Result<CredentialDeleteOutcome, CredentialError> {
panic!("card commands must not delete credentials")
}
fn lock_mutation(
&self,
_target: &CredentialTarget,
) -> Result<Box<dyn CredentialMutation + Send + '_>, CredentialError> {
self.lock_count.fetch_add(1, Ordering::SeqCst);
if let Some(replacement) = self.replacement_on_lock.lock().unwrap().take() {
*self.record.lock().unwrap() = Some(replacement);
}
Ok(Box::new(FakeCredentialMutation { store: self }))
}
}
struct FakeCredentialMutation<'a> {
store: &'a FakeCredentialStore,
}
impl CredentialMutation for FakeCredentialMutation<'_> {
fn exists(&self) -> Result<bool, CredentialError> {
Ok(self.store.record.lock().unwrap().is_some())
}
fn load(&self) -> Result<Option<CredentialRecord>, CredentialError> {
Ok(self.store.record.lock().unwrap().clone())
}
fn create(
&self,
_record: &CredentialRecord,
) -> Result<CredentialCreateOutcome, CredentialError> {
panic!("card commands must not store credentials through a mutation guard")
}
fn delete(&self) -> Result<bool, CredentialError> {
panic!("card commands must not delete credentials through a mutation guard")
}
fn delete_if_matches(
&self,
_expected: &CredentialRecord,
) -> Result<CredentialDeleteOutcome, CredentialError> {
panic!("card commands must not delete credentials through a mutation guard")
}
}
fn factory(server: &MockServer) -> WekanClientFactory {
WekanClientFactory::for_profile(format!("{}/", server.uri()), "default".to_owned(), false)
}
async fn dispatch(
command: CardCommand,
client_factory: &WekanClientFactory,
credential_store: &dyn CredentialStore,
confirmation: &dyn ConfirmationProvider,
) -> Result<CommandSuccess, AppError> {
super::dispatch(command, client_factory, credential_store, confirmation).await
}
fn card_command(arguments: &[&str]) -> CardCommand {
let mut all = vec!["wekan", "--server", "https://wekan.example", "card"];
all.extend_from_slice(arguments);
let cli = Cli::try_parse_from(all).unwrap();
let RootCommand::Card(args) = cli.command else {
panic!("expected card command")
};
args.command
}
#[test]
fn parser_exposes_crud_and_enforces_update_contract() {
for arguments in [
vec!["list", "--board", "board-1", "--list", "list-1"],
vec!["get", "--board", "board-1", "--list", "list-1", "card-1"],
vec![
"create",
"--board",
"board-1",
"--list",
"list-1",
"--title",
"Todo",
"--swimlane-id",
"swimlane-1",
],
vec![
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--clear-due-at",
],
vec![
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--sort",
"0",
"--spent-time",
"0",
"--is-over-time",
"false",
],
vec![
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--label",
"label-1",
"--member",
"user-1",
"--due-complete",
"false",
],
vec![
"delete", "--board", "board-1", "--list", "list-1", "card-1", "--yes",
],
] {
card_command(&arguments);
}
for invalid in [
vec![
"wekan", "card", "update", "--board", "board-1", "--list", "list-1", "card-1",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--due-at",
"2030-01-01T00:00:00Z",
"--clear-due-at",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--label",
"label-1",
"--clear-labels",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--member",
"user-1",
"--clear-members",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--assignee",
"user-1",
"--clear-assignees",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--spent-time",
"NaN",
],
vec![
"wekan", "card", "update", "--board", "board-1", "--list", "list-1", "card-1",
"--sort", "NaN",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--is-over-time",
"maybe",
],
vec![
"wekan", "card", "update", "--board", "board-1", "--list", "list-1", "card-1",
"--color", "belize",
],
vec![
"wekan", "card", "get", "--board", "board-1", "--list", "list-1", "card-1", "--yes",
],
vec!["wekan", "card", "move", "board-1", "list-1", "card-1"],
vec!["wekan", "card", "archive", "board-1", "list-1", "card-1"],
] {
assert!(Cli::try_parse_from(invalid).is_err());
}
}
#[test]
fn parser_validates_identifiers_text_and_dates_without_masking_title_truncation() {
for invalid in [
vec!["wekan", "card", "list", "--board", " ", "--list", "list-1"],
vec![
"wekan",
"card",
"create",
"--board",
"board-1",
"--list",
"list-1",
"--title",
" ",
"--swimlane-id",
"swimlane-1",
],
vec![
"wekan",
"card",
"create",
"--board",
"board-1",
"--list",
"list-1",
"--title",
"Todo",
"--swimlane-id",
"swimlane-1",
"--due-at",
"tomorrow",
],
vec![
"wekan",
"card",
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--description",
" ",
],
] {
assert!(Cli::try_parse_from(invalid).is_err());
}
let accepted = "😀".repeat(501);
card_command(&[
"update", "--board", "board-1", "--list", "list-1", "card-1", "--title", &accepted,
]);
}
#[tokio::test]
async fn commands_return_typed_results_and_canonical_submitted_fields() {
let server = MockServer::start().await;
Mock::given(method("GET"))
.and(path("/api/boards/board%2F1/lists/list%2F1/cards"))
.respond_with(ResponseTemplate::new(200).set_body_json(json!([{
"_id": "card-1",
"title": "Todo",
"assignees": []
}])))
.expect(1)
.mount(&server)
.await;
Mock::given(method("GET"))
.and(path("/api/boards/board%2F1/lists/list%2F1/cards/card%2F1"))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({
"_id": "card/1",
"title": "Todo",
"archived": false,
"swimlaneId": "swimlane-1",
"createdAt": "2030-01-02T03:04:05Z",
"modifiedAt": "2030-01-02T03:04:05Z",
"dateLastActivity": "2030-01-02T03:04:05Z",
"userId": "user-1",
"type": "cardType-card",
"showActivities": false
})))
.expect(1)
.mount(&server)
.await;
Mock::given(method("POST"))
.and(path("/api/boards/board%2F1/lists/list%2F1/cards"))
.and(body_json(json!({
"title": "Todo",
"swimlaneId": "swimlane-1",
"members": ["user-1"],
"dueAt": "2030-01-03T00:00:00Z"
})))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({"_id": "card-2"})))
.expect(1)
.mount(&server)
.await;
Mock::given(method("PUT"))
.and(path("/api/boards/board%2F1/lists/list%2F1/cards/card%2F1"))
.and(body_json(json!({
"title": "Doing",
"sort": 3.5,
"labelIds": [],
"dueAt": "",
"isOverTime": true,
"members": ["user-2"],
"assignees": [],
"dueComplete": false
})))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({"_id": "card/1"})))
.expect(1)
.mount(&server)
.await;
let store = FakeCredentialStore::authenticated(&server);
let confirmation = FakeConfirmationProvider::accepting();
let CommandSuccess::CardCollection(collection) = dispatch(
card_command(&["list", "--board", "board/1", "--list", "list/1"]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap() else {
panic!("expected card collection")
};
assert_eq!(collection.board_id, "board/1");
assert_eq!(collection.list_id, "list/1");
assert_eq!(collection.cards[0].card_id, "card-1");
let CommandSuccess::CardShown(card) = dispatch(
card_command(&["get", "--board", "board/1", "--list", "list/1", "card/1"]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap() else {
panic!("expected card detail")
};
assert_eq!(card.card_id, "card/1");
assert!(card.members.is_empty());
let CommandSuccess::CardCreated(created) = dispatch(
card_command(&[
"create",
"--board",
"board/1",
"--list",
"list/1",
"--title",
" Todo ",
"--swimlane-id",
"swimlane-1",
"--member",
"user-1",
"--due-at",
"2030-01-03T00:00:00Z",
]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap() else {
panic!("expected card creation")
};
assert_eq!(created.card_id, "card-2");
let CommandSuccess::CardUpdated(updated) = dispatch(
card_command(&[
"update",
"--board",
"board/1",
"--list",
"list/1",
"card/1",
"--due-complete",
"false",
"--clear-assignees",
"--member",
"user-2",
"--clear-due-at",
"--clear-labels",
"--title",
"Doing",
"--sort",
"3.5",
"--is-over-time",
"true",
]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap() else {
panic!("expected card update")
};
assert_eq!(
updated.submitted_fields,
[
CardSubmittedField::Title,
CardSubmittedField::Sort,
CardSubmittedField::LabelIds,
CardSubmittedField::DueAt,
CardSubmittedField::IsOverTime,
CardSubmittedField::Members,
CardSubmittedField::Assignees,
CardSubmittedField::DueComplete,
]
);
}
#[tokio::test]
async fn card_errors_map_not_found_and_id_mismatches() {
let server = MockServer::start().await;
Mock::given(method("GET"))
.and(path("/api/boards/board-1/lists/list-1/cards/missing"))
.respond_with(ResponseTemplate::new(200))
.expect(1)
.mount(&server)
.await;
Mock::given(method("PUT"))
.and(path("/api/boards/board-1/lists/list-1/cards/card-1"))
.respond_with(ResponseTemplate::new(404).set_body_json(json!({"error": "not-found"})))
.expect(1)
.mount(&server)
.await;
Mock::given(method("PUT"))
.and(path("/api/boards/board-1/lists/list-1/cards/card-2"))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({"_id": "different"})))
.expect(1)
.mount(&server)
.await;
let store = FakeCredentialStore::authenticated(&server);
let confirmation = FakeConfirmationProvider::accepting();
let missing = dispatch(
card_command(&["get", "--board", "board-1", "--list", "list-1", "missing"]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap_err();
assert_eq!(missing.code(), ErrorCode::NotFound);
assert_eq!(missing.details().http_status, Some(200));
assert_eq!(missing.details().wekan_status_code, Some(404));
let update_missing = dispatch(
card_command(&[
"update",
"--board",
"board-1",
"--list",
"list-1",
"card-1",
"--title",
"Doing",
"--parent-id",
"missing-parent",
]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap_err();
assert_eq!(update_missing.code(), ErrorCode::NotFound);
assert_eq!(update_missing.details().outcome_unknown, Some(true));
let mismatch = dispatch(
card_command(&[
"update", "--board", "board-1", "--list", "list-1", "card-2", "--title", "Doing",
]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap_err();
assert_eq!(mismatch.code(), ErrorCode::ProtocolError);
assert_eq!(mismatch.details().outcome_unknown, Some(true));
}
#[tokio::test]
async fn delete_confirms_guards_credentials_and_verifies_the_returned_id() {
let server = MockServer::start().await;
let store = FakeCredentialStore::authenticated(&server);
let declining = FakeConfirmationProvider::declining();
let cancelled = dispatch(
card_command(&[
"delete",
"--board",
"board-1",
"--list",
"list-1",
"card\u{1b}]52;c;x\u{7}",
]),
&factory(&server),
&store,
&declining,
)
.await
.unwrap();
assert_eq!(
cancelled,
CommandSuccess::Cancelled(crate::command_result::CancellationSuccess::new(
DestructiveOperation::CardDelete
))
);
assert!(!declining.requests()[0].prompt().contains('\u{1b}'));
assert!(declining.requests()[0].prompt().contains("cascade"));
let failing = FakeConfirmationProvider::failing(AppError::invalid_input("prompt failed"));
let prompt_error = dispatch(
card_command(&[
"delete",
"--board",
"board-1",
"--list",
"list-1",
"card-prompt",
]),
&factory(&server),
&store,
&failing,
)
.await
.unwrap_err();
assert_eq!(prompt_error.code(), ErrorCode::InvalidInput);
Mock::given(method("DELETE"))
.and(path("/api/boards/board-1/lists/list-1/cards/card-1"))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({"_id": "card-1"})))
.expect(1)
.mount(&server)
.await;
let CommandSuccess::CardDeleted(deleted) = dispatch(
card_command(&[
"delete", "--board", "board-1", "--list", "list-1", "card-1", "--yes",
]),
&factory(&server),
&store,
&FakeConfirmationProvider::declining(),
)
.await
.unwrap() else {
panic!("expected card deletion")
};
assert!(deleted.deleted);
assert_eq!(deleted.delete_mode, CardDeleteMode::Hard);
Mock::given(method("DELETE"))
.and(path("/api/boards/board-1/lists/list-1/cards/card-2"))
.respond_with(ResponseTemplate::new(200).set_body_json(json!({"_id": "different"})))
.expect(1)
.mount(&server)
.await;
let mismatch = dispatch(
card_command(&[
"delete", "--board", "board-1", "--list", "list-1", "card-2", "--yes",
]),
&factory(&server),
&store,
&FakeConfirmationProvider::declining(),
)
.await
.unwrap_err();
assert_eq!(mismatch.code(), ErrorCode::ProtocolError);
assert_eq!(mismatch.details().outcome_unknown, Some(true));
let replacement = CredentialRecord::new(
format!("{}/", server.uri()),
"user-2".to_owned(),
SecretString::from("replacement-token".to_owned()),
OffsetDateTime::now_utc() + Duration::hours(1),
);
store.replace_on_next_lock(replacement);
let confirmation = FakeConfirmationProvider::accepting();
let changed = dispatch(
card_command(&["delete", "--board", "board-1", "--list", "list-1", "card-3"]),
&factory(&server),
&store,
&confirmation,
)
.await
.unwrap_err();
assert_eq!(changed.code(), ErrorCode::CredentialStoreFailed);
assert!(confirmation.requests().is_empty());
assert_eq!(store.lock_count.load(Ordering::SeqCst), 5);
}