Civi/Test/TAP - Replace Symfony YAML dependency

This allows two paths:

1. If you have the YAML PECL extension, then that will be used.
   To get the best output, you can install it.
2. If you don't, then it'll output JSON. JSON is considered valid YAML.
   It's not clear that everyone reading the document will handle
   the full range YAML (incl JSON), but... we don't actually have any
   listeners that care to parse the TAP output...

tools/scripts/phpunit - Flip back to preferring 'php' for subprocess

Rationale:

* On the test servers, `php` is wrapper script.
* Outside the test servers, you should be able to use `phpunit` directly
  (rather than calling `tools/scripts/phpunit`), and then it's
  not our problem.

Merge pull request #14511 from eileenmcnaughton/jungle

Add Unit test for testing smart group flow

Add Unit test for testing smart group flow

Merge pull request #14527 from totten/5.15-yaml-dg

(REF) Convert YAML fixtures to JSON. Reset symfony/yaml dependency.

(REF) Convert YAML fixtures to JSON fixtures. Reset symfony/yaml dependency.

Overview
--------

The test suite includes some tests that read fixture data files (i.e. YAML files). This patch
provides a safer way to address a dependency-management issue with parsing YAML files.

Before
------

* In the last public release (`5.14`), the `symfony/yaml` package was not included with `civicrm-core`.
  However, it was loaded during testing by virtue of running `phpunit6`.
* In the `5.15` RC code, `symfony/yaml` was added -- so that the test suite worked in `phpunit6`.
  However, this also added `symfony/yaml` to the general runtime distribution - which is liable to create conflicts
  with Civi extensions and CMS modules that also bundle `symfony/yaml`. (It's a popular package.)

After
-----

* The test suite no longer uses YAML, and the build no longer requires `symfony/yaml`. Instead, it uses JSON.

Merge pull request #14516 from eileenmcnaughton/except

[test] Add PrematureExitException to support unit testing where the code would otherwise exit

Add PrematureExitException to support unit testing where the code would otherwise exit

Merge pull request #14468 from seamuslee001/e2e_asset_builder_test_wordpress

Add CRM_Utils_System::sendResponse(). Fix AssetBuilder's status-code on WP.

Incorporate Tim's Cleanup

Merge pull request #14482 from seamuslee001/update_exception_dao

[NFC] Update Exception DAO class

Update Exception DAO class

Slimplify functions down as per Tim's review comments

Update to use PSR HTTP Message Response Interface as per @totten

Move content of returning response into CRM_Utils_System and have user_system based method to set HTTP Status code

Set version to 5.15.beta1

Merge pull request #14472 from eileenmcnaughton/from_to_test

[nfc] [test] Skip testGetFromTo on date transition

Merge pull request #14470 from eileenmcnaughton/alike

[NFC][test-fix] centralise use of assertLike for comparing sql

Merge pull request #14448 from eileenmcnaughton/exception2

Fix Exception api to save lower id number as contact 1

Merge pull request #14473 from eileenmcnaughton/custom_filter

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fix Exception api to save lower id number as contact 1

This is already actioned (& tested) via ajax fn markNonDuplicates - this moves the functionality from
the ajax layer to the BAO layer & stdises it under unit tests

Merge pull request #14447 from eileenmcnaughton/exception

Fix Exception to remove result from duplicates

Merge pull request #14475 from eileenmcnaughton/contact_def

Permit sort_name as a url parameter on advanced search

Convert sort_name to use search form methodology to pass by url

Merge pull request #14476 from eileenmcnaughton/basic_search

[REF] Move retrieval of basicSearchFields to 'get' fn

Move retrieval of basicSearchFields to 'get' fn

This is a partial of https://github.com/civicrm/civicrm-core/pull/14418

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fixes a bug where contact.get does not retrieve results if a custom field is requested and an activity custom field exists

This bug came about because a recent fix made metadata more consistent but it turns out this class was filtering the
metadata by the passed in  - and then iterating through the metadata. Now it iterates the ids instead.

Merge pull request #14469 from eileenmcnaughton/pledge_msg

Fix version for pledge message update, also remove 'please print'

Skip testGetFromTo on date transition

This avoids fails when the expectations were calculated right at the end of a day but then are compared with the new day's values.

Results in this case look like

CRM_Report_FormTest::testGetFromTo with data set #1 ('20190606000000', '20190606235959', 'this.day', '', '')
fail on data set [ this.day ,  ,  ]. Local php time is 2019-06-07 00:00:13 and mysql time is 2019-06-07 00:00:13
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => '20190606000000'
-    1 => '20190606235959'
+    0 => '20190607000000'
+    1 => '20190607235959'

[NFC][test-fix] centralise use of assertLike for comparing sql

This function makes the tests cope with minor changes to whitespace in sql strings being compared

Merge pull request #14461 from colemanw/displayPrefsStyle

Use crm-checkbox-list styling in display prefs tpl

Merge pull request #14450 from eileenmcnaughton/pay_rs

Wrap titles in metadata in ts on payment.create api

Merge pull request #14433 from eileenmcnaughton/mailin_settings

[Outbound mail admin form] fix some more notices resulting from recent change

Merge pull request #14427 from eileenmcnaughton/search_upgrade

Fix upgrade on participant_register date

Fix version for pledge message update, also remove 'please print'

Merge pull request #14467 from mlutfy/ts-fixes-5-16

Misc translation 'ts' fixes

Merge pull request #14465 from philmb/dev/core#1010

dev/core#1010 pledge_acknowledge template - capitalization and greeting

Merge pull request #14442 from colemanw/drupal8jquery

Conditionally use Drupal's jQuery only if available

Merge pull request #14464 from mattwire/smartgroupeditcriteria

Don't run search when editing smartgroup criteria

Misc translation 'ts' fixes

dev/core#1010 pledge_acknowledge template - capitalization and greeting

Pledge Acknowledgements have incorrect capitalization in greetings, and greetings do not use email_greeting.

Conditionally use Drupal's jQuery only if available

Don't search before showing smartgroup criteria

Whitespace formatting in CRM/Group/Form/Edit.tpl

Merge pull request #14463 from seamuslee001/5_14_master

5.14

Merge in 5.14

Use crm-checkbox-list styling in display prefs tpl

Merge pull request #14394 from eileenmcnaughton/merge_get_conflicts

Add Contact.get_merge_conflicts api

Update 5.14.0.md

install/index.php - Fix misleading install text

During GUI install, there's a health-check based on the PHP version.  Now
that PHP 7 is required, this message *always* talks about PHP 7.x. But
it's presented under a bucket labeled "PHP5".

Update 5.14.0.md

Set version to 5.14.0

Merge pull request #14451 from eileenmcnaughton/format

[NFC] array formatting in api_v3_paymentTest class

Merge pull request #14458 from eileenmcnaughton/cachekey

Test tweaks that *might* help with intermittant test weirdness

Update 5.14.0.md

Merge pull request #14201 from seamuslee001/5.14

Fix AssetBuilderTest when running on WordPress. Use Guzzle.

Merge pull request #14449 from demeritcowboy/bulksms-enotice

dev/core#1023 - E_NOTICE when creating bulk SMS

Merge pull request #14457 from civicrm/5.14

5.14

AssetBuilderTest - Add more assertions for Guzzle variant of test

Test tweaks that *might* help with intermittant test weirdness

It's technically possible for the 2 cache keys to be the same - which would break stuff - although it seems
a bit too frequent to be the reason...

Also we clear one cache in tear down & not the other.

I'm not optimistic either of these will solve the intermittant fails but they seem more correct changed

Also I added verbosity

Merge pull request #14445 from demeritcowboy/deletedActivitiesSearchFilter

dev/core#1022 - Manage Case deleted activities search filter no longer working

Fix Invalid asset builder test by using guzzle

Revert "Fix Asset builder test by altering error_reporting level to match Drupal and changing regex to not care about HTTP version"

This reverts commit be441aa368ef7f1e5a8dd115c065caa172f95a6a.

Update 5.14.0.md

Update 5.14.0.md

Merge pull request #14212 from seamuslee001/wordpress_e2e_assetbuilder_fix

Fix AssetBuilderTest by altering error_reporting level to match Dru…

Merge pull request #14439 from seamuslee001/5_14_0_release_notes_update

Update 5.14.0 release notes with latest changes

Merge pull request #14455 from civicrm/5.14

5.14 to master

Merge pull request #14454 from eileenmcnaughton/php72

(NFC) Remove without param tests

Merge pull request #14453 from eileenmcnaughton/export_sc_514

dev/core#1015 fix regression on exporting soft credits

Remove without param tests

These really date back to when we were creating the api wrapper. I don't think
they add much value now - esp since php is not quiet if we break the wrapper
contract

dev/core#1015 fix regression on exporting soft credits

Cut down from https://github.com/civicrm/civicrm-core/pull/14440/commits due to test fails

Update Release 5.14.0 release notes with latest changes

Update release notes for PR #14440

Withdraw soft credits item

fix E_NOTICE

[NFC] array formatting in api_v3_paymentTest class

Wrap titles in metadata in ts on payment.create api

Fix Exception to remove result from duplicates

This updates, and tests, the Exception.create function (via the api) to remove duplicates from the
prevnext_cache. This feels like a better fix than the cartwheels the prevnext cache code does to avoid
retrieving cached duplicates.

Merge pull request #14441 from colemanw/sortable

dev/core#628 - Make Quicksearch options sortable

Merge pull request #14411 from eileenmcnaughton/merge_get_duplicates

Add api for Dedupe.getduplicates

make search filter work again

Update get_merge_conflicts api

Increase result nuance in response to review

dev/core#628 - Make Quicksearch options sortable

Merge pull request #14438 from eileenmcnaughton/display

(dev/core#1000) Fixes contact's displayname not appearing in membership edit, if no registered email is found

Merge pull request #14363 from mfb/error-msg

Avoid logging error message if status check query fails.

Merge pull request #14407 from eileenmcnaughton/514

Fix checksum failure on editing recurring contributions

Merge pull request #14435 from eileenmcnaughton/5.14

Revert "Create contribution before taking payment, per contribution page workflow"

Fix checksum failure on editing recurring contributions

Fixes contact's displayname not appearing in membership edit, if no registered email is found

Add a unit test to lock in fix to dev/core#1000

5.14.0 release notes (#14245)

* 5.14.0 release notes: raw from script

* 5.14.0 release notes: add boilerplate

* sorting misc and deleting 5.13 commits

* fixing up contributors

* fixing up gitlab issues titles

* annotating

* rearranging sections

* editing and clean up

* 5.14.0 release notes: edits

* 5.14.0 release notes: incorporated late changes

Merge pull request #14437 from agh1/no-php-56

Minimum supported PHP version is 7.0

Merge pull request #14426 from eileenmcnaughton/cont_notice

dev/core#1014 fix e-notice regression

Add api for Dedupe.getduplicates

Minimum supported PHP version is 7.0

Improve handling of 1st option on search prefs

Merge pull request #14247 from pradpnayak/ActvityTags

Change Activity tag to use select2

Revert "Create contribution before taking payment, per contribution page workflow"

This reverts commit 9257f129

Merge pull request #14434 from eileenmcnaughton/activity_form

[REF] Activity form - very minor tidy up

Remove fatal errors in favour of status bounces

[REF] very small code cleanup

activityTName is only ever set in the proceeding IF clause so moving this chunk of code into that
clause makes that a little more readable

[Outbound mail admin form] fix some more notices resulting from recent change