Merge pull request #14 from gokulk16/r-11

Fix: security vulnerability from snyk; Removing homecurrency test since github workflows run in different location

js/currency.js

@@ -77,7 +77,7 @@ export async function getConversionRates() {
       const conversionRate =
         (amountToConvert / currencies[baseCurrency]) *
         currencies[targetCurrency];
-      convertedRates[baseCurrency][targetCurrency] = conversionRate;
+      _.set(convertedRates, [baseCurrency, targetCurrency], conversionRate);
     });
   });
 

test/editor.test.js

@@ -1,6 +1,5 @@
 import {
   setupEvaluator,
-  setupHomeCurrency,
   generateDocID,
   evaluate,
   loadPlaceholderData,
@@ -19,13 +18,6 @@ describe("testing generateDocID", () => {
   });
 });
 
-describe("testing setupHomeCurrency", () => {
-  test("setting up Home Currency", async () => {
-    const homeCurr = await setupHomeCurrency();
-    expect(homeCurr).toBe("INR");
-  });
-});
-
 describe("testing evaluate", () => {
   test("Evaluate empty string", async () => {
     const evalValues = await evaluate("");