feature(views): cleanup.

experimental/packages/opentelemetry-exporter-metrics-otlp-grpc/test/metricsHelper.ts

@@ -23,7 +23,8 @@ import {
   AggregationTemporality,
   ExplicitBucketHistogramAggregation,
   MeterProvider,
-  MetricReader, View,
+  MetricReader,
+  View,
 } from '@opentelemetry/sdk-metrics-base';
 import { IKeyValue, IMetric, IResource } from '@opentelemetry/otlp-transformer';
 

experimental/packages/opentelemetry-exporter-metrics-otlp-http/test/node/CollectorMetricExporter.test.ts

@@ -35,7 +35,10 @@ import {
   mockCounter,
   mockObservableGauge,
   mockHistogram,
-  collect, shutdown, setUp, HISTOGRAM_AGGREGATION_VIEW,
+  collect,
+  shutdown,
+  setUp,
+  HISTOGRAM_AGGREGATION_VIEW,
 } from '../metricsHelper';
 import { MockedResponse } from './nodeHelpers';
 import { AggregationTemporality, ResourceMetrics } from '@opentelemetry/sdk-metrics-base';

experimental/packages/opentelemetry-exporter-metrics-otlp-proto/test/metricsHelper.ts

@@ -27,7 +27,8 @@ import {
   AggregationTemporality,
   ExplicitBucketHistogramAggregation,
   MeterProvider,
-  MetricReader, View
+  MetricReader,
+  View
 } from '@opentelemetry/sdk-metrics-base';
 import { IExportMetricsServiceRequest, IKeyValue, IMetric } from '@opentelemetry/otlp-transformer';
 import { Stream } from 'stream';

experimental/packages/opentelemetry-exporter-prometheus/test/PrometheusSerializer.test.ts

@@ -24,7 +24,8 @@ import {
   DataPointType,
   ExplicitBucketHistogramAggregation,
   SumAggregation,
-  Histogram, View,
+  Histogram,
+  View,
 } from '@opentelemetry/sdk-metrics-base';
 import * as sinon from 'sinon';
 import { PrometheusSerializer } from '../src';

experimental/packages/opentelemetry-sdk-metrics-base/test/view/View.test.ts

@@ -39,14 +39,15 @@ describe('View', () => {
     });
 
     it('without at least one selector option should throw', () => {
+      // would do nothing
       assert.throws(() => new View({}));
       // would implicitly rename all instruments to 'name'
       assert.throws(() => new View({ name: 'name' }));
       // would implicitly drop all attribute keys on all instruments except 'key'
       assert.throws(() => new View({ attributeKeys: ['key'] }));
       // would implicitly rename all instruments to description
       assert.throws(() => new View({ description: 'description' }));
-      // would implicitly change all instruments to use histogram aggregation.
+      // would implicitly change all instruments to use histogram aggregation
       assert.throws(() => new View({
         aggregation: new ExplicitBucketHistogramAggregation([1, 100])
       }));