chore(test): move handling_spec off of the control flow (#5010)

Author: CrispusDH

spec/basic/handling_spec.js

@@ -1,11 +1,10 @@
-
-describe('handling timeout errors', function() {
-
-  it('should call error handler on a timeout', function() {
-    browser.get('http://dummyUrl', 1).then(function() {
+describe('handling timeout errors', () => {
+  it('should call error handler on a timeout', async () => {
+    try {
+      await browser.get('http://dummyUrl', 1);
       throw 'did not handle error';
-    }, function(err) {
+    } catch (err) {
       expect(err instanceof Error).toBeTruthy();
-    });
+    }
   });
 });

spec/basicConf.js

@@ -10,7 +10,8 @@ exports.config = {
   // Spec patterns are relative to this directory.
   specs: [
     'basic/elements_spec.js',
-    'basic/lib_spec.js'
+    'basic/lib_spec.js',
+    'basic/handling_spec.js'
   ],
 
   // Exclude patterns are relative to this directory.

spec/ciFullConf.js

@@ -11,7 +11,8 @@ exports.config = {
   // TODO(selenium4): revert back to basic/*_spec.js
   specs: [
     'basic/elements_spec.js',
-    'basic/lib_spec.js'
+    'basic/lib_spec.js',
+    'basic/handling_spec.js'
   ],
 
   // Exclude patterns are relative to this directory.