Review fixes

x-pack/plugins/ingest_manager/common/types/rest_spec/common.ts

@@ -3,7 +3,6 @@
  * or more contributor license agreements. Licensed under the Elastic License;
  * you may not use this file except in compliance with the Elastic License.
  */
-import { HttpFetchQuery } from 'src/core/public';
 
 export interface ListWithKuery {
  page?: number;

x-pack/plugins/ingest_manager/server/services/epm/packages/get.ts

@@ -61,17 +61,15 @@ export async function getLimitedPackages(options: {
  (pkg) => (pkg.status = InstallationStatus.installed)
  );
  const installedPackagesInfo = await Promise.all(
- installedPackages.map(async (pkgInstall) => {
+ installedPackages.map((pkgInstall) => {
  return getPackageInfo({
  savedObjectsClient,
  pkgName: pkgInstall.name,
  pkgVersion: pkgInstall.version,
  });
  })
  );
- return installedPackagesInfo
- .filter((pkgInfo) => isPackageLimited(pkgInfo))
- .map((pkgInfo) => pkgInfo.name);
+ return installedPackagesInfo.filter((pkgInfo) => isPackageLimited).map((pkgInfo) => pkgInfo.name);
 }
 
 export async function getPackageSavedObjects(savedObjectsClient: SavedObjectsClientContract) {

x-pack/plugins/ingest_manager/server/services/package_config.ts

@@ -46,16 +46,18 @@ class PackageConfigService {
  ): Promise<PackageConfig> {
  // Make sure the associated package is installed
  if (packageConfig.package?.name) {
- await ensureInstalledPackage({
- savedObjectsClient: soClient,
- pkgName: packageConfig.package.name,
- callCluster,
- });
- const pkgInfo = await getPackageInfo({
- savedObjectsClient: soClient,
- pkgName: packageConfig.package.name,
- pkgVersion: packageConfig.package.version,
- });
+ const [, pkgInfo] = await Promise.all([
+ ensureInstalledPackage({
+ savedObjectsClient: soClient,
+ pkgName: packageConfig.package.name,
+ callCluster,
+ }),
+ getPackageInfo({
+ savedObjectsClient: soClient,
+ pkgName: packageConfig.package.name,
+ pkgVersion: packageConfig.package.version,
+ }),
+ ]);
 
  // Check if it is a limited package, and if so, check that the corresponding agent config does not
  // already contain a package config for this package

x-pack/test/ingest_manager_api_integration/apis/index.js

@@ -10,7 +10,7 @@ export default function ({ loadTestFile }) {
 
  // EPM
  loadTestFile(require.resolve('./epm/list'));
- loadTestFile(require.resolve('./epm/list'));
+ loadTestFile(require.resolve('./epm/file'));
  //loadTestFile(require.resolve('./epm/template'));
  loadTestFile(require.resolve('./epm/ilm'));