Fix grant error related to mysql syntax error

cgreatbanks commented 6 years ago

What does this PR change?

Fix grant error related to mysql syntax error reported in #55

Details

Latest release v1.5.0 introduced Mysql syntax error when run on existing deployments
When not specifying a table, grant entries were missing or adding `` around

Testing

make testacc
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v  -timeout 120m
?       github.com/terraform-providers/terraform-provider-mysql [no test files]
=== RUN   TestProvider
--- PASS: TestProvider (0.00s)
=== RUN   TestProvider_impl
--- PASS: TestProvider_impl (0.00s)
=== RUN   TestAccDatabase
--- PASS: TestAccDatabase (0.05s)
=== RUN   TestAccDatabase_collationChange
--- PASS: TestAccDatabase_collationChange (0.11s)
=== RUN   TestAccGrant
--- PASS: TestAccGrant (0.20s)
=== RUN   TestAccGrant_role
--- SKIP: TestAccGrant_role (0.01s)
    resource_grant_test.go:68: Roles require MySQL 8+
=== RUN   TestAccGrant_roleToUser
--- SKIP: TestAccGrant_roleToUser (0.01s)
    resource_grant_test.go:103: Roles require MySQL 8+
=== RUN   TestAccRole_basic
--- SKIP: TestAccRole_basic (0.01s)
    resource_role_test.go:32: Roles require MySQL 8+
=== RUN   TestAccUserPassword_basic
--- PASS: TestAccUserPassword_basic (0.78s)
=== RUN   TestAccUser_basic
--- PASS: TestAccUser_basic (0.22s)
=== RUN   TestAccUser_auth
--- PASS: TestAccUser_auth (0.13s)
=== RUN   TestAccUser_deprecated
--- PASS: TestAccUser_deprecated (0.35s)
PASS
ok      github.com/terraform-providers/terraform-provider-mysql/mysql   (cached)