Skip to content

Commit 278ce03

Browse files
gioddiggitoddbaert
authored andcommitted
fix: address gemini suggestions
Signed-off-by: Giovanni De Giorgio <[email protected]>
1 parent 72a9985 commit 278ce03

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

libs/providers/aws-ssm/src/lib/aws-ssm-provider.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ describe('aws-ssm-provider.ts - AwsSsmProvider', () => {
196196
reason: StandardResolutionReasons.STATIC,
197197
});
198198

199-
const result = await provider.resolveObjectEvaluation('test-object-success', -1, {});
199+
const result = await provider.resolveObjectEvaluation('test-object-success', {}, {});
200200

201201
expect(result).toEqual({
202202
value: { default: true },

libs/providers/aws-ssm/src/lib/cache.spec.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,10 @@ describe('cache.ts - Cache', () => {
2525
expect(cache.get('test')).toEqual({ value: true, reason: 'test' });
2626
});
2727

28-
it('should not set any value when cache is disabled', () => {
29-
expect(setSpy).not.toHaveBeenCalled();
28+
it('should not set a value when cache is disabled', () => {
29+
const cache = new Cache({ enabled: false });
30+
cache.set('test', { value: true, reason: 'test' });
31+
expect(cache.get('test')).toBeUndefined();
3032
});
3133

3234
it('should clear the cache when cache is enabled and .clear() is called', () => {

0 commit comments

Comments
 (0)