PromucFlow_constructor/app/client/src/utils/testPropertyPaneConfig.test.ts
Valera Melnikov 10a98f9563
fix: use wds tokens in sections and zones (#30479)
## Description
- Added tokens to the anvil config
- Removed redundant functions for calculating widget sizes
- I cleaned the widget configs
- Added a zero-state to the input component
- Optimised the responsiveness of the action group component


https://github.com/appsmithorg/appsmith/assets/11555074/e9e0513a-cf3f-42ef-b960-820bb4980858

#### PR fixes following issue(s)
Fixes #29507 

#### Type of change
> Please delete options that are not relevant.
- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Chore (housekeeping or task changes that don't impact user perception)

## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress
>
>
#### Test Plan
> Add Testsmith test cases links that relate to this PR
>
>
#### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking
(copy link from comments dropped on this PR)
>
>
>
## Checklist:
#### Dev activity
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag


#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Enhanced `MenuItem` display with additional text attribute for better
accessibility.
- Improved user interaction with `TextArea` and `TextInput` components
through more effective handling of default and read-only states.
- Extended `Flex` component to include click event handling
capabilities.
- Refined `ActionGroup` styling and dynamic sizing for a more polished
UI experience.

- **Enhancements**
- Updated `Button` styling to ensure consistency across different screen
sizes.
- Optimized layout and styling of `Menu` items for improved visual
hierarchy and responsiveness.

- **Refactor**
- Simplified `WidgetProvider` and `AnvilEditorWidget` configurations for
more streamlined layout management.
- Refined `ZoneColumn` rendering for better alignment with layout system
principles.

- **Bug Fixes**
- Fixed minimum width calculation in `useZoneMinWidth` hook to ensure
correct layout behavior.
- Corrected `AnvilDSLTransformer` padding value for consistent widget
spacing.

- **Documentation**
- Removed redundant Storybook addon to streamline developer experience.

- **Style**
- Adjusted CSS for various components to align with updated design
system standards.

- **Tests**
- Updated widget property pane configuration tests for better coverage
and reliability.

- **Chores**
- Cleaned up unused imports and methods across multiple widget
components for improved code maintenance.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2024-01-24 11:02:42 +03:00

221 lines
7.4 KiB
TypeScript

import type {
PropertyPaneConfig,
PropertyPaneControlConfig,
PropertyPaneSectionConfig,
ValidationConfig,
} from "constants/PropertyControlConstants";
import { ValidationTypes } from "constants/WidgetValidation";
import { isFunction } from "lodash";
import widgets from "widgets";
import WidgetFactory from "WidgetProvider/factory";
import { registerWidgets } from "WidgetProvider/factory/registrationHelper";
function validatePropertyPaneConfig(
config: PropertyPaneConfig[],
isWidgetHidden: boolean,
) {
for (const sectionOrControlConfig of config) {
const sectionConfig = sectionOrControlConfig as PropertyPaneSectionConfig;
if (sectionConfig.sectionName && isFunction(sectionConfig.sectionName)) {
return ` SectionName should be a string and not a function. Search won't work for functions at the moment`;
}
if (sectionOrControlConfig.children) {
for (const propertyControlConfig of sectionOrControlConfig.children) {
const propertyControlValidation = validatePropertyControl(
propertyControlConfig,
isWidgetHidden,
);
if (propertyControlValidation !== true)
return propertyControlValidation;
}
}
}
return true;
}
function validatePropertyControl(
config: PropertyPaneConfig,
isWidgetHidden: boolean,
): boolean | string {
const _config = config as PropertyPaneControlConfig;
const controls = ["INPUT_TEXT"];
if (_config.label && isFunction(_config.label)) {
return `${_config.propertyName}: Label should be a string and not a function. Search won't work for functions at the moment`;
}
if (
!isWidgetHidden &&
_config.label &&
!_config.invisible &&
!(_config.helpText || _config.helperText)
) {
return `${_config.propertyName} (${_config.label}): Help text or Helper text is mandatory for property controls`;
}
if (
(_config.isJSConvertible || controls.includes(_config.controlType)) &&
!_config.isTriggerProperty
) {
if (!_config.isBindProperty)
return `${
_config.propertyName
}: isBindProperty should be true if isJSConvertible is true or when control type is [${controls.join(
" | ",
)}]`;
if (!_config.validation)
return `${
_config.propertyName
}: validation should be defined if isJSConvertible is true or when control type is [${controls.join(
" | ",
)}]`;
}
if (controls.includes(_config.controlType) && _config.isJSConvertible) {
return `${
_config.propertyName
}: No need of setting isJSConvertible since users can write JS inside [${controls.join(
" | ",
)}]`;
}
if (_config.validation !== undefined) {
const res = validateValidationStructure(_config.validation);
if (res !== true) return `${_config.propertyName}: ${res}`;
}
if (_config.children) {
for (const child of _config.children) {
const res = validatePropertyControl(child, isWidgetHidden);
if (res !== true) return `${_config.propertyName}.${res}`;
}
}
if (_config.panelConfig) {
if (_config.panelConfig.children) {
const res = validatePropertyPaneConfig(
_config.panelConfig.children,
isWidgetHidden,
);
if (res !== true) return `${_config.propertyName}.${res}`;
}
if (_config.panelConfig.contentChildren) {
const res = validatePropertyPaneConfig(
_config.panelConfig.contentChildren,
isWidgetHidden,
);
if (res !== true) return `${_config.propertyName}.${res}`;
}
if (_config.panelConfig.styleChildren) {
const res = validatePropertyPaneConfig(
_config.panelConfig.styleChildren,
isWidgetHidden,
);
if (res !== true) return `${_config.propertyName}.${res}`;
}
}
return true;
}
function validateValidationStructure(
config: ValidationConfig,
): boolean | string {
if (
config.type === ValidationTypes.FUNCTION &&
config.params &&
config.params.fn
) {
if (!config.params.expected)
return `For a ${ValidationTypes.FUNCTION} type validation, expected type and example are mandatory`;
}
return true;
}
const isNotFloat = (n: any) => {
return Number(n) === n && n % 1 === 0;
};
describe("Tests all widget's propertyPane config", () => {
beforeAll(() => {
registerWidgets(widgets);
});
widgets
// Exclude WDS widgets from the tests, since they work differently
.filter((widget) => !widget.type.includes("WDS"))
.forEach((widget) => {
const config = widget.getConfig();
it(`Checks ${widget.type}'s propertyPaneConfig`, () => {
const propertyPaneConfig = widget.getPropertyPaneConfig();
expect(
validatePropertyPaneConfig(propertyPaneConfig, !!config.hideCard),
).toStrictEqual(true);
const propertyPaneContentConfig = widget.getPropertyPaneContentConfig();
expect(
validatePropertyPaneConfig(
propertyPaneContentConfig,
!!config.isDeprecated,
),
).toStrictEqual(true);
const propertyPaneStyleConfig = widget.getPropertyPaneStyleConfig();
expect(
validatePropertyPaneConfig(
propertyPaneStyleConfig,
!!config.isDeprecated,
),
).toStrictEqual(true);
});
it(`Check if ${widget.type}'s dimensions are always integers`, () => {
const defaults = widget.getDefaults();
expect(isNotFloat(defaults.rows)).toBe(true);
expect(isNotFloat(defaults.columns)).toBe(true);
});
if (config.isDeprecated) {
it(`Check if ${widget.type}'s deprecation config has a proper replacement Widget`, () => {
const widgetType = widget.type;
if (config.replacement === undefined) {
fail(`${widgetType}'s replacement widget is not defined`);
}
const replacementWidgetType = config.replacement;
const replacementWidget = WidgetFactory.get(replacementWidgetType);
const replacementWidgetConfig = replacementWidget?.getConfig();
if (replacementWidgetConfig === undefined) {
fail(
`${widgetType}'s replacement widget ${replacementWidgetType} does not resolve to an actual widget Config`,
);
}
if (replacementWidgetConfig?.isDeprecated) {
fail(
`${widgetType}'s replacement widget ${replacementWidgetType} itself is deprecated. Cannot have a deprecated widget as a replacement for another deprecated widget`,
);
}
if (replacementWidgetConfig?.hideCard) {
fail(
`${widgetType}'s replacement widget ${replacementWidgetType} should be available in the entity Explorer`,
);
}
});
}
it(`Check if ${widget.type}'s setter method are configured correctly`, () => {
const setterConfig = widget.getSetterConfig();
if (setterConfig) {
expect(setterConfig).toHaveProperty("__setters");
const setters = setterConfig.__setters;
for (const [setterName, config] of Object.entries(setters)) {
expect(config).toHaveProperty("type");
expect(config).toHaveProperty("path");
expect(setterName).toContain("set");
const type = config.type;
const path = config.path;
expect(typeof type).toBe("string");
expect(typeof path).toBe("string");
}
}
});
});
});