-
Notifications
You must be signed in to change notification settings - Fork 62
added password encrypt/decrypt stubs, recursive frame/iframe searches us... #122
base: master
Are you sure you want to change the base?
Conversation
… using xpath or CSS, contextMenu handling
A shame about all the whitespace changes - I added #147 to address that separately |
Does |
Where is |
It would be nice to have a example/test demonstrating the added features |
It is a recursive call, but I hardly noticed anything.
|
Sure. It is used within an OASIS REST call. There is also a decrypt as well.
|
@@ -456,6 +524,81 @@ private boolean executeDoCommand(final String methodName, final String[] values) | |||
} | |||
} | |||
|
|||
private String encryptString(String password) { //encrypt exposed passwords or information stub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the encryption/decryption methods should move to their own fixture, outside of Xebium.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it!
On Jun 28, 2015 5:50 PM, "Arnout Engelen" notifications@github.com wrote:
In src/main/java/com/xebia/incubator/xebium/SeleniumDriverFixture.java
#122 (comment):@@ -456,6 +524,81 @@ private boolean executeDoCommand(final String methodName, final String[] values)
}
}
- private String encryptString(String password) { //encrypt exposed passwords or information stub
I think the encryption/decryption methods should move to their own
fixture, outside of Xebium.—
Reply to this email directly or view it on GitHub
https://github.com/xebia/Xebium/pull/122/files#r33427425.
Added encrypt/decrypt stubs, recursive frame search capabilities (no more need for selectFrame) and contextMenu support.