search

crate / crate-qa

CrateDB Quality Assurance
4 stars 1 forks source link

Ensure user exists before rolling upgrade #302

Closed mfussenegger closed 7 months ago

mfussenegger commented 7 months ago

To catch regressions like https://github.com/crate/crate/pull/15425 in the future

matriv commented 7 months ago

Thanks a lot! Shouldn't we also assert the user and privs like we did in the upgrade test:

cursor.execute("select name from sys.users order by 1")
self.assertEqual(cursor.fetchall(), [["crate"], ["trillian"]])

cursor.execute("select * from sys.privileges")
self.assertEqual(cursor.fetchall(), [["TABLE", "trillian", "crate", "doc.t1", "GRANT", "DQL"]])
mfussenegger commented 7 months ago

Good idea. Added a fixup