• Type: Task
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 4.0M10
• Affects Version/s: 4.0M8
• Component/s: None
• Labels:

  None

1. Ametys - 01 Runtime - copie.patch

   73 kB

   01/Sep/16 14:42

2. Ametys - 03 Plugin Repository.patch

   1 kB

   01/Sep/16 14:42

3. Ametys - 04 Workflow.patch

   17 kB

   01/Sep/16 14:42

4. Ametys - 07 CMS.patch

   3 kB

   01/Sep/16 14:42

[RUNTIME-2062] Tests must always drop all tables before running the application

Collapse comment: Simon Prieul (Inactive) added a comment - 30/Aug/16 15:13

Simon Prieul (Inactive) added a comment - 30/Aug/16 15:13

But it has to not be.

For every test, even those not for testing a JDBC feature, we have to ensure all tables needed are dropped. Then, when starting applciation, the SqlTablesInit will create them automatically. this prevents from having existing tables, but with incorrect columns (for instance, tables created from another branch test...)
So, for every test, setUp will do a #_dropTables(), then #_startApplication(), and then #_populateTables() if needed (only for group, user and right tests normally)

Expand comment: Simon Prieul (Inactive) added a comment - 30/Aug/16 15:13

Simon Prieul (Inactive) added a comment - 30/Aug/16 15:13 But it has to not be. For every test, even those not for testing a JDBC feature, we have to ensure all tables needed are dropped. Then, when starting applciation, the SqlTablesInit will create them automatically. this prevents from having existing tables, but with incorrect columns (for instance, tables created from another branch test...) So, for every test, setUp will do a #_dropTables(), then #_startApplication(), and then #_populateTables() if needed (only for group, user and right tests normally)

Collapse comment: Simon Prieul (Inactive) added a comment - 30/Aug/16 15:08

Simon Prieul (Inactive) added a comment - 30/Aug/16 15:08

Currently, the feature creating the READER profile was excluded from all tests

Expand comment: Simon Prieul (Inactive) added a comment - 30/Aug/16 15:08

Simon Prieul (Inactive) added a comment - 30/Aug/16 15:08 Currently, the feature creating the READER profile was excluded from all tests