Conversation
This helps in testing the analytics of apinfw
Have updated the installation guide, based on the review inputs.
…on review comments
APInf Test Environment Setup Guide along with Test Scripts
Common.js
…to testfix/viewapibackend-tests # Conflicts: # test/viewapibackend.js
Hotfix/adding api
Testfix/viewapibackend tests
Hotfix/registration by daria
…nto feature/move-tests
test/common.js
Outdated
| }, | ||
| signIn: function(driver) { | ||
| driver.get(this.url); | ||
| driver.findElement(By.linkText('login')).click(); |
There was a problem hiding this comment.
@jykae We will have problem in this place because now navbar has 'Sign in' button for login.
There was a problem hiding this comment.
@marla-singer well noticed, and that's what tests are good for, they fail if something is different. :)
|
Ran the whole test set: |
|
@jykae 8.1 test doesn't work as expected because we have an e-mail verification on nightly. Maybe it is also relating to 1.1 test |
|
@marla-singer I ran this one against localhost as noticed email verifications are ON nightly. Changing url of instance we are testing against should be simpler, as well as the login user that is expected to be "robot@test.com". We might need to come up with decision what is basically the instance we want to run these tests. If it's nightly we need to turn off email verifications OFF for registration tests. I can't think of possibility how we could test that automatically. Basically here the task was to move tests & make it possible to run them using "npm test". |
|
@marla-singer assign yourself if you would like to review this one. I propose that fixing tests would be in separate PRs, if something is needed. |
|
Github login above failed possibly because I had settings for Github wrong in localhost.. |
|
@jykae Merge it. |
|
🔌 🍏 ⛵ |
6 participants
Closes #1802
Proposed changes