feat: add DELETE /data/foo/:oid to remove blob from storage

2 files changed, 16 insertions, 2 deletions

diff --git a/server-rack.rb b/server-rack.rb
index fbfd62a..2f22a4b 100644
--- a/server-rack.rb
+++ b/server-rack.rb
@@ -22,7 +22,7 @@ class DataStorageServer
   # {object data}
   # Objects that are not on the server will return a 404 Not Found.
   def get(path)
-    oid = path.split('/')[-1]
+    oid = id_from(path)
     if @storage.key?(oid)
       ['200', {}, [@storage[oid]]]
     else
@@ -37,6 +37,12 @@ class DataStorageServer
     ['201', {}, [JSON.generate({ size: data.size, oid: oid })]]
   end
 
+  def destroy(path)
+    oid = id_from(path)
+    @storage.delete(oid)
+    ['200', {}, []]
+  end
+
   def call(env)
     path = env['PATH_INFO']
     case env['REQUEST_METHOD']
@@ -44,10 +50,18 @@ class DataStorageServer
       get(path)
     when 'PUT'
       put(env)
+    when 'DELETE'
+      destroy(path)
     else
       raise env.inspect
     end
   end
+
+  private
+
+  def id_from(path)
+    path.split('/')[-1]
+  end
 end
 
 # This starts the server if the script is invoked from the command line. No
diff --git a/test.rb b/test.rb
index 7c0d2ab..429fa80 100644
--- a/test.rb
+++ b/test.rb
@@ -40,7 +40,7 @@ class DataStorageServerTest < Minitest::Test
     assert_equal 404, last_response.status
   end
 
-  def xtest_delete
+  def test_delete
     put '/data/foo', 'some object'
     res = JSON.parse(last_response.body)