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.

Sign up for GitHub

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

Jump to bottom

dev/core#746 Add in unit tests to ensure that where clause is as is w… #13685

Merged
merged 1 commit into from
Feb 23, 2019
Merged

dev/core#746 Add in unit tests to ensure that where clause is as is w… #13685

merged 1 commit into from
Feb 23, 2019

Conversation

seamuslee001
Copy link
Contributor

…hen multiple smart group where clauses are used in search builder

Add in a check of from clauses too

Overview

Adds in unit tests where there are 2 separate group clauses in a search builder

Before

no test on this specific configuration

After

test

ping @eileenmcnaughton @monishdeb @jitendrapurohit

@civibot
Copy link

civibot bot commented Feb 23, 2019

(Standard links)

@civibot civibot bot added the master label Feb 23, 2019
@seamuslee001
dev/core#746 Add in unit tests to ensure that where clause is as is w…
e24f938 
…hen multiple smart group where clauses are used in search builder

Add in a check of from clauses too
@seamuslee001
Copy link
Contributor Author

Merging as per the tag

@seamuslee001 seamuslee001 merged commit 753d647 into civicrm:master Feb 23, 2019
@seamuslee001 seamuslee001 deleted the tests_746 branch February 23, 2019 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master merge on pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@seamuslee001 @eileenmcnaughton