MySQL tests are green so this is mean codebase is stabilized for MySQL based sites.
The goal of this issue is to make tests for PostgreSQL and MySQL to be green.

CommentFileSizeAuthor
#5 fix_postgresql_db_driver-2981682-5.patch672 bytesthedut

Comments

voleger created an issue. See original summary.

voleger’s picture

Title: Stabilize module for usage with PostgreSQL DB driver » [META] Stabilize module for usage with PostgreSQL DB driver
Category: Task » Plan
Issue summary: View changes
voleger’s picture

thedut’s picture

This issue may be connected with a Drupal Core issue, not fixed yet, concerning postgre and the email colomn : #2490294.

thedut’s picture

StatusFileSize
new672 bytes

I have not installed postgre on my dev platform.
Just testing a patch here.

thedut’s picture

I hide #5 since it doesn't work.
Seems there is only 1 functionnal test on 10 that fails, the one that compare all kind of queries with and without the dbee module.
And inside this test, 32 kind of queries are tested, and it seems that only one fails.
It may help looking at the html verbose on running DbeeQueryUsersTest, witch is not available for me, untill I install a dev platform with postgre.
it seems that the bug come from the simpletest, and not the main module itself. Maybe assertEquals may have diffculties to compare multidimentionnal array ?

thedut’s picture

#1870786 issue may help regarding assertEquals.

  • thedut committed e4e8a86 on 8.x-2.x
    Issue #2981682 by thedut: [META] Stabilize module for usage with...
thedut’s picture

Assigned: Unassigned » thedut
Priority: Major » Normal
Status: Needs work » Fixed

Hello,
I have just fixed this issue.
Note that D8 with postgreSQL is buggy concerning email sensitive case features !
As an example, on a D8 fresh install with Postgre:

  1. Add user with the email abc@abc.com
  2. Create another user with the same exact email abc@abc.com : The system reject it : great.
  3. but create another user with the email aBc@abc.com (sensitive case) : the system accepts it but it should not ! And with Mysql, the system reject it as expected !

So I downgrade this issue priority to Normal.

Thanks to the dbee module, in such case, dbee module will dectect the problem, even on Postgre.
So I still have an error on my simpletests on postgre (the one than compare queries result with or without the dbee module) but I keep it that way, since the dbee module does the right thing and not D8 yet.

voleger’s picture

Thanks for clarifying and updates.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.