fix: return enum type

[aaqilniz]

lb4 discover doesn't handle enum properly. This PR implements this workaround for the enum in lb4 discover. For that to work, MySQL connector needs to return the enum type as it is.

Related GitHub issues & PRs:

PR loopbackio/loopback-next#10065

Checklist

• DCO (Developer Certificate of Origin) signed in all commits
• npm test passes on your machine
• New tests added or existing tests modified to cover all changes
• Code conforms with the style guide
• Commit messages are following our guidelines

[aaqilniz]

Hi, @dhmlau. Please have a look at the PR. I am not sure what's causing the issue but the tests are failing for this PR too like this one.

And please have a look at the related PR with the fix here.

[samarpanB]

CI check should pass after merging #539

[samarpanB]

@aaqilniz can you please rebase this PR ?

[aaqilniz]

Hi, @samarpanB. The tests are still failing.

[samarpanB]

@aaqilniz I did the same changes in a new PR #541 and it seemed to be clearing the checks. Can you try by creating a fresh new PR ? Or since you are now part of maintainers you can raise a PR in a new branch here itself without forking the repo. Try that. It seems something is wrong with github action on this PR or on this fork.

[aaqilniz]

Hi, @samarpanB. I created a new PR (with the forked one) but it still fails.

I tried pushing a new branch at the original branch (not the forked one), but the request failed with the permission denied error.

[samarpanB]

@aaqilniz rebase once more please, so that we can check if the CI action passes now.

[aaqilniz]

Hi, @samarpanB. The tests are finally passing. Thanks for the help.

[aaqilniz]

Hi, @dhmlau. The tests are finally passing for this PR too. Can you please have a look at the PR please?