Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: NotAttribute, Text\ElementNotPresent #32

Closed
cwpm opened this issue Feb 22, 2018 · 5 comments
Closed

Feature: NotAttribute, Text\ElementNotPresent #32

cwpm opened this issue Feb 22, 2018 · 5 comments

Comments

@cwpm
Copy link

cwpm commented Feb 22, 2018

New Feature. In the past there were requirements to add a detection to check that a value \ script \ attribute \ text isn't present on websites.

We usually have a TextNotPresent to rule out bad characters, and NotAttribute to detect NoIndex,NoFollow isn't set on websites. I presume it's possible to do an ELSE or IF to accommodate this situation but it leads to writing further commands
A straight forward NotAttribute, TextNotPresent or ElementNotPresent would be quicker and less entries to include. This was present in the old Selenium IDE FF version.

Could this be considered?

@cwpm
Copy link
Author

cwpm commented Feb 26, 2018

Thank you for the feedback via e-mail

"NOT" commands: As you guessed, inverting a command with "if" is quite easy - and more flexible then the old "not" versions of the commands:

verifyChecked | id=xxx
if | {!lastCommandOk}
ECHO | “element is NOT CHECKED” | red
endif

Need to work on IF and ENDIF for each NotAttribute. I'll look to work that in.

@A9T9
Copy link
Owner

A9T9 commented Feb 26, 2018

In addition, the last upgrade brings the ThrowError command. This allows you stop the macro, just like an AssertNot... would do.

verifyChecked | id=xxx
if | {!lastCommandOk}
ThrowError “element is NOT CHECKED”
endif

@A9T9
Copy link
Owner

A9T9 commented Feb 26, 2018

We might still add the whole family of ...NOT... commands, but are unsure. Feedback if they are really needed, or if the if/endif solution works, is welcome.

@cwpm
Copy link
Author

cwpm commented Mar 6, 2018

I'll vote 👍

@A9T9
Copy link
Owner

A9T9 commented Feb 8, 2019

Closing this, as we use the forum now to track feature request.

@A9T9 A9T9 closed this as completed Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants