Skip to content

need live tests for queries to 'information schema.tables' table #541

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

Open
1 of 6 tasks
klirichek opened this issue May 22, 2025 · 0 comments
Open
1 of 6 tasks

need live tests for queries to 'information schema.tables' table #541

klirichek opened this issue May 22, 2025 · 0 comments
Assignees

Comments

@klirichek
Copy link

Proposal:

Files in src/Plugin/Select implement responses to different tools speaking with mysql proto.
Since some funcs going to move to daemon's code, query tests are necessary.
That is all real kinds of 'select ... from information_schema.tables' - with correct answers, which budy generates.

It should be not just general queries, but concrete specific queries dumped from the tools in real sessions.

There are some comments in the code - like 'HeidiSQL tool doesn't recognize lowercase data types so we add this extra transformation', or 'Adding the character set columns with fake data since they're mandatory for HeidiSQL'. They explain some of details, but that is much better to have whole picture. Ideally - just written sessions from different apps with buddy's answers, what is happened 'underneath'.

For now only queries to INFORMATION_SCHEMA.TABLES is necessary

As an example you can see test/test_272/test.xml. It contains real sessions from mysqldump with different params. Here is request to make same one with the tools tested with 'information_schema.tables mentioned HeidiSQL, and also all the other which was used to create the plugin.

Checklist:

To be completed by the assignee. Check off tasks that have been completed or are not applicable.

  • Implementation completed
  • Tests developed
  • Documentation updated
  • Documentation reviewed
  • Changelog updated
  • OpenAPI YAML updated and issue created to rebuild clients
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

No branches or pull requests

2 participants