chore(test): move restart_spec off of the control flow (#5014)

Author: CrispusDH

spec/basic/restart_spec.js

@@ -1,14 +1,14 @@
-describe('browser.restart', function() {
-  it('doesn\'t break ignoreSynchronization', function() {
-    browser.get('index.html#/polling');
-    browser.restart();
+describe('browser.restart', () => {
+  it('doesn\'t break ignoreSynchronization', async () => {
+    await browser.get('index.html#/polling');
+    await browser.restart();
 
     browser.ignoreSynchronization = true;
     // Get a non-angular page. It shouldn't fail if ignoreSynchronization is on.
-    browser.get('https://google.com/');
+    await browser.get('https://google.com/');
   });
 
-  afterAll(function() {
+  afterAll(() => {
     browser.ignoreSynchronization = false;
   });
 });

spec/basicConf.js

@@ -15,6 +15,7 @@ exports.config = {
     // 'basic/handling_spec.js',
     // 'basic/mockmodule_spec.js',
     // 'basic/navigation_spec.js',
+    // 'basic/restart_spec.js',
   ],
 
   // Exclude patterns are relative to this directory.

spec/ciFullConf.js

@@ -17,6 +17,7 @@ exports.config = {
     // 'basic/handling_spec.js'
     // 'basic/mockmodule_spec.js',
     // 'basic/navigation_spec.js',
+    // 'basic/restart_spec.js',
   ],
 
   // Exclude patterns are relative to this directory.