Opened 8 years ago

Closed 8 years ago

#7736 closed enhancement (implemented)

additional test cases for module

Reported by: robinson Owned by: atagar
Priority: Medium Milestone:
Component: Archived/Stem Version:
Severity: Keywords: testing
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:



Please pull my exp-event-module-tests branch[0], which contains test cases covering most of the events module. This started as tests for the Event class, but I was on a roll. I can make changes or consolidate commits, if you want.

This patch[1] exercises the Event class where its sub-classes do not touch. It highlights the quoted positional arguments discussion[2] we had earlier in the month.

Only AuthDirNewDescEvent has lines which are not tested.. It appears from internal comments that it is not fully working due to an insufficient spec?


Child Tickets

Change History (1)

comment:1 Changed 8 years ago by atagar

Resolution: implemented
Status: newclosed

This is fantastic, thanks Sean! Pushed with a few minor revisions...

Note: See TracTickets for help on using tickets.