Modified in accordance with the comments

Author: Konstantin Bulavin

jdi-light-angular-tests/src/test/java/io/github/epam/angular/tests/elements/complex/ChipsTests.java

@@ -46,9 +46,9 @@ public void before() {
     public void basicChipsTest() {
         basicChips.show();
         basicChips.is().displayed().and().enabled();
-        basicChips.selectByText(TWOFISH);
+        basicChips.selectByLabel(TWOFISH);
         basicChips.getElement(TWOFISH).is().selected();
-        basicChips.selectByText(WARNFISH);
+        basicChips.selectByLabel(WARNFISH);
         basicChips.getElement(WARNFISH).is().selected();
 
         ChipsPage.basicSelectedValue.has().text(format("You clicked: %s", WARNFISH));
@@ -121,7 +121,7 @@ public void chipsRemovableTest() {
         inputChips.show();
         inputChips.is().displayed();
         inputChips.getElement(LEMON).is().removable();
-        inputChips.getElement(LEMON).close();
+        inputChips.getElement(LEMON).remove();
     }
 
     @Test(description = "Test checks that chips can't be removed")
@@ -159,8 +159,8 @@ public void chipsMultipleSelectionTest() {
         multipleChips.show();
         multipleChips.is().displayed();
         multipleChips.is().multiselectable();
-        multipleChips.selectByText("multiple color none");
-        multipleChips.selectByText("multiple color Primary");
+        multipleChips.selectByLabel("multiple color none");
+        multipleChips.selectByLabel("multiple color Primary");
         multipleChips.getElement("multiple color none").is().selected();
         multipleChips.getElement("multiple color Primary").is().selected();
     }
@@ -170,8 +170,8 @@ public void chipsNotMultipleSelectionTest() {
         basicChips.show();
         basicChips.is().displayed();
         basicChips.is().notMultiselectable();
-        basicChips.selectByText(TWOFISH);
-        basicChips.selectByText(WARNFISH);
+        basicChips.selectByLabel(TWOFISH);
+        basicChips.selectByLabel(WARNFISH);
         basicChips.getElement(TWOFISH).is().deselected();
         basicChips.getElement(WARNFISH).is().selected();
 

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/common/Chip.java

@@ -20,7 +20,7 @@ public boolean selected() {
     }
 
     @JDIAction("Close '{name}'")
-    public void close() {
+    public void remove() {
         find(MAT_CHIP_REMOVE).click();
     }
 
@@ -44,6 +44,20 @@ public boolean highlighted() {
         return hasClass("mat-mdc-chip-highlighted");
     }
 
+    @JDIAction()
+    public void select() {
+        if (!selected()) {
+            label().click();
+        }
+    }
+
+    @JDIAction()
+    public void unselect() {
+        if (selected()) {
+            label().click();
+        }
+    }
+
     @Override
     public Label label() {
         return new Label().setCore(Label.class, find(".mdc-evolution-chip__text-label"));

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/ChipGroup.java

@@ -2,6 +2,7 @@
 
 import com.epam.jdi.light.angular.asserts.ChipGroupAssert;
 import com.epam.jdi.light.angular.elements.common.Chip;
+import com.epam.jdi.light.angular.elements.interfaces.HasErrorState;
 import com.epam.jdi.light.angular.elements.interfaces.HasOrientation;
 import com.epam.jdi.light.angular.elements.interfaces.IsGroupElement;
 import com.epam.jdi.light.common.JDIAction;
@@ -11,7 +12,7 @@
 import java.util.List;
 import java.util.stream.Collectors;
 
-public class ChipGroup extends UIBaseElement<ChipGroupAssert> implements IsGroupElement<Chip>, HasOrientation {
+public class ChipGroup extends UIBaseElement<ChipGroupAssert> implements IsGroupElement<Chip>, HasOrientation, HasErrorState {
 
     public static final String CHIP_LOCATOR = ".mat-mdc-chip";
 
@@ -39,13 +40,6 @@ public boolean hasElement(String value) {
                 .anyMatch(chip -> chip.labelText().equals(value));
     }
 
-    @Override
-    @JDIAction("Get if all chips elements in '{name}' is displayed")
-    public boolean isDisplayed() {
-        return groupElements().stream()
-                .allMatch(ICoreElement::isDisplayed);
-    }
-
     @Override
     @JDIAction("Get if all chips elements in '{name}' is enabled")
     public boolean isEnabled() {
@@ -54,19 +48,33 @@ public boolean isEnabled() {
     }
 
     @JDIAction("Select chip element in '{name}' by '{0}'")
-    public void selectByText(String text) {
+    public void selectByLabel(String text) {
         Chip chip = getElement(text);
-        if (!chip.selected()) {
-            chip.click();
-        }
+        chip.select();
+    }
+
+    @JDIAction("Select chip element in '{name}' by '{0}'")
+    public void selectByLabel(List<String> values) {
+        values.forEach(value -> {
+            Chip chip = getElement(value);
+            chip.select();
+        });
+
     }
 
     @JDIAction("Deselect chip element in '{name}' by '{0}'")
-    public void deselectByText(String text) {
+    public void unselectByLabel(String text) {
         Chip chip = getElement(text);
-        if (chip.selected()) {
-            chip.click();
-        }
+        chip.unselect();
+    }
+
+    @JDIAction("Deselect chip element in '{name}' by '{0}'")
+    public void unselectByLabel(List<String> values) {
+        values.forEach(value -> {
+            Chip chip = getElement(value);
+            chip.unselect();
+        });
+
     }
 
     @Override
@@ -80,11 +88,6 @@ public String multiselectable() {
         return attr("aria-multiselectable");
     }
 
-    @JDIAction("Get if '{name}' has error state")
-    public boolean errorState() {
-        return core().hasClass("mat-form-field-invalid");
-    }
-
     @Override
     public ChipGroupAssert is() {
         return new ChipGroupAssert().set(this);