Merge pull request #15 from tylerwalts/develop

Develop

README.md

@@ -16,7 +16,7 @@ This approach was inspired by: http://stackoverflow.com/questions/10268583/how-t
 
 Currently Supported:
 *  RedHat Family (RedHat, Fedora, CentOS)
-*  Java 6, 7
+*  Java 6, 7, 8
 
 *  This may work on other linux flavors but more testing is needed.  Please seend feedback!
 

manifests/init.pp

@@ -46,6 +46,10 @@
     }
 
     case $version {
+        '8': {
+            $javaDownloadURI = "http://download.oracle.com/otn-pub/java/jdk/8-b132/jdk-8-linux-${plat_filename}.tar.gz"
+            $java_home = "${install_dir}/jdk1.8.0"
+        }
         '7': {
             $javaDownloadURI = "http://download.oracle.com/otn-pub/java/jdk/7/jdk-7-linux-${plat_filename}.tar.gz"
             $java_home = "${install_dir}/jdk1.7.0"
@@ -86,8 +90,8 @@
         }
     }
 
-    # Java 7 comes in a tarball so just extract it.
-    if ( $version == '7' ) {
+    # Java 7/8 comes in a tarball so just extract it.
+    if ( $version in [ '7', '8' ] ) {
         exec { 'extract_jdk':
             cwd     => "${install_dir}/",
             command => "tar -xf ${installerFilename}",

spec/classes/jdk_oracle_init_spec.rb

@@ -39,6 +39,25 @@
             }
         end
 
+        context 'specifying version 8' do
+            let :params do {
+                :version => '8',
+            } end
+
+            it {
+                should contain_exec( 'get_jdk_installer').with_creates('/opt/jdk-8-linux-x64.tar.gz')
+                should contain_file('/opt/jdk-8-linux-x64.tar.gz')
+                should contain_exec('extract_jdk').with_creates('/opt/jdk1.8.0')
+                should contain_file('/etc/alternatives/java').with({
+                    :ensure  => 'link',
+                    :target  => '/opt/jdk1.8.0/bin/java',
+                })
+                should contain_file('/opt/jdk-8').with({
+                    :ensure  => 'link',
+                })
+            }
+        end
+
         context 'using custom installation directory' do
             let :params do {
                 :install_dir => '/my/path',