[APM] Don't use transaction metrics if set to never (elastic#117370) (e…

…lastic#117414)

Closes elastic#117226.

Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>

x-pack/plugins/apm/server/lib/helpers/transactions/index.ts

@@ -63,18 +63,23 @@ export async function getSearchAggregatedTransactions({
  apmEventClient: APMEventClient;
  kuery: string;
 }): Promise<boolean> {
- const searchAggregatedTransactions = config.searchAggregatedTransactions;
+ switch (config.searchAggregatedTransactions) {
+ case SearchAggregatedTransactionSetting.always:
+ return kuery
+ ? getHasAggregatedTransactions({ start, end, apmEventClient, kuery })
+ : true;
 
- if (
- kuery ||
- searchAggregatedTransactions === SearchAggregatedTransactionSetting.auto
- ) {
- return getHasAggregatedTransactions({ start, end, apmEventClient, kuery });
- }
+ case SearchAggregatedTransactionSetting.auto:
+ return getHasAggregatedTransactions({
+ start,
+ end,
+ apmEventClient,
+ kuery,
+ });
 
- return (
- searchAggregatedTransactions === SearchAggregatedTransactionSetting.always
- );
+  case SearchAggregatedTransactionSetting.never:
+  return false;
+ }
 }
 
 export function getTransactionDurationFieldForTransactions(