Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I think the module should include Simple Tests to ensure that any patches submitted don't break module features.
Comment | File | Size | Author |
---|---|---|---|
#6 | session_limit-add_simpletests-2005782-7496616.patch | 19.62 KB | johnennew |
#4 | session_limit-add_simpletests-2005782-7469308.patch | 19.63 KB | johnennew |
#2 | session_limit-add_simpletests-2005782-7466620.patch | 19.34 KB | johnennew |
#1 | session_limit-add_simpletests-2005782-1.patch | 13.47 KB | johnennew |
Comments
Comment #1
johnennew CreditAttribution: johnennew commentedPlease find some initial tests attached for review. These test the default workings of the module. I'll look at adding role based tests if these are accepted.
Comment #2
johnennew CreditAttribution: johnennew commentedAnother testing patch with additional tests for role and user session limit.
Comment #3
deekayen CreditAttribution: deekayen commentedAdded to 7.x-2.x in 977e154.
Comment #4
johnennew CreditAttribution: johnennew commentedSimple tests back ported to 6.x-2.x branch
Comment #6
johnennew CreditAttribution: johnennew commentedRemoved devel code, trying again ...
Comment #7
johnennew CreditAttribution: johnennew commentedTests pass - committing to 6.x-2.x