Merge pull request #4 from thom8/firefox

Use boolean type for selenium_install_firefox.
arknoll · Feb 29, 2016 · 5a2914f · 5a2914f
2 parents 7fa6781 + 6d64d2f
commit 5a2914f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/defaults/main.yml b/defaults/main.yml
@@ -2,4 +2,4 @@
 # defaults file for selenium
 selenium_install_dir: /opt
 selenium_version: 2.46.0
-selenium_install_firefox: true
+selenium_install_firefox: yes
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -18,7 +18,7 @@
   apt: name={{item}}
   with_items:
     - firefox
-  when: ansible_os_family == 'Debian' and selenium_install_firefox == 'true'
+  when: ansible_os_family == 'Debian' and selenium_install_firefox
 
 - name: Install browser Xvfb
   yum: name={{item}}
@@ -30,7 +30,7 @@
   yum: name={{item}}
   with_items:
     - firefox
-  when: ansible_os_family == 'RedHat' and selenium_install_firefox == 'true'
+  when: ansible_os_family == 'RedHat' and selenium_install_firefox
 
 
 - name: install