gets rid of assertNil() in tests in favor of deepStrictEqual(), given…

… that error arrays are now always present
jacoscaz · Sep 9, 2024 · e9c6b7b · e9c6b7b
1 parent 94c7cda
commit e9c6b7b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/src/common/utils.test.ts b/src/common/utils.test.ts
diff --git a/src/ocpp16/call.test.ts b/src/ocpp16/call.test.ts
@@ -3,8 +3,6 @@ import { OCPP16 } from './index.js';
 import { describe, it } from 'node:test';
 import { deepStrictEqual } from 'node:assert';
 
-import { assertNil } from '../common/utils.test.js';
-
 const { MessageType, Action, validate } = OCPP16;
 
 describe('OCPP16 - Call', () => {
@@ -28,7 +26,7 @@ describe('OCPP16 - Call', () => {
  chargePointModel: 'test',
  chargePointVendor: '55',
  }]), true);
- assertNil(validate.errors);
+ deepStrictEqual(validate.errors, []);
  });
 
  it('invalid notification (missing model)', () => {

diff --git a/src/ocpp20/call.test.ts b/src/ocpp20/call.test.ts
@@ -3,8 +3,6 @@ import { OCPP20 } from './index.js';
 import { describe, it } from 'node:test';
 import { deepStrictEqual } from 'node:assert';
 
-import { assertNil } from '../common/utils.test.js';
-
 const { MessageType, Action, validate } = OCPP20;
 
 describe('OCPP20 - Call', () => {
@@ -33,7 +31,7 @@ describe('OCPP20 - Call', () => {
  },
  reason: 'PowerUp',
  }]), true);
- assertNil(validate.errors);
+ deepStrictEqual(validate.errors, []);
  });
 
  it('invalid notification (missing model)', () => {