Branch: refs/heads/master Home: https://github.com/jenkinsci/acceptance-test-harness
Commit: 346961bb68da3fb058a01176a6788146985d90e2 https://github.com/jenkinsci/acceptance-test-harness/commit/346961bb68da3fb058a01176a6788146985d90e2 Author: James Nord <jnord+git...@cloudbees.com> Date: 2019-08-17 (Sat, 17 Aug 2019) Changed paths: M src/main/java/org/jenkinsci/test/acceptance/po/CapybaraPortingLayerImpl.java M src/main/java/org/jenkinsci/test/acceptance/po/Control.java M src/main/java/org/jenkinsci/test/acceptance/po/Login.java Log Message: ----------- login does not wait for the user to be logged in If you have a large delay on your Authentication then login will return but the login process will not have completed. The upshot is you must never use WebElement.click() for anything that will be gauranteed to change the current page (URL) https://blog.codeship.com/get-selenium-to-wait-for-page-load/ Whilst this is a form I tried changing click() to submit() but this caused a worse issue and did not even work. So for now add sime methods that will wait until the page has changed (ie the current element will be stale. This will occur even if the same page is returned) Commit: 42830de4436c2fd24c299ffff78e9c136005ec41 https://github.com/jenkinsci/acceptance-test-harness/commit/42830de4436c2fd24c299ffff78e9c136005ec41 Author: James Nord <jnord+git...@cloudbees.com> Date: 2019-09-03 (Tue, 03 Sep 2019) Changed paths: M src/main/java/org/jenkinsci/test/acceptance/po/Control.java Log Message: ----------- I forgot to add the change to use click() after trying submit() Commit: 40a41c7ec44979cb117003eacfd7db8d86b95e26 https://github.com/jenkinsci/acceptance-test-harness/commit/40a41c7ec44979cb117003eacfd7db8d86b95e26 Author: Oliver Gondža <ogon...@gmail.com> Date: 2019-09-11 (Wed, 11 Sep 2019) Changed paths: M src/main/java/org/jenkinsci/test/acceptance/po/CapybaraPortingLayerImpl.java M src/main/java/org/jenkinsci/test/acceptance/po/Control.java M src/main/java/org/jenkinsci/test/acceptance/po/Login.java Log Message: ----------- Merge pull request #521 from jtnord/though-shalt-not-click login does not wait for the user to be logged in Commit: 5606d3bb3a645311711b74833d5d3a4fdc4d713d https://github.com/jenkinsci/acceptance-test-harness/commit/5606d3bb3a645311711b74833d5d3a4fdc4d713d Author: Oliver Gondža <ogon...@gmail.com> Date: 2019-09-11 (Wed, 11 Sep 2019) Changed paths: M src/main/java/org/jenkinsci/test/acceptance/po/Control.java Log Message: ----------- Do not expose the experimental API Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/a9be9e3b9efd...5606d3bb3a64 -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/acceptance-test-harness/push/refs/heads/master/a9be9e-5606d3%40github.com.