downgrade bak SQL version and add Airline test users (#15)

* downgrade bak SQL version and add Airline test users

* fix some messages and README

* update sqlmlutils zip

* reenable toplevel tests

Python/README.md

@@ -209,7 +209,7 @@ pkgmanager.uninstall("tensorflow")
 1. Make sure a SQL Server with an updated ML Services Python is running on localhost. 
 2. Restore the AirlineTestDB from the .bak file in this repo 
 3. Make sure Trusted (Windows) authentication works for connecting to the database
-4. Setup a user with db_owner role with uid: "Tester" and password "FakeTesterPwd"
+4. Setup a user with db_owner role with uid: "Tester" and password "FakeT3sterPwd!"
     
 ### Notable TODOs and open issues
 

Python/sqlmlutils/packagemanagement/sqlpackagemanager.py

@@ -78,7 +78,7 @@ class SQLPackageManager:
         db_owner role, you can also specify scope as public. This will uninstall packages from a public path for all
         users. Note: if you connect as dbo, you can only uninstall packages from the public path.
         """
-        print("Uninstalling " + package_name + "only, not dependencies")
+        print("Uninstalling " + package_name + " only, not dependencies")
         self._drop_sql_package(package_name, scope)
 
     def list(self):

R/DESCRIPTION

@@ -15,5 +15,5 @@ License: MIT + file LICENSE
 Copyright: Copyright 2016 Microsoft Corporation
 RoxygenNote: 6.1.0
 Encoding: UTF-8
-Suggests: testthat,
+Suggests: testthat (>= 2.0.0),
     roxygen2

R/tests/testthat-sqlPackage-toplevel.R

@@ -4,4 +4,4 @@
 library(testthat)
 library(sqlmlutils)
 
-#test_check("sqlmlutils", filter = "toplevel")
+test_check("sqlmlutils", filter = "toplevel")

R/tests/testthat/test.sqlPackage.scope.R

@@ -9,7 +9,7 @@ library(testthat)
 context("Tests for sqlmlutils package management scope")
 
 test_that("dbo cannot install package into private scope", {
-    #skip("temporaly_disabled")
+    #skip("temporarily_disabled")
     skip_if(helper_isServerLinux(), "Linux tests do not have support for Trusted user." )
 
     connectionStringDBO <- helper_getSetting("connectionStringDBO")
@@ -22,7 +22,7 @@ test_that("dbo cannot install package into private scope", {
 })
 
 test_that( "package install and remove by scope", {
-    #skip("temporaly_disabled")
+    #skip("temporarily_disabled")
     skip_if(helper_isServerLinux(), "Linux tests do not have support for Trusted user." )
 
     connectionStringDBO <- helper_getSetting("connectionStringDBO")