Skip to content

Commit 893a87c

Browse files
author
Marco Franceschi
committed
chore: Added missing unit tests for comparison operators
1 parent 632ba76 commit 893a87c

File tree

3 files changed

+237
-8
lines changed

3 files changed

+237
-8
lines changed

‎src/rules-engine/evaluators/json-evaluator.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -138,10 +138,10 @@ export default class JsonEvaluator implements RuleEvaluator<JsonRule> {
138138
this.isCondition(otherArgs) &&
139139
otherArgs.path
140140
) {
141-
const otherData = { ..._data }
142-
const elementPath = this.calculatePath(otherData, otherArgs.path)
143-
otherData.elementPath = elementPath
144-
otherArgs = this.resolvePath(otherData, elementPath)
141+
const resourceData = { ..._data }
142+
const elementPath = this.calculatePath(resourceData, otherArgs.path)
143+
resourceData.elementPath = elementPath
144+
otherArgs = this.resolvePath(resourceData, elementPath)
145145
}
146146

147147
if (path) {

‎tests/json-evaluator.test.ts

Lines changed: 74 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,7 @@ describe('JsonEvaluator', () => {
422422
expect(finding.result).toBe(Result.PASS)
423423
})
424424

425-
test('Should passed using the same value for the equal operator with the same path', async () => {
425+
test('Should pass using the same value for the equal operator with the same path', async () => {
426426
const data = {
427427
data: { a: { b: [0, { e: 'same value', d: 'same value' }] } },
428428
}
@@ -441,9 +441,9 @@ describe('JsonEvaluator', () => {
441441
expect(finding.result).toBe(Result.PASS)
442442
})
443443

444-
test('Should failed using two different values for the equal operator', async () => {
444+
test('Should fail using two different values for the equal operator', async () => {
445445
const data = {
446-
data: { a: { b: [0, { e: 'not the same', d: 'value' }] } },
446+
data: { a: { b: [0, { e: 'not the same', d: 'values' }] } },
447447
}
448448
const rule = { path: 'a.b[1].d', equal: { path: 'a.b[1].e' } }
449449

@@ -459,4 +459,75 @@ describe('JsonEvaluator', () => {
459459

460460
expect(finding.result).toBe(Result.FAIL)
461461
})
462+
463+
test('Should fail using the same value for the equal operator but using a wrong path', async () => {
464+
const data = {
465+
data: { a: { b: [0, { e: 'values', d: 'values' }] } },
466+
}
467+
const rule = { path: 'a.b[1].d', equal: { path: 'b[0].e' } }
468+
469+
const finding = await evaluator.evaluateSingleResource(
470+
{
471+
conditions: rule,
472+
resource: {
473+
id: cuid(),
474+
},
475+
} as any,
476+
data
477+
)
478+
479+
expect(finding.result).toBe(Result.FAIL)
480+
})
481+
482+
test('Should pass comparing a date greater than other', async () => {
483+
const data = {
484+
data: {
485+
a: {
486+
b: [
487+
0,
488+
{ e: '2022-03-14T20:42:58.510Z', d: '2022-03-15T20:42:58.510Z' },
489+
],
490+
},
491+
},
492+
}
493+
const rule = { path: 'a.b[1].d', greaterThan: { path: 'a.b[1].e' } }
494+
495+
const finding = await evaluator.evaluateSingleResource(
496+
{
497+
conditions: rule,
498+
resource: {
499+
id: cuid(),
500+
},
501+
} as any,
502+
data
503+
)
504+
505+
expect(finding.result).toBe(Result.PASS)
506+
})
507+
508+
test('Should fail comparing a date after than other', async () => {
509+
const data = {
510+
data: {
511+
a: {
512+
b: [
513+
0,
514+
{ e: '2022-03-15T20:42:58.510Z', d: new Date().toISOString() },
515+
],
516+
},
517+
},
518+
}
519+
const rule = { path: 'a.b[1].d', lessThan: { path: 'a.b[1].e' } }
520+
521+
const finding = await evaluator.evaluateSingleResource(
522+
{
523+
conditions: rule,
524+
resource: {
525+
id: cuid(),
526+
},
527+
} as any,
528+
data
529+
)
530+
531+
expect(finding.result).toBe(Result.FAIL)
532+
})
462533
})

‎tests/operators/comparison.test.ts

Lines changed: 159 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,164 @@ import ComparisonOperators from '../../src/rules-engine/operators/comparison'
22

33
describe('Comparison Operators', () => {
44
describe('Equal Operator', () => {
5-
test.todo('Should pass given two equal values')
5+
test('Should pass given two equal strings', () => {
6+
const result = ComparisonOperators.equal('Switch', 'Switch')
7+
expect(result).toBeTruthy()
8+
})
9+
10+
test('Should fail given two different strings', () => {
11+
const result = ComparisonOperators.equal('PS5', 'Xbox Series X')
12+
expect(result).toBeFalsy()
13+
})
14+
test('Should pass given two different numbers', () => {
15+
const result = ComparisonOperators.equal(360, 360)
16+
expect(result).toBeTruthy()
17+
})
18+
test('Should fail given two different numbers', () => {
19+
const result = ComparisonOperators.equal(360, 90)
20+
expect(result).toBeFalsy()
21+
})
22+
test('Should pass given two equal values with different types', () => {
23+
const result = ComparisonOperators.equal(180, '180')
24+
expect(result).toBeTruthy()
25+
})
26+
})
27+
28+
describe('NotEqual Operator', () => {
29+
test('Should fail given two equal strings', () => {
30+
const result = ComparisonOperators.notEqual('Switch', 'Switch')
31+
expect(result).toBeFalsy()
32+
})
33+
34+
test('Should pass given two different strings', () => {
35+
const result = ComparisonOperators.notEqual('PS5', 'Xbox Series X')
36+
expect(result).toBeTruthy()
37+
})
38+
test('Should fail given two different numbers', () => {
39+
const result = ComparisonOperators.notEqual(360, 360)
40+
expect(result).toBeFalsy()
41+
})
42+
test('Should pass given two different numbers', () => {
43+
const result = ComparisonOperators.notEqual(360, 90)
44+
expect(result).toBeTruthy()
45+
})
46+
test('Should pass given two equal values with different types', () => {
47+
const result = ComparisonOperators.notEqual(180, '180')
48+
expect(result).toBeTruthy()
49+
})
50+
})
51+
52+
describe('LessThan and LessThanInclusive Operator', () => {
53+
test('Should fail if the first string is less than the second one', () => {
54+
const result = ComparisonOperators.lessThan('X', 'A')
55+
expect(result).toBeFalsy()
56+
})
57+
58+
test('Should fail comparing the same string', () => {
59+
const result = ComparisonOperators.lessThan('A', 'A')
60+
expect(result).toBeFalsy()
61+
})
62+
63+
test('Should pass if the first string is greater than the second one', () => {
64+
const result = ComparisonOperators.lessThan('A', 'X')
65+
expect(result).toBeTruthy()
66+
})
67+
68+
test('Should pass if the first number is less than the second one', () => {
69+
const result = ComparisonOperators.lessThan(10, 100)
70+
expect(result).toBeTruthy()
71+
})
72+
73+
test('Should fail comparing the same number', () => {
74+
const result = ComparisonOperators.lessThan(10, 10)
75+
expect(result).toBeFalsy()
76+
})
77+
78+
test('Should fail if the first number is greater than the second one', () => {
79+
const result = ComparisonOperators.lessThan(100, 10)
80+
expect(result).toBeFalsy()
81+
})
82+
83+
test('Should pass if the first number is less or equal than the second one', () => {
84+
expect(ComparisonOperators.lessThanInclusive(1, 10)).toBeTruthy()
85+
expect(ComparisonOperators.lessThanInclusive(10, 10)).toBeTruthy()
86+
})
87+
88+
test('Should pass if the first string is less or equal than the second one', () => {
89+
expect(ComparisonOperators.lessThanInclusive('a', 'x')).toBeTruthy()
90+
expect(ComparisonOperators.lessThanInclusive('a', 'a')).toBeTruthy()
91+
})
92+
93+
test('Should pass if the first date is before than the second one', () => {
94+
const result = ComparisonOperators.lessThan(
95+
'2022-03-15T20:42:58.510Z',
96+
new Date().toISOString()
97+
)
98+
expect(result).toBeTruthy()
99+
})
100+
test('Should fail if the first date is after than the second one', () => {
101+
const result = ComparisonOperators.lessThan(
102+
new Date().toISOString(),
103+
'2022-03-15T20:42:58.510Z'
104+
)
105+
expect(result).toBeFalsy()
106+
})
107+
})
108+
109+
describe('GreaterThan and GreaterThanInclusive Operator', () => {
110+
test('Should fail if the first string is greater than the second one', () => {
111+
const result = ComparisonOperators.greaterThan('A', 'X')
112+
expect(result).toBeFalsy()
113+
})
114+
115+
test('Should fail comparing the same string', () => {
116+
const result = ComparisonOperators.greaterThan('A', 'A')
117+
expect(result).toBeFalsy()
118+
})
119+
120+
test('Should pass if the first string is less than the second one', () => {
121+
const result = ComparisonOperators.greaterThan('X', 'A')
122+
expect(result).toBeTruthy()
123+
})
124+
125+
test('Should pass if the first number is greater than the second one', () => {
126+
const result = ComparisonOperators.greaterThan(100, 10)
127+
expect(result).toBeTruthy()
128+
})
129+
130+
test('Should fail comparing the same number', () => {
131+
const result = ComparisonOperators.greaterThan(10, 10)
132+
expect(result).toBeFalsy()
133+
})
134+
135+
test('Should fail if the first number is less than the second one', () => {
136+
const result = ComparisonOperators.greaterThan(10, 100)
137+
expect(result).toBeFalsy()
138+
})
139+
140+
test('Should pass if the first number is greater or equal than the second one', () => {
141+
expect(ComparisonOperators.greaterThanInclusive(10, 1)).toBeTruthy()
142+
expect(ComparisonOperators.greaterThanInclusive(10, 10)).toBeTruthy()
143+
})
144+
145+
test('Should pass if the first string is greater or equal than the second one', () => {
146+
expect(ComparisonOperators.greaterThanInclusive('x', 'a')).toBeTruthy()
147+
expect(ComparisonOperators.greaterThanInclusive('a', 'a')).toBeTruthy()
148+
})
149+
150+
test('Should pass if the first date is after than the second one', () => {
151+
const result = ComparisonOperators.greaterThan(
152+
new Date().toISOString(),
153+
'2022-03-15T20:42:58.510Z'
154+
)
155+
expect(result).toBeTruthy()
156+
})
157+
test('Should fail if the first date is before than the second one', () => {
158+
const result = ComparisonOperators.greaterThan(
159+
'2022-03-15T20:42:58.510Z',
160+
new Date().toISOString()
161+
)
162+
expect(result).toBeFalsy()
163+
})
6164
})
7165
})

0 commit comments

Comments
 (0)