Skip to content

Commit 86835a7

Browse files
committed
Revert "CONSOLE-3769: Remove usage of legacy Extension"
This reverts commit 4087fcb.
1 parent 2b444de commit 86835a7

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

43 files changed

+183
-210
lines changed

frontend/packages/console-app/src/__tests__/plugin-test-utils.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
11
import { List as ImmutableList } from 'immutable';
22
import * as _ from 'lodash';
3-
import type { ExtensionDeclaration } from '@console/dynamic-plugin-sdk/src/types';
4-
import { PluginStore } from '@console/plugin-sdk';
3+
import { Extension, PluginStore } from '@console/plugin-sdk';
54
import { loadActivePluginsForTestPurposes } from '@console/plugin-sdk/src/codegen/active-plugins';
65
import { resolvePluginPackages } from '@console/plugin-sdk/src/codegen/plugin-resolver';
76

87
const testedPlugins = loadActivePluginsForTestPurposes(resolvePluginPackages());
98
const testedPluginStore = new PluginStore(testedPlugins);
109

11-
export const testedExtensions = ImmutableList<ExtensionDeclaration>(
12-
testedPluginStore.getExtensions(),
13-
);
10+
export const testedExtensions = ImmutableList<Extension>(testedPluginStore.getExtensions());
1411

1512
export const getDuplicates = (values: string[]) => {
1613
return _.transform(

frontend/packages/console-dynamic-plugin-sdk/src/coderefs/__tests__/coderef-resolver.spec.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as _ from 'lodash';
2-
import { ExtensionDeclaration, EncodedCodeRef, CodeRef } from '../../types';
2+
import { Extension, EncodedCodeRef, CodeRef } from '../../types';
33
import {
44
getExecutableCodeRefMock,
55
getEntryModuleMocks,
@@ -208,7 +208,7 @@ describe('loadReferencedObject', () => {
208208

209209
describe('resolveEncodedCodeRefs', () => {
210210
it('replaces encoded code references with CodeRef functions', async () => {
211-
const extensions: ExtensionDeclaration[] = [
211+
const extensions: Extension[] = [
212212
{
213213
type: 'Foo',
214214
properties: {
@@ -247,7 +247,7 @@ describe('resolveEncodedCodeRefs', () => {
247247
});
248248

249249
it('clones the provided extensions array and its elements', () => {
250-
const extensions: ExtensionDeclaration[] = [
250+
const extensions: Extension[] = [
251251
{ type: 'Foo', properties: { test: true } },
252252
{ type: 'Bar', properties: { test: [1] } },
253253
];
@@ -274,7 +274,7 @@ describe('resolveEncodedCodeRefs', () => {
274274

275275
describe('resolveExtension', () => {
276276
it('replaces CodeRef functions with referenced objects', async () => {
277-
const extensions: ExtensionDeclaration[] = [
277+
const extensions: Extension[] = [
278278
{
279279
type: 'Foo',
280280
properties: {
@@ -309,7 +309,7 @@ describe('resolveExtension', () => {
309309
});
310310

311311
it('logs a warning if the referenced object resolves to null or undefined', async () => {
312-
const extensions: ExtensionDeclaration[] = [
312+
const extensions: Extension[] = [
313313
{
314314
type: 'Foo',
315315
properties: {
@@ -343,7 +343,7 @@ describe('resolveExtension', () => {
343343
});
344344

345345
it('returns the same extension instance', async () => {
346-
const extensions: ExtensionDeclaration[] = [
346+
const extensions: Extension[] = [
347347
{
348348
type: 'Foo',
349349
properties: { test: true },
@@ -361,7 +361,7 @@ describe('resolveExtension', () => {
361361
it('continuously reject code refs which have failed to resolve', async () => {
362362
const errorCodeRef = getErrorExecutableCodeRefMock();
363363

364-
const extension: ExtensionDeclaration = {
364+
const extension: Extension = {
365365
type: 'Foo',
366366
properties: { test: true, qux: errorCodeRef },
367367
};

frontend/packages/console-dynamic-plugin-sdk/src/extensions/actions.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import * as React from 'react';
22
import { ExtensionHook, ExtensionK8sKindVersionModel } from '../api/common-types';
33
import { ActionContext } from '../api/internal-types';
4-
import { ExtensionDeclaration, CodeRef } from '../types';
4+
import { Extension, ExtensionDeclaration, CodeRef } from '../types';
55
import { AccessReviewResourceAttributes } from './console-types';
66

77
/** ActionProvider contributes a hook that returns list of actions for specific context */
@@ -74,19 +74,19 @@ export type SupportedActionExtensions =
7474

7575
// Type Guards
7676

77-
export const isActionProvider = (e: ExtensionDeclaration): e is ActionProvider => {
77+
export const isActionProvider = (e: Extension): e is ActionProvider => {
7878
return e.type === 'console.action/provider';
7979
};
8080

81-
export const isResourceActionProvider = (e: ExtensionDeclaration): e is ResourceActionProvider => {
81+
export const isResourceActionProvider = (e: Extension): e is ResourceActionProvider => {
8282
return e.type === 'console.action/resource-provider';
8383
};
8484

85-
export const isActionGroup = (e: ExtensionDeclaration): e is ActionGroup => {
85+
export const isActionGroup = (e: Extension): e is ActionGroup => {
8686
return e.type === 'console.action/group';
8787
};
8888

89-
export const isActionFilter = (e: ExtensionDeclaration): e is ActionFilter => {
89+
export const isActionFilter = (e: Extension): e is ActionFilter => {
9090
return e.type === 'console.action/filter';
9191
};
9292

frontend/packages/console-dynamic-plugin-sdk/src/extensions/add-actions.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as React from 'react';
2-
import { ExtensionDeclaration, CodeRef } from '../types';
2+
import { Extension, ExtensionDeclaration, CodeRef } from '../types';
33
import { AccessReviewResourceAttributes } from './console-types';
44

55
/** This extension allows plugins to contribute an add action item to the add page of developer perspective.
@@ -48,10 +48,10 @@ export type AddActionGroup = ExtensionDeclaration<
4848

4949
// Type guards
5050

51-
export const isAddAction = (e: ExtensionDeclaration): e is AddAction => {
51+
export const isAddAction = (e: Extension): e is AddAction => {
5252
return e.type === 'dev-console.add/action';
5353
};
5454

55-
export const isAddActionGroup = (e: ExtensionDeclaration): e is AddActionGroup => {
55+
export const isAddActionGroup = (e: Extension): e is AddActionGroup => {
5656
return e.type === 'dev-console.add/action-group';
5757
};

frontend/packages/console-dynamic-plugin-sdk/src/extensions/alerts.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { Rule, PrometheusRulesResponse } from '../api/common-types';
2-
import { ExtensionDeclaration, CodeRef } from '../types';
2+
import { CodeRef, Extension, ExtensionDeclaration } from '../types';
33

44
type AlertingRulesFetch = (namespace?: string) => Promise<PrometheusRulesResponse>;
55

@@ -31,8 +31,8 @@ export type AlertingRuleChartExtension = ExtensionDeclaration<
3131

3232
// Type guards
3333

34-
export const isAlertingRulesSource = (e: ExtensionDeclaration): e is AlertingRulesSourceExtension =>
34+
export const isAlertingRulesSource = (e: Extension): e is AlertingRulesSourceExtension =>
3535
e.type === 'console.alerts/rules-source';
3636

37-
export const isAlertingRuleChart = (e: ExtensionDeclaration): e is AlertingRuleChartExtension =>
37+
export const isAlertingRuleChart = (e: Extension): e is AlertingRuleChartExtension =>
3838
e.type === 'console.alerts/rules-chart';

frontend/packages/console-dynamic-plugin-sdk/src/extensions/breadcrumbs.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { K8sModel } from '../api/common-types';
2-
import { ExtensionDeclaration, CodeRef } from '../types';
2+
import { CodeRef, Extension, ExtensionDeclaration } from '../types';
33

44
export type DetailsPageBreadCrumbsHook = (
55
kind: K8sModel,
@@ -22,5 +22,5 @@ export type DetailPageBreadCrumbs = ExtensionDeclaration<
2222

2323
// Type guards
2424

25-
export const isDetailPageBreadCrumbs = (e: ExtensionDeclaration): e is DetailPageBreadCrumbs =>
25+
export const isDetailPageBreadCrumbs = (e: Extension): e is DetailPageBreadCrumbs =>
2626
e.type === 'dev-console.detailsPage/breadcrumbs';

frontend/packages/console-dynamic-plugin-sdk/src/extensions/catalog.ts

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import * as React from 'react';
22
import { ExtensionHook } from '../api/common-types';
3-
import { ExtensionDeclaration, CodeRef } from '../types';
3+
import { Extension, ExtensionDeclaration, CodeRef } from '../types';
44

55
/** This extension allows plugins to contribute a new type of catalog item. For example, a Helm plugin can define
66
a new catalog item type as HelmCharts that it wants to contribute to the Software Catalog. */
@@ -109,33 +109,27 @@ export type SupportedCatalogExtensions =
109109

110110
// Type guards
111111

112-
export const isCatalogItemType = (e: ExtensionDeclaration): e is CatalogItemType => {
112+
export const isCatalogItemType = (e: Extension): e is CatalogItemType => {
113113
return e.type === 'console.catalog/item-type';
114114
};
115115

116-
export const isCatalogItemTypeMetadata = (
117-
e: ExtensionDeclaration,
118-
): e is CatalogItemTypeMetadata => {
116+
export const isCatalogItemTypeMetadata = (e: Extension): e is CatalogItemTypeMetadata => {
119117
return e.type === 'console.catalog/item-type-metadata';
120118
};
121119

122-
export const isCatalogItemProvider = (e: ExtensionDeclaration): e is CatalogItemProvider => {
120+
export const isCatalogItemProvider = (e: Extension): e is CatalogItemProvider => {
123121
return e.type === 'console.catalog/item-provider';
124122
};
125123

126-
export const isCatalogItemFilter = (e: ExtensionDeclaration): e is CatalogItemFilter => {
124+
export const isCatalogItemFilter = (e: Extension): e is CatalogItemFilter => {
127125
return e.type === 'console.catalog/item-filter';
128126
};
129127

130-
export const isCatalogItemMetadataProvider = (
131-
e: ExtensionDeclaration,
132-
): e is CatalogItemMetadataProvider => {
128+
export const isCatalogItemMetadataProvider = (e: Extension): e is CatalogItemMetadataProvider => {
133129
return e.type === 'console.catalog/item-metadata';
134130
};
135131

136-
export const isCatalogCategoriesProvider = (
137-
e: ExtensionDeclaration,
138-
): e is CatalogCategoriesProvider => {
132+
export const isCatalogCategoriesProvider = (e: Extension): e is CatalogCategoriesProvider => {
139133
return e.type === 'console.catalog/categories-provider';
140134
};
141135

frontend/packages/console-dynamic-plugin-sdk/src/extensions/cluster-configuration.ts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import * as React from 'react';
22
import { JSONSchema7Type } from 'json-schema';
3-
import { ExtensionDeclaration, CodeRef } from '../types';
3+
import { Extension, ExtensionDeclaration, CodeRef } from '../types';
44
import { AccessReviewResourceAttributes } from './console-types';
55

66
export type Resource = {
@@ -114,14 +114,10 @@ export type ClusterConfigurationItem = ExtensionDeclaration<
114114

115115
// Type guards
116116

117-
export const isClusterConfigurationGroup = (
118-
e: ExtensionDeclaration,
119-
): e is ClusterConfigurationGroup => {
117+
export const isClusterConfigurationGroup = (e: Extension): e is ClusterConfigurationGroup => {
120118
return e.type === 'console.cluster-configuration/group';
121119
};
122120

123-
export const isClusterConfigurationItem = (
124-
e: ExtensionDeclaration,
125-
): e is ClusterConfigurationItem => {
121+
export const isClusterConfigurationItem = (e: Extension): e is ClusterConfigurationItem => {
126122
return e.type === 'console.cluster-configuration/item';
127123
};

frontend/packages/console-dynamic-plugin-sdk/src/extensions/cluster-overview.ts

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import * as React from 'react';
22
import { OverviewDetailItemProps } from '@openshift-console/plugin-shared/src';
3-
import { ExtensionDeclaration, CodeRef } from '../types';
3+
import { CodeRef, Extension, ExtensionDeclaration } from '../types';
44
import { Humanize, TopConsumerPopoverProps, QueryWithDescription } from './console-types';
55

66
/** Adds a new inventory item into cluster overview page. */
@@ -77,22 +77,20 @@ export type CustomOverviewDetailItem = ExtensionDeclaration<
7777

7878
// Type guards
7979

80-
export const isClusterOverviewInventoryItem = (
81-
e: ExtensionDeclaration,
82-
): e is ClusterOverviewInventoryItem => e.type === 'console.cluster-overview/inventory-item';
80+
export const isClusterOverviewInventoryItem = (e: Extension): e is ClusterOverviewInventoryItem =>
81+
e.type === 'console.cluster-overview/inventory-item';
8382

8483
export const isClusterOverviewUtilizationItem = (
85-
e: ExtensionDeclaration,
84+
e: Extension,
8685
): e is ClusterOverviewUtilizationItem => e.type === 'console.cluster-overview/utilization-item';
8786

8887
export const isClusterOverviewMultilineUtilizationItem = (
89-
e: ExtensionDeclaration,
88+
e: Extension,
9089
): e is ClusterOverviewMultilineUtilizationItem =>
9190
e.type === 'console.cluster-overview/multiline-utilization-item';
9291

93-
export const isOverviewDetailItem = (e: ExtensionDeclaration): e is OverviewDetailItem =>
92+
export const isOverviewDetailItem = (e: Extension): e is OverviewDetailItem =>
9493
e.type === 'console.dashboards/overview/detail/item';
9594

96-
export const isCustomOverviewDetailItem = (
97-
e: ExtensionDeclaration,
98-
): e is CustomOverviewDetailItem => e.type === 'console.dashboards/custom/overview/detail/item';
95+
export const isCustomOverviewDetailItem = (e: Extension): e is CustomOverviewDetailItem =>
96+
e.type === 'console.dashboards/custom/overview/detail/item';

frontend/packages/console-dynamic-plugin-sdk/src/extensions/cluster-settings.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { ExtensionK8sModel } from '../api/common-types';
2-
import { ExtensionDeclaration } from '../types';
2+
import { Extension, ExtensionDeclaration } from '../types';
3+
34
/** This extension identifies a resource used to manage the configuration of the cluster.
45
A link to the resource will be added to the Administration - Cluster Settings - Configuration page. */
56
export type ClusterGlobalConfig = ExtensionDeclaration<
@@ -18,5 +19,5 @@ export type ClusterGlobalConfig = ExtensionDeclaration<
1819

1920
// Type guards
2021

21-
export const isClusterGlobalConfig = (e: ExtensionDeclaration): e is ClusterGlobalConfig =>
22+
export const isClusterGlobalConfig = (e: Extension): e is ClusterGlobalConfig =>
2223
e.type === 'console.global-config';

0 commit comments

Comments
 (0)