Simplify Travis CI testing by only using mysql root

[photodude]

Pull Request for Simplify Travis CI testing by only using mysql root .

Summary of Changes

Simplify Travis CI testing by only using mysql root

This eliminates the HHVM and non-HHVM specific mysql database creations. By using only the root option we have a simpler testing setup.

Testing Instructions

merge by code review

Documentation Changes Required

none

[andrepereiradasilva]

I have tested this item ✅ successfully on 8754ce0

seems logic to me.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11703.}

[photodude]

@rdeutz, @mbabker, @wilsonge, can this get a bump for merge consideration

[rdeutz]

I have tested this item ✅ successfully on 8754ce0

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11703.}

[rdeutz]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11703.}

[yvesh]

@photodude how about moving that create database into the sql file? That would simplify it even more ;)

[mbabker]

👎 the SQL file is used for more than Travis and the database name is configurable via the DSN strings for each instance.

[rdeutz]

set to needs review for double checking the patch

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11703.}

[photodude]

@rdeutz I'm not sure I'm following, what needs to be reviewed with the patch?

[rdeutz]

@photodude the comment @mbabker makes me wonder if this the right thing to do, so I removed RTC to make sure it doesn't gets merged by accident and give me some time to check again.

[mbabker]

My comment was in response to @yvesh suggesting the CREATE DATABASE statement be moved into the SQL files, not the root user change.

[rdeutz]

Ah, thought better look twice than sorry :-) So we can merge this.

[photodude]

Thanks @rdeutz