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/Unit Test] Remove wombats and bananas from sites/default/files/civicrm/custom #21768

Merged
merged 1 commit into from
Oct 8, 2021

Conversation

demeritcowboy
Copy link
Contributor

Overview

If you run tests against the whole BAO/CaseTest.php file it leaves behind 4 files and then another 2 files each subsequent time you run.

@civibot civibot bot added the master label Oct 8, 2021
@civibot
Copy link

civibot bot commented Oct 8, 2021

(Standard links)

@colemanw
Copy link
Member

colemanw commented Oct 8, 2021

I had no idea wombats and bananas could be such pests on a unit test.

@demeritcowboy
Copy link
Contributor Author

jenkins retest this please

Some kind of mysql issue. For the record this test passed.

<p>Initialization Error</p><p><pre>Array
(
    [callback] =&gt; Array
        (
            [0] =&gt; CRM_Core_Error
            [1] =&gt; simpleHandler
        )

    [code] =&gt; -14
    [message] =&gt; DB Error: no database selected

@eileenmcnaughton
Copy link
Contributor

Wombats are really very cute. Also they poo cubes

@demeritcowboy
Copy link
Contributor Author

That sounds made up. Pics or it didn't happen.

@demeritcowboy
Copy link
Contributor Author

Don't post wombat poo pics :)

@demeritcowboy
Copy link
Contributor Author

Probably 800 people just clicked on a link about wombat poo. I think we have a new clickbait subject...

@eileenmcnaughton
Copy link
Contributor

:-)

@eileenmcnaughton eileenmcnaughton merged commit 19369a2 into civicrm:master Oct 8, 2021
@demeritcowboy demeritcowboy deleted the remove-wombats branch October 8, 2021 16:57
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.

3 participants