Make sure per is set to something when visiting expand and collapse

[antvaset]

Currently, the translator fails to build the call context for collapse { null } returning

Could not determine signature for invocation of operator System.Collapse.

This is because the per argument is not instantiated inside visitSetAggregateExpressionTerm when it's not explicitly set and source is not a list of intervals. In the case of collapse { null }, source is of type List<System.Any>. The fallback expression for per in all cases should be null of type System.Quantity.

I noticed this issue while working on #1424. collapse { null } still won't work due to a downstream error (which is being addressed in PR 1424):

Could not resolve call to operator Collapse with signature (list<System.Any>,System.Quantity).

[github-actions]

Formatting check succeeded!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.35%. Comparing base (8390d7a) to head (b816b4b).
Report is 1 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1425      +/-   ##
============================================
- Coverage     64.35%   64.35%   -0.01%     
- Complexity     1920     1921       +1     
============================================
  Files           494      494              
  Lines         28134    28131       -3     
  Branches       5588     5588              
============================================
- Hits          18106    18103       -3     
+ Misses         7776     7775       -1     
- Partials       2252     2253       +1     

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud