ApiGatewayMetricFactory.metricTps and metricInvocationRate do not pass ".with(properties)" to underlying metricInvocationCount

[jackbnz-aws]

Version

5.0.0

Steps and/or minimal code example to reproduce

1. Create a MetricFactory with a tps metric.
2. use .with(properties); on the tps metric
3. Observe that properties aren't included on the tps metric itself.

Relevant line of code:

cdk-monitoring-constructs/lib/monitoring/aws-apigateway/ApiGatewayMetricFactory.ts

Line 77 in 064d54f

metricInvocationRate() {

Expected behavior

I would expect that properties are passed to the underlying metric. This functionality works on metrics that do not use the "toRate" method as the metric is directly returned.

For example, .with(properties) works correctly on metric4XXErrorCount:

cdk-monitoring-constructs/lib/monitoring/aws-apigateway/ApiGatewayMetricFactory.ts

Line 98 in 064d54f

metric4XXErrorCount() {

Actual behavior

Properties are not passed to underlying TPS metric.

Other details

We are using this to show a widget for TPS, but need to specify the account. The account for the TPS metric we specify in the properties is not being used.