Azure / sql-action

🚀 Deploy changes to your SQL database easily with SQL projects or SQL scripts and sql-action for GitHub workflows
MIT License
103 stars 58 forks source link

Fix tcp in server name when adding firewall rule #233

Closed zijchen closed 4 months ago

zijchen commented 4 months ago

Connection string parser doesn't remove "tcp:" from server name, which fails the request to add firewall rule. The automated test run can't reliably check this since the IP address isn't always blocked.

I updated the unit tests to use a more specific expect.toBe instead of expect.toMatch which may not match the whole string.