test: un-ignore tests that now pass

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

3 files changed, 0 insertions, 11 deletions

diff --git a/crates/shirabe/tests/dependency_resolver/rule_set_test.rs b/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
index f1e573e..c1faa11 100644
--- a/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
@@ -45,7 +45,6 @@ fn iterator_count(mut iterator: shirabe::dependency_resolver::RuleSetIterator) -
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_add() {
     let request0 = rule(vec![1]);
     let request1 = rule(vec![2]);
@@ -78,7 +77,6 @@ fn test_add() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_add_ignores_duplicates() {
     let mut rule_set = RuleSet::new();
     rule_set.add(rule(vec![]), RuleSet::TYPE_REQUEST).unwrap();
@@ -100,7 +98,6 @@ fn test_add_when_type_is_not_recognized() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_count() {
     let mut rule_set = RuleSet::new();
     rule_set.add(rule(vec![1]), RuleSet::TYPE_REQUEST).unwrap();
@@ -110,7 +107,6 @@ fn test_count() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_rule_by_id() {
     let mut rule_set = RuleSet::new();
     let rule = rule(vec![]);
@@ -120,7 +116,6 @@ fn test_rule_by_id() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_get_iterator() {
     let mut rule_set = RuleSet::new();
     let rule1 = rule(vec![1]);
@@ -135,7 +130,6 @@ fn test_get_iterator() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_get_iterator_for() {
     let mut rule_set = RuleSet::new();
     let rule1 = rule(vec![1]);
@@ -148,7 +142,6 @@ fn test_get_iterator_for() {
 }
 
 #[test]
-#[ignore = "RuleSet::add computes Rule::get_hash, which reaches hash_raw (todo!()) in the php-shim"]
 fn test_get_iterator_without() {
     let mut rule_set = RuleSet::new();
     let rule1 = rule(vec![1]);
@@ -164,7 +157,6 @@ fn test_get_iterator_without() {
 // "No package found" branch is taken; the RepositorySet/Request collaborators are never
 // actually consulted (PHP mocks them with the constructor disabled).
 #[test]
-#[ignore]
 fn test_pretty_string() {
     let p = get_package("foo", "2.1");
     let mut pool = Pool::new(
diff --git a/crates/shirabe/tests/dependency_resolver/rule_test.rs b/crates/shirabe/tests/dependency_resolver/rule_test.rs
index 156931c..8216848 100644
--- a/crates/shirabe/tests/dependency_resolver/rule_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/rule_test.rs
@@ -26,7 +26,6 @@ fn generic_rule(literals: Vec<i64>) -> Rule {
     ))
 }
 
-#[ignore]
 #[test]
 fn test_get_hash() {
     let rule = generic_rule(vec![123]);
@@ -108,7 +107,6 @@ fn test_is_assertions() {
 
 // PHP mocks RepositorySet and Request with the constructor disabled; the RULE_PACKAGE_REQUIRES
 // branch with a non-empty requires list never consults them, so real minimal instances suffice.
-#[ignore]
 #[test]
 fn test_pretty_string() {
     let p1 = get_package("foo", "2.1");
diff --git a/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs b/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs
index 1232241..98ac131 100644
--- a/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs
@@ -32,7 +32,6 @@ fn test_dont_filter_packages_with_block_insecure_disabled() {
     todo!()
 }
 
-#[ignore]
 #[test]
 fn test_dont_filter_packages_with_abandoned_package() {
     let package_name_ignore_abandoned = "acme/ignore-abandoned";