fixed broken spec (now handling nil namespace correctly)

markryall · Apr 6, 2012 · 23b99e3 · 23b99e3
1 parent 99ad4e8
commit 23b99e3
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/lib/blast_furnace/nodes.rb b/lib/blast_furnace/nodes.rb
@@ -1,7 +1,6 @@
 class ClassNode < Struct.new :name
   def eval context
-    context.namespace ||= ''
+    path = build_path context.destination, context.namespace
-    path = "#{context.destination}/#{context.namespace}/#{name}.java"
     FileUtils.mkdir_p File.dirname path
     File.open(path, 'w') do |io|
     io.puts <<EOF
@@ -11,8 +10,14 @@ def eval context
     end
   end
 
+  def build_path destination, namespace
+    namespace ?
+      "#{destination}/#{namespace}/#{name}.java" :
+      "#{destination}/#{name}.java"
+  end
+
   def package_declaration namespace
-    namespace.empty? ? '' : <<EOF
+    namespace ? <<EOF : ''
 package #{namespace.gsub('/','.')};
 
 EOF