Broken storage specs.

lib/pwnbus/configdb.rb

@@ -32,7 +32,7 @@ def self.open(name, options = {})
     begin
       return_value = yield db.proxy
       if db.dirty?
-        Files.write_file db_path do |wf|
+        Files.write_db db_path, options do |wf|
           db.write wf
         end
       end

lib/pwnbus/configdb/db.rb

@@ -57,7 +57,12 @@ def [](key)
   # Returns value.
   def []=(key, value)
     @dirty = true
-    @data[key.to_s] = value.dup
+    if value.nil?
+      @data.delete key.to_s
+    else
+      @data[key.to_s] = (value.kind_of?(Numeric) || value.kind_of?(Symbol) ||
+                         value == true || value == false) ? value : value.dup
+    end
     value
   end
 
@@ -75,7 +80,8 @@ def dirty?
   # key doesn't exist. This makes it possible to have .-separated keys, like
   # db.user.name = 'abc'.
   def proxy_get(key)
-    (value = self[key]) ? value : Proxy.new(self, key + '.')
+    value = self[key]
+    value.nil? ? Proxy.new(self, key + '.') : value
   end
 
   # Inserts or updates

spec/db_spec.rb

@@ -73,6 +73,39 @@
     end
   end
 
+  describe "with saved db" do
+    before do
+      Configdb.open('persistence') do |p|
+        p.really.long.flag = true
+        p.really.long.number = 41
+        p.really.long.string = 'something'
+      end
+    end
+
+    it "should report variables correctly" do
+      Configdb.open('persistence') do |p|
+        p.really.long.flag.should == true
+        p.really.long.number.should == 41
+        p.really.long.string.should == 'something'
+      end
+    end
+
+    describe "after overwriting some vars" do
+      Configdb.open('persistence') do |p|
+        p.really.long.number = 42
+        p.really.long.flag = false
+      end
+
+      it "should report changes correctly" do
+        Configdb.open('persistence') do |p|
+          p.really.long.flag.should == false
+          p.really.long.number.should == 42
+          p.really.long.string.should == 'something'        
+        end
+      end
+    end
+  end
+
   describe "after crash" do
     before do
       @crash_path = @user_dir + '/crash.yml'