Test script testUserMgmt.sql is incomplete and runs unnecessary tests (ME)

DASSL / ClassDB

An open-source system to let students experiment with relational data

https://dassl.github.io/ClassDB/

Other

7 stars 2 forks source link

Closed smurthys closed 6 years ago

smurthys commented 6 years ago

The script testUserMgmt.sql has two problems:

Does not test the key functionality in addUserMgmtCore.sql: rejecting inserts associated with non-ClassDB users, and rejecting all updates
Unnecessarily tests row counts in activity tables, including initially testing if the activity tables are empty. Neither of the row-count tests are necessary.