error.rs

//! Error types for the mpp library.
//!
//! This module provides:
//! - [`MppError`]: The main error enum for all mpp operations
//! - [`PaymentErrorDetails`]: RFC 9457 Problem Details format for HTTP error responses
//! - [`PaymentError`]: Trait for converting errors to Problem Details

use thiserror::Error;

/// Result type alias for mpp operations.
pub type Result<T> = std::result::Result<T, MppError>;

// ==================== RFC 9457 Problem Details ====================

/// Base URI for core payment-auth problem types.
pub const CORE_PROBLEM_TYPE_BASE: &str = "https://paymentauth.org/problems";

/// Base URI for session/channel problem types.
pub const SESSION_PROBLEM_TYPE_BASE: &str = "https://paymentauth.org/problems/session";

/// Deprecated: use [`SESSION_PROBLEM_TYPE_BASE`] instead. Remove in next major version.
#[deprecated(since = "0.5.0", note = "renamed to SESSION_PROBLEM_TYPE_BASE")]
pub const STREAM_PROBLEM_TYPE_BASE: &str = SESSION_PROBLEM_TYPE_BASE;

/// RFC 9457 Problem Details structure for payment errors.
///
/// This struct provides a standardized format for HTTP error responses,
/// following [RFC 9457](https://www.rfc-editor.org/rfc/rfc9457.html).
///
/// # Example
///
/// ```
/// use mpp::error::PaymentErrorDetails;
///
/// let problem = PaymentErrorDetails::core("verification-failed")
///     .with_title("VerificationFailedError")
///     .with_status(402)
///     .with_detail("Payment verification failed: insufficient amount.");
///
/// // Serialize to JSON for HTTP response body
/// let json = serde_json::to_string(&problem).unwrap();
/// ```
#[derive(Debug, Clone, serde::Serialize, serde::Deserialize)]
pub struct PaymentErrorDetails {
    /// A URI reference that identifies the problem type.
    #[serde(rename = "type")]
    pub problem_type: String,

    /// A short, human-readable summary of the problem type.
    pub title: String,

    /// The HTTP status code for this problem.
    pub status: u16,

    /// A human-readable explanation specific to this occurrence.
    pub detail: String,

    /// The challenge ID associated with this error, if applicable.
    #[serde(rename = "challengeId", skip_serializing_if = "Option::is_none")]
    pub challenge_id: Option<String>,
}

impl PaymentErrorDetails {
    /// Create a new PaymentErrorDetails with a full problem type URI.
    pub fn new(type_uri: impl Into<String>) -> Self {
        Self {
            problem_type: type_uri.into(),
            title: String::new(),
            status: 402,
            detail: String::new(),
            challenge_id: None,
        }
    }

    /// Create a PaymentErrorDetails for a core payment-auth problem.
    ///
    /// The full type URI will be `{CORE_PROBLEM_TYPE_BASE}/{suffix}`.
    pub fn core(suffix: impl std::fmt::Display) -> Self {
        Self::new(format!("{}/{}", CORE_PROBLEM_TYPE_BASE, suffix))
    }

    /// Create a PaymentErrorDetails for a session/channel problem.
    ///
    /// The full type URI will be `{SESSION_PROBLEM_TYPE_BASE}/{suffix}`.
    pub fn session(suffix: impl std::fmt::Display) -> Self {
        Self::new(format!("{}/{}", SESSION_PROBLEM_TYPE_BASE, suffix))
    }

    /// Deprecated: use [`PaymentErrorDetails::session`] instead. Remove in next major version.
    #[deprecated(since = "0.5.0", note = "renamed to session()")]
    pub fn stream(suffix: impl std::fmt::Display) -> Self {
        Self::session(suffix)
    }

    /// Set the title.
    pub fn with_title(mut self, title: impl Into<String>) -> Self {
        self.title = title.into();
        self
    }

    /// Set the HTTP status code.
    pub fn with_status(mut self, status: u16) -> Self {
        self.status = status;
        self
    }

    /// Set the detail message.
    pub fn with_detail(mut self, detail: impl Into<String>) -> Self {
        self.detail = detail.into();
        self
    }

    /// Set the associated challenge ID.
    pub fn with_challenge_id(mut self, id: impl Into<String>) -> Self {
        self.challenge_id = Some(id.into());
        self
    }

    /// Standard RFC 9457 problem for a bare `402` payment challenge (no prior
    /// error). Used when a server advertises a challenge without a specific
    /// failure to report.
    pub fn payment_required(challenge_id: impl Into<String>) -> Self {
        Self::core("payment-required")
            .with_title("Payment Required")
            .with_status(402)
            .with_detail("Payment is required.")
            .with_challenge_id(challenge_id)
    }
}

/// Trait for errors that can be converted to RFC 9457 Problem Details.
///
/// Implement this trait to enable automatic conversion of payment errors
/// to standardized HTTP error responses.
///
/// # Example
///
/// ```
/// use mpp::error::{PaymentError, PaymentErrorDetails};
///
/// struct MyError {
///     reason: String,
/// }
///
/// impl PaymentError for MyError {
///     fn to_problem_details(&self, challenge_id: Option<&str>) -> PaymentErrorDetails {
///         PaymentErrorDetails::core("my-error")
///             .with_title("MyError")
///             .with_status(402)
///             .with_detail(&self.reason)
///     }
/// }
/// ```
pub trait PaymentError {
    /// Convert this error to RFC 9457 Problem Details format.
    ///
    /// # Arguments
    ///
    /// * `challenge_id` - Optional challenge ID to include in the response
    fn to_problem_details(&self, challenge_id: Option<&str>) -> PaymentErrorDetails;
}

#[derive(Error, Debug)]
pub enum MppError {
    /// Required amount exceeds user's maximum allowed
    #[error("Required amount ({required}) exceeds maximum allowed ({max})")]
    AmountExceedsMax { required: u128, max: u128 },

    /// Invalid amount format
    #[error("Invalid amount: {0}")]
    InvalidAmount(String),

    /// Configuration is invalid
    #[error("Invalid configuration: {0}")]
    InvalidConfig(String),

    // ==================== HTTP Errors ====================
    /// HTTP request/response error
    #[error("HTTP error: {0}")]
    Http(String),

    /// Chain ID mismatch between the expected (pinned) chain and the one found
    #[error("Chain ID mismatch: expected {expected}, got {got}")]
    ChainIdMismatch { expected: u64, got: u64 },

    /// JSON serialization/deserialization error
    #[error("JSON error: {0}")]
    Json(#[from] serde_json::Error),

    /// Hex decoding error
    #[cfg(feature = "utils")]
    #[error("Hex decoding error: {0}")]
    HexDecode(#[from] hex::FromHexError),

    /// Base64 decoding error
    #[cfg(feature = "utils")]
    #[error("Base64 decoding error: {0}")]
    Base64Decode(#[from] base64::DecodeError),

    // ==================== Web Payment Auth Errors ====================
    /// Unsupported payment method
    #[error("Unsupported payment method: {0}")]
    UnsupportedPaymentMethod(String),

    /// Missing required header
    #[error("Missing required header: {0}")]
    MissingHeader(String),

    /// Invalid base64url encoding
    #[error("Invalid base64url: {0}")]
    InvalidBase64Url(String),

    // ==================== RFC 9457 Payment Problems ====================
    // These variants can be converted to RFC 9457 Problem Details format.
    /// Credential is malformed (invalid base64url, bad JSON structure).
    #[error("{}", format_malformed_credential(.0))]
    MalformedCredential(Option<String>),

    /// Challenge ID is unknown, expired, or already used.
    #[error("{}", format_invalid_challenge(.id, .reason))]
    InvalidChallenge {
        id: Option<String>,
        reason: Option<String>,
    },

    /// Payment proof is invalid or verification failed.
    #[error("{}", format_verification_failed(.0))]
    VerificationFailed(Option<String>),

    /// Payment has expired.
    #[error("{}", format_payment_expired(.0))]
    PaymentExpired(Option<String>),

    /// No credential was provided but payment is required.
    #[error("{}", format_payment_required(.realm, .description))]
    PaymentRequired {
        realm: Option<String>,
        description: Option<String>,
    },

    /// Credential payload does not match the expected schema.
    #[error("{}", format_invalid_payload(.0))]
    InvalidPayload(Option<String>),

    /// Request is malformed or contains invalid parameters.
    #[error("{}", format_bad_request(.0))]
    BadRequest(Option<String>),

    /// Payment requires additional action (e.g., 3D Secure confirmation).
    #[error("{}", format_payment_action_required(.0))]
    PaymentActionRequired(Option<String>),

    /// Payment amount is insufficient for the requested resource.
    #[error("{}", format_payment_insufficient(.0))]
    PaymentInsufficient(Option<String>),

    // ==================== Session/Channel Errors ====================
    /// Insufficient balance in payment channel.
    #[error("{}", format_insufficient_balance(.0))]
    InsufficientBalance(Option<String>),

    /// Invalid cryptographic signature.
    #[error("{}", format_invalid_signature(.0))]
    InvalidSignature(Option<String>),

    /// Signer does not match expected address.
    #[error("{}", format_signer_mismatch(.0))]
    SignerMismatch(Option<String>),

    /// Voucher amount exceeds channel deposit.
    #[error("{}", format_amount_exceeds_deposit(.0))]
    AmountExceedsDeposit(Option<String>),

    /// Voucher delta is below the minimum threshold.
    #[error("{}", format_delta_too_small(.0))]
    DeltaTooSmall(Option<String>),

    /// Payment channel not found.
    #[error("{}", format_channel_not_found(.0))]
    ChannelNotFound(Option<String>),

    /// Payment channel has been closed.
    #[error("{}", format_channel_closed(.0))]
    ChannelClosed(Option<String>),

    // ==================== Method-Specific Client Errors ====================
    /// Tempo-specific client error (transaction reverts, keychain issues, etc.).
    ///
    /// See [`crate::client::tempo::TempoClientError`] for variants.
    #[cfg(all(feature = "client", feature = "tempo"))]
    #[error("{0}")]
    Tempo(#[from] crate::client::tempo::TempoClientError),

    // ==================== External Library Errors ====================
    /// IO error
    #[error("IO error: {0}")]
    Io(#[from] std::io::Error),

    /// Invalid UTF-8 in response
    #[error("Invalid UTF-8 in response body")]
    InvalidUtf8(#[from] std::string::FromUtf8Error),

    /// System time error
    #[error("System time error: {0}")]
    SystemTime(#[from] std::time::SystemTimeError),
}

// ==================== Result Extension Trait ====================

/// Extension trait for mapping errors into [`MppError`] with a contextual message.
pub(crate) trait ResultExt<T> {
    /// Map the error into [`MppError::Http`] with the given context prefix.
    fn mpp_http(self, context: &str) -> std::result::Result<T, MppError>;

    /// Map the error into [`MppError::InvalidConfig`] with the given context prefix.
    fn mpp_config(self, context: &str) -> std::result::Result<T, MppError>;
}

impl<T, E: std::fmt::Display> ResultExt<T> for std::result::Result<T, E> {
    fn mpp_http(self, context: &str) -> std::result::Result<T, MppError> {
        self.map_err(|e| MppError::Http(format!("{context}: {e}")))
    }

    fn mpp_config(self, context: &str) -> std::result::Result<T, MppError> {
        self.map_err(|e| MppError::InvalidConfig(format!("{context}: {e}")))
    }
}

// ==================== RFC 9457 Format Helpers ====================

fn format_malformed_credential(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Credential is malformed: {}.", r),
        None => "Credential is malformed.".to_string(),
    }
}

fn format_invalid_challenge(id: &Option<String>, reason: &Option<String>) -> String {
    let id_part = id
        .as_ref()
        .map(|id| format!(" \"{}\"", id))
        .unwrap_or_default();
    let reason_part = reason
        .as_ref()
        .map(|r| format!(": {}", r))
        .unwrap_or_default();
    format!("Challenge{} is invalid{}.", id_part, reason_part)
}

fn format_verification_failed(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Payment verification failed: {}.", r),
        None => "Payment verification failed.".to_string(),
    }
}

fn format_payment_expired(expires: &Option<String>) -> String {
    match expires {
        Some(e) => format!("Payment expired at {}.", e),
        None => "Payment has expired.".to_string(),
    }
}

fn format_payment_required(realm: &Option<String>, description: &Option<String>) -> String {
    let mut s = "Payment is required".to_string();
    if let Some(r) = realm {
        s.push_str(&format!(" for \"{}\"", r));
    }
    if let Some(d) = description {
        s.push_str(&format!(" ({})", d));
    }
    s.push('.');
    s
}

fn format_invalid_payload(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Credential payload is invalid: {}.", r),
        None => "Credential payload is invalid.".to_string(),
    }
}

fn format_bad_request(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Bad request: {}.", r),
        None => "Bad request.".to_string(),
    }
}

fn format_payment_action_required(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Payment requires action: {}.", r),
        None => "Payment requires action.".to_string(),
    }
}

fn format_payment_insufficient(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Payment insufficient: {}.", r),
        None => "Payment amount is insufficient.".to_string(),
    }
}

fn format_insufficient_balance(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Insufficient balance: {}.", r),
        None => "Insufficient balance.".to_string(),
    }
}

fn format_invalid_signature(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Invalid signature: {}.", r),
        None => "Invalid signature.".to_string(),
    }
}

fn format_signer_mismatch(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Signer mismatch: {}.", r),
        None => "Signer is not authorized for this channel.".to_string(),
    }
}

fn format_amount_exceeds_deposit(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Amount exceeds deposit: {}.", r),
        None => "Voucher amount exceeds channel deposit.".to_string(),
    }
}

fn format_delta_too_small(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Delta too small: {}.", r),
        None => "Amount increase below minimum voucher delta.".to_string(),
    }
}

fn format_channel_not_found(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Channel not found: {}.", r),
        None => "No channel with this ID exists.".to_string(),
    }
}

fn format_channel_closed(reason: &Option<String>) -> String {
    match reason {
        Some(r) => format!("Channel closed: {}.", r),
        None => "Channel is closed.".to_string(),
    }
}

impl MppError {
    /// Create an unsupported payment method error
    pub fn unsupported_method(method: &impl std::fmt::Display) -> Self {
        Self::UnsupportedPaymentMethod(format!("Payment method '{}' is not supported", method))
    }

    // ==================== RFC 9457 Payment Problem Constructors ====================

    /// Create a malformed credential error.
    pub fn malformed_credential(reason: impl Into<String>) -> Self {
        Self::MalformedCredential(Some(reason.into()))
    }

    /// Create a malformed credential error without a reason.
    pub fn malformed_credential_default() -> Self {
        Self::MalformedCredential(None)
    }

    /// Create an invalid challenge error with ID.
    pub fn invalid_challenge_id(id: impl Into<String>) -> Self {
        Self::InvalidChallenge {
            id: Some(id.into()),
            reason: None,
        }
    }

    /// Create an invalid challenge error with reason.
    pub fn invalid_challenge_reason(reason: impl Into<String>) -> Self {
        Self::InvalidChallenge {
            id: None,
            reason: Some(reason.into()),
        }
    }

    /// Create an invalid challenge error with ID and reason.
    pub fn invalid_challenge(id: impl Into<String>, reason: impl Into<String>) -> Self {
        Self::InvalidChallenge {
            id: Some(id.into()),
            reason: Some(reason.into()),
        }
    }

    /// Create an invalid challenge error without details.
    pub fn invalid_challenge_default() -> Self {
        Self::InvalidChallenge {
            id: None,
            reason: None,
        }
    }

    /// Create a verification failed error.
    pub fn verification_failed(reason: impl Into<String>) -> Self {
        Self::VerificationFailed(Some(reason.into()))
    }

    /// Create a verification failed error without a reason.
    pub fn verification_failed_default() -> Self {
        Self::VerificationFailed(None)
    }

    /// Create a payment expired error with expiration timestamp.
    pub fn payment_expired(expires: impl Into<String>) -> Self {
        Self::PaymentExpired(Some(expires.into()))
    }

    /// Create a payment expired error without timestamp.
    pub fn payment_expired_default() -> Self {
        Self::PaymentExpired(None)
    }

    /// Create a payment required error with realm.
    pub fn payment_required_realm(realm: impl Into<String>) -> Self {
        Self::PaymentRequired {
            realm: Some(realm.into()),
            description: None,
        }
    }

    /// Create a payment required error with description.
    pub fn payment_required_description(description: impl Into<String>) -> Self {
        Self::PaymentRequired {
            realm: None,
            description: Some(description.into()),
        }
    }

    /// Create a payment required error with realm and description.
    pub fn payment_required(realm: impl Into<String>, description: impl Into<String>) -> Self {
        Self::PaymentRequired {
            realm: Some(realm.into()),
            description: Some(description.into()),
        }
    }

    /// Create a payment required error without details.
    pub fn payment_required_default() -> Self {
        Self::PaymentRequired {
            realm: None,
            description: None,
        }
    }

    /// Create an invalid payload error.
    pub fn invalid_payload(reason: impl Into<String>) -> Self {
        Self::InvalidPayload(Some(reason.into()))
    }

    /// Create an invalid payload error without a reason.
    pub fn invalid_payload_default() -> Self {
        Self::InvalidPayload(None)
    }

    /// Create a bad request error.
    pub fn bad_request(reason: impl Into<String>) -> Self {
        Self::BadRequest(Some(reason.into()))
    }

    /// Create a bad request error without a reason.
    pub fn bad_request_default() -> Self {
        Self::BadRequest(None)
    }

    /// Create a payment action required error.
    pub fn payment_action_required(reason: impl Into<String>) -> Self {
        Self::PaymentActionRequired(Some(reason.into()))
    }

    /// Create a payment action required error without a reason.
    pub fn payment_action_required_default() -> Self {
        Self::PaymentActionRequired(None)
    }

    /// Create a payment insufficient error.
    pub fn payment_insufficient(reason: impl Into<String>) -> Self {
        Self::PaymentInsufficient(Some(reason.into()))
    }

    /// Create a payment insufficient error without a reason.
    pub fn payment_insufficient_default() -> Self {
        Self::PaymentInsufficient(None)
    }

    /// Returns the RFC 9457 problem type suffix if this is a payment problem.
    pub fn problem_type_suffix(&self) -> Option<&'static str> {
        match self {
            Self::MalformedCredential(_) => Some("malformed-credential"),
            Self::InvalidChallenge { .. } => Some("invalid-challenge"),
            Self::VerificationFailed(_) => Some("verification-failed"),
            Self::PaymentExpired(_) => Some("payment-expired"),
            Self::PaymentRequired { .. } => Some("payment-required"),
            Self::InvalidPayload(_) => Some("invalid-payload"),
            Self::BadRequest(_) => Some("bad-request"),
            Self::UnsupportedPaymentMethod(_) => Some("method-unsupported"),
            Self::PaymentActionRequired(_) => Some("payment-action-required"),
            Self::PaymentInsufficient(_) => Some("payment-insufficient"),
            Self::InsufficientBalance(_) => Some("session/insufficient-balance"),
            Self::InvalidSignature(_) => Some("session/invalid-signature"),
            Self::SignerMismatch(_) => Some("session/signer-mismatch"),
            Self::AmountExceedsDeposit(_) => Some("session/amount-exceeds-deposit"),
            Self::DeltaTooSmall(_) => Some("session/delta-too-small"),
            Self::ChannelNotFound(_) => Some("session/channel-not-found"),
            Self::ChannelClosed(_) => Some("session/channel-finalized"),
            _ => None,
        }
    }

    /// Returns true if this error is an RFC 9457 payment problem.
    pub fn is_payment_problem(&self) -> bool {
        self.problem_type_suffix().is_some()
    }
}

impl PaymentError for MppError {
    fn to_problem_details(&self, challenge_id: Option<&str>) -> PaymentErrorDetails {
        let mut problem = match self {
            // Core payment-auth errors
            Self::MalformedCredential(_) => PaymentErrorDetails::core("malformed-credential")
                .with_title("MalformedCredentialError")
                .with_status(402),
            Self::InvalidChallenge { .. } => PaymentErrorDetails::core("invalid-challenge")
                .with_title("InvalidChallengeError")
                .with_status(402),
            Self::VerificationFailed(_) => PaymentErrorDetails::core("verification-failed")
                .with_title("VerificationFailedError")
                .with_status(402),
            Self::PaymentExpired(_) => PaymentErrorDetails::core("payment-expired")
                .with_title("PaymentExpiredError")
                .with_status(402),
            Self::PaymentRequired { .. } => PaymentErrorDetails::core("payment-required")
                .with_title("PaymentRequiredError")
                .with_status(402),
            Self::InvalidPayload(_) => PaymentErrorDetails::core("invalid-payload")
                .with_title("InvalidPayloadError")
                .with_status(402),
            Self::BadRequest(_) => PaymentErrorDetails::core("bad-request")
                .with_title("BadRequestError")
                .with_status(400),
            Self::UnsupportedPaymentMethod(_) => PaymentErrorDetails::core("method-unsupported")
                .with_title("PaymentMethodUnsupportedError")
                .with_status(400),
            Self::PaymentActionRequired(_) => PaymentErrorDetails::core("payment-action-required")
                .with_title("PaymentActionRequiredError")
                .with_status(402),
            Self::PaymentInsufficient(_) => PaymentErrorDetails::core("payment-insufficient")
                .with_title("PaymentInsufficientError")
                .with_status(402),
            // Session/channel errors
            Self::InsufficientBalance(_) => PaymentErrorDetails::session("insufficient-balance")
                .with_title("InsufficientBalanceError")
                .with_status(402),
            Self::InvalidSignature(_) => PaymentErrorDetails::session("invalid-signature")
                .with_title("InvalidSignatureError")
                .with_status(402),
            Self::SignerMismatch(_) => PaymentErrorDetails::session("signer-mismatch")
                .with_title("SignerMismatchError")
                .with_status(402),
            Self::AmountExceedsDeposit(_) => PaymentErrorDetails::session("amount-exceeds-deposit")
                .with_title("AmountExceedsDepositError")
                .with_status(402),
            Self::DeltaTooSmall(_) => PaymentErrorDetails::session("delta-too-small")
                .with_title("DeltaTooSmallError")
                .with_status(402),
            Self::ChannelNotFound(_) => PaymentErrorDetails::session("channel-not-found")
                .with_title("ChannelNotFoundError")
                .with_status(410),
            Self::ChannelClosed(_) => PaymentErrorDetails::session("channel-finalized")
                .with_title("ChannelClosedError")
                .with_status(410),
            // Non-payment-problem errors get a generic problem type
            _ => PaymentErrorDetails::core("internal-error")
                .with_title("InternalError")
                .with_status(402),
        }
        .with_detail(self.to_string());

        // Use embedded challenge ID from InvalidChallenge, or the provided one
        let embedded_id = match self {
            Self::InvalidChallenge { id, .. } => id.as_deref(),
            _ => None,
        };
        if let Some(id) = challenge_id.or(embedded_id) {
            problem = problem.with_challenge_id(id);
        }
        problem
    }
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn test_amount_exceeds_max_display() {
        let err = MppError::AmountExceedsMax {
            required: 1000,
            max: 500,
        };
        let display = err.to_string();
        assert!(display.contains("Required amount (1000) exceeds maximum allowed (500)"));
    }

    #[test]
    fn test_invalid_amount_display() {
        let err = MppError::InvalidAmount("not a number".to_string());
        assert_eq!(err.to_string(), "Invalid amount: not a number");
    }

    #[test]
    fn test_invalid_config_display() {
        let err = MppError::InvalidConfig("invalid rpc url".to_string());
        assert_eq!(err.to_string(), "Invalid configuration: invalid rpc url");
    }

    #[test]
    fn test_http_display() {
        let err = MppError::Http("404 Not Found".to_string());
        assert_eq!(err.to_string(), "HTTP error: 404 Not Found");
    }

    #[test]
    fn test_unsupported_payment_method_display() {
        let err = MppError::UnsupportedPaymentMethod("bitcoin".to_string());
        assert_eq!(err.to_string(), "Unsupported payment method: bitcoin");
    }

    #[test]
    fn test_invalid_challenge_display() {
        let err = MppError::invalid_challenge_reason("Malformed challenge");
        assert_eq!(
            err.to_string(),
            "Challenge is invalid: Malformed challenge."
        );
    }

    #[test]
    fn test_missing_header_display() {
        let err = MppError::MissingHeader("WWW-Authenticate".to_string());
        assert_eq!(err.to_string(), "Missing required header: WWW-Authenticate");
    }

    #[test]
    fn test_invalid_base64_url_display() {
        let err = MppError::InvalidBase64Url("Invalid padding".to_string());
        assert_eq!(err.to_string(), "Invalid base64url: Invalid padding");
    }

    #[test]
    fn test_challenge_expired_display() {
        let err = MppError::payment_expired("2025-01-15T12:00:00Z");
        assert_eq!(err.to_string(), "Payment expired at 2025-01-15T12:00:00Z.");
    }

    #[test]
    fn test_unsupported_method_constructor() {
        let err = MppError::unsupported_method(&"bitcoin");
        assert!(matches!(err, MppError::UnsupportedPaymentMethod(_)));
        assert!(err.to_string().contains("bitcoin"));
        assert!(err.to_string().contains("not supported"));
    }

    // ==================== RFC 9457 Problem Details Tests ====================

    #[test]
    fn test_problem_details_core() {
        let problem = PaymentErrorDetails::core("test-error")
            .with_title("TestError")
            .with_status(400)
            .with_detail("Something went wrong");

        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/test-error"
        );
        assert_eq!(problem.title, "TestError");
        assert_eq!(problem.status, 400);
        assert_eq!(problem.detail, "Something went wrong");
        assert!(problem.challenge_id.is_none());
    }

    #[test]
    fn test_problem_details_session() {
        let problem = PaymentErrorDetails::session("insufficient-balance")
            .with_title("InsufficientBalanceError")
            .with_status(402)
            .with_detail("Insufficient balance.");

        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/insufficient-balance"
        );
        assert_eq!(problem.title, "InsufficientBalanceError");
        assert_eq!(problem.status, 402);
    }

    #[test]
    fn test_problem_details_with_challenge_id() {
        let problem = PaymentErrorDetails::core("test-error")
            .with_title("TestError")
            .with_challenge_id("abc123");

        assert_eq!(problem.challenge_id, Some("abc123".to_string()));
    }

    #[test]
    fn test_problem_details_serialize() {
        let problem = PaymentErrorDetails::core("verification-failed")
            .with_title("VerificationFailedError")
            .with_status(402)
            .with_detail("Payment verification failed.")
            .with_challenge_id("abc123");

        let json = serde_json::to_string(&problem).unwrap();
        assert!(json.contains("\"type\":"));
        assert!(json.contains("verification-failed"));
        assert!(json.contains("\"challengeId\":\"abc123\""));
    }

    #[test]
    fn test_malformed_credential_error() {
        let err = MppError::malformed_credential_default();
        assert_eq!(err.to_string(), "Credential is malformed.");

        let err = MppError::malformed_credential("invalid base64url");
        assert_eq!(
            err.to_string(),
            "Credential is malformed: invalid base64url."
        );

        let problem = err.to_problem_details(Some("test-id"));
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/malformed-credential"
        );
        assert_eq!(problem.title, "MalformedCredentialError");
        assert_eq!(problem.challenge_id, Some("test-id".to_string()));
    }

    #[test]
    fn test_invalid_challenge_error() {
        let err = MppError::invalid_challenge_default();
        assert_eq!(err.to_string(), "Challenge is invalid.");

        let err = MppError::invalid_challenge_id("abc123");
        assert_eq!(err.to_string(), "Challenge \"abc123\" is invalid.");

        let err = MppError::invalid_challenge_reason("expired");
        assert_eq!(err.to_string(), "Challenge is invalid: expired.");

        let err = MppError::invalid_challenge("abc123", "already used");
        assert_eq!(
            err.to_string(),
            "Challenge \"abc123\" is invalid: already used."
        );

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/invalid-challenge"
        );
        assert_eq!(problem.challenge_id, Some("abc123".to_string()));
    }

    #[test]
    fn test_verification_failed_error() {
        let err = MppError::verification_failed_default();
        assert_eq!(err.to_string(), "Payment verification failed.");

        let err = MppError::verification_failed("insufficient amount");
        assert_eq!(
            err.to_string(),
            "Payment verification failed: insufficient amount."
        );

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/verification-failed"
        );
        assert_eq!(problem.title, "VerificationFailedError");
    }

    #[test]
    fn test_payment_expired_error() {
        let err = MppError::payment_expired_default();
        assert_eq!(err.to_string(), "Payment has expired.");

        let err = MppError::payment_expired("2025-01-15T12:00:00Z");
        assert_eq!(err.to_string(), "Payment expired at 2025-01-15T12:00:00Z.");

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/payment-expired"
        );
    }

    #[test]
    fn test_payment_required_error() {
        let err = MppError::payment_required_default();
        assert_eq!(err.to_string(), "Payment is required.");

        let err = MppError::payment_required_realm("api.example.com");
        assert_eq!(
            err.to_string(),
            "Payment is required for \"api.example.com\"."
        );

        let err = MppError::payment_required_description("Premium content access");
        assert_eq!(
            err.to_string(),
            "Payment is required (Premium content access)."
        );

        let err = MppError::payment_required("api.example.com", "Premium access");
        assert_eq!(
            err.to_string(),
            "Payment is required for \"api.example.com\" (Premium access)."
        );

        let problem = err.to_problem_details(Some("chal-id"));
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/payment-required"
        );
        assert_eq!(problem.challenge_id, Some("chal-id".to_string()));
    }

    #[test]
    fn test_bad_request_error() {
        let err = MppError::bad_request_default();
        assert_eq!(err.to_string(), "Bad request.");

        let err = MppError::bad_request("invalid parameters");
        assert_eq!(err.to_string(), "Bad request: invalid parameters.");

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/bad-request"
        );
        assert_eq!(problem.title, "BadRequestError");
        assert_eq!(problem.status, 400);
    }

    #[test]
    fn test_unsupported_payment_method_problem_details() {
        let err = MppError::unsupported_method(&"lightning");
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/method-unsupported"
        );
        assert_eq!(problem.title, "PaymentMethodUnsupportedError");
        assert_eq!(problem.status, 400);
    }

    #[test]
    fn test_payment_action_required_error() {
        let err = MppError::payment_action_required_default();
        assert_eq!(err.to_string(), "Payment requires action.");

        let err = MppError::payment_action_required("requires_action");
        assert_eq!(err.to_string(), "Payment requires action: requires_action.");

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/payment-action-required"
        );
        assert_eq!(problem.title, "PaymentActionRequiredError");
        assert_eq!(problem.status, 402);
    }

    #[test]
    fn test_payment_insufficient_error() {
        let err = MppError::payment_insufficient_default();
        assert_eq!(err.to_string(), "Payment amount is insufficient.");

        let err = MppError::payment_insufficient("expected 1000, received 500");
        assert_eq!(
            err.to_string(),
            "Payment insufficient: expected 1000, received 500."
        );

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/payment-insufficient"
        );
        assert_eq!(problem.title, "PaymentInsufficientError");
        assert_eq!(problem.status, 402);
    }

    #[test]
    fn test_insufficient_balance_problem_details() {
        let err = MppError::InsufficientBalance(Some("requested 500, available 100".to_string()));
        assert_eq!(
            err.to_string(),
            "Insufficient balance: requested 500, available 100."
        );
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/insufficient-balance"
        );
        assert_eq!(problem.title, "InsufficientBalanceError");
        assert_eq!(problem.status, 402);

        let err = MppError::InsufficientBalance(None);
        assert_eq!(err.to_string(), "Insufficient balance.");
    }

    #[test]
    fn test_invalid_signature_problem_details() {
        let err = MppError::InvalidSignature(Some("ECDSA recovery failed".to_string()));
        assert_eq!(err.to_string(), "Invalid signature: ECDSA recovery failed.");
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/invalid-signature"
        );
        assert_eq!(problem.title, "InvalidSignatureError");
        assert_eq!(problem.status, 402);

        let err = MppError::InvalidSignature(None);
        assert_eq!(err.to_string(), "Invalid signature.");
    }

    #[test]
    fn test_signer_mismatch_problem_details() {
        let err = MppError::SignerMismatch(Some("expected 0x123, got 0x456".to_string()));
        assert_eq!(
            err.to_string(),
            "Signer mismatch: expected 0x123, got 0x456."
        );
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/signer-mismatch"
        );
        assert_eq!(problem.title, "SignerMismatchError");
        assert_eq!(problem.status, 402);

        let err = MppError::SignerMismatch(None);
        assert_eq!(
            err.to_string(),
            "Signer is not authorized for this channel."
        );
    }

    #[test]
    fn test_amount_exceeds_deposit_problem_details() {
        let err = MppError::AmountExceedsDeposit(Some("voucher exceeds deposit".to_string()));
        assert_eq!(
            err.to_string(),
            "Amount exceeds deposit: voucher exceeds deposit."
        );
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/amount-exceeds-deposit"
        );
        assert_eq!(problem.title, "AmountExceedsDepositError");
        assert_eq!(problem.status, 402);

        let err = MppError::AmountExceedsDeposit(None);
        assert_eq!(err.to_string(), "Voucher amount exceeds channel deposit.");
    }

    #[test]
    fn test_delta_too_small_problem_details() {
        let err = MppError::DeltaTooSmall(Some("increase below minimum".to_string()));
        assert_eq!(err.to_string(), "Delta too small: increase below minimum.");
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/delta-too-small"
        );
        assert_eq!(problem.title, "DeltaTooSmallError");
        assert_eq!(problem.status, 402);

        let err = MppError::DeltaTooSmall(None);
        assert_eq!(
            err.to_string(),
            "Amount increase below minimum voucher delta."
        );
    }

    #[test]
    fn test_channel_not_found_problem_details() {
        let err = MppError::ChannelNotFound(Some("no such channel".to_string()));
        assert_eq!(err.to_string(), "Channel not found: no such channel.");
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/channel-not-found"
        );
        assert_eq!(problem.title, "ChannelNotFoundError");
        assert_eq!(problem.status, 410);

        let err = MppError::ChannelNotFound(None);
        assert_eq!(err.to_string(), "No channel with this ID exists.");
    }

    #[test]
    fn test_channel_closed_problem_details() {
        let err = MppError::ChannelClosed(Some("channel is finalized on-chain".to_string()));
        assert_eq!(
            err.to_string(),
            "Channel closed: channel is finalized on-chain."
        );
        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/session/channel-finalized"
        );
        assert_eq!(problem.title, "ChannelClosedError");
        assert_eq!(problem.status, 410);

        let err = MppError::ChannelClosed(None);
        assert_eq!(err.to_string(), "Channel is closed.");
    }

    #[test]
    fn test_invalid_payload_error() {
        let err = MppError::invalid_payload_default();
        assert_eq!(err.to_string(), "Credential payload is invalid.");

        let err = MppError::invalid_payload("missing signature field");
        assert_eq!(
            err.to_string(),
            "Credential payload is invalid: missing signature field."
        );

        let problem = err.to_problem_details(None);
        assert_eq!(
            problem.problem_type,
            "https://paymentauth.org/problems/invalid-payload"
        );
        assert_eq!(problem.title, "InvalidPayloadError");
    }

    // --- Tempo error wrapping ---

    #[cfg(all(feature = "client", feature = "tempo"))]
    #[test]
    fn test_tempo_error_wraps_through_from() {
        use crate::client::tempo::TempoClientError;

        let tempo_err = TempoClientError::AccessKeyNotProvisioned;
        let mpp_err: MppError = tempo_err.into();
        assert!(matches!(mpp_err, MppError::Tempo(_)));
        assert_eq!(mpp_err.to_string(), "Access key not provisioned on wallet");
    }
}