test: add test for non-integer cert revoked reason #864

Author: jaredhendrickson13

pfSense-pkg-RESTAPI/files/usr/local/pkg/RESTAPI/Tests/APIModelsCertificateRevocationListRevokedCertificateTestCase.inc

@@ -130,4 +130,27 @@ class APIModelsCertificateRevocationListRevokedCertificateTestCase extends TestC
             },
         );
     }
+
+    /**
+     * Ensures revoked certificates with non-numeric reasons are converted to -1 (unspecified) when reading #864
+     */
+    public function test_non_numeric_reason_conversion(): void
+    {
+        # Mock a revoked certificate with a non-numeric reason
+        Model::set_config(
+            "crl/{$this->crl->id}/cert/0",
+            [
+                'certref' => $this->cert->refid->value,
+                'reason' => 'some-non-numeric-reason',
+                'revoke_time' => time(),
+            ],
+        );
+
+        # Read the CRL and ensure the reason was converted to -1
+        $crl_revoked_cert = CertificateRevocationListRevokedCertificate::query(
+            parent_id: $this->crl->id,
+            id: 0
+        )->first();
+        $this->assert_equals(-1, $crl_revoked_cert->reason->value);
+    }
 }