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

[NFC] [Test] Cleanup on membership type test #20649

Merged
merged 1 commit into from
Jun 18, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jun 18, 2021

Overview

[NFC] [Test] Cleanup on membership type test

Rely on tearDown to delete created memberships

@civibot
Copy link

civibot bot commented Jun 18, 2021

(Standard links)

@civibot civibot bot added the master label Jun 18, 2021
@@ -1915,9 +1916,10 @@ public function quickCleanUpFinancialEntities() {
'civicrm_price_field',
];
$this->quickCleanup($tablesToTruncate);
$this->disableTaxAndInvoicing();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually this line was removed for no reason in the end - doesn't it look prettier here though?

@eileenmcnaughton eileenmcnaughton changed the title [NFC] [Test] Ensure consistent membership types are in db [NFC] [Test] Cleanup on membership type test Jun 18, 2021
@eileenmcnaughton eileenmcnaughton force-pushed the mem_clean branch 2 times, most recently from 22e2a2a to d0b895d Compare June 18, 2021 05:40
@seamuslee001 seamuslee001 merged commit 35b3936 into civicrm:master Jun 18, 2021
@seamuslee001 seamuslee001 deleted the mem_clean branch June 18, 2021 21:15
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