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
[WIP] Find tests that cover admin forms so that they can be converted too 😈 #28558
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
So this is good to merge then.... |
f32fb71
to
d85c336
Compare
Haha! |
d85c336
to
229091b
Compare
229091b
to
2c553ea
Compare
@@ -65,6 +65,7 @@ public function getDefaultEntity(): string { | |||
* @throws \CRM_Core_Exception | |||
*/ | |||
public function preProcess(): void { | |||
throw new \Exception('aha!'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you considered the right exception here might be CRM_Core_Exception()
- any total site breakage should be Civi-specific IMHO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think from here on targeting the specific forms that were converted might be a better way to find them.
This is a good enough start, and I made the mixin tests fail for REAL earlier, which is a bonus.
Will transfer to a lab ticket. EDIT: https://lab.civicrm.org/dev/core/-/issues/4850
CRM_Activity_Form_SearchTest.testSearch
CRM_Admin_Form_PaymentProcessorTest.testUpdateAcceptCreditCard
CRM_Core_FormTest.testOpeningForms with data set "Location Type"
CRM_Core_FormTest.testOpeningForms with data set "Message Templates"
CRM_Core_FormTest.testOpeningForms with data set "Scheduled Jobs"
CRM_Core_OptionGroupTest.testSanitizeFromEmailAddress with data set #0
CRM_Core_OptionGroupTest.testSanitizeFromEmailAddress with data set #1
CRM_Core_OptionGroupTest.testSanitizeFromEmailAddress with data set #2
Overview
Sledgehammer approach that won't find everything but might find some.
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