fix(sdk-metrics-base): misbehaving aggregation temporality selector tolerance

[legendecas]

Which problem is this PR solving?

When the AggregationTemporalitySelector returns different AggregationTemporality for the same instrument type, the collection result can be very misleading and incorrect.

Short description of the changes

• For a given MetricReader/MetricCollector and metric storage mapped by views, the AggregationTemporalitySelector is only invoked once.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

• TemporalMetricProcessor.buildMetrics is tested with AggregationTemporalitySelector call count verification

Checklist:

• Followed the style guidelines of this project
• Unit tests have been added

[codecov]

Codecov Report

Merging #2958 (add238d) into main (479321c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2958   +/-   ##
=======================================
  Coverage   92.53%   92.53%           
=======================================
  Files         183      183           
  Lines        5958     5959    +1     
  Branches     1266     1266           
=======================================
+ Hits         5513     5514    +1     
  Misses        445      445           

Impacted Files	Coverage Δ
...-metrics-base/src/state/TemporalMetricProcessor.ts	97.95% <100.00%> (+0.04%)	⬆️

[seemk]

Completely glanced over this section when you initially mentioned the issue, thanks! 👍

[dyladan]

Thanks for moving the changelog entry to the correct section

[pichlermarc]

Looks good! 🙂