remotable 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c5772a5edc3139fabf4602d96c59fc69fe3ac83ae016d286a2257deac26ae8e0
-  data.tar.gz: e0e6a52da707287733bcc0bb22a4d34e515bec9e2e869f0a2ac4f49696e07b19
+  metadata.gz: a78225c824ca7d175cf92ee2b9249c8f7cb564b40b9cb9e517afb23c60942a33
+  data.tar.gz: c9ccfb561fcf0589e907868f62bc0208677addd29be93bf69556d84e0a1e7997
 SHA512:
-  metadata.gz: ec594e5275ecb8a7dbd7df17f5c1addfc16f3b982391d8bb88079dc3922f3d62549ef4422bad8d9c9113d7528077490947d3999ca02d32cc2d12a262acad8980
-  data.tar.gz: 5d4c975e43ab1e62af43e257eacd0d5f28a32729ed31de556ef8bf950890db1bf7117d06c0d669cb9ceb8a189f24e77fafda0dd177ff643a41fef4b2615d6117
+  metadata.gz: f59286e3951a20031978f3077f95a3d93fb9d4a69cbeb37d35ae7375233b10ddf9ed28f09c7a4d95aa8a2c38bc10b942b6f48f6a451a4a3c1222262f44d976c6
+  data.tar.gz: 114ee2c2f19c42c85b81e629a540aa8bfee1cc7b269462e62743e41ed09273627cdc2f21a8532a8d4f9cf681cc6bb9e74f9f46ebc91404c78aa67b181a8fc1d2

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+### 0.6.1
+* **Bugfix** Tweaked the behavior of `Remotable.nosync?` to defer to a value set on the main thread if no value has been set for the current thread.
+
 ### 0.6.0
 * **Bugfix** Completely removed dependence on `activeresource`'s `ThreadsafeAttributes`. This makes multithreaded behavior more deterministic and involves fewer edgecases, since things meant to be global state are reverted back to being so, while things meant to have a per-thread scope are constrained as they should be.
 

data/lib/remotable/nosync.rb

@@ -61,11 +61,17 @@ module Remotable
     private
 
       def _nosync
-        Thread.current.thread_variable_get "remotable.nosync.#{self.object_id}"
+        return Thread.current.thread_variable_get "remotable.nosync.#{self.object_id}" if nosync_defined_on?(Thread.current)
+        return Thread.main.thread_variable_get "remotable.nosync.#{self.object_id}" if nosync_defined_on?(Thread.main)
       end
 
       def _nosync=(value)
         Thread.current.thread_variable_set "remotable.nosync.#{self.object_id}", value
+        Thread.current.thread_variable_set "remotable.nosync.#{self.object_id}.defined", true
+      end
+
+      def nosync_defined_on?(thread)
+        !!thread.thread_variable_get("remotable.nosync.#{self.object_id}.defined")
       end
 
     end

data/lib/remotable/version.rb

@@ -1,3 +1,3 @@
 module Remotable
-  VERSION = "0.6.0"
+  VERSION = "0.6.1"
 end

data/test/nosync_test.rb

@@ -48,6 +48,18 @@ class NoSyncTest < ActiveSupport::TestCase
     assert_equal true, Tenant.nosync?
   end
 
+  test "nosync? should defer to the main thread's value if set there but not on the current thread" do
+    Remotable.nosync!
+    subthread = Thread.new do
+      assert Remotable.nosync?
+      Remotable.reset_nosync!
+      refute Remotable.nosync?
+      Thread.stop
+    end
+    sleep 0.1 while subthread.status != "sleep"
+    assert Remotable.nosync?
+  end
+
 
 
   # ========================================================================= #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: remotable
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - Robert Lail