Refactor service tests to remove unnecessary whitespace and improve readability

Author: iampopovich

javascript/selenium-webdriver/test/chrome/service_test.js

@@ -60,7 +60,7 @@ test.suite(
         it('uses SE_CHROMEDRIVER environment variable when set', function () {
           const testPath = '/custom/path/to/chromedriver'
           process.env.SE_CHROMEDRIVER = testPath
-          
+
           const serviceBuilder = new chrome.ServiceBuilder()
           const service = serviceBuilder.build()
           assert.strictEqual(service.getExecutable(), testPath)
@@ -69,17 +69,17 @@ test.suite(
         it('explicit path overrides environment variable', function () {
           const envPath = '/env/path/to/chromedriver'
           const explicitPath = '/explicit/path/to/chromedriver'
-          
+
           process.env.SE_CHROMEDRIVER = envPath
           const serviceBuilder = new chrome.ServiceBuilder(explicitPath)
           const service = serviceBuilder.build()
-          
+
           assert.strictEqual(service.getExecutable(), explicitPath)
         })
 
         it('falls back to default behavior when environment variable is not set', function () {
           delete process.env.SE_CHROMEDRIVER
-          
+
           const serviceBuilder = new chrome.ServiceBuilder()
           const service = serviceBuilder.build()
           // Should be null/undefined when no explicit path and no env var

javascript/selenium-webdriver/test/edge/service_test.js

@@ -58,7 +58,7 @@ test.suite(
         it('uses SE_EDGEDRIVER environment variable when set', function () {
           const testPath = '/custom/path/to/edgedriver'
           process.env.SE_EDGEDRIVER = testPath
-          
+
           const serviceBuilder = new edge.ServiceBuilder()
           const service = serviceBuilder.build()
           assert.strictEqual(service.getExecutable(), testPath)
@@ -67,17 +67,17 @@ test.suite(
         it('explicit path overrides environment variable', function () {
           const envPath = '/env/path/to/edgedriver'
           const explicitPath = '/explicit/path/to/edgedriver'
-          
+
           process.env.SE_EDGEDRIVER = envPath
           const serviceBuilder = new edge.ServiceBuilder(explicitPath)
           const service = serviceBuilder.build()
-          
+
           assert.strictEqual(service.getExecutable(), explicitPath)
         })
 
         it('falls back to default behavior when environment variable is not set', function () {
           delete process.env.SE_EDGEDRIVER
-          
+
           const serviceBuilder = new edge.ServiceBuilder()
           const service = serviceBuilder.build()
           // Should be null/undefined when no explicit path and no env var

javascript/selenium-webdriver/test/firefox/service_test.js

@@ -58,7 +58,7 @@ test.suite(
         it('uses SE_GECKODRIVER environment variable when set', function () {
           const testPath = '/custom/path/to/geckodriver'
           process.env.SE_GECKODRIVER = testPath
-          
+
           const serviceBuilder = new firefox.ServiceBuilder()
           const service = serviceBuilder.build()
           assert.strictEqual(service.getExecutable(), testPath)
@@ -67,17 +67,17 @@ test.suite(
         it('explicit path overrides environment variable', function () {
           const envPath = '/env/path/to/geckodriver'
           const explicitPath = '/explicit/path/to/geckodriver'
-          
+
           process.env.SE_GECKODRIVER = envPath
           const serviceBuilder = new firefox.ServiceBuilder(explicitPath)
           const service = serviceBuilder.build()
-          
+
           assert.strictEqual(service.getExecutable(), explicitPath)
         })
 
         it('falls back to default behavior when environment variable is not set', function () {
           delete process.env.SE_GECKODRIVER
-          
+
           const serviceBuilder = new firefox.ServiceBuilder()
           const service = serviceBuilder.build()
           // Should be null/undefined when no explicit path and no env var

javascript/selenium-webdriver/test/ie/service_test.js

@@ -39,7 +39,7 @@ test.suite(
           this.skip()
           return
         }
-        
+
         service = new ie.ServiceBuilder().build()
         service.setExecutable(getBinaryPaths(new ie.Options()).driverPath)
         let url = await service.start()
@@ -64,24 +64,24 @@ test.suite(
         it('uses SE_IEDRIVER environment variable when set', function () {
           const testPath = '/custom/path/to/iedriver'
           process.env.SE_IEDRIVER = testPath
-          
+
           const serviceBuilder = new ie.ServiceBuilder()
           assert.strictEqual(serviceBuilder.getExecutable(), testPath)
         })
 
         it('explicit path overrides environment variable', function () {
           const envPath = '/env/path/to/iedriver'
           const explicitPath = '/explicit/path/to/iedriver'
-          
+
           process.env.SE_IEDRIVER = envPath
           const serviceBuilder = new ie.ServiceBuilder(explicitPath)
-          
+
           assert.strictEqual(serviceBuilder.getExecutable(), explicitPath)
         })
 
         it('falls back to default behavior when environment variable is not set', function () {
           delete process.env.SE_IEDRIVER
-          
+
           const serviceBuilder = new ie.ServiceBuilder()
           // Should be null/undefined when no explicit path and no env var
           assert.ok(!serviceBuilder.getExecutable())

javascript/selenium-webdriver/test/safari_test.js

@@ -57,7 +57,7 @@ test.suite(
         it('uses SE_SAFARIDRIVER environment variable when set', function () {
           const testPath = '/custom/path/to/safaridriver'
           process.env.SE_SAFARIDRIVER = testPath
-          
+
           const serviceBuilder = new safari.ServiceBuilder()
           const service = serviceBuilder.build()
           assert.strictEqual(service.getExecutable(), testPath)
@@ -66,17 +66,17 @@ test.suite(
         it('explicit path overrides environment variable', function () {
           const envPath = '/env/path/to/safaridriver'
           const explicitPath = '/explicit/path/to/safaridriver'
-          
+
           process.env.SE_SAFARIDRIVER = envPath
           const serviceBuilder = new safari.ServiceBuilder(explicitPath)
           const service = serviceBuilder.build()
-          
+
           assert.strictEqual(service.getExecutable(), explicitPath)
         })
 
         it('falls back to default behavior when environment variable is not set', function () {
           delete process.env.SE_SAFARIDRIVER
-          
+
           const serviceBuilder = new safari.ServiceBuilder()
           const service = serviceBuilder.build()
           // Should be null/undefined when no explicit path and no env var