[Fleet] Fix package archive bug + reenable Endpoint tests (elastic#14…

…3120) * Reenable endpoint tests to debug in CI * Revert removal of skip in trusted apps * Try enabling Fleet debug logs * Fix missing archive path in bundled packages * Remove Fleet debug logs in endpoint tests * Reenable endpoint suites Co-authored-by: Kibana Machine <[email protected]>
kibanamachine · Oct 14, 2022 · d9d10f2 · d9d10f2
1 parent acdfb23
commit d9d10f2
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/x-pack/plugins/fleet/server/services/epm/registry/index.ts b/x-pack/plugins/fleet/server/services/epm/registry/index.ts
@@ -246,6 +246,7 @@ export async function getInfo(name: string, version: string) {
       // input type packages must get their pkg info from the archive
       if (packageInfo.type === 'integration') setPackageInfo({ name, version, packageInfo });
     }
+
     return packageInfo as RegistryPackage;
   });
 }
@@ -348,9 +349,17 @@ export async function fetchArchiveBuffer({
   verificationResult?: PackageVerificationResult;
 }> {
   const logger = appContextService.getLogger();
-  const { download: archivePath } = await getInfo(pkgName, pkgVersion);
+  let { download: archivePath } = await getInfo(pkgName, pkgVersion);
+
+  // Bundled packages don't have a download path when they're installed, as they're
+  // ArchivePackage objects - so we fake the download path here instead
+  if (!archivePath) {
+    archivePath = `/epr/${pkgName}/${pkgName}-${pkgVersion}.zip`;
+  }
+
   const archiveUrl = `${getRegistryUrl()}${archivePath}`;
   const archiveBuffer = await getResponseStream(archiveUrl).then(streamToBuffer);
+
   if (shouldVerify) {
     const verificationResult = await verifyPackageArchiveSignature({
       pkgName,

diff --git a/x-pack/test/security_solution_endpoint/apps/endpoint/artifact_entries_list.ts b/x-pack/test/security_solution_endpoint/apps/endpoint/artifact_entries_list.ts
@@ -22,7 +22,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
   const esClient = getService('es');
   const unzipPromisify = promisify(unzip);
 
-  describe.skip('For each artifact list under management', function () {
+  describe('For each artifact list under management', function () {
     let indexedData: IndexedHostsAndAlertsResponse;
 
     const checkFleetArtifacts = async (

diff --git a/x-pack/test/security_solution_endpoint/apps/endpoint/endpoint_permissions.ts b/x-pack/test/security_solution_endpoint/apps/endpoint/endpoint_permissions.ts
@@ -20,7 +20,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
   const endpointTestResources = getService('endpointTestResources');
   const policyTestResources = getService('policyTestResources');
 
-  describe.skip('Endpoint permissions:', () => {
+  describe('Endpoint permissions:', () => {
     let indexedData: IndexedHostsAndAlertsResponse;
 
     before(async () => {

diff --git a/x-pack/test/security_solution_endpoint/apps/endpoint/responder.ts b/x-pack/test/security_solution_endpoint/apps/endpoint/responder.ts
@@ -76,7 +76,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
     );
   };
 
-  describe.skip('Response Actions Responder', function () {
+  describe('Response Actions Responder', function () {
     let indexedData: IndexedHostsAndAlertsResponse;
     let endpointAgentId: string;