crosslink 0.8.0

use anyhow::Result;

use crate::db::Database;
use crate::shared_writer::SharedWriter;
use crate::utils::format_issue_id;

pub fn add(db: &Database, writer: Option<&SharedWriter>, issue_id: i64, label: &str) -> Result<()> {
    db.require_issue(issue_id)?;

    if let Some(w) = writer {
        w.add_label(db, issue_id, label)?;
        println!(
            "Added label '{}' to issue {}",
            label,
            format_issue_id(issue_id)
        );
    } else if db.add_label(issue_id, label)? {
        println!(
            "Added label '{}' to issue {}",
            label,
            format_issue_id(issue_id)
        );
    } else {
        println!(
            "Label '{}' already exists on issue {}",
            label,
            format_issue_id(issue_id)
        );
    }
    Ok(())
}

pub fn remove(
    db: &Database,
    writer: Option<&SharedWriter>,
    issue_id: i64,
    label: &str,
) -> Result<()> {
    db.require_issue(issue_id)?;

    if let Some(w) = writer {
        w.remove_label(db, issue_id, label)?;
        println!(
            "Removed label '{}' from issue {}",
            label,
            format_issue_id(issue_id)
        );
    } else if db.remove_label(issue_id, label)? {
        println!(
            "Removed label '{}' from issue {}",
            label,
            format_issue_id(issue_id)
        );
    } else {
        println!(
            "Label '{}' not found on issue {}",
            label,
            format_issue_id(issue_id)
        );
    }
    Ok(())
}

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

    fn setup_test_db() -> (Database, tempfile::TempDir) {
        let dir = tempfile::tempdir().unwrap();
        let db_path = dir.path().join("test.db");
        let db = Database::open(&db_path).unwrap();
        (db, dir)
    }

    // ==================== Add Label Tests ====================

    #[test]
    fn test_add_label_to_existing_issue() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let result = add(&db, None, issue_id, "bug");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert!(labels.contains(&"bug".to_string()));
    }

    #[test]
    fn test_add_label_to_nonexistent_issue() {
        let (db, _dir) = setup_test_db();

        let result = add(&db, None, 99999, "bug");
        assert!(result.is_err());
        assert!(result.unwrap_err().to_string().contains("not found"));
    }

    #[test]
    fn test_add_duplicate_label() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        add(&db, None, issue_id, "bug").unwrap();
        let result = add(&db, None, issue_id, "bug"); // Duplicate
        assert!(result.is_ok()); // Should succeed but not add duplicate

        let labels = db.get_labels(issue_id).unwrap();
        assert_eq!(labels.len(), 1);
    }

    #[test]
    fn test_add_multiple_labels() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        add(&db, None, issue_id, "bug").unwrap();
        add(&db, None, issue_id, "urgent").unwrap();
        add(&db, None, issue_id, "backend").unwrap();

        let labels = db.get_labels(issue_id).unwrap();
        assert_eq!(labels.len(), 3);
        assert!(labels.contains(&"bug".to_string()));
        assert!(labels.contains(&"urgent".to_string()));
        assert!(labels.contains(&"backend".to_string()));
    }

    #[test]
    fn test_add_empty_label() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let result = add(&db, None, issue_id, "");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert!(labels.contains(&String::new()));
    }

    #[test]
    fn test_add_unicode_label() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let result = add(&db, None, issue_id, "バグ");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert!(labels.contains(&"バグ".to_string()));
    }

    #[test]
    fn test_add_label_with_special_chars() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let result = add(&db, None, issue_id, "high-priority");
        assert!(result.is_ok());

        let result = add(&db, None, issue_id, "v2.0");
        assert!(result.is_ok());

        let result = add(&db, None, issue_id, "team:backend");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert_eq!(labels.len(), 3);
    }

    #[test]
    fn test_add_label_sql_injection() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let malicious = "'; DROP TABLE labels; --";
        let result = add(&db, None, issue_id, malicious);
        assert!(result.is_ok());

        // Verify label was stored literally
        let labels = db.get_labels(issue_id).unwrap();
        assert!(labels.contains(&malicious.to_string()));

        // Verify database integrity
        let issues = db.list_issues(None, None, None).unwrap();
        assert!(!issues.is_empty());
    }

    // ==================== Remove Label Tests ====================

    #[test]
    fn test_remove_existing_label() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        add(&db, None, issue_id, "bug").unwrap();
        let result = remove(&db, None, issue_id, "bug");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert!(!labels.contains(&"bug".to_string()));
    }

    #[test]
    fn test_remove_nonexistent_label() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        let result = remove(&db, None, issue_id, "nonexistent");
        assert!(result.is_ok()); // Should succeed but report not found
    }

    #[test]
    fn test_remove_label_from_nonexistent_issue() {
        let (db, _dir) = setup_test_db();

        let result = remove(&db, None, 99999, "bug");
        assert!(result.is_err());
        assert!(result.unwrap_err().to_string().contains("not found"));
    }

    #[test]
    fn test_remove_one_of_many_labels() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();

        add(&db, None, issue_id, "bug").unwrap();
        add(&db, None, issue_id, "urgent").unwrap();
        add(&db, None, issue_id, "backend").unwrap();

        remove(&db, None, issue_id, "urgent").unwrap();

        let labels = db.get_labels(issue_id).unwrap();
        assert_eq!(labels.len(), 2);
        assert!(labels.contains(&"bug".to_string()));
        assert!(labels.contains(&"backend".to_string()));
        assert!(!labels.contains(&"urgent".to_string()));
    }

    #[test]
    fn test_add_label_to_closed_issue() {
        let (db, _dir) = setup_test_db();
        let issue_id = db.create_issue("Test issue", None, "medium").unwrap();
        db.close_issue(issue_id).unwrap();

        let result = add(&db, None, issue_id, "bug");
        assert!(result.is_ok());

        let labels = db.get_labels(issue_id).unwrap();
        assert!(labels.contains(&"bug".to_string()));
    }

    // ==================== Property-Based Tests ====================

    proptest! {
        #[test]
        fn prop_add_label_roundtrip(label in "[a-zA-Z0-9_\\-]{1,30}") {
            let (db, _dir) = setup_test_db();
            let issue_id = db.create_issue("Test", None, "medium").unwrap();

            add(&db, None, issue_id, &label).unwrap();

            let labels = db.get_labels(issue_id).unwrap();
            prop_assert!(labels.contains(&label));
        }

        #[test]
        fn prop_remove_label_works(label in "[a-zA-Z0-9_\\-]{1,30}") {
            let (db, _dir) = setup_test_db();
            let issue_id = db.create_issue("Test", None, "medium").unwrap();

            add(&db, None, issue_id, &label).unwrap();
            remove(&db, None, issue_id, &label).unwrap();

            let labels = db.get_labels(issue_id).unwrap();
            prop_assert!(!labels.contains(&label));
        }

        #[test]
        fn prop_nonexistent_issue_fails(issue_id in 1000i64..10000) {
            let (db, _dir) = setup_test_db();

            let add_result = add(&db, None, issue_id, "label");
            prop_assert!(add_result.is_err());

            let remove_result = remove(&db, None, issue_id, "label");
            prop_assert!(remove_result.is_err());
        }

        #[test]
        fn prop_multiple_labels_independent(
            labels in proptest::collection::vec("[a-zA-Z]{1,10}", 1..5)
        ) {
            let (db, _dir) = setup_test_db();
            let issue_id = db.create_issue("Test", None, "medium").unwrap();

            // Add all labels
            for label in &labels {
                add(&db, None, issue_id, label).unwrap();
            }

            // Remove first label
            if !labels.is_empty() {
                remove(&db, None, issue_id, &labels[0]).unwrap();

                let remaining = db.get_labels(issue_id).unwrap();
                prop_assert!(!remaining.contains(&labels[0]));

                // Others should still exist (unless they were the same as first)
                for label in labels.iter().skip(1) {
                    if label != &labels[0] {
                        prop_assert!(remaining.contains(label));
                    }
                }
            }
        }

        #[test]
        fn prop_unicode_labels_work(
            label in "[\\p{L}]{1,20}"
        ) {
            let (db, _dir) = setup_test_db();
            let issue_id = db.create_issue("Test", None, "medium").unwrap();

            let result = add(&db, None, issue_id, &label);
            prop_assert!(result.is_ok());

            let labels = db.get_labels(issue_id).unwrap();
            prop_assert!(labels.contains(&label));
        }
    }
}