Fixed AdminAPI test failures in windows:

In windows platforms some tests were failing due of an error in
dba.deleteSandboxInstance() calls as the directories were still being
accessed. As dba.stopSandboxInstance() returns without full guarantee
that the mysqld process if stopped leading to a possible failure in
deleting the sandbox dir aftewards.
The fix was to include a retry method with a 60 seconds timeout for
the deleteSandboxInstance calls in the cleanup_sandbox() function.

Reverted commit 6b2190ebbce76edc354e06169fcee8f488aa5006 and included
patch to disable by default the Shell_py_dba_tests

Author: miguelaraujo

unittest/scripts/js_devapi/setup/setup.js

@@ -226,7 +226,6 @@ function start_sandbox(params) {
 }
 
 function cleanup_sandbox(port) {
-
     println ('Stopping the sandbox at ' + port + ' to delete it...');
     try {
       stop_options = {}
@@ -239,17 +238,30 @@ function cleanup_sandbox(port) {
       println(err.message);
     }
 
-    println ('Deleting the sandbox at ' + port);
-    try {
-      options = {}
-      if (__sandbox_dir != '')
-        options['sandboxDir'] = __sandbox_dir;
+    options = {}
+    if (__sandbox_dir != '')
+      options['sandboxDir'] = __sandbox_dir;
 
-      dba.deleteSandboxInstance(port, options);
-    } catch (err) {
-      println(err.message);
+    var deleted = false;
 
+    print('Try deleting sandbox at: ' + port);
+    deleted = wait(10, 1, function() {
+      try {
+        dba.deleteSandboxInstance(port, options);
+
+        println(' succeeded');
+        return true;
+      } catch (err) {
+        println(' failed: ' + err.message);
+        return false;
+      }
+    });
+    if (deleted) {
+      println('Delete succeeded at: ' + port);
+    } else {
+      println('Delete failed at: ' + port);
     }
+
 }
 
 // Smart deployment routines

unittest/scripts/py_devapi/setup/setup.py

@@ -227,16 +227,24 @@ def cleanup_sandbox(port):
       print err.message
       pass
 
-    print 'Deleting the sandbox at %s' % port
-    try:
       options = {}
       if __sandbox_dir != '':
         options['sandboxDir'] = __sandbox_dir
 
-      dba.delete_sandbox_instance(port, options)
-    except Exception, err:
-      print err.message
-      pass
+      print 'Try deleting sandbox at: %s' % port
+      def try_delete():
+        try:
+          dba.delete_sandbox_instance(port, options)
+          print "succeeded"
+          return True
+        except Exception, err:
+          print "failed: %s" % str(err)
+          return False
+
+      if wait(10, 1, try_delete):
+        print 'Delete succeeded at: %s' % port
+      else:
+        print 'Delete failed at: %s' % port
 
 def reset_or_deploy_sandbox(port):
   deployed_here = False

unittest/test_main.cc

@@ -103,8 +103,9 @@ int main(int argc, char **argv) {
       new_flags = "Shell_py_dba_tests.no_interactive_deploy*:Shell_py_dba_tests.interactive_classic_*";
     else if (flags == "ALLBUTDBA")
       new_flags = "*:-Shell_py_dba_tests.*:Shell_js_dba_tests.*";
+    else if (flags == "*")
+      new_flags = "*:-Shell_py_dba_tests.*";  // Exclude DBAPY tests by default
 
-    new_flags = "*:-Shell_py_dba_tests.*";
     if (!new_flags.empty())
       ::testing::GTEST_FLAG(filter) = new_flags.c_str();
   }