Fixes for localIds and Nary function resolution

[JPercival]

• Added more tests to verify that compiled ELM does contain localIds.
• Fixed cases where nodes created via reflection did not have Ids assigned.
• Fixed cases where NaryFunctions where resolved as BinaryFunctions
• Added assertions to the CQL compiler to validate all nodes have ids
• Refactored/Removed cases where localIds could be null.

[github-actions]

Formatting check succeeded!

[codecov]

Codecov Report

Attention: Patch coverage is 83.33333% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 63.18%. Comparing base (ec84b9e) to head (4e5d22f).

Files	Patch %	Lines
.../preprocessor/CqlPreprocessorElmCommonVisitor.java	28.57%	4 Missing and 1 partial ⚠️
...qframework/cql/cql2elm/SystemFunctionResolver.java	91.42%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1356      +/-   ##
============================================
- Coverage     63.18%   63.18%   -0.01%     
+ Complexity     6128     6126       -2     
============================================
  Files           468      468              
  Lines         25336    25315      -21     
  Branches       4716     4713       -3     
============================================
- Hits          16009    15995      -14     
+ Misses         7170     7166       -4     
+ Partials       2157     2154       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.