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

dev/core#2829 - unit test for nonempty dashboard #21437

Merged
merged 1 commit into from
Sep 11, 2021

Conversation

demeritcowboy
Copy link
Contributor

Overview

https://lab.civicrm.org/dev/core/-/issues/2829

Test demonstrating dashboard fail.

My only question is whether $explicitJoin['bridge'] in Api4SelectQuery is expected to always be set or whether it's ok that it's missing. I'm thinking the latter.

Before

What is the old user-interface or technical-contract (as appropriate)?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

After

What changed? What is new old user-interface or technical-contract?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

Technical Details

If the PR involves technical details/changes/considerations which would not be manifest to a casual developer skimming the above sections, please describe the details here.

Comments

Anything else you would like the reviewer to note

@civibot
Copy link

civibot bot commented Sep 11, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor

@demeritcowboy was this meant to fail? or is the fix already merged?

@demeritcowboy
Copy link
Contributor Author

Oh, it failed to fail because the test node is php 7.2. It succeeds in failing on 7.4.

@eileenmcnaughton
Copy link
Contributor

lol ok - I'll merge & it can fail as successfully as it can

@demeritcowboy
Copy link
Contributor Author

Thanks.
You can see now on the MAX matrix tests it fails to not fail and you get the same stack trace as in the lab ticket: https://test.civicrm.org/job/CiviCRM-Core-Matrix/10039/BKPROF=max,CIVIVER=master,SUITES=phpunit-crm,label=bknix-tmp/testReport/junit/(root)/CRM_Core_InvokeTest/testInvokeDashboardWithGettingStartedDashlet/

Fix is at #21441

@eileenmcnaughton
Copy link
Contributor

Does is succeed in succeeding while failing to not fail?

@demeritcowboy
Copy link
Contributor Author

Well, this outputs "yes".

$failure = 0;
$success = 1;
if ($failure = $success) {
  echo 'yes!';
}

He he.

@eileenmcnaughton
Copy link
Contributor

My brain just broke

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants