Form Smart Paste: Handle possible issues in smartPaste method#30957
Form Smart Paste: Handle possible issues in smartPaste method#30957juleari merged 11 commits intoDevExpress:25_2from
Conversation
packages/devextreme/js/__internal/ui/form/form.layout_manager.ts
Outdated
Show resolved
Hide resolved
packages/devextreme/testing/tests/DevExpress.ui.widgets.form/form.ai.utils.tests.js
Outdated
Show resolved
Hide resolved
packages/devextreme/testing/tests/DevExpress.ui.widgets.form/form.ai.utils.tests.js
Outdated
Show resolved
Hide resolved
packages/devextreme/testing/tests/DevExpress.ui.widgets.form/form.ai.utils.tests.js
Outdated
Show resolved
Hide resolved
packages/devextreme/js/__internal/ui/form/form.layout_manager.ts
Outdated
Show resolved
Hide resolved
| assert.deepEqual(smartPaste1.getCall(0).args, smartPaste1.getCall(0).args, 'new smartPaste invoked with same text and fields data'); | ||
| }); | ||
|
|
||
| QUnit.test('Default smartPaste button should be enabled if aiIntegration is set', function(assert) { |
There was a problem hiding this comment.
these test can be combined with forEach, using editorType, value, expected format variables
There was a problem hiding this comment.
the tests you referring to don't use editorType or value
they are about default smart button
at the end of file there are dxCheckBox, dxSwitch, dxHtmlEditor and dxDateRangeBox tests.
dxCheckBox and dxSwitch are similar between each other (and differ from others because the resulted value is not string|[string]), but they also different from each other since one of them have default value of undefined, and other false.
dxHtmlEditor has html wrapper to its value. so it is also different from the others.
mistakes are similar though, but to me it does not look right to group mistakes, and not editors
3 participants
No description provided.