From 6d175727b54cf96a95008537b40e64eef584cb3f Mon Sep 17 00:00:00 2001
From: MarwaOuledElHadjAli
 <30328476+MarwaOuledElHadjAli@users.noreply.github.com>
Date: Mon, 20 Jan 2020 15:00:30 +0100
Subject: [PATCH] Update GridStarter.java

Remove EdgeWebDriver form Grid starter command for Edge higher than 44 and automatically Add EdgeWebDriver Path in the command to start grid node for Edge prior than 44
---
 .../seleniumgridextras/grid/GridStarter.java  | 37 ++++++++++++++++++-
 1 file changed, 36 insertions(+), 1 deletion(-)

diff --git a/SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java b/SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java
index 6dadd92c..dfa20a6c 100644
--- a/SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java
+++ b/SeleniumGridExtras/src/main/java/com/groupon/seleniumgridextras/grid/GridStarter.java
@@ -17,6 +17,7 @@
 import java.util.LinkedList;
 import java.util.List;
 import java.util.ArrayList;
+import java.util.Scanner;
 
 public class GridStarter {
 
@@ -200,9 +201,43 @@ protected static List<String> getWebNodeStartCommand(String configFile, Boolean
         command.add(option);
       }
     }
+    // Check if MicrosoftEdge is declared as browser in  node config
+    File file = new File(configFile);
+    Boolean edgeExist = false;
+    Boolean edgePrior44 = false;
+    String fullVersion;
+    int version = 44;
+    try {
+      Scanner scanner = new Scanner(file);
+      int lineNum = 0;
+      while (scanner.hasNextLine()) {
+        String line = scanner.nextLine();
+        lineNum ++;
+        // Store version
+        if(line.contains("version")) {
+          fullVersion = line.trim().replaceAll("[^\\d.]", "");
+          version = Integer.parseInt(fullVersion.split("\\.")[0]);
+        }
+        if(line.contains("MicrosoftEdge")) {
+          edgeExist = true;
+        }
+      }
+      // If MicrosoftEdge is declared in node capabilities
+      // Check it's version prior than 44 or no
+      if (edgeExist && version < 44) {
+          logger.info("Edge exist and it's version is prior than 44");
+          edgePrior44 = true;
+      }
+    } catch(Exception e) {
+      System.out.println("file doesn't exist");
+    }
+
     if (windows) {
       command.add(getIEDriverExecutionPathParam(config));
-      command.add(getEdgeDriverExecutionPathParam(config));
+      if(edgePrior44){
+        logger.info("Add Edge Driver to the command to start Grid Node");
+        command.add(getEdgeDriverExecutionPathParam(config));
+      }
     }
 
     command.add(getChromeDriverExecutionPathParam(config));