Skip to content

Commit 6ff959f

Browse files
committed
Add test with original value; udpate test for expected audit event types
1 parent 0b60d24 commit 6ff959f

2 files changed

Lines changed: 30 additions & 10 deletions

File tree

packages/components/src/internal/components/auditlog/AuditDetails.test.tsx

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,17 +89,37 @@ describe('AuditDetails', () => {
8989
test('changeDetails', () => {
9090
renderWithAppContext(
9191
<AuditDetails
92-
rowId={1}
93-
user={TEST_USER_APP_ADMIN}
9492
changeDetails={AuditDetailsModel.create({
9593
oldData: { a: 1 },
9694
newData: { a: 2 },
9795
})}
96+
rowId={1}
97+
user={TEST_USER_APP_ADMIN}
9898
/>,
9999
{ serverContext: { user: TEST_USER_APP_ADMIN } }
100100
);
101101
expect(document.querySelectorAll('.table-responsive')).toHaveLength(0);
102102
expect(document.querySelectorAll('.user-link')).toHaveLength(0);
103103
expect(document.querySelector('.panel-body').textContent).toBe('a12');
104104
});
105+
test('original data', () => {
106+
renderWithAppContext(
107+
<AuditDetails
108+
changeDetails={AuditDetailsModel.create({
109+
oldData: { a: 'file.txt' },
110+
newData: { a: 'new-1.txt' },
111+
originalValues: {
112+
a: 'new.txt',
113+
},
114+
})}
115+
rowId={1}
116+
user={TEST_USER_APP_ADMIN}
117+
/>,
118+
{ serverContext: { user: TEST_USER_APP_ADMIN } }
119+
);
120+
expect(document.querySelectorAll('.table-responsive')).toHaveLength(0);
121+
expect(document.querySelectorAll('.user-link')).toHaveLength(0);
122+
expect(document.querySelector('.panel-body').textContent).toBe('afile.txtnew-1.txt');
123+
expect(document.querySelector('.original-value-icon')).toBeInTheDocument();
124+
});
105125
});

packages/components/src/internal/components/auditlog/utils.test.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,26 +28,26 @@ import {
2828
describe('getAuditQueries', () => {
2929
test('LKS starter', () => {
3030
const auditQueries = getAuditQueries(TEST_LKS_STARTER_MODULE_CONTEXT);
31-
expect(auditQueries.length).toBe(12);
31+
expect(auditQueries.length).toBe(13);
3232
expect(auditQueries.findIndex(entry => entry === ASSAY_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
33-
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(10);
33+
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(11);
3434
expect(auditQueries.findIndex(entry => entry === WORKFLOW_AUDIT_QUERY)).toBe(-1);
3535
expect(auditQueries.findIndex(entry => entry === SOURCE_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
3636
});
3737

3838
test('LKSM starter', () => {
3939
const auditQueries = getAuditQueries(TEST_LKSM_STARTER_MODULE_CONTEXT);
40-
expect(auditQueries.length).toBe(11);
41-
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(9);
40+
expect(auditQueries.length).toBe(12);
41+
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(10);
4242
expect(auditQueries.findIndex(entry => entry === ASSAY_AUDIT_QUERY)).toBe(-1);
4343
expect(auditQueries.findIndex(entry => entry === WORKFLOW_AUDIT_QUERY)).toBe(-1);
4444
expect(auditQueries.findIndex(entry => entry === SOURCE_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
4545
});
4646

4747
test('LKSM professional', () => {
4848
const auditQueries = getAuditQueries(TEST_LKSM_PROFESSIONAL_MODULE_CONTEXT);
49-
expect(auditQueries.length).toBe(15);
50-
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(13);
49+
expect(auditQueries.length).toBe(16);
50+
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(14);
5151
expect(auditQueries.findIndex(entry => entry === ASSAY_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
5252
expect(auditQueries.findIndex(entry => entry === WORKFLOW_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
5353
expect(auditQueries.findIndex(entry => entry === SOURCE_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
@@ -69,8 +69,8 @@ describe('getAuditQueries', () => {
6969
},
7070
};
7171
const auditQueries = getAuditQueries(moduleContext);
72-
expect(auditQueries.length).toBe(16);
73-
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(14);
72+
expect(auditQueries.length).toBe(17);
73+
expect(auditQueries.findIndex(entry => entry === INVENTORY_AUDIT_QUERY)).toBe(15);
7474
expect(auditQueries.findIndex(entry => entry === ASSAY_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
7575
expect(auditQueries.findIndex(entry => entry === WORKFLOW_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);
7676
expect(auditQueries.findIndex(entry => entry === NOTEBOOK_AUDIT_QUERY)).toBeGreaterThanOrEqual(0);

0 commit comments

Comments
 (0)