bindcar 0.6.0

// Copyright (c) 2025 Erick Bourgeois, firestoned
// SPDX-License-Identifier: MIT

//! Unit tests for zones module

use super::zones::*;
use std::collections::HashMap;

#[test]
fn test_zone_config_to_zone_file() {
    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec![
            "ns1.example.com.".to_string(),
            "ns2.example.com.".to_string(),
        ],
        name_server_ips: HashMap::new(),
        records: vec![
            DnsRecord {
                name: "www".to_string(),
                record_type: "A".to_string(),
                value: "192.0.2.1".to_string(),
                ttl: Some(300),
                priority: None,
            },
            DnsRecord {
                name: "@".to_string(),
                record_type: "MX".to_string(),
                value: "mail.example.com.".to_string(),
                ttl: None,
                priority: Some(10),
            },
        ],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();

    assert!(zone_file.contains("$TTL 3600"));
    assert!(zone_file.contains("@ IN SOA ns1.example.com. admin.example.com."));
    assert!(zone_file.contains("2025010101"));
    assert!(zone_file.contains("@ IN NS ns1.example.com."));
    assert!(zone_file.contains("@ IN NS ns2.example.com."));
    assert!(zone_file.contains("www 300 IN A 192.0.2.1"));
    assert!(zone_file.contains("@ IN MX 10 mail.example.com."));
}

#[test]
fn test_soa_record_with_defaults() {
    let json = r#"{
        "primaryNs": "ns1.example.com.",
        "adminEmail": "admin.example.com.",
        "serial": 2025010101
    }"#;

    let soa: SoaRecord = serde_json::from_str(json).unwrap();
    assert_eq!(soa.primary_ns, "ns1.example.com.");
    assert_eq!(soa.admin_email, "admin.example.com.");
    assert_eq!(soa.serial, 2025010101);
    assert_eq!(soa.refresh, 3600);
    assert_eq!(soa.retry, 600);
    assert_eq!(soa.expire, 604_800);
    assert_eq!(soa.negative_ttl, 86400);
}

#[test]
fn test_soa_record_default_serial() {
    // Test that serial defaults to YYYYMMDD01 format when not provided
    let json = r#"{
        "primaryNs": "ns1.example.com.",
        "adminEmail": "admin.example.com."
    }"#;

    let soa: SoaRecord = serde_json::from_str(json).unwrap();
    assert_eq!(soa.primary_ns, "ns1.example.com.");
    assert_eq!(soa.admin_email, "admin.example.com.");

    // Serial should be in YYYYMMDD01 format (10 digits, ending in 01)
    let serial_str = soa.serial.to_string();
    assert_eq!(serial_str.len(), 10, "Serial should be 10 digits");
    assert!(serial_str.ends_with("01"), "Serial should end with 01");

    // Verify default values for other fields
    assert_eq!(soa.refresh, 3600);
    assert_eq!(soa.retry, 600);
    assert_eq!(soa.expire, 604_800);
    assert_eq!(soa.negative_ttl, 86400);
}

#[test]
fn test_create_zone_request_deserialization() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "primary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {
                "ns1.example.com.": "192.0.2.1"
            },
            "records": []
        },
        "updateKeyName": "test-key"
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_name, "example.com");
    assert_eq!(request.zone_type, "primary");
    assert_eq!(request.zone_config.ttl, 3600);
    assert_eq!(request.update_key_name, Some("test-key".to_string()));
}

#[test]
fn test_create_zone_request_without_update_key() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "primary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {}
        }
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_name, "example.com");
    assert!(request.update_key_name.is_none());
}

#[test]
fn test_dns_record_without_optional_fields() {
    let json = r#"{
        "name": "www",
        "type": "A",
        "value": "192.0.2.1"
    }"#;

    let record: DnsRecord = serde_json::from_str(json).unwrap();
    assert_eq!(record.name, "www");
    assert_eq!(record.record_type, "A");
    assert_eq!(record.value, "192.0.2.1");
    assert!(record.ttl.is_none());
    assert!(record.priority.is_none());
}

#[test]
fn test_zone_response_serialization() {
    let response = ZoneResponse {
        success: true,
        message: "Zone created".to_string(),
        details: Some("Output".to_string()),
    };

    let json = serde_json::to_string(&response).unwrap();
    assert!(json.contains("\"success\":true"));
    assert!(json.contains("\"message\":\"Zone created\""));
}

// Negative test cases

#[test]
fn test_zone_config_empty_name_servers() {
    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec![],
        name_server_ips: HashMap::new(),
        records: vec![],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();
    assert!(zone_file.contains("$TTL 3600"));
    assert!(zone_file.contains("@ IN SOA"));
    // Should not have any NS records
    assert!(!zone_file.contains("@ IN NS"));
}

#[test]
fn test_zone_config_special_characters_in_names() {
    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: HashMap::new(),
        records: vec![DnsRecord {
            name: "_dmarc".to_string(),
            record_type: "TXT".to_string(),
            value: "v=DMARC1; p=none".to_string(),
            ttl: None,
            priority: None,
        }],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();
    assert!(zone_file.contains("_dmarc"));
    assert!(zone_file.contains("TXT"));
}

#[test]
fn test_zone_config_zero_ttl() {
    let config = ZoneConfig {
        ttl: 0,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 0,
            retry: 0,
            expire: 0,
            negative_ttl: 0,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: HashMap::new(),
        records: vec![],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();
    assert!(zone_file.contains("$TTL 0"));
}

#[test]
fn test_dns_record_mx_with_priority_zero() {
    let record = DnsRecord {
        name: "@".to_string(),
        record_type: "MX".to_string(),
        value: "mail.example.com.".to_string(),
        ttl: None,
        priority: Some(0),
    };

    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: HashMap::new(),
        records: vec![record],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();
    assert!(zone_file.contains("@ IN MX 0 mail.example.com."));
}

#[test]
fn test_multiple_records_same_name() {
    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: HashMap::new(),
        records: vec![
            DnsRecord {
                name: "@".to_string(),
                record_type: "A".to_string(),
                value: "192.0.2.1".to_string(),
                ttl: None,
                priority: None,
            },
            DnsRecord {
                name: "@".to_string(),
                record_type: "A".to_string(),
                value: "192.0.2.2".to_string(),
                ttl: None,
                priority: None,
            },
        ],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();
    assert!(zone_file.contains("@ IN A 192.0.2.1"));
    assert!(zone_file.contains("@ IN A 192.0.2.2"));
}

#[test]
fn test_zone_config_with_nameserver_glue_records() {
    let mut ns_ips = HashMap::new();
    ns_ips.insert("ns1.example.com.".to_string(), "192.0.2.1".to_string());
    ns_ips.insert("ns2.example.com.".to_string(), "192.0.2.2".to_string());

    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec![
            "ns1.example.com.".to_string(),
            "ns2.example.com.".to_string(),
        ],
        name_server_ips: ns_ips,
        records: vec![],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();

    // Check that NS records are present
    assert!(zone_file.contains("@ IN NS ns1.example.com."));
    assert!(zone_file.contains("@ IN NS ns2.example.com."));

    // Check that glue records (A records for nameservers) are present with trailing dots
    assert!(zone_file.contains("ns1.example.com. IN A 192.0.2.1"));
    assert!(zone_file.contains("ns2.example.com. IN A 192.0.2.2"));
}

#[test]
fn test_zone_config_glue_records_serialization() {
    let mut ns_ips = HashMap::new();
    ns_ips.insert("ns1.example.com.".to_string(), "192.0.2.1".to_string());

    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: ns_ips,
        records: vec![],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    // Test that it can be serialized to JSON
    let json = serde_json::to_string(&config).unwrap();
    assert!(json.contains("nameServerIps"));

    // Test that it can be deserialized from JSON
    let deserialized: ZoneConfig = serde_json::from_str(&json).unwrap();
    assert!(!deserialized.name_server_ips.is_empty());
    assert_eq!(
        deserialized.name_server_ips.get("ns1.example.com."),
        Some(&"192.0.2.1".to_string())
    );
}

#[test]
fn test_zone_config_without_nameserver_ips() {
    let config = ZoneConfig {
        ttl: 3600,
        soa: SoaRecord {
            primary_ns: "ns1.example.com.".to_string(),
            admin_email: "admin.example.com.".to_string(),
            serial: 2025010101,
            refresh: 3600,
            retry: 600,
            expire: 604_800,
            negative_ttl: 86400,
        },
        name_servers: vec!["ns1.example.com.".to_string()],
        name_server_ips: HashMap::new(),
        records: vec![],
        also_notify: None,
        allow_transfer: None,
        primaries: None,
        dnssec_policy: None,
        inline_signing: None,
    };

    let zone_file = config.to_zone_file();

    // Should have NS records
    assert!(zone_file.contains("@ IN NS ns1.example.com."));

    // Should NOT have A records for nameservers when name_server_ips is empty
    assert!(!zone_file.contains("ns1.example.com IN A"));
}

#[test]
fn test_zone_config_with_also_notify() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "alsoNotify": ["10.244.2.101", "10.244.2.102"]
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.also_notify.is_some());
    let also_notify = config.also_notify.unwrap();
    assert_eq!(also_notify.len(), 2);
    assert_eq!(also_notify[0], "10.244.2.101");
    assert_eq!(also_notify[1], "10.244.2.102");
}

#[test]
fn test_zone_config_with_allow_transfer() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "allowTransfer": ["10.244.2.101", "10.244.2.102", "10.244.2.103"]
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.allow_transfer.is_some());
    let allow_transfer = config.allow_transfer.unwrap();
    assert_eq!(allow_transfer.len(), 3);
    assert_eq!(allow_transfer[0], "10.244.2.101");
    assert_eq!(allow_transfer[1], "10.244.2.102");
    assert_eq!(allow_transfer[2], "10.244.2.103");
}

#[test]
fn test_zone_config_with_both_notify_and_transfer() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "alsoNotify": ["10.244.2.101", "10.244.2.102"],
        "allowTransfer": ["10.244.2.101", "10.244.2.102"]
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.also_notify.is_some());
    assert!(config.allow_transfer.is_some());

    let also_notify = config.also_notify.unwrap();
    let allow_transfer = config.allow_transfer.unwrap();

    assert_eq!(also_notify.len(), 2);
    assert_eq!(allow_transfer.len(), 2);
    assert_eq!(also_notify, allow_transfer); // Same IPs
}

#[test]
fn test_zone_config_without_notify_and_transfer() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": []
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.also_notify.is_none());
    assert!(config.allow_transfer.is_none());
}

#[test]
fn test_zone_config_with_empty_notify_and_transfer() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "alsoNotify": [],
        "allowTransfer": []
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.also_notify.is_some());
    assert!(config.allow_transfer.is_some());
    assert_eq!(config.also_notify.unwrap().len(), 0);
    assert_eq!(config.allow_transfer.unwrap().len(), 0);
}

#[test]
fn test_create_zone_request_with_zone_transfer_fields() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "primary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {},
            "records": [],
            "alsoNotify": ["10.244.2.101", "10.244.2.102"],
            "allowTransfer": ["10.244.2.101", "10.244.2.102"]
        }
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_name, "example.com");
    assert_eq!(request.zone_type, "primary");
    assert!(request.zone_config.also_notify.is_some());
    assert!(request.zone_config.allow_transfer.is_some());

    let also_notify = request.zone_config.also_notify.unwrap();
    let allow_transfer = request.zone_config.allow_transfer.unwrap();

    assert_eq!(also_notify.len(), 2);
    assert_eq!(allow_transfer.len(), 2);
}

// Tests for secondary zones with primaries

#[test]
fn test_zone_config_with_primaries() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "primaries": ["192.0.2.1", "192.0.2.2"]
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.primaries.is_some());
    let primaries = config.primaries.unwrap();
    assert_eq!(primaries.len(), 2);
    assert_eq!(primaries[0], "192.0.2.1");
    assert_eq!(primaries[1], "192.0.2.2");
}

#[test]
fn test_create_secondary_zone_request() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "secondary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {},
            "records": [],
            "primaries": ["192.0.2.1", "192.0.2.2"]
        }
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_name, "example.com");
    assert_eq!(request.zone_type, "secondary");
    assert!(request.zone_config.primaries.is_some());

    let primaries = request.zone_config.primaries.unwrap();
    assert_eq!(primaries.len(), 2);
    assert_eq!(primaries[0], "192.0.2.1");
    assert_eq!(primaries[1], "192.0.2.2");
}

#[test]
fn test_secondary_zone_with_single_primary() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "secondary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {},
            "records": [],
            "primaries": ["192.0.2.1"]
        }
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_type, "secondary");
    assert!(request.zone_config.primaries.is_some());

    let primaries = request.zone_config.primaries.unwrap();
    assert_eq!(primaries.len(), 1);
    assert_eq!(primaries[0], "192.0.2.1");
}

#[test]
fn test_zone_config_without_primaries() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": []
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.primaries.is_none());
}

#[test]
fn test_zone_config_with_empty_primaries() {
    let json = r#"{
        "ttl": 3600,
        "soa": {
            "primaryNs": "ns1.example.com.",
            "adminEmail": "admin.example.com.",
            "serial": 2025010101
        },
        "nameServers": ["ns1.example.com."],
        "nameServerIps": {},
        "records": [],
        "primaries": []
    }"#;

    let config: ZoneConfig = serde_json::from_str(json).unwrap();
    assert!(config.primaries.is_some());
    assert_eq!(config.primaries.unwrap().len(), 0);
}

#[test]
fn test_secondary_zone_with_all_transfer_fields() {
    let json = r#"{
        "zoneName": "example.com",
        "zoneType": "secondary",
        "zoneConfig": {
            "ttl": 3600,
            "soa": {
                "primaryNs": "ns1.example.com.",
                "adminEmail": "admin.example.com.",
                "serial": 2025010101
            },
            "nameServers": ["ns1.example.com."],
            "nameServerIps": {},
            "records": [],
            "primaries": ["192.0.2.1", "192.0.2.2"],
            "alsoNotify": ["10.244.2.101"],
            "allowTransfer": ["10.244.2.101", "10.244.2.102"]
        }
    }"#;

    let request: CreateZoneRequest = serde_json::from_str(json).unwrap();
    assert_eq!(request.zone_type, "secondary");
    assert!(request.zone_config.primaries.is_some());
    assert!(request.zone_config.also_notify.is_some());
    assert!(request.zone_config.allow_transfer.is_some());

    let primaries = request.zone_config.primaries.unwrap();
    let also_notify = request.zone_config.also_notify.unwrap();
    let allow_transfer = request.zone_config.allow_transfer.unwrap();

    assert_eq!(primaries.len(), 2);
    assert_eq!(also_notify.len(), 1);
    assert_eq!(allow_transfer.len(), 2);
}

// Tests for ModifyZoneRequest

#[test]
fn test_modify_zone_request_with_both_fields() {
    let json = r#"{
        "alsoNotify": ["10.244.2.101", "10.244.2.102"],
        "allowTransfer": ["10.244.2.103", "10.244.2.104"]
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_some());
    assert!(request.allow_transfer.is_some());

    let also_notify = request.also_notify.unwrap();
    let allow_transfer = request.allow_transfer.unwrap();

    assert_eq!(also_notify.len(), 2);
    assert_eq!(also_notify[0], "10.244.2.101");
    assert_eq!(also_notify[1], "10.244.2.102");

    assert_eq!(allow_transfer.len(), 2);
    assert_eq!(allow_transfer[0], "10.244.2.103");
    assert_eq!(allow_transfer[1], "10.244.2.104");
}

#[test]
fn test_modify_zone_request_only_also_notify() {
    let json = r#"{
        "alsoNotify": ["10.244.2.101"]
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_some());
    assert!(request.allow_transfer.is_none());

    let also_notify = request.also_notify.unwrap();
    assert_eq!(also_notify.len(), 1);
    assert_eq!(also_notify[0], "10.244.2.101");
}

#[test]
fn test_modify_zone_request_only_allow_transfer() {
    let json = r#"{
        "allowTransfer": ["10.244.2.101", "10.244.2.102", "10.244.2.103"]
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_none());
    assert!(request.allow_transfer.is_some());

    let allow_transfer = request.allow_transfer.unwrap();
    assert_eq!(allow_transfer.len(), 3);
    assert_eq!(allow_transfer[0], "10.244.2.101");
    assert_eq!(allow_transfer[1], "10.244.2.102");
    assert_eq!(allow_transfer[2], "10.244.2.103");
}

#[test]
fn test_modify_zone_request_with_empty_arrays() {
    let json = r#"{
        "alsoNotify": [],
        "allowTransfer": []
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_some());
    assert!(request.allow_transfer.is_some());

    assert_eq!(request.also_notify.unwrap().len(), 0);
    assert_eq!(request.allow_transfer.unwrap().len(), 0);
}

#[test]
fn test_modify_zone_request_empty_json() {
    let json = r#"{}"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_none());
    assert!(request.allow_transfer.is_none());
}

#[test]
fn test_modify_zone_request_serialization() {
    let request = ModifyZoneRequest {
        also_notify: Some(vec!["10.244.2.101".to_string()]),
        allow_transfer: Some(vec!["10.244.2.102".to_string()]),
        allow_update: Some(vec!["10.244.2.103".to_string()]),
    };

    let json = serde_json::to_string(&request).unwrap();
    assert!(json.contains("alsoNotify"));
    assert!(json.contains("allowTransfer"));
    assert!(json.contains("allowUpdate"));
    assert!(json.contains("10.244.2.101"));
    assert!(json.contains("10.244.2.102"));
    assert!(json.contains("10.244.2.103"));
}

#[test]
fn test_modify_zone_request_serialization_skip_none() {
    let request = ModifyZoneRequest {
        also_notify: Some(vec!["10.244.2.101".to_string()]),
        allow_transfer: None,
        allow_update: None,
    };

    let json = serde_json::to_string(&request).unwrap();
    assert!(json.contains("alsoNotify"));
    assert!(!json.contains("allowTransfer")); // Should be skipped when None
    assert!(!json.contains("allowUpdate")); // Should be skipped when None
}

#[test]
fn test_modify_zone_request_with_ipv6_addresses() {
    let json = r#"{
        "alsoNotify": ["2001:db8::1", "2001:db8::2"],
        "allowTransfer": ["2001:db8::3"]
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    assert!(request.also_notify.is_some());
    assert!(request.allow_transfer.is_some());

    let also_notify = request.also_notify.unwrap();
    let allow_transfer = request.allow_transfer.unwrap();

    assert_eq!(also_notify.len(), 2);
    assert_eq!(also_notify[0], "2001:db8::1");
    assert_eq!(allow_transfer[0], "2001:db8::3");
}

#[test]
fn test_modify_zone_request_with_mixed_ip_versions() {
    let json = r#"{
        "alsoNotify": ["10.244.2.101", "2001:db8::1"],
        "allowTransfer": ["192.168.1.1", "fe80::1"]
    }"#;

    let request: ModifyZoneRequest = serde_json::from_str(json).unwrap();
    let also_notify = request.also_notify.unwrap();
    let allow_transfer = request.allow_transfer.unwrap();

    assert_eq!(also_notify.len(), 2);
    assert_eq!(also_notify[0], "10.244.2.101");
    assert_eq!(also_notify[1], "2001:db8::1");

    assert_eq!(allow_transfer.len(), 2);
    assert_eq!(allow_transfer[0], "192.168.1.1");
    assert_eq!(allow_transfer[1], "fe80::1");
}

// Tests for journal file cleanup behavior

#[tokio::test]
async fn test_journal_file_cleanup_on_zone_creation() {
    use tempfile::TempDir;

    // Create a temporary directory for testing
    let temp_dir = TempDir::new().unwrap();
    let zone_name = "test-journal-create.com";

    // Create a fake old journal file
    let journal_file_name = format!("{}.zone.jnl", zone_name);
    let journal_file_path = temp_dir.path().join(&journal_file_name);
    tokio::fs::write(&journal_file_path, "old journal data")
        .await
        .unwrap();

    // Verify the journal file exists
    assert!(journal_file_path.exists());

    // Simulate the cleanup logic from create_zone
    if journal_file_path.exists() {
        tokio::fs::remove_file(&journal_file_path).await.unwrap();
    }

    // Verify the journal file was removed
    assert!(!journal_file_path.exists());
}

#[tokio::test]
async fn test_journal_file_cleanup_on_zone_deletion() {
    use tempfile::TempDir;

    // Create a temporary directory for testing
    let temp_dir = TempDir::new().unwrap();
    let zone_name = "test-journal-delete.com";

    // Create fake zone and journal files
    let zone_file_name = format!("{}.zone", zone_name);
    let zone_file_path = temp_dir.path().join(&zone_file_name);
    tokio::fs::write(&zone_file_path, "zone data")
        .await
        .unwrap();

    let journal_file_name = format!("{}.zone.jnl", zone_name);
    let journal_file_path = temp_dir.path().join(&journal_file_name);
    tokio::fs::write(&journal_file_path, "journal data")
        .await
        .unwrap();

    // Verify both files exist
    assert!(zone_file_path.exists());
    assert!(journal_file_path.exists());

    // Simulate the cleanup logic from delete_zone
    if zone_file_path.exists() {
        tokio::fs::remove_file(&zone_file_path).await.unwrap();
    }

    if journal_file_path.exists() {
        tokio::fs::remove_file(&journal_file_path).await.unwrap();
    }

    // Verify both files were removed
    assert!(!zone_file_path.exists());
    assert!(!journal_file_path.exists());
}

#[tokio::test]
async fn test_journal_file_cleanup_when_journal_does_not_exist() {
    use tempfile::TempDir;

    // Create a temporary directory for testing
    let temp_dir = TempDir::new().unwrap();
    let zone_name = "test-no-journal.com";

    // Create only the zone file, no journal
    let zone_file_name = format!("{}.zone", zone_name);
    let zone_file_path = temp_dir.path().join(&zone_file_name);
    tokio::fs::write(&zone_file_path, "zone data")
        .await
        .unwrap();

    let journal_file_name = format!("{}.zone.jnl", zone_name);
    let journal_file_path = temp_dir.path().join(&journal_file_name);

    // Verify journal doesn't exist
    assert!(!journal_file_path.exists());

    // Simulate the cleanup logic - should not error when journal doesn't exist
    if journal_file_path.exists() {
        tokio::fs::remove_file(&journal_file_path).await.unwrap();
    }

    // Should complete without errors
    assert!(!journal_file_path.exists());
}

#[test]
fn test_journal_file_naming_convention() {
    // Test that journal file names follow the expected pattern
    let zone_name = "example.com";
    let expected_zone_file = format!("{}.zone", zone_name);
    let expected_journal_file = format!("{}.zone.jnl", zone_name);

    assert_eq!(expected_zone_file, "example.com.zone");
    assert_eq!(expected_journal_file, "example.com.zone.jnl");

    // Test with subdomain
    let zone_name = "sub.example.com";
    let expected_journal_file = format!("{}.zone.jnl", zone_name);
    assert_eq!(expected_journal_file, "sub.example.com.zone.jnl");
}