Add getFunction to the Context (#338)

Fixes #335 by adding a getFunction() method to the Context class which goes up the parent context tree until it gets to the Library and is able to execute getFunction.

This fixes the issue of an overloaded function that has the same name as one of its arguments and calls itself (or another overload). The broad context.get() method was being used to get the possible FunctionDefs was returning the first thing that matches the name of the function. So it was ending up with the value of the argument passed in which had the same name. Tests are included that replicate this issue with multiple modes of execution.

This PR also updates the cql-to-elm version for test data generation to 3.22.0 and updates the translator options to roughly match the options used by MADiE. 

Included (squashed) commits:

* initial adding of test for fluent overloading issue

* updated cql-to-elm to 3.22.0 and added some translator options that MADiE uses

* Added test for overloaded functions calling themselves

* resolved npm audit concern with babel and fixed prettier issue with test-data file

* reworked test to have an argument name matching the overloaded function name. fixed issue by adding getFunction to Context

* add missing newline to package-lock.json that npm insists on making

* added info on translator settings. cql fix for ProperIncludedIn test. re-built test, fixing some post rebase stuff. added new test library to prettier ignore

* renamed new tests. rebuilt spec-test data with new translator and added workaround for ProperIncludedIn null test

Author: hossenlopp

.prettierignore

@@ -13,3 +13,4 @@ test/elm/library/Common.js
 test/elm/library/Common2.js
 test/elm/library/data-with-namespace.js
 test/elm/message/Included.js
+test/elm/reusable/FluentFunctionsOverloadCallingSelf.js

examples/browser/cql4browsers.js

@@ -6876,7 +6876,7 @@ class FunctionRef extends expression_1.Expression {
             child_ctx = libCtx ? libCtx.childContext() : undefined;
         }
         else {
-            functionDefs = ctx.get(this.name);
+            functionDefs = ctx.getFunction(this.name);
             child_ctx = ctx.childContext();
         }
         const args = await this.execArgs(ctx);
@@ -8082,6 +8082,9 @@ class Context {
     getConcept(name) {
         return this.parent && this.parent.getConcept(name);
     }
+    getFunction(name) {
+        return this.parent && this.parent.getFunction(name);
+    }
     get(identifier) {
         // Check for undefined because if its null, we actually *do* want to return null (rather than
         // looking at parent), but if it's really undefined, *then* look at the parent

src/elm/reusable.ts

@@ -74,7 +74,7 @@ export class FunctionRef extends Expression {
       const libCtx = ctx.getLibraryContext(this.library);
       child_ctx = libCtx ? libCtx.childContext() : undefined;
     } else {
-      functionDefs = ctx.get(this.name);
+      functionDefs = ctx.getFunction(this.name);
       child_ctx = ctx.childContext();
     }
     const args = await this.execArgs(ctx);

src/runtime/context.ts

@@ -155,6 +155,10 @@ export class Context {
     return this.parent && this.parent.getConcept(name);
   }
 
+  getFunction(name: string) {
+    return this.parent && this.parent.getFunction(name);
+  }
+
   get(identifier: string): any {
     // Check for undefined because if its null, we actually *do* want to return null (rather than
     // looking at parent), but if it's really undefined, *then* look at the parent