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

Add Dusk tests for Queries tab #1687

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Conversation

mccahan
Copy link
Contributor

@mccahan mccahan commented Oct 9, 2024

NOTE: This requires the regression test fix from PR #1686 before it will pass

Adds testing that would cover the thing I broke:

  • Confirms the Queries tab is counting
  • Confirms that a test query appears
  • Confirms clicking the query expands it and shows bindings and trace

The two screenshots it grabs:

queries-expanded
queries-tab

@barryvdh
Copy link
Owner

Can you merge master? It seems I don't have permissions.

@mccahan
Copy link
Contributor Author

mccahan commented Oct 10, 2024

Are you able to re-run the failed checks? They should fire now with that other PR merged. If you can't, I should just be able to force push this one

@barryvdh
Copy link
Owner

Yeah I tried to run them but didn’t work. Thanks!

@barryvdh barryvdh merged commit b705c56 into barryvdh:master Oct 10, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants