refactor(metrics): rename option property from raiseOnEmptyMetrics to throwOnEmptyMetrics

Author: kozub

Diff for: docs/core/metrics.md

@@ -433,7 +433,7 @@ The `logMetrics` decorator of the metrics utility can be used when your Lambda h
 
 #### Throwing a RangeError when no metrics are emitted
 
-If you want to ensure that at least one metric is emitted before you flush them, you can use the `raiseOnEmptyMetrics` parameter and pass it to the middleware or decorator:
+If you want to ensure that at least one metric is emitted before you flush them, you can use the `throwOnEmptyMetrics` parameter and pass it to the middleware or decorator:
 
 ```typescript hl_lines="11"
     import { Metrics, MetricUnits, logMetrics } from '@aws-lambda-powertools/metrics';
@@ -446,7 +446,7 @@ If you want to ensure that at least one metric is emitted before you flush them,
     }
 
     export const handler = middy(lambdaHandler)
-        .use(logMetrics(metrics, { raiseOnEmptyMetrics: true }));
+        .use(logMetrics(metrics, { throwOnEmptyMetrics: true }));
 ```
 
 ### Capturing a cold start invocation as metric

Diff for: packages/metrics/examples/decorator/empty-metrics.ts

@@ -16,10 +16,10 @@ const metrics = new Metrics();
 class Lambda implements LambdaInterface {
 
   // Be default, we will not throw any error if there is no metrics. Use this property to override and throw an exception
-  @metrics.logMetrics({ raiseOnEmptyMetrics: true })
+  @metrics.logMetrics({ throwOnEmptyMetrics: true })
   public handler<TEvent, TResult>(_event: TEvent, _context: Context, _callback: Callback<TResult>): void | Promise<TResult> {
     // Notice that no metrics are added
-    // Since the raiseOnEmptyMetrics parameter is set to true, the Powertool throw an Error
+    // Since the throwOnEmptyMetrics parameter is set to true, the Powertool throw an Error
   }
 
 }

Diff for: packages/metrics/examples/empty-metrics.ts

@@ -14,10 +14,10 @@ const metrics = new Metrics();
 
 const lambdaHandler = async (): Promise<void> => {
   // Notice that no metrics are added
-  // Since the raiseOnEmptyMetrics parameter is set to true, the Powertool throw an Error
+  // Since the throwOnEmptyMetrics parameter is set to true, the Powertool throw an Error
 };
 
 const handlerWithMiddleware = middy(lambdaHandler)
-  .use(logMetrics(metrics, { raiseOnEmptyMetrics: true }));
+  .use(logMetrics(metrics, { throwOnEmptyMetrics: true }));
 
 handlerWithMiddleware(dummyEvent, dummyContext, () => console.log('Lambda invoked!'));

Diff for: packages/metrics/src/Metrics.ts

@@ -49,7 +49,7 @@ const DEFAULT_NAMESPACE = 'default_namespace';
  *
  *   // FYI: Decorator might not render properly in VSCode mouse over due to https://github.com/microsoft/TypeScript/issues/39371 and might show as *@metrics* instead of `@metrics.logMetrics`
  *
- *   @metrics.logMetrics({captureColdStartMetric: true, raiseOnEmptyMetrics: true, })
+ *   @metrics.logMetrics({captureColdStartMetric: true, throwOnEmptyMetrics: true, })
  *   public handler(_event: any, _context: Context, _callback: Callback<any>): void | Promise<any> {
  *    // ...
  *    metrics.addMetric('test-metric', MetricUnits.Count, 10);
@@ -89,7 +89,7 @@ class Metrics implements MetricsInterface {
   private isSingleMetric: boolean = false;
   private metadata: { [key: string]: string } = {};
   private namespace?: string;
-  private shouldRaiseOnEmptyMetrics: boolean = false;
+  private shouldThrowOnEmptyMetrics: boolean = false;
   private storedMetrics: StoredMetrics = {};
 
   public constructor(options: MetricsOptions = {}) {
@@ -227,9 +227,9 @@ class Metrics implements MetricsInterface {
    * @decorator Class
    */
   public logMetrics(options: ExtraOptions = {}): HandlerMethodDecorator {
-    const { raiseOnEmptyMetrics, defaultDimensions, captureColdStartMetric } = options;
-    if (raiseOnEmptyMetrics) {
-      this.raiseOnEmptyMetrics();
+    const { throwOnEmptyMetrics, defaultDimensions, captureColdStartMetric } = options;
+    if (throwOnEmptyMetrics) {
+      this.throwOnEmptyMetrics();
     }
     if (defaultDimensions !== undefined) {
       this.setDefaultDimensions(defaultDimensions);
@@ -293,13 +293,13 @@ class Metrics implements MetricsInterface {
    * const metrics = new Metrics({namespace:"serverlessAirline", serviceName:"orders"});
    *
    * export const handler = async (event: any, context: Context) => {
-   *     metrics.raiseOnEmptyMetrics();
+   *     metrics.throwOnEmptyMetrics();
    *     metrics.publishStoredMetrics(); // will throw since no metrics added.
    * }
    * ```
    */
-  public raiseOnEmptyMetrics(): void {
-    this.shouldRaiseOnEmptyMetrics = true;
+  public throwOnEmptyMetrics(): void {
+    this.shouldThrowOnEmptyMetrics = true;
   }
 
   /**
@@ -312,7 +312,7 @@ class Metrics implements MetricsInterface {
       Name: metricDefinition.name,
       Unit: metricDefinition.unit,
     }));
-    if (metricDefinitions.length === 0 && this.shouldRaiseOnEmptyMetrics) {
+    if (metricDefinitions.length === 0 && this.shouldThrowOnEmptyMetrics) {
       throw new RangeError('The number of metrics recorded must be higher than zero');
     }
 

Diff for: packages/metrics/src/middleware/middy.ts

@@ -8,9 +8,9 @@ const logMetrics = (target: Metrics | Metrics[], options: ExtraOptions = {}): mi
   const logMetricsBefore = async (request: middy.Request): Promise<void> => {
     metricsInstances.forEach((metrics: Metrics) => {
       metrics.setFunctionName(request.context.functionName);
-      const { raiseOnEmptyMetrics, defaultDimensions, captureColdStartMetric } = options;
-      if (raiseOnEmptyMetrics !== undefined) {
-        metrics.raiseOnEmptyMetrics();
+      const { throwOnEmptyMetrics, defaultDimensions, captureColdStartMetric } = options;
+      if (throwOnEmptyMetrics !== undefined) {
+        metrics.throwOnEmptyMetrics();
       }
       if (defaultDimensions !== undefined) {
         metrics.setDefaultDimensions(defaultDimensions);

Diff for: packages/metrics/src/types/Metrics.ts

@@ -39,7 +39,7 @@ type HandlerMethodDecorator = (
  * ```typescript
  *
  * const metricsOptions: MetricsOptions = {
- *   raiseOnEmptyMetrics: true,
+ *   throwOnEmptyMetrics: true,
  *   defaultDimensions: {'environment': 'dev'},
  *   captureColdStartMetric: true,
  * }
@@ -51,7 +51,7 @@ type HandlerMethodDecorator = (
  * ```
  */
 type ExtraOptions = {
-  raiseOnEmptyMetrics?: boolean
+  throwOnEmptyMetrics?: boolean
   defaultDimensions?: Dimensions
   captureColdStartMetric?: boolean
 };

Diff for: packages/metrics/tests/e2e/decorator.test.MyFunction.ts

@@ -18,7 +18,7 @@ const metrics = new Metrics({ namespace: namespace, service: serviceName });
 
 class Lambda implements LambdaInterface {
 
-  @metrics.logMetrics({ captureColdStartMetric: true, defaultDimensions: JSON.parse(defaultDimensions), raiseOnEmptyMetrics: true })
+  @metrics.logMetrics({ captureColdStartMetric: true, defaultDimensions: JSON.parse(defaultDimensions), throwOnEmptyMetrics: true })
   public async handler(_event: unknown, _context: Context): Promise<void> {
     metrics.addMetric(metricName, metricUnit, parseInt(metricValue));
     metrics.addDimension(

Diff for: packages/metrics/tests/e2e/standardFunctions.test.MyFunction.ts

@@ -17,7 +17,7 @@ const metrics = new Metrics({ namespace: namespace, serviceName: serviceName });
 
 export const handler = async (_event: unknown, _context: Context): Promise<void> => {
   metrics.captureColdStartMetric();
-  metrics.raiseOnEmptyMetrics();
+  metrics.throwOnEmptyMetrics();
   metrics.setDefaultDimensions(JSON.parse(defaultDimensions));
   metrics.addMetric(metricName, metricUnit, parseInt(metricValue));
   metrics.addDimension(

Diff for: packages/metrics/tests/unit/Metrics.test.ts

@@ -346,13 +346,13 @@ describe('Class: Metrics', () => {
     });
   });
 
-  describe('Feature: raiseOnEmptyMetrics', () => {
-    test('Error should be thrown on empty metrics when raiseOnEmptyMetrics is passed', async () => {
+  describe('Feature: throwOnEmptyMetrics', () => {
+    test('Error should be thrown on empty metrics when throwOnEmptyMetrics is passed', async () => {
       expect.assertions(1);
 
       const metrics = new Metrics({ namespace: 'test' });
       class LambdaFunction implements LambdaInterface {
-        @metrics.logMetrics({ raiseOnEmptyMetrics: true })
+        @metrics.logMetrics({ throwOnEmptyMetrics: true })
         // eslint-disable-next-line @typescript-eslint/ban-ts-comment
         // @ts-ignore
         public handler<TEvent, TResult>(
@@ -371,12 +371,12 @@ describe('Class: Metrics', () => {
       }
     });
 
-    test('Error should be thrown on empty metrics when raiseOnEmptyMetrics() is callse', async () => {
+    test('Error should be thrown on empty metrics when throwOnEmptyMetrics() is callse', async () => {
       expect.assertions(1);
 
       const metrics = new Metrics({ namespace: 'test' });
       const handler = async (_event: DummyEvent, _context: Context): Promise<void> => {
-        metrics.raiseOnEmptyMetrics();
+        metrics.throwOnEmptyMetrics();
         // Logic goes here
         metrics.publishStoredMetrics();
       };

Diff for: packages/metrics/tests/unit/middleware/middy.test.ts

@@ -86,7 +86,7 @@ describe('Middy middleware', () => {
         metrics.addMetric('successfulBooking', MetricUnits.Count, 1);
       };
       const metricsOptions: ExtraOptions = {
-        raiseOnEmptyMetrics: true,
+        throwOnEmptyMetrics: true,
         defaultDimensions: { environment : 'prod', aws_region: 'eu-central-1' },
         captureColdStartMetric: true
       };
@@ -173,7 +173,7 @@ describe('Middy middleware', () => {
         metrics.addMetric('successfulBooking', MetricUnits.Count, 1);
       };
       const metricsOptions: ExtraOptions = {
-        raiseOnEmptyMetrics: true
+        throwOnEmptyMetrics: true
       };
       const handler = middy(lambdaHandler).use(logMetrics([metrics], metricsOptions));