Java Selenium: Custom Assert Message for Multiple Checkbox

Question

I'm working with an automated script for multiple checkboxes that has dependent permissions.

Given there are multiple checkboxes

• Checkbox A
• Checkbox B
• Checkbox C
• Checkbox D
• Checkbox E

When I select Checkbox A Then Checkbox C and Checkbox E is selected

I've handled this using a switch statement because there are multiple combinations but its kind of messy because certain checkbox(es) are not only selected but disabled as well due to the dependent permissions:

e.g.

 public boolean multipleCheckboxVerification(String roleName) {
 switch (roleName) {
 case CheckboxA:
 return permissionPage.isSelectedChkA() && permissionPage.isSelectedChkC()
 && permissionPage.isSelectedChkE() && !permissionPage.isEnabledChkC() && !permissionPage.isEnabledChk();}

May I kindly know if there is a cleaner way to return which checkbox isn't as expected?

I'm currently just using assertTrue from TestNG but the output seems to be vague of which checkbox is actually failing.

I was looking for an output where the error message would display "Checkbox E is enabled where it should be disabled"

Answer 1

You can Replace Conditional with Polymorphism.

class Checkboxes {
 // ...
 Boolean multipleCheckboxVerification() {
 switch (roleName) {
 case CheckboxA:
 return permissionPage.isSelectedChkA() && permissionPage.isSelectedChkC()
 && permissionPage.isSelectedChkE() && !permissionPage.isEnabledChkC() && !permissionPage.isEnabledChk();
 case CheckboxB:
 return !permissionPage.isSelectedChkA() && permissionPage.isSelectedChkC()
 && permissionPage.isSelectedChkE() && !permissionPage.isEnabledChkC() && !permissionPage.isEnabledChk();
 ....
 }
 }
}

Refactoring...

abstract class Checkbox {
 // ...
 abstract Boolean isValid(Page permissionPage);
 abstract String getErrorMessage();
}
class CheckboxA extends Checkbox {
 Boolean isValid(Page permissionpage) {
 return permissionPage.isSelectedChkA() && permissionPage.isSelectedChkC()
 && permissionPage.isSelectedChkE() && !permissionPage.isEnabledChkC() && !permissionPage.isEnabledChk();
 }
 String getErrorMessage() { return "Checkbox A is enabled where it should be disabled"; }
}
class CheckboxB extends Checkbox {
 Boolean isValid(Page permissionpage) {
 return !permissionPage.isSelectedChkA() && permissionPage.isSelectedChkC()
 && permissionPage.isSelectedChkE() && !permissionPage.isEnabledChkC() && !permissionPage.isEnabledChk();
 }
 String getErrorMessage() { return "Checkbox B is enabled where it should be disabled"; }
}

This way, you can keep your swtich statements only as new CheckboxA().isValid() or new CheckboxA().getErrorMessage(), e.g.

If you need to create a new checkbox, you just need to create a new class and add it to switch statement.

If you need to create further specific cases of CheckboxA, e.g., you can just create a new class and override the methods.

Answer 2

For meaningful message, You can use Assert.fail("Checkbox E is enabled where it should be disabled") instead of AssertTrue inside your function permissionPage.isSelectedChkE().