Skip to content

Commit

Permalink
feat: fixing comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Patricio Albizu authored and Patricio Albizu committed Nov 9, 2023
1 parent 5497c5d commit b2d03da
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ export abstract class ExploreCartesianDataSourceModel extends GraphQlDataSourceM
const metricAggregationData = result.getMetricAggregationSeriesData(spec.name, spec.aggregation);

Check warning on line 178 in projects/observability/src/shared/dashboard/data/graphql/explore/explore-cartesian-data-source.model.ts

View check run for this annotation

Codecov / codecov/patch

projects/observability/src/shared/dashboard/data/graphql/explore/explore-cartesian-data-source.model.ts#L177-L178

Added lines #L177 - L178 were not covered by tests

return {

Check warning on line 180 in projects/observability/src/shared/dashboard/data/graphql/explore/explore-cartesian-data-source.model.ts

View check run for this annotation

Codecov / codecov/patch

projects/observability/src/shared/dashboard/data/graphql/explore/explore-cartesian-data-source.model.ts#L180

Added line #L180 was not covered by tests
data: metricAggregationData?.value ? [['', metricAggregationData.value]] : [],
data: metricAggregationData.value ? [['', metricAggregationData.value]] : [],
spec: spec
};
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,11 +82,7 @@ export class ExploreResult {
): MetricAggregationData {
return {

Check warning on line 83 in projects/observability/src/shared/dashboard/data/graphql/explore/explore-result.ts

View check run for this annotation

Codecov / codecov/patch

projects/observability/src/shared/dashboard/data/graphql/explore/explore-result.ts#L83

Added line #L83 was not covered by tests
key: spec.name,
...(result
? {
value: result?.[spec.resultAlias()].value as number
}
: {})
value: result?.[spec.resultAlias()].value as number
};
}

Expand Down Expand Up @@ -168,5 +164,5 @@ interface GroupData {

interface MetricAggregationData {
key: string;
value?: number;
value: number | undefined;
}

0 comments on commit b2d03da

Please sign in to comment.