Unskip flaky suites in agent service jest tests

Closes elastic#172114 Closes elastic#171536 Closes elastic#171160 Closes elastic#171052 Closes elastic#171541
kpollich · Nov 29, 2023 · 1dc9f70 · 1dc9f70
1 parent d1402af
commit 1dc9f70
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 7 deletions.
diff --git a/x-pack/plugins/fleet/server/services/agents/crud.test.ts b/x-pack/plugins/fleet/server/services/agents/crud.test.ts
@@ -27,6 +27,14 @@ jest.mock('../audit_logging');
 jest.mock('../../../common/services/is_agent_upgradeable', () => ({
   isAgentUpgradeable: jest.fn().mockImplementation((agent: Agent) => agent.id.includes('up')),
 }));
+jest.mock('./versions', () => {
+  return {
+    getAvailableVersions: jest
+      .fn()
+      .mockResolvedValue(['8.4.0', '8.5.0', '8.6.0', '8.7.0', '8.8.0']),
+    getLatestAvailableVersion: jest.fn().mockResolvedValue('8.8.0'),
+  };
+});
 
 const mockedAuditLoggingService = auditLoggingService as jest.Mocked<typeof auditLoggingService>;
 
@@ -151,8 +159,7 @@ describe('Agents CRUD test', () => {
     });
   });
 
-  // FLAKY: https://github.com/elastic/kibana/issues/171541
-  describe.skip('getAgentsByKuery', () => {
+  describe('getAgentsByKuery', () => {
     it('should return upgradeable on first page', async () => {
       searchMock
         .mockImplementationOnce(() => Promise.resolve(getEsResponse(['1', '2', '3', '4', '5'], 7)))

diff --git a/x-pack/plugins/fleet/server/services/agents/upgrade.test.ts b/x-pack/plugins/fleet/server/services/agents/upgrade.test.ts
@@ -15,6 +15,15 @@ import { sendUpgradeAgentsActions } from './upgrade';
 import { createClientMock } from './action.mock';
 import { getRollingUpgradeOptions, upgradeBatch } from './upgrade_action_runner';
 
+jest.mock('./versions', () => {
+  return {
+    getAvailableVersions: jest
+      .fn()
+      .mockResolvedValue(['8.4.0', '8.5.0', '8.6.0', '8.7.0', '8.8.0']),
+    getLatestAvailableVersion: jest.fn().mockResolvedValue('8.8.0'),
+  };
+});
+
 jest.mock('./action_status', () => {
   return {
     getCancelledActions: jest.fn().mockResolvedValue([
@@ -25,11 +34,7 @@ jest.mock('./action_status', () => {
   };
 });
 
-// FLAKY: https://github.com/elastic/kibana/issues/171052
-// FLAKY: https://github.com/elastic/kibana/issues/172114
-// FLAKY: https://github.com/elastic/kibana/issues/171536
-// FLAKY: https://github.com/elastic/kibana/issues/171160
-describe.skip('sendUpgradeAgentsActions (plural)', () => {
+describe('sendUpgradeAgentsActions (plural)', () => {
   beforeEach(async () => {
     appContextService.start(createAppContextStartContractMock());
   });