Skip to content

Commit

Permalink
[Spaces] Filtering out features that do not have space scope (#194157)
Browse files Browse the repository at this point in the history
## Summary

Filtering out features that do not have space scope in
`create_space_page`


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

__Fixes: https://github.com/elastic/kibana/issues/194134__

## Release Note
Filtering out features that do not have space scope in
`create_space_page`
  • Loading branch information
elena-shostak authored Sep 26, 2024
1 parent 4591aa2 commit f8416b8
Show file tree
Hide file tree
Showing 2 changed files with 67 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import { act } from 'react-dom/test-utils';

import { DEFAULT_APP_CATEGORIES } from '@kbn/core/public';
import { notificationServiceMock, scopedHistoryMock } from '@kbn/core/public/mocks';
import { KibanaFeatureScope } from '@kbn/features-plugin/common';
import { KibanaFeature } from '@kbn/features-plugin/public';
import { featuresPluginMock } from '@kbn/features-plugin/public/mocks';
import { findTestSubject, mountWithIntl } from '@kbn/test-jest-helpers';
Expand Down Expand Up @@ -48,6 +49,15 @@ featuresStart.getFeatures.mockResolvedValue([
app: [],
category: DEFAULT_APP_CATEGORIES.kibana,
privileges: null,
scope: [KibanaFeatureScope.Spaces, KibanaFeatureScope.Security],
}),
new KibanaFeature({
id: 'feature-2',
name: 'feature 2',
app: [],
category: DEFAULT_APP_CATEGORIES.kibana,
privileges: null,
scope: [KibanaFeatureScope.Security],
}),
]);

Expand Down Expand Up @@ -641,6 +651,54 @@ describe('ManageSpacePage', () => {

expect(spacesManager.updateSpace).toHaveBeenCalledTimes(1);
});

it('shows only features with space scope', async () => {
const spacesManager = spacesManagerMock.create();
spacesManager.getSpace = jest.fn().mockResolvedValue({
id: 'my-space',
name: 'Existing Space',
description: 'hey an existing space',
color: '#aabbcc',
initials: 'AB',
disabledFeatures: [],
});
spacesManager.getActiveSpace = jest.fn().mockResolvedValue(space);

const wrapper = mountWithIntl(
<CreateSpacePage
spaceId={'my-space'}
spacesManager={spacesManager as unknown as SpacesManager}
getFeatures={featuresStart.getFeatures}
notifications={notificationServiceMock.createStartContract()}
history={history}
capabilities={{
navLinks: {},
management: {},
catalogue: {},
spaces: { manage: true },
}}
eventTracker={eventTracker}
allowFeatureVisibility
allowSolutionVisibility
/>
);

await waitFor(() => {
wrapper.update();
expect(spacesManager.getSpace).toHaveBeenCalledWith('my-space');
});

expect(wrapper.state('features')).toEqual([
new KibanaFeature({
id: 'feature-1',
name: 'feature 1',
app: [],
category: DEFAULT_APP_CATEGORIES.kibana,
privileges: null,
scope: [KibanaFeatureScope.Spaces, KibanaFeatureScope.Security],
}),
]);
});
});

function updateSpace(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ import React, { Component } from 'react';

import type { Capabilities, NotificationsStart, ScopedHistory } from '@kbn/core/public';
import { SectionLoading } from '@kbn/es-ui-shared-plugin/public';
import { KibanaFeatureScope } from '@kbn/features-plugin/common';
import type { FeaturesPluginStart, KibanaFeature } from '@kbn/features-plugin/public';
import { i18n } from '@kbn/i18n';
import { FormattedMessage } from '@kbn/i18n-react';
Expand Down Expand Up @@ -91,6 +92,10 @@ export class CreateSpacePage extends Component<Props, State> {
};
}

private filterSpaceFeatures = (features: KibanaFeature[]) => {
return features.filter((feature) => feature.scope?.includes(KibanaFeatureScope.Spaces));
};

public async componentDidMount() {
if (!this.props.capabilities.spaces.manage) {
return;
Expand All @@ -103,7 +108,8 @@ export class CreateSpacePage extends Component<Props, State> {
await this.loadSpace(spaceId, getFeatures());
} else {
const features = await getFeatures();
this.setState({ isLoading: false, features });

this.setState({ isLoading: false, features: this.filterSpaceFeatures(features) });
}
} catch (e) {
notifications.toasts.addError(e, {
Expand Down Expand Up @@ -410,6 +416,7 @@ export class CreateSpacePage extends Component<Props, State> {
spacesManager.getSpace(spaceId),
featuresPromise,
]);

if (space) {
if (onLoadSpace) {
onLoadSpace(space);
Expand All @@ -426,7 +433,7 @@ export class CreateSpacePage extends Component<Props, State> {
!!space.initials && getSpaceInitials({ name: space.name }) !== space.initials,
customAvatarColor: !!space.color && getSpaceColor({ name: space.name }) !== space.color,
},
features,
features: this.filterSpaceFeatures(features),
originalSpace: space,
isLoading: false,
});
Expand Down

0 comments on commit f8416b8

Please sign in to comment.