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

Problem when starting a test without a Selenium driver set #16

Open
simone36050 opened this issue Oct 10, 2024 · 1 comment · Fixed by #17
Open

Problem when starting a test without a Selenium driver set #16

simone36050 opened this issue Oct 10, 2024 · 1 comment · Fixed by #17
Assignees
Labels
bug Something isn't working

Comments

@simone36050
Copy link
Collaborator

If a user runs a test without setting a Selenium driver, the test will fail because the driver cannot be used. The user will then probably select the correct driver and re-run the test. The result is that the test may not run correctly. In particular, in multi-session tests where the instructions of the first session are supposed to sleep until the second session reaches a specific instruction, the sleep function is not performed correctly.

Steps to reproduce:

  • run Burp with MIG-T;
  • choose the wrong driver file;
  • import test and sessions (for my tests, I used OP-Token response-refresh_token-already-used.json);
  • run the test;
  • select the correct driver;
  • re-run the test;
  • wait for the end.

Expected result: pass
Actual result: not applied

@mattebit
Copy link
Collaborator

I added the PR above, it basically add a more complex verification to the selection of the executable before allowing the user to use it. This partially solves the initial problem of using a wrong executable during test executions. I was not yet able to identify where the second issue originates, I will do it.

@mattebit mattebit added the bug Something isn't working label Oct 10, 2024
@mattebit mattebit self-assigned this Oct 10, 2024
@mattebit mattebit reopened this Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants