NO-ISSUE: Fix security issues on images (#2829) #37
Annotations
3 errors, 11 warnings, and 5 notices
FULL → Build:
packages/dmn-editor/tests-e2e/deleteConnectedNodes.spec.ts#L110
1) [webkit] › deleteConnectedNodes.spec.ts:146:11 › Delete node - Connected nodes › Connected with multiple nodes › Two "from"s related to one "to" › should delete "to"s and not affect "from"
Test timeout of 60000ms exceeded while running "beforeEach" hook.
108 |
109 | test.describe('Two "from"s related to one "to"', () => {
> 110 | test.beforeEach(async ({ diagram, palette, nodes }) => {
| ^
111 | await palette.dragNewNode({
112 | type: NodeType.INPUT_DATA,
113 | targetPosition: { x: 100, y: 100 },
at /home/runner/work/incubator-kie-tools/incubator-kie-tools/packages/dmn-editor/tests-e2e/deleteConnectedNodes.spec.ts:110:12
|
FULL → Build:
packages/dmn-editor/tests-e2e/__fixtures__/nodes.ts#L231
1) [webkit] › deleteConnectedNodes.spec.ts:146:11 › Delete node - Connected nodes › Connected with multiple nodes › Two "from"s related to one "to" › should delete "to"s and not affect "from"
Error: page.waitForFunction: Test timeout of 60000ms exceeded.
at __fixtures__/nodes.ts:231
229 | // This methods waits until it gets the focus.
230 | public async waitForNodeToBeFocused(args: { name: string }) {
> 231 | return this.page.waitForFunction(
| ^
232 | (nodeName) => (document.activeElement as HTMLInputElement)?.value === nodeName,
233 | args.name
234 | );
at Nodes.waitForNodeToBeFocused (/home/runner/work/incubator-kie-tools/incubator-kie-tools/packages/dmn-editor/tests-e2e/__fixtures__/nodes.ts:231:22)
at Palette.dragNewNode (/home/runner/work/incubator-kie-tools/incubator-kie-tools/packages/dmn-editor/tests-e2e/__fixtures__/palette.ts:37:22)
at /home/runner/work/incubator-kie-tools/incubator-kie-tools/packages/dmn-editor/tests-e2e/deleteConnectedNodes.spec.ts:111:9
|
FULL → Build:
packages/dmn-editor/tests-e2e/drgElements/modelDecisionService.spec.ts#L235
2) [webkit] › drgElements/modelDecisionService.spec.ts:222:11 › Model Decision Service › Model Decision Service - Signature › Model Decision Service - Signature - Inputs Order › Decision Service Inputs Signature should be reordered
Error: expect(received).toEqual(expected) // deep equality
Expected: "New Decision Service(A, B, AA, BB)"
Received: "New Decision Service(A, B, BB, AA)"
233 | await decisionServicePropertiesPanel.moveInputData({ fromIndex: 0, toIndex: 1 });
234 | await decisionServicePropertiesPanel.moveInputDecision({ fromIndex: 0, toIndex: 1 });
> 235 | expect(await decisionServicePropertiesPanel.getInvokingThisDecisionServiceInFeel()).toEqual(
| ^
236 | "New Decision Service(A, B, AA, BB)"
237 | );
238 | expect(await decisionServicePropertiesPanel.getInputDecisions()).toEqual(["A", "B"]);
at /home/runner/work/incubator-kie-tools/incubator-kie-tools/packages/dmn-editor/tests-e2e/drgElements/modelDecisionService.spec.ts:235:93
|
Setup environment
Restore cache failed: Dependencies file is not found in /home/runner/work/incubator-kie-tools/incubator-kie-tools. Supported file pattern: go.sum
|
FULL → Build:
packages/envelope/src/embedded/EmbeddedEnvelopeFactory.tsx#L96
React Hook useMemo has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useMemo call and refer to those specific props inside useMemo
|
FULL → Build:
packages/envelope/src/embedded/EmbeddedEnvelopeFactory.tsx#L101
React Hook useImperativeHandle has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useImperativeHandle call and refer to those specific props inside useImperativeHandle
|
FULL → Build:
packages/editor/src/embedded/embedded/EmbeddedEditor.tsx#L87
React Hook useMemo has an unnecessary dependency: 'props.file.getFileContents'. Either exclude it or remove the dependency array
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L66
React Hook useMemo has missing dependencies: 'envelopeContext.operatingSystem' and 'envelopeContext.services.keyboardShortcuts'. Either include them or remove the dependency array
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L66
React Hook useMemo has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L76
React Hook useEffect has a missing dependency: 'envelopeContext.services.keyboardShortcuts'. Either include it or remove the dependency array
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L85
React Hook useEffect has a missing dependency: 'envelopeContext.services.keyboardShortcuts'. Either include it or remove the dependency array
|
FULL → Build:
packages/import-java-classes-component/src/components/ImportJavaClasses/ImportJavaClassesWizardSecondStep.tsx#L53
React Hook useCallback has a missing dependency: 'accessorPrefixesList'. Either include it or remove the dependency array
|
FULL → Build:
packages/import-java-classes-component/src/components/ImportJavaClasses/ImportJavaClassesWizardSecondStep.tsx#L69
React Hook useCallback has a missing dependency: 'accessorPrefixesList'. Either include it or remove the dependency array
|
FULL → Build:
packages/import-java-classes-component/src/components/ImportJavaClasses/ImportJavaClassesWizardSecondStep.tsx#L82
React Hook useCallback has a missing dependency: 'denyList'. Either include it or remove the dependency array
|
FULL → Build
77 skipped
805 passed (26.8m)
|
FULL → Build
2 flaky
[webkit] › deleteConnectedNodes.spec.ts:146:11 › Delete node - Connected nodes › Connected with multiple nodes › Two "from"s related to one "to" › should delete "to"s and not affect "from"
[webkit] › drgElements/modelDecisionService.spec.ts:222:11 › Model Decision Service › Model Decision Service - Signature › Model Decision Service - Signature - Inputs Order › Decision Service Inputs Signature should be reordered
99 skipped
1357 passed (1.3h)
|
FULL → Build
46 skipped
131 passed (7.4m)
|
FULL → Build
39 passed (5.8m)
|
FULL → Build
72 skipped
24 passed (3.7m)
|
Loading