turn off tests rather than merge fixes at this stage as the PR would be too big

for comfortable review. If PR is merged before this issue goes cold for me I'll submit the fixes to
master for these

diff --git a/tests/phpunit/api/v3/SyntaxConformanceTest.php b/tests/phpunit/api/v3/SyntaxConformanceTest.php
index 5cb03abd136e81f3ca404f724e9147ba7f6ba23a..1992bac3b3fd04df6305b6d0744845c1d06143d7 100644 (file)
--- a/tests/phpunit/api/v3/SyntaxConformanceTest.php
+++ b/tests/phpunit/api/v3/SyntaxConformanceTest.php
@@ -397,8 +397,12 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
   public static function toBeSkipped_getSqlOperators() {
     $entitiesWithout = array(
       'Case',//case api has non-std mandatory fields one of (case_id, contact_id, activity_id, contact_id)
-      'Contact', // on the todo list!
       'EntityTag', // non-standard api - has inappropriate mandatory fields & doesn't implement limit
+      'Contact', // as with the next 3 a fix is tested & working for this but trying to keep code for review small hence just recording
+      //https://github.com/eileenmcnaughton/civicrm-core/commit/da3d6dcfd73a7e6e5f5a553e98b4608724d33765
+      'Event',// fix is https://github.com/eileenmcnaughton/civicrm-core/commit/cfd3114bd3215fed26c6e933f621b8e893cd7b3e
+      'Participant',//fix is https://github.com/eileenmcnaughton/civicrm-core/commit/6ebae38eb8999756f62259818af8827f50e8d4f3
+      'Pledge',// same fix as participant - keeping out of 4.5 as a conservative move
       'Extension', // can't handle creating 25
       'Note', // note has a default get that isn't implemented in createTestObject -meaning you don't 'get' them
       'MailingGroup', // no get call on MailingGroup