diff --git a/packages/bruno-js/tests/utils.spec.js b/packages/bruno-js/tests/utils.spec.js index 6a454fde7..b1ecc7db7 100644 --- a/packages/bruno-js/tests/utils.spec.js +++ b/packages/bruno-js/tests/utils.spec.js @@ -7,7 +7,7 @@ describe('utils', () => { res: { data: { pets: ['bruno', 'max'] }, context: 'testContext', - _BrunoNewFunctionInnerContext: 0 + __bruno__functionInnerContext: 0 } }; @@ -47,32 +47,32 @@ describe('utils', () => { it('should identify top level variables', () => { const expr = 'res.data.pets[0].toUpperCase()'; evaluateJsExpression(expr, context); - expect(cache.get(expr).toString()).toContain('let { res } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res } = __bruno__functionInnerContext;'); }); it('should not duplicate variables', () => { const expr = 'res.data.pets[0] + res.data.pets[1]'; evaluateJsExpression(expr, context); - expect(cache.get(expr).toString()).toContain('let { res } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res } = __bruno__functionInnerContext;'); }); it('should exclude js keywords like true false from vars', () => { const expr = 'res.data.pets.length > 0 ? true : false'; evaluateJsExpression(expr, context); - expect(cache.get(expr).toString()).toContain('let { res } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res } = __bruno__functionInnerContext;'); }); it('should exclude numbers from vars', () => { const expr = 'res.data.pets.length + 10'; evaluateJsExpression(expr, context); - expect(cache.get(expr).toString()).toContain('let { res } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res } = __bruno__functionInnerContext;'); }); it('should pick variables from complex expressions', () => { const expr = 'res.data.pets.map(pet => pet.length)'; const result = evaluateJsExpression(expr, context); expect(result).toEqual([5, 3]); - expect(cache.get(expr).toString()).toContain('let { res, pet } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res, pet } = __bruno__functionInnerContext;'); }); it('should be ok picking extra vars from strings', () => { @@ -80,7 +80,7 @@ describe('utils', () => { const result = evaluateJsExpression(expr, context); expect(result).toBe('hello bruno'); // extra var hello is harmless - expect(cache.get(expr).toString()).toContain('let { hello, res } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { hello, res } = __bruno__functionInnerContext;'); }); it('should evaluate expressions referencing globals', () => { @@ -118,14 +118,14 @@ describe('utils', () => { it('should allow "context" as a var name', () => { const expr = 'res["context"].toUpperCase()'; evaluateJsExpression(expr, context); - expect(cache.get(expr).toString()).toContain('let { res, context } = _BrunoNewFunctionInnerContext;'); + expect(cache.get(expr).toString()).toContain('let { res, context } = __bruno__functionInnerContext;'); }); - it('should throw an error when we use "_BrunoNewFunctionInnerContext" as a var name', () => { - const expr = 'res["_BrunoNewFunctionInnerContext"].toUpperCase()'; + it('should throw an error when we use "__bruno__functionInnerContext" as a var name', () => { + const expr = 'res["__bruno__functionInnerContext"].toUpperCase()'; expect(() => evaluateJsExpression(expr, context)).toThrow(SyntaxError); expect(() => evaluateJsExpression(expr, context)).toThrow( - "Identifier '_BrunoNewFunctionInnerContext' has already been declared" + "Identifier '__bruno__functionInnerContext' has already been declared" ); }); });