From 2164d59caf70c3a4b05d876cf67048f230ac98df Mon Sep 17 00:00:00 2001
From: Joshua Hoblitt <jhoblitt@cpan.org>
Date: Tue, 1 Oct 2013 15:37:45 -0700
Subject: [PATCH] mv user setup from selenium::server -> selenium::install

To break a dependency cycle between wget::fetch and user.
---
 manifests/install.pp | 5 +++++
 manifests/server.pp  | 5 -----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/manifests/install.pp b/manifests/install.pp
index 15af89c..9a98a4b 100644
--- a/manifests/install.pp
+++ b/manifests/install.pp
@@ -29,6 +29,11 @@
 
   include wget
 
+  user { $selenium::server::user:
+    gid => [$selenium::server::group],
+  }
+  group { $selenium::server::group: }
+
   $jar_name = "selenium-server-standalone-${version}.jar"
 
   if $url {
diff --git a/manifests/server.pp b/manifests/server.pp
index e9b787a..ec6fe6c 100644
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -15,11 +15,6 @@
   validate_string($group)
   validate_string($install_path)
 
-  user { $user:
-    gid => [$group],
-  }
-  group { $group: }
-
   class { 'selenium::install': } ->
   class { 'selenium::config': } ->
   class { 'selenium::service': } ->