Allow jdk_version to be a string or number

Previously, the case statements checking the jdk_version failed
because they were assuming jdk_version was a string. For users overriding
this value and setting it to be a number, the case statement would never
match. This commit ensures we are always using a string when comparing
jdk_version.

Author: carmstrong

libraries/helpers.rb

@@ -27,7 +27,7 @@ class OpenJDK
     def initialize(node)
       @node = node.to_hash
       @java_home = @node['java']['java_home'] || '/usr/lib/jvm/default-java'
-      @jdk_version = @node['java']['jdk_version'] || '6'
+      @jdk_version = @node['java']['jdk_version'].to_s || '6'
     end
 
     def java_location

recipes/ibm.rb

@@ -52,7 +52,7 @@
 
 java_alternatives 'set-java-alternatives' do
   java_location node['java']['java_home']
-  case node['java']['jdk_version']
+  case node['java']['jdk_version'].to_s
   when "6"
     bin_cmds node['java']['ibm']['6']['bin_cmds']
   when "7"

recipes/ibm_tar.rb

@@ -50,7 +50,7 @@
 
 java_alternatives 'set-java-alternatives' do
   java_location node['java']['java_home']
-  case node['java']['jdk_version']
+  case node['java']['jdk_version'].to_s
   when "6"
     bin_cmds node['java']['ibm']['6']['bin_cmds']
   when "7"

recipes/openjdk.rb

@@ -50,7 +50,7 @@
   java_alternatives 'set-java-alternatives' do
     java_location jdk.java_home
     priority jdk.alternatives_priority
-    case node['java']['jdk_version']
+    case node['java']['jdk_version'].to_s
     when "6"
       bin_cmds node['java']['jdk']['6']['bin_cmds']
     when "7"

recipes/oracle.rb

@@ -28,14 +28,8 @@
 
 java_home = node['java']["java_home"]
 arch = node['java']['arch']
-jdk_version = node['java']['jdk_version']
 
-#convert version number to a string if it isn't already
-if jdk_version.instance_of? Fixnum
-  jdk_version = jdk_version.to_s
-end
-
-case jdk_version
+case node['java']['jdk_version'].to_s
 when "6"
   tarball_url = node['java']['jdk']['6'][arch]['url']
   tarball_checksum = node['java']['jdk']['6'][arch]['checksum']

recipes/oracle_i386.rb

@@ -28,7 +28,7 @@
 
 java_home = node['java']["java_home"]
 
-case node['java']['jdk_version']
+case node['java']['jdk_version'].to_s
 when "6"
   tarball_url = node['java']['jdk']['6']['i586']['url']
   tarball_checksum = node['java']['jdk']['6']['i586']['checksum']

spec/libraries/helpers_spec.rb

@@ -23,7 +23,7 @@
     {
       'java' => {
         'java_home' => '/usr/lib/jvm/default-java',
-        'jdk_version' => 6
+        'jdk_version' => '6'
       },
       'kernel' => {
         'machine' => 'x86_64'