Ignore component removal if the component has no parent

[denis-anisimov]

Fixes #4528

---

This change is 

[caalador]

Reviewable status: 1 unresolved discussion, 0 of 1 LGTMs obtained

---

flow-server/src/main/java/com/vaadin/flow/component/HasComponents.java, line 68 at r1 (raw file):

                    "Component to remove cannot be null");
            Element parent = component.getElement().getParent();
            if (parent == null) {

This now fails the UITest.removeNotChildComponent test as it expects an IllegalArgumentException instead of a silent logging of missing parent.

The test should be removed as it is tested in the remove_removeComponentWithNoParent that is close to identical.

[pekam]

Reviewable status: 1 unresolved discussion, 0 of 1 LGTMs obtained

---

flow-server/src/main/java/com/vaadin/flow/component/HasComponents.java, line 68 at r1 (raw file):

Previously, caalador wrote…

This now fails the UITest.removeNotChildComponent test as it expects an IllegalArgumentException instead of a silent logging of missing parent.

The test should be removed as it is tested in the remove_removeComponentWithNoParent that is close to identical.

Done.

[caalador]

Reviewed 2 of 2 files at r1, 1 of 1 files at r2.
Reviewable status: all discussions resolved, 0 of 1 LGTMs obtained, and 1 stale