Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FAILS ON MKI] Serverless Search API Integration Tests - Common Group 1.x-pack/test_serverless/api_integration/test_suites/common/platform_security/authorization.ts #192282

Closed
wayneseymour opened this issue Sep 6, 2024 · 3 comments · Fixed by #195584
Assignees
Labels
impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. skipped-test Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test-failure-flaky

Comments

@wayneseymour
Copy link
Member

Summary

Changes introduced earlier today have caused failures on MKI, real serverless.

Error Message

Owners
elastic/kibana-security
Error message
Error: expected { statusCode: 400,
  error: 'Bad Request',
  message: '[request body]: expected a plain object value, but found [null] instead.' } to sort of equal { statusCode: 404,
  error: 'Not Found',
  message: 'Not Found' }
    at Assertion.assert (expect.js:100:11)
    at Assertion.eql (expect.js:244:8)
    at Object.assertApiNotFound (svl_common_api.ts:44:23)
    at Context. (authorization.ts:70:26)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.apply (wrap_function.js:73:16) {
  actual: '{\n' +
    '  "error": "Bad Request"\n' +
    '  "message": "[request body]: expected a plain object value, but found [null] instead."\n' +
    '  "statusCode": 400\n' +
    '}',
  expected: '{\n  "error": "Not Found"\n  "message": "Not Found"\n  "statusCode": 404\n}',
  showDiff: true
}
@wayneseymour wayneseymour added impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test-failure-flaky labels Sep 6, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

wayneseymour added a commit to wayneseymour/kibana that referenced this issue Sep 6, 2024
pheyos pushed a commit that referenced this issue Sep 6, 2024
…on/platform_security/authorization.ts (#192284)

## Summary

See details: #192282
@pheyos
Copy link
Member

pheyos commented Sep 6, 2024

Skipped for MKI runs in #192284

@jeramysoucy
Copy link
Contributor

This failure is tied to the enabling of custom roles in QA. It was not caught earlier because there was a typo in the endpoint path, which was fixed in #191729. Thus these tests have been temporarily skipped in MKI.

When custom roles are enabled in production, these tests can be unskipped. Also, occurrences of "roleAuthc" in test descriptions should be fixed (to "role") as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. skipped-test Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test-failure-flaky
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants