use jest-specific functions instead of reaching into mock calls for e…

…xepcted values
elastic · Aug 9, 2021 · ba1857e · ba1857e
1 parent 8dd966f
commit ba1857e
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/x-pack/plugins/rule_registry/server/alert_data_client/tests/bulk_update.test.ts b/x-pack/plugins/rule_registry/server/alert_data_client/tests/bulk_update.test.ts
@@ -209,8 +209,8 @@ describe('bulkUpdate()', () => {
             status: 'closed',
           })
         ).rejects.toThrowErrorMatchingInlineSnapshot(`"Unauthorized for fake.rule and apm"`);
-        expect(auditLogger.log.mock.calls).toHaveLength(2);
-        expect(auditLogger.log.mock.calls[0][0]).toEqual({
+        expect(auditLogger.log).toHaveBeenCalledTimes(2);
+        expect(auditLogger.log).toHaveBeenNthCalledWith(1, {
           message: `Failed attempt to update alert [id=${unsuccessfulAuthzHit}]`,
           event: {
             action: 'alert_update',
@@ -223,7 +223,7 @@ describe('bulkUpdate()', () => {
             message: 'Unauthorized for fake.rule and apm',
           },
         });
-        expect(auditLogger.log.mock.calls[1][0]).toEqual({
+        expect(auditLogger.log).toHaveBeenNthCalledWith(2, {
           message: `Failed attempt to update alert [id=${successfulAuthzHit}]`,
           event: {
             action: 'alert_update',
@@ -422,8 +422,8 @@ describe('bulkUpdate()', () => {
                 Error: Error: Unauthorized for fake.rule and apm"
               `);
 
-        expect(auditLogger.log.mock.calls).toHaveLength(2);
-        expect(auditLogger.log.mock.calls[0][0]).toEqual({
+        expect(auditLogger.log).toHaveBeenCalledTimes(2);
+        expect(auditLogger.log).toHaveBeenNthCalledWith(1, {
           message: `Failed attempt to update alert [id=${unsuccessfulAuthzHit}]`,
           event: {
             action: 'alert_update',
@@ -436,7 +436,7 @@ describe('bulkUpdate()', () => {
             message: 'Unauthorized for fake.rule and apm',
           },
         });
-        expect(auditLogger.log.mock.calls[1][0]).toEqual({
+        expect(auditLogger.log).toHaveBeenNthCalledWith(2, {
           message: `Failed attempt to update alert [id=${successfulAuthzHit}]`,
           event: {
             action: 'alert_update',