saviour 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bc893ccb84bdea17562e885649be535589247449
-  data.tar.gz: b626abd923dbcde6e9a44f0a6c6e5f608d9e7f85
+  metadata.gz: d67c7eda9780fbf27c6fe5c4c1b621839b81b82a
+  data.tar.gz: 634ea69a334a72f9690fbc1102dd7ea6b98f3683
 SHA512:
-  metadata.gz: 76b2b7986d9c44f006396a445ec188973a44ffc5c631a48a7b089ce3585428ad9565e29cc42d6c1153d0aaf662c0d1039abc218d0121d37b3fd7357a9fe166b8
-  data.tar.gz: 42e2ca1a05a2e57233e80c4f4f90909095ec7b64b8858bc40b0bf2448d951391765521d618146f115b5bb61ac648a77568ec7d25196950f3316bc30bd7b74683
+  metadata.gz: 3df7814a85bd316abe2d2a0c82436fdc66bec1860867344faccd834b581d38a81dc9172ba61eff57ffe649d068ede726d1e25ef652a6a4ef8738bc63c4765c64
+  data.tar.gz: 67b818e120b6f389fecebb883fdc3df26cd2f536222d7e10f0a2ed53b1b22ec0cc43a8f023193ec19129e3407e31d108697931d44375ab67c57ff3245756a2cd

data/lib/saviour/config.rb

@@ -1,3 +1,5 @@
+require 'thread'
+
 module Saviour
   class Config
     class NotImplemented
@@ -6,29 +8,35 @@ module Saviour
       end
     end
 
+    @semaphore = Mutex.new
+
     class << self
       def processing_enabled
-        Thread.current["Saviour::Config"] ||= {}
-        Thread.current["Saviour::Config"][:processing_enabled] || true
+        Thread.current.thread_variable_set("Saviour::Config", {}) unless Thread.current.thread_variable_get("Saviour::Config")
+        Thread.current.thread_variable_get("Saviour::Config")[:processing_enabled] || true
       end
 
       def processing_enabled=(value)
-        Thread.current["Saviour::Config"] ||= {}
-        Thread.current["Saviour::Config"][:processing_enabled] = value
+        Thread.current.thread_variable_set("Saviour::Config", {}) unless Thread.current.thread_variable_get("Saviour::Config")
+        Thread.current.thread_variable_get("Saviour::Config")[:processing_enabled] = value
       end
 
       def storage
-        Thread.current["Saviour::Config"] ||= {}
-        Thread.current["Saviour::Config"][:storage] || (Thread.main["Saviour::Config"] && Thread.main["Saviour::Config"][:storage]) || NotImplemented.new
+        @semaphore.synchronize do
+          Thread.current.thread_variable_set("Saviour::Config", {}) unless Thread.current.thread_variable_get("Saviour::Config")
+          Thread.current.thread_variable_get("Saviour::Config")[:storage] || (Thread.main.thread_variable_get("Saviour::Config") && Thread.main.thread_variable_get("Saviour::Config")[:storage]) || NotImplemented.new
+        end
       end
 
       def storage=(value)
-        Thread.current["Saviour::Config"] ||= {}
-        Thread.current["Saviour::Config"][:storage] = value
+        @semaphore.synchronize do
+          Thread.current.thread_variable_set("Saviour::Config", {}) unless Thread.current.thread_variable_get("Saviour::Config")
+          Thread.current.thread_variable_get("Saviour::Config")[:storage] = value
 
-        if Thread.main["Saviour::Config"].nil?
-          Thread.main["Saviour::Config"] ||= {}
-          Thread.main["Saviour::Config"][:storage] = value
+          if Thread.main.thread_variable_get("Saviour::Config").nil?
+            Thread.main.thread_variable_set("Saviour::Config", {})
+            Thread.main.thread_variable_get("Saviour::Config")[:storage] = value
+          end
         end
       end
     end

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.4.0"
+  VERSION = "0.4.1"
 end

data/spec/models/config_spec.rb

@@ -28,7 +28,7 @@ describe Saviour::Config do
       end
 
       it "raises correct exception if the main thread is not yet configured" do
-        Thread.main["Saviour::Config"] = nil
+        Thread.main.thread_variable_set("Saviour::Config", nil)
 
         Thread.new {
           expect { Saviour::Config.storage.anything }.to raise_error(RuntimeError)
@@ -36,7 +36,7 @@ describe Saviour::Config do
       end
 
       it "forwards configuration to main thread if not configured" do
-        Thread.main["Saviour::Config"] = nil
+        Thread.main.thread_variable_set("Saviour::Config", nil)
 
         Thread.new {
           Saviour::Config.storage = 'config_set_on_thread_1'
@@ -55,6 +55,18 @@ describe Saviour::Config do
         end.join
         expect(Saviour::Config.storage).to eq 12
       end
+
+      it "shares values per-fiber" do
+        Thread.new {
+          Saviour::Config.storage = "value_#{Thread.current.object_id}"
+
+          f = Fiber.new do
+            Saviour::Config.storage
+          end
+
+          expect(f.resume).to eq "value_#{Thread.current.object_id}"
+        }.join
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Roger Campos