[Security Solution] fix/unskip endpoint metering tests

elastic · Aug 7, 2024 · dd12054 · dd12054
1 parent ae1ff9e
commit dd12054
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/x-pack/plugins/security/server/authentication/authenticator.ts b/x-pack/plugins/security/server/authentication/authenticator.ts
@@ -811,6 +811,9 @@ export class Authenticator {
       });
       existingSessionValue = null;
     } else if (usernameHasChanged) {
+      this.logger.warn(
+        `~~~old ${existingSessionValue!.username} | ~~~new ${authenticationResult.user!.username}`
+      );
       this.logger.warn('Username has changed, existing session will be invalidated.');
       await this.invalidateSessionValue({ request, sessionValue: existingSessionValue });
       existingSessionValue = null;

diff --git a/x-pack/plugins/security_solution/public/management/cypress/cypress_base.config.ts b/x-pack/plugins/security_solution/public/management/cypress/cypress_base.config.ts
@@ -77,7 +77,7 @@ export const getCypressBaseConfig = (
         // baseUrl: To override, set Env. variable `CYPRESS_BASE_URL`
         baseUrl: 'http://localhost:5601',
         supportFile: 'public/management/cypress/support/e2e.ts',
-        specPattern: 'public/management/cypress/e2e/**/*.cy.{js,jsx,ts,tsx}',
+        specPattern: 'public/management/cypress/e2e/**/metering.cy.{js,jsx,ts,tsx}',
         experimentalRunAllSpecs: true,
         experimentalMemoryManagement: true,
         experimentalInteractiveRunEvents: true,

diff --git a/x-pack/plugins/security_solution/public/management/cypress/e2e/serverless/metering.cy.ts b/x-pack/plugins/security_solution/public/management/cypress/e2e/serverless/metering.cy.ts
@@ -17,8 +17,7 @@ import type { ReturnTypeFromChainable } from '../../types';
 import { indexEndpointHeartbeats } from '../../tasks/index_endpoint_heartbeats';
 import { login, ROLE } from '../../tasks/login';
 
-// Failing: See https://github.com/elastic/kibana/issues/187083
-describe.skip(
+describe(
   'Metering',
   {
     tags: ['@serverless', '@skipInServerlessMKI'],

diff --git a/x-pack/plugins/security_solution/public/management/cypress/support/e2e.ts b/x-pack/plugins/security_solution/public/management/cypress/support/e2e.ts
@@ -107,4 +107,5 @@ before(() => {
   login(ROLE.soc_manager);
   loadPage('/app/security/alerts');
   cy.getByTestSubj('manage-alert-detection-rules').should('exist');
+  cy.visit('/logout');
 });