chore: linting

packages/core/src/credentials/V1/KiltAttestationProofV1.ts

@@ -735,6 +735,7 @@ export async function issue(
     )
   }
 
+  /* eslint-disable @typescript-eslint/no-non-null-assertion -- we've checked the appropriate combination of parameters above, but typescript does not follow */
   const didSigned = authorizeTx
     ? await authorizeTx(call)
     : await didAuthorizeTx(
@@ -748,6 +749,7 @@ export async function issue(
   const transactionPromise = submitTx
     ? submitTx(didSigned)
     : defaultTxSubmit(didSigned, submitterAccount!, signers!, api)
+  /* eslint-enable @typescript-eslint/no-non-null-assertion */
 
   const {
     includedAt: { blockHash, blockTime },

tests/testUtils/TestUtils.ts

@@ -140,7 +140,10 @@ export async function makeStoreDidSigner(
     signers,
     Signers.select.verifiableOnChain()
   )
-  return signer!
+  if (!signer) {
+    throw new Error('Failed to derive DID creation signer from keypair')
+  }
+  return signer
 }
 
 export interface KeyTool {

tests/testUtils/testData.ts

@@ -28,10 +28,10 @@ const attestationPalletIndex = 62
 mockedApi.once('ready', () => {
   const idx = mockedApi.runtimeMetadata.asLatest.pallets.find((x) =>
     x.name.match(/attestation/i)
-  )!.index
-  if (!idx.eqn(attestationPalletIndex)) {
+  )?.index
+  if (!idx?.eqn(attestationPalletIndex)) {
     console.warn(
-      `The attestation pallet index is expected to be ${attestationPalletIndex}, but the metadata used lists it as ${idx.toNumber()}. This may lead to tests not behaving as expected!`
+      `The attestation pallet index is expected to be ${attestationPalletIndex}, but the metadata used lists it as ${idx?.toNumber()}. This may lead to tests not behaving as expected!`
     )
   }
 })