ignore uncovered else conditions

Signed-off-by: Brian DeHamer <[email protected]>

Author: bdehamer

packages/core/src/x509/cert.ts

@@ -143,6 +143,7 @@ export class X509Certificate {
     const ca = this.extBasicConstraints?.isCA || false;
 
     // If the KeyUsage extension is present, keyCertSign must be set
+    /* istanbul ignore else */
     if (this.extKeyUsage) {
       return ca && this.extKeyUsage.keyCertSign;
     }

packages/oci/src/image.ts

@@ -62,6 +62,7 @@ export class OCIImage {
     };
 
     try {
+      /* istanbul ignore else */
       if (this.#credentials) {
         await this.#client.signIn(this.#credentials);
       }
@@ -127,6 +128,7 @@ export class OCIImage {
 
   async getDigest(tag: string): Promise<string> {
     try {
+      /* istanbul ignore else */
       if (this.#credentials) {
         await this.#client.signIn(this.#credentials);
       }
@@ -175,6 +177,7 @@ export class OCIImage {
 
     // If the artifact is not already in the index, add it to the list and
     // re-upload the index
+    /* istanbul ignore else */
     if (
       !referrerManifest.manifests.some(
         (manifest) => manifest.digest === opts.artifact.digest

packages/tuf/src/client.ts

@@ -97,6 +97,7 @@ function initTufCache(cachePath: string): void {
     fs.mkdirSync(cachePath, { recursive: true });
   }
 
+  /* istanbul ignore else */
   if (!fs.existsSync(targetsPath)) {
     fs.mkdirSync(targetsPath);
   }
@@ -120,10 +121,11 @@ function seedCache({
 
   // If the root.json file does not exist (or we're forcing re-initialization),
   // populate it either from the supplied rootPath or from one of the repo seeds.
+  /* istanbul ignore else */
   if (!fs.existsSync(cachedRootPath) || forceInit) {
     if (tufRootPath) {
       fs.copyFileSync(tufRootPath, cachedRootPath);
-    } else { 
+    } else {
       const seeds: RepoSeeds = require('../seeds.json');
       const repoSeed = seeds[mirrorURL];
 

packages/verify/src/key/certificate.ts

@@ -184,6 +184,7 @@ export class CertificateChainVerifier {
     // or issuer/subject. Potential issuers are added to the result array.
     this.localCerts.forEach((possibleIssuer) => {
       if (keyIdentifier) {
+        /* istanbul ignore else */
         if (possibleIssuer.extSubjectKeyID) {
           if (
             possibleIssuer.extSubjectKeyID.keyIdentifier.equals(keyIdentifier)

packages/verify/src/verifier.ts

@@ -162,6 +162,7 @@ export class Verifier {
     identity: CertificateIdentity
   ) {
     // Check the subject alternative name of the signer matches the policy
+    /* istanbul ignore else */
     if (policy.subjectAlternativeName) {
       verifySubjectAlternativeName(
         policy.subjectAlternativeName,
@@ -170,6 +171,7 @@ export class Verifier {
     }
 
     // Check that the extensions of the signer match the policy
+    /* istanbul ignore else */
     if (policy.extensions) {
       verifyExtensions(policy.extensions, identity.extensions);
     }