- wmf_civicrm_civicrm_merge
- wmf_civicrm_civicrm_post
- wmf_civicrm_civicrm_contactListQuery
- wmf_civicrm_civicrm_buildForm
- wmf_civicrm_civicrm_validate
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Declined | None | T90630 Recurring Payments Reporting | |||
Resolved | • mepps | T97372 Enable ability to look up contacts by phone number | |||
Resolved | None | T77910 [epic] Upgrade Civi to 4.6 & integrate new reporting | |||
Resolved | awight | T99844 Install Civi 4.6 to production | |||
Resolved | Eileenmcnaughton | T99842 [epic] Install 4.6 to staging and test like hell | |||
Resolved | Eileenmcnaughton | T115426 All Civi hook code paths must be covered by PHPUnit tests |
Event Timeline
Change 246450 had a related patch set uploaded (by Eileen):
Remove custom quicksearch hook
Change 249028 had a related patch set uploaded (by Eileen):
Add test for contact.merge action
Change 249034 had a related patch set uploaded (by Eileen):
add test for contact.merge action
Hmm - I have found I need to set some permissions (merge contacts & delete contacts) on the test user. Not sure if there is a precedent for that in the suite
We are going to look at merge separately & the contactListQuery has been removed but the others are worth doing.
Change 252481 had a related patch set uploaded (by Eileen):
add test for contact.merge action
I'm setting this to resolved cos I think that's what we agreed to do with it. Big picture adding tests = good
Change 249028 abandoned by Eileen:
Add test for contact.merge action
Reason:
I'm going to ditch this - it fixes some enotices that block tests - but until we are actively doing those tests it feels like noise
Change 285774 had a related patch set uploaded (by Eileen):
add test for contact.merge action