Skip to content

Commit 42d22fe

Browse files
[8.19] Remove skipped user API tests that are no longer accurate (#239786) (#239843)
# Backport This will backport the following commits from `main` to `8.19`: - [Remove skipped user API tests that are no longer accurate (#239786)](#239786) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Larry Gregory","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-10-21T06:42:12Z","message":"Remove skipped user API tests that are no longer accurate (#239786)\n\n## Summary\n\nRemoves skipped user api tests that are no longer relevant.\nThese tests were slated for inclusion when we expected the user apis to\nbe disabled in Serverless environments. These APIs are to remain enabled\nin Serverless, with no current plans to disable them.","sha":"6cdb76c3c2cdc24e544617ef8550567816f02b9c","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Security","release_note:skip","backport:all-open","v9.3.0"],"title":"Remove skipped user API tests that are no longer accurate","number":239786,"url":"https://github.com/elastic/kibana/pull/239786","mergeCommit":{"message":"Remove skipped user API tests that are no longer accurate (#239786)\n\n## Summary\n\nRemoves skipped user api tests that are no longer relevant.\nThese tests were slated for inclusion when we expected the user apis to\nbe disabled in Serverless environments. These APIs are to remain enabled\nin Serverless, with no current plans to disable them.","sha":"6cdb76c3c2cdc24e544617ef8550567816f02b9c"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239786","number":239786,"mergeCommit":{"message":"Remove skipped user API tests that are no longer accurate (#239786)\n\n## Summary\n\nRemoves skipped user api tests that are no longer relevant.\nThese tests were slated for inclusion when we expected the user apis to\nbe disabled in Serverless environments. These APIs are to remain enabled\nin Serverless, with no current plans to disable them.","sha":"6cdb76c3c2cdc24e544617ef8550567816f02b9c"}}]}] BACKPORT--> Co-authored-by: Larry Gregory <[email protected]>
1 parent 56ccab8 commit 42d22fe

File tree

1 file changed

+0
-59
lines changed
  • x-pack/platform/test/serverless/api_integration/test_suites/platform_security

1 file changed

+0
-59
lines changed

x-pack/platform/test/serverless/api_integration/test_suites/platform_security/users.ts

Lines changed: 0 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import { FtrProviderContext } from '../../ftr_provider_context';
1212
export default function ({ getService }: FtrProviderContext) {
1313
const samlAuth = getService('samlAuth');
1414
const supertestWithoutAuth = getService('supertestWithoutAuth');
15-
const svlCommonApi = getService('svlCommonApi');
1615
const roleScopedSupertest = getService('roleScopedSupertest');
1716
let supertestAdminWithCookieCredentials: SupertestWithRoleScopeType;
1817

@@ -28,64 +27,6 @@ export default function ({ getService }: FtrProviderContext) {
2827
});
2928

3029
describe('route access', () => {
31-
// ToDo: uncomment when we disable user APIs
32-
describe.skip('disabled', () => {
33-
it('get', async () => {
34-
const { body, status } = await supertestAdminWithCookieCredentials.get(
35-
'/internal/security/users/elastic'
36-
);
37-
svlCommonApi.assertApiNotFound(body, status);
38-
});
39-
40-
it('get all', async () => {
41-
const { body, status } = await supertestAdminWithCookieCredentials.get(
42-
'/internal/security/users'
43-
);
44-
svlCommonApi.assertApiNotFound(body, status);
45-
});
46-
47-
it('create/update', async () => {
48-
const { body, status } = await supertestWithoutAuth
49-
.post(`/internal/security/users/some_testuser`)
50-
.set(samlAuth.getInternalRequestHeader())
51-
.send({ username: 'some_testuser', password: 'testpassword', roles: [] });
52-
svlCommonApi.assertApiNotFound(body, status);
53-
});
54-
55-
it('delete', async () => {
56-
const { body, status } = await supertestAdminWithCookieCredentials.delete(
57-
`/internal/security/users/elastic`
58-
);
59-
svlCommonApi.assertApiNotFound(body, status);
60-
});
61-
62-
it('disable', async () => {
63-
const { body, status } = await supertestAdminWithCookieCredentials.post(
64-
`/internal/security/users/elastic/_disable`
65-
);
66-
svlCommonApi.assertApiNotFound(body, status);
67-
});
68-
69-
it('enable', async () => {
70-
const { body, status } = await supertestAdminWithCookieCredentials.post(
71-
`/internal/security/users/elastic/_enable`
72-
);
73-
svlCommonApi.assertApiNotFound(body, status);
74-
});
75-
76-
it('set password', async () => {
77-
const { body, status } = await supertestWithoutAuth
78-
.post(`/internal/security/users/{username}/password`)
79-
.set(samlAuth.getInternalRequestHeader())
80-
.send({
81-
password: 'old_pw',
82-
newPassword: 'new_pw',
83-
});
84-
svlCommonApi.assertApiNotFound(body, status);
85-
});
86-
});
87-
88-
// ToDo: remove when we disable user APIs
8930
describe('internal', () => {
9031
it('get', async () => {
9132
const { status } = await supertestAdminWithCookieCredentials.get(

0 commit comments

Comments
 (0)