Skip to content

Commit d46011b

Browse files
authored
Merge pull request #44 from auth0-lab/expected_update_undefined
fix: avoid undefined/null values in validityInfo.expectedUpdate
2 parents 6a3f223 + 7eeef77 commit d46011b

File tree

5 files changed

+96
-375
lines changed

5 files changed

+96
-375
lines changed

__tests__/issuing/deviceResponse.tests.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ describe('issuing a device response', () => {
138138
expect(validityInfo.signed).toEqual(signed);
139139
expect(validityInfo.validFrom).toEqual(signed);
140140
expect(validityInfo.validUntil).toEqual(validUntil);
141-
expect(validityInfo.expectedUpdate).toBeUndefined();
141+
expect(validityInfo).not.toHaveProperty('expectedUpdate');
142142
});
143143

144144
it('should contain the device namespaces', () => {
@@ -223,7 +223,7 @@ describe('issuing a device response', () => {
223223
expect(validityInfo.signed).toEqual(signed);
224224
expect(validityInfo.validFrom).toEqual(signed);
225225
expect(validityInfo.validUntil).toEqual(validUntil);
226-
expect(validityInfo.expectedUpdate).toBeUndefined();
226+
expect(validityInfo).not.toHaveProperty('expectedUpdate');
227227
});
228228

229229
it('should contain the device namespaces', () => {

__tests__/issuing/deviceResponseWithMac.tests.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ curves.forEach((c) => {
168168
expect(validityInfo.signed).toEqual(signed);
169169
expect(validityInfo.validFrom).toEqual(signed);
170170
expect(validityInfo.validUntil).toEqual(validUntil);
171-
expect(validityInfo.expectedUpdate).toBeUndefined();
171+
expect(validityInfo).not.toHaveProperty('expectedUpdate');
172172
});
173173

174174
it('should contain the device namespaces', () => {
@@ -252,7 +252,7 @@ curves.forEach((c) => {
252252
expect(validityInfo.signed).toEqual(signed);
253253
expect(validityInfo.validFrom).toEqual(signed);
254254
expect(validityInfo.validUntil).toEqual(validUntil);
255-
expect(validityInfo.expectedUpdate).toBeUndefined();
255+
expect(validityInfo).not.toHaveProperty('expectedUpdate');
256256
});
257257

258258
it('should contain the device namespaces', () => {

__tests__/issuing/issuingMDoc.tests.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ describe('issuing an MDOC', () => {
2020
validFrom.setMinutes(signed.getMinutes() + 5);
2121
const validUntil = new Date(signed);
2222
validUntil.setFullYear(signed.getFullYear() + 30);
23+
const expectedUpdate = new Date(signed);
24+
expectedUpdate.setFullYear(signed.getFullYear() + 1);
2325

2426
beforeAll(async () => {
2527
const issuerPrivateKey = ISSUER_PRIVATE_KEY_JWK;
@@ -53,6 +55,7 @@ describe('issuing an MDOC', () => {
5355
signed,
5456
validFrom,
5557
validUntil,
58+
expectedUpdate,
5659
})
5760
.addDeviceKeyInfo({ deviceKey: publicKeyJWK })
5861
.sign({
@@ -86,7 +89,7 @@ describe('issuing an MDOC', () => {
8689
expect(validityInfo.signed).toEqual(signed);
8790
expect(validityInfo.validFrom).toEqual(validFrom);
8891
expect(validityInfo.validUntil).toEqual(validUntil);
89-
expect(validityInfo.expectedUpdate).toBeUndefined();
92+
expect(validityInfo.expectedUpdate).toEqual(expectedUpdate);
9093
});
9194

9295
it('should use the correct digest alg', () => {

0 commit comments

Comments
 (0)