Missing integration test coverage for holding revoked credential

[lenkan]

Previously, we had these lines commented out in the credentials test:

signify-ts/examples/integration-scripts/credentials.test.ts

Lines 311 to 336 in 2a95620

	// Recipient check revoked credential
	// let revoked = false
	// while (!revoked) {
	// let cred2 = await client2.credentials().get(holderAidName, creds1[0].sad.d)
	// if (cred2.status.s == "1") {
	// revoked = true
	// }
	// await new Promise((resolve) => setTimeout(resolve, 250));
	// }
	// assert.equal(creds2.length, 1)
	// assert.equal(creds2[0].sad.s, schemaSAID)
	// assert.equal(creds2[0].sad.i, aid1.prefix)
	// assert.equal(creds2[0].status.s, "1") // 1 = revoked
	// console.log("Revocation received by recipient")
	// Present revoked credential
	// await client1
	// .credentials()
	// .present(issuerAidName, creds1[0].sad.d, verifierAidName, true);
	// await new Promise((resolve) => setTimeout(resolve, 5000));
	// creds3 = await client3.credentials().list(verifierAidName);
	// assert.equal(creds3.length, 1);
	// assert.equal(creds3[0].sad.s, schemaSAID);
	// assert.equal(creds3[0].sad.i, aid1.prefix);
	// assert.equal(creds3[0].status.s, '1'); // 1 = revoked
	// console.log('Revocation presented and received by verifier');

.

We should introduce coverage for this feature again. However, just adding the same check does not work currently with new IPEX.

[pfeairheller]

Fixed with PR #225

[lenkan]

Fixed with PR #225

@pfeairheller It looks like #225 only includes the revocation itself and only involves the issuer? The issue above was about how the holder of a credential is made aware of a revocation.

Perhaps this should be closed anyway as a "working as intended". But I do think it would be good with an example of how a holder would be made aware that their credential revocation.