Merge pull request #149 from dbt-msft/swanderz-patch-2

drop unecessary& broken code from ADIntegrated connection option

Author: dataders

dbt/adapters/sqlserver/connections.py

@@ -270,9 +270,6 @@ def open(cls, connection):
                     con_str.append(f"PWD={{{credentials.PWD}}}")
                 elif type_auth == "ActiveDirectoryInteractive":
                     con_str.append(f"UID={{{credentials.UID}}}")
-                elif type_auth == "ActiveDirectoryIntegrated":
-                    # why is this necessary???
-                    con_str.remove("UID={None}")
                 elif type_auth == "ActiveDirectoryMsi":
                     raise ValueError("ActiveDirectoryMsi is not supported yet")