Fix qualified wildcard stringifying

[thomas-jeepe]

Issue #52

[coveralls]

Pull Request Test Coverage Report for Build 157

• 29 of 30 (96.67%) changed or added relevant lines in 4 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+7.3%) to 91.973%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
tests/sqlparser_generic.rs	14	15	93.33%

Totals
Change from base Build 148:	7.3%
Covered Lines:	951
Relevant Lines:	1034

---

💛 - Coveralls

[nickolay]

I'd move the roundtrip_qualified_wildcard test from sqlparser_ansi to sqlparser_generic, as the latter has helper functions that let you test the round-tripping easily:

let sql = "SELECT COUNT(Employee.*) FROM northwind.Order JOIN northwind.Employee ON Order.employee = Employee.id";
let _select = verified_only_select(&sql);

(This would also avoid the conflict with the change to SQLStatement::SQLSelect in #51)

See also my question in #52 about validity of this SQL; I think a comment about whether this SQL is valid in some dialects would be helpful.

For the second patch, you might want to run cargo fmt, which we use for auto-formatting the code -- sorry it's not mentioned anywhere!

[nickolay]

Excellent, thank you.

[andygrove]

Thanks @thomas-jeepe !