Passing the timeout as parameter for Selenium class

jhoblitt · Jan 16, 2014 · 7739a50 · 7739a50
1 parent cdc6f18
commit 7739a50
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 11 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -16,6 +16,7 @@
 #   java         => 'java',
 #   version      => '2.39.0',
 #   url          => undef,
+#   timeout      => '90'
 # }
 # ```
 #
@@ -74,6 +75,7 @@
   $java         = $selenium::params::java,
   $version      = $selenium::params::version,
   $url          = undef,
+  $timeout      = $selenium::params::download_timeout
 ) inherits selenium::params {
   validate_string($user)
   validate_string($group)
@@ -129,7 +131,7 @@
   wget::fetch { 'selenium-server-standalone':
     source      => $jar_url,
     destination => "${jar_path}/${jar_name}",
-    timeout     => 90,
+    timeout     => "${timeout}",
     execuser    => $user,
     require     => File[$jar_path],
   }

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -4,16 +4,17 @@
 #
 #
 class selenium::params {
-  $display         = ':0'
-  $user            = 'selenium'
-  $group           = $user
-  $install_root    = '/opt/selenium'
-  $server_options  = '-Dwebdriver.enable.native.events=1'
-  $hub_options     = '-role hub'
-  $node_options    = "${server_options} -role node"
-  $java            = 'java'
-  $version         = '2.39.0'
-  $default_hub     = 'http://localhost:4444/grid/register'
+  $display          = ':0'
+  $user             = 'selenium'
+  $group            = $user
+  $install_root     = '/opt/selenium'
+  $server_options   = '-Dwebdriver.enable.native.events=1'
+  $hub_options      = '-role hub'
+  $node_options     = "${server_options} -role node"
+  $java             = 'java'
+  $version          = '2.39.0'
+  $default_hub      = 'http://localhost:4444/grid/register'
+  $download_timeout = '90' 
 
   case $::osfamily {
     'redhat': {}