Record after playback resets URL

SeleniumHQ / selenium-ide

Open Source record and playback test automation for the web.

https://selenium.dev/selenium-ide/

Apache License 2.0

2.73k stars 739 forks source link

Record after playback resets URL #1753

Closed coinzdude closed 5 months ago

coinzdude commented 5 months ago

Alpha 78 build

Record a script, starting with some home page

Then record and navigate to another location in the script

Then play back the script, ending at the expected last URL/location

Then click record. URL is reset to initial URL

Expected behavior is that the current URL would remain and not change to initial URL (which is the prior behavior)

coinzdude commented 5 months ago

log

11:15:03 AM [info] [ 'Instantiating driver builder for ', 'electron' ] 11:15:03 AM [info] [ 'Building driver for electron' ] 11:15:04 AM [info] [ 'Built driver for electron' ] 11:15:05 AM [debug] [ 'Playing state changed prep for test New Test', '' ] 11:15:05 AM [debug] [ 'Playing state changed playing for test New Test', '' ] 11:15:05 AM [debug] [ 'executing open|/' ] 11:15:06 AM [debug] [ 'passed open|/' ] 11:15:06 AM [debug] [ 'executing click|linkText=About' ] 11:15:07 AM [debug] [ 'passed click|linkText=About' ] 11:15:07 AM [debug] [ 'Playing state changed finished for test New Test', '' ] 11:15:11 AM [info] [ 'Instantiating driver builder for ', 'electron' ] 11:15:11 AM [info] [ 'Building driver for electron' ] 11:15:11 AM [info] [ 'Built driver for electron' ]

toddtarsi commented 5 months ago

@coinzdude - v .79 will be live shortly, and should have this resolved:

After playing through, the test remains on the final command instead of resetting back to the first.
The command pane will do a best effort scrolling to keep the active command in the viewport. This is just because it seems unpleasant to have the recorder commands scroll out of view mid recording.

coinzdude commented 5 months ago

@toddtarsi Just did try https://github.com/SeleniumHQ/selenium-ide/releases/tag/4.0.1-alpha.79-windows but it has the same issue

toddtarsi commented 5 months ago

@coinzdude - I will try reproducing your exact test. I used a suite in side-testkit as my test, and didn't cover this. Thanks for testing!

toddtarsi commented 5 months ago

@coinzdude - Will be resolved in next version, should be out later tonight

toddtarsi commented 5 months ago

@coinzdude - Resolved in x.80

coinzdude commented 5 months ago

Working, thanks!

github-actions[bot] commented 4 months ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.