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

Query generation bug on QBE feature caught by our Selenium test suite #16657

Closed
williamdes opened this issue Feb 17, 2021 · 1 comment
Closed
Assignees
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
Milestone

Comments

@williamdes
Copy link
Member

williamdes commented Feb 17, 2021

Describe the bug

The query is wrong and the highlight is broken

To Reproduce

https://demo.phpmyadmin.net/QA_5_0/db_qbe.php?db=sakila&server=1

MariaDB

MariaDB

MySQL

MySQL

Expected behavior

Like in MariaDB

Server configuration

  • phpMyAdmin version: 5.0, 5.1, master

Dev

It seems like on MySQL the checkbox does not auto-tick "show" the column

#16479 (comment)

There was 1 failure:

1) PhpMyAdmin\Tests\Selenium\Database\QueryByExampleTest::testQueryByExample
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'SELECT `test_table`.`id` AS `ID`, `test_table`.`val` AS `VAL`\n
-FROM `test_table`\n
+'FROM `test_table`\n
 WHERE ((`test_table`.`id` > 1) AND (`test_table`.`val` < 6))\n
 ORDER BY `test_table`.`val` ASC, `test_table`.`id` DESC'

/home/runner/work/phpmyadmin/phpmyadmin/test/selenium/Database/QueryByExampleTest.php:121
@williamdes williamdes added Bug A problem or regression with an existing feature ui Issues relating to the user interface labels Feb 17, 2021
@williamdes
Copy link
Member Author

This is because of this line:

$this->addScriptFiles(['database/qbe.js']);

if the storage is not installed, the script is not loaded

@williamdes williamdes added this to the 5.1.0 milestone Feb 17, 2021
@williamdes williamdes added this to Needs triage in issues via automation Feb 17, 2021
@williamdes williamdes moved this from Needs triage to Reproduced in issues Feb 17, 2021
@williamdes williamdes self-assigned this Feb 17, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Feb 17, 2021
issues automation moved this from Reproduced to Closed Feb 17, 2021
williamdes added a commit that referenced this issue Feb 17, 2021
Signed-off-by: William Desportes <williamdes@wdes.fr>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
issues
  
Closed
Development

No branches or pull requests

1 participant