Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test to lock in obscure custom join handling #15715

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Unit test preliminary to cleaning up the CustomQuery join handling

Before

Search builder custom address field scenario untested

After

Tested

Technical Details

Some tweaks to test helper class - will have to see if this upsets any existing tests as it tweaks the identifiers

Comments

@civibot
Copy link

civibot bot commented Nov 4, 2019

(Standard links)

@civibot civibot bot added the master label Nov 4, 2019
@@ -108,7 +113,7 @@ public function testSearchCustomDataDateFromTo() {

$params[$dateCustomField['id']] = CRM_Contact_BAO_Query::convertFormValues($formValues);
$queryObj = new CRM_Core_BAO_CustomQuery($params);
$queryObj->Query();
$queryObj->query();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correction of case to match fn

@seamuslee001
Copy link
Contributor

Only affects tests and Jenkins is happen then i am too

@seamuslee001 seamuslee001 merged commit 299eed8 into civicrm:master Nov 4, 2019
@seamuslee001 seamuslee001 deleted the custom_query branch November 4, 2019 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants