Use the new getInteger() method also for integer configuration values…

… for Selenium.
sebastianbergmann · Oct 8, 2011 · 0df8fa1 · 0df8fa1
1 parent a9aa2b5
commit 0df8fa1
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/PHPUnit/Util/Configuration.php b/PHPUnit/Util/Configuration.php
@@ -718,13 +718,17 @@ public function getSeleniumBrowserConfiguration()
             }
 
             if ($config->hasAttribute('port')) {
-                $port = (int)$config->getAttribute('port');
+                $port = $this->getInteger(
+                  (string)$config->getAttribute('port'), 4444
+                );
             } else {
                 $port = 4444;
             }
 
             if ($config->hasAttribute('timeout')) {
-                $timeout = (int)$config->getAttribute('timeout');
+                $timeout = $this->getInteger(
+                  (string)$config->getAttribute('timeout'), 30000
+                );
             } else {
                 $timeout = 30000;
             }