request_recorder 0.0.3 → 0.0.4

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    request_recorder (0.0.3)
+    request_recorder (0.0.4)
       rack
 
 GEM

data/Readme.md

@@ -8,11 +8,14 @@ Install
 Add to your middleware stack:
 
     require "request_recorder"
-    use RequestRecorder::Middleware, :store => RequestRecorder::RedisLogger.new(Redis.new)
 
-### No Redis, No problem
-If you do not have redis, you can write your own logger, you only need a .write method,
-see [RedisLogger](https://github.com/grosser/request_recorder/blob/master/lib/request_recorder/redis_logger.rb)
+    require "request_recorder/cache_logger"
+    use RequestRecorder::Middleware, :store => RequestRecorder::CacheLogger.new(Rails.cache)
+
+    -- or --
+
+    require "request_recorder/redis_logger"
+    use RequestRecorder::Middleware, :store => RequestRecorder::RedisLogger.new(Redis.new)
 
 Usage
 =====

data/lib/request_recorder/cache_logger.rb

@@ -0,0 +1,15 @@
+module RequestRecorder
+  class CacheLogger
+    KEY = "request_recorder"
+
+    def initialize(store)
+      @store = store
+    end
+
+    def write(id, text)
+      id = "#{KEY}.#{Time.now.utc.strftime("%Y-%m-%d %H:%M:%S.%L")}_#{Process.pid}" unless id
+      @store.write(id, @store.read(id).to_s + text)
+      id
+    end
+  end
+end

data/lib/request_recorder/redis_logger.rb

@@ -8,7 +8,7 @@ module RequestRecorder
 
     def write(id, text)
       old = (id ? @redis.hget(KEY, id) : "")
-      id = "#{Time.now.utc.strftime("%Y-%m-%d %H:%M:%S")}_#{Process.pid}" unless id
+      id = "#{Time.now.utc.strftime("%Y-%m-%d %H:%M:%S.%L")}_#{Process.pid}" unless id
       @redis.hset(KEY, id, old.to_s + text)
       id
     end

data/lib/request_recorder/version.rb

@@ -1,3 +1,3 @@
 module RequestRecorder
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/lib/request_recorder.rb

@@ -1,6 +1,5 @@
 require "request_recorder/version"
 require "request_recorder/middleware"
-require "request_recorder/redis_logger"
 
 module RequestRecorder
 end

data/spec/request_recorder/cache_logger_spec.rb

@@ -0,0 +1,43 @@
+require "spec_helper"
+
+describe RequestRecorder::CacheLogger do
+  class Store
+    def initialize
+      @data = {}
+    end
+
+    def read(x)
+      @data[x]
+    end
+
+    def write(x,y)
+      @data[x] = y
+    end
+  end
+
+  let(:store){ Store.new }
+  let(:logger){ RequestRecorder::CacheLogger.new(store) }
+
+  context "#write" do
+    it "returns a unique id" do
+      old = logger.write(nil, "X")
+      sleep 0.01
+      old.should_not == logger.write(nil, "X")
+    end
+
+    it "returns existing ids" do
+      "1111".should == logger.write("1111", "X")
+    end
+
+    it "creates a new entry" do
+      logger.write("1111", "X")
+      store.read("1111").should == "X"
+    end
+
+    it "appends to an existing key" do
+      logger.write("1111", "X")
+      logger.write("1111", "X")
+      store.read("1111").should == "XX"
+    end
+  end
+end

data/spec/request_recorder/redis_logger_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+describe RequestRecorder::RedisLogger do
+  let(:store){ FakeRedis::Redis.new }
+  let(:key){ RequestRecorder::RedisLogger::KEY }
+  let(:logger){ RequestRecorder::RedisLogger.new(store) }
+
+  context "#write" do
+    it "returns a unique id" do
+      old = logger.write(nil, "X")
+      sleep 0.01
+      old.should_not == logger.write(nil, "X")
+    end
+
+    it "returns existing ids" do
+      "1111".should == logger.write("1111", "X")
+    end
+
+    it "creates a new entry" do
+      logger.write("1111", "X")
+      store.hget(key, "1111").should == "X"
+    end
+
+    it "appends to an existing key" do
+      logger.write("1111", "X")
+      logger.write("1111", "X")
+      store.hget(key, "1111").should == "XX"
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,4 +1,6 @@
 require "request_recorder"
+require "request_recorder/redis_logger"
+require "request_recorder/cache_logger"
 
 require "active_record"
 require "fakeredis"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: request_recorder
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -39,10 +39,13 @@ files:
 - Rakefile
 - Readme.md
 - lib/request_recorder.rb
+- lib/request_recorder/cache_logger.rb
 - lib/request_recorder/middleware.rb
 - lib/request_recorder/redis_logger.rb
 - lib/request_recorder/version.rb
 - request_recorder.gemspec
+- spec/request_recorder/cache_logger_spec.rb
+- spec/request_recorder/redis_logger_spec.rb
 - spec/request_recorder_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/grosser/request_recorder
@@ -60,7 +63,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 4362379118001674709
+      hash: 4604048949302387428
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -69,7 +72,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 4362379118001674709
+      hash: 4604048949302387428
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24