Skip to content

Commit 63d3cb3

Browse files
authored
Merge pull request #3806 from Pragadesh-45/fix/handle-assert-results
Chore/ Remove Unused `RemoveQuotes` function
2 parents 6abd063 + 10a5935 commit 63d3cb3

File tree

2 files changed

+0
-20
lines changed

2 files changed

+0
-20
lines changed

packages/bruno-js/src/sandbox/quickjs/index.js

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,6 @@ const toNumber = (value) => {
2222
return Number.isInteger(num) ? parseInt(value, 10) : parseFloat(value);
2323
};
2424

25-
// const removeQuotes = (str) => {
26-
// if ((str.startsWith('"') && str.endsWith('"')) || (str.startsWith("'") && str.endsWith("'"))) {
27-
// return str.slice(1, -1);
28-
// }
29-
// return str;
30-
// };
3125

3226
const executeQuickJsVm = ({ script: externalScript, context: externalContext, scriptType = 'template-literal' }) => {
3327
if (!externalScript?.length || typeof externalScript !== 'string') {
@@ -44,8 +38,6 @@ const executeQuickJsVm = ({ script: externalScript, context: externalContext, sc
4438
if (externalScript === 'null') return null;
4539
if (externalScript === 'undefined') return undefined;
4640

47-
// This is commented out as part of the fix for #3758
48-
// externalScript = removeQuotes(externalScript);
4941

5042
const vm = QuickJSSyncContext;
5143

@@ -95,8 +87,6 @@ const executeQuickJsVmAsync = async ({ script: externalScript, context: external
9587
if (externalScript === 'null') return null;
9688
if (externalScript === 'undefined') return undefined;
9789

98-
// This is commented out as part of the fix for #3758
99-
// externalScript = removeQuotes(externalScript);
10090

10191
try {
10292
const module = await newQuickJSWASMModule();

packages/bruno-js/src/utils.js

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -85,16 +85,6 @@ const evaluateJsTemplateLiteral = (templateLiteral, context) => {
8585
return undefined;
8686
}
8787

88-
// This is commented out as part of the fix for #3758
89-
// if (templateLiteral.startsWith('"') && templateLiteral.endsWith('"')) {
90-
// return templateLiteral.slice(1, -1);
91-
// }
92-
93-
// This is commented out as part of the fix for #3758
94-
// if (templateLiteral.startsWith("'") && templateLiteral.endsWith("'")) {
95-
// return templateLiteral.slice(1, -1);
96-
// }
97-
9888
if (!isNaN(templateLiteral)) {
9989
const number = Number(templateLiteral);
10090
// Check if the number is too high. Too high number might get altered, see #1000

0 commit comments

Comments
 (0)