semian 0.3.0 → 0.4.0

data/test/redis_test.rb

@@ -79,7 +79,7 @@ class TestRedis < MiniTest::Unit::TestCase
   end
 
   def test_resource_acquisition_for_connect
-    client = connect_to_redis!
+    connect_to_redis!
 
     Semian[:redis_testing].acquire do
       error = assert_raises Redis::ResourceBusyError do
@@ -213,11 +213,11 @@ class TestRedis < MiniTest::Unit::TestCase
   def connect_to_redis!(semian_options = {})
     redis = Redis.new(
       host: '127.0.0.1',
-      port: 16379,
+      port: 16_379,
       reconnect_attempts: 0,
       db: 1,
       timeout: 0.5,
-      semian: SEMIAN_OPTIONS.merge(semian_options)
+      semian: SEMIAN_OPTIONS.merge(semian_options),
     )
     redis.client.connect
     redis

data/test/resource_test.rb

@@ -2,7 +2,9 @@ require 'test_helper'
 
 class TestResource < MiniTest::Unit::TestCase
   def setup
-    Semian.destroy(:testing) rescue nil
+    Semian.destroy(:testing)
+  rescue
+    nil
   end
 
   def teardown
@@ -109,10 +111,10 @@ class TestResource < MiniTest::Unit::TestCase
     resource = create_resource :testing, tickets: 2, timeout: 0.5
 
     resource.acquire do
-      pid = fork do
+      fork do
         resource.acquire do
           assert_raises Semian::TimeoutError do
-            resource.acquire {  }
+            resource.acquire {}
           end
         end
       end
@@ -122,35 +124,33 @@ class TestResource < MiniTest::Unit::TestCase
   end
 
   def test_acquire_releases_on_kill
-    begin
-      resource = create_resource :testing, tickets: 1, timeout: 0.1
-      acquired = false
+    resource = create_resource :testing, tickets: 1, timeout: 0.1
+    acquired = false
 
-      # Ghetto process synchronization
-      file = Tempfile.new('semian')
-      path = file.path
-      file.close!
+    # Ghetto process synchronization
+    file = Tempfile.new('semian')
+    path = file.path
+    file.close!
 
-      pid = fork do
-        resource.acquire do
-          FileUtils.touch(path)
-          sleep 1000
-        end
+    pid = fork do
+      resource.acquire do
+        FileUtils.touch(path)
+        sleep 1000
       end
+    end
 
-      sleep 0.1 until File.exists?(path)
-      assert_raises Semian::TimeoutError do
-        resource.acquire {}
-      end
+    sleep 0.1 until File.exist?(path)
+    assert_raises Semian::TimeoutError do
+      resource.acquire {}
+    end
 
-      Process.kill("KILL", pid)
-      resource.acquire { acquired = true }
-      assert acquired
+    Process.kill("KILL", pid)
+    resource.acquire { acquired = true }
+    assert acquired
 
-      Process.wait
-    ensure
-      FileUtils.rm_f(path) if path
-    end
+    Process.wait
+  ensure
+    FileUtils.rm_f(path) if path
   end
 
   def test_count
@@ -184,7 +184,7 @@ class TestResource < MiniTest::Unit::TestCase
     resource = create_resource :testing, tickets: 1
     resource.destroy
     assert_raises Semian::SyscallError do
-      resource.acquire { }
+      resource.acquire {}
     end
   end
 
@@ -287,9 +287,7 @@ class TestResource < MiniTest::Unit::TestCase
 
       f.flock(File::LOCK_UN)
 
-      while children.any? do
-        children.delete(Process.wait)
-      end
+      children.delete(Process.wait) while children.any?
 
       assert_equal 5, create_resource(:testing, tickets: 0).count
     ensure
@@ -307,7 +305,11 @@ class TestResource < MiniTest::Unit::TestCase
   def destroy_resources
     return unless @resources
     @resources.each do |resource|
-      resource.destroy rescue nil
+      begin
+        resource.destroy
+      rescue
+        nil
+      end
     end
     @resources = []
   end

data/test/semian_test.rb

@@ -2,7 +2,9 @@ require 'test_helper'
 
 class TestSemian < MiniTest::Unit::TestCase
   def setup
-    Semian.destroy(:testing) rescue nil
+    Semian.destroy(:testing)
+  rescue
+    nil
   end
 
   def test_unsupported_acquire_yields
@@ -27,5 +29,4 @@ class TestSemian < MiniTest::Unit::TestCase
   ensure
     ENV.delete('SEMIAN_SEMAPHORES_DISABLED')
   end
-
 end

data/test/simple_integer_test.rb

@@ -0,0 +1,49 @@
+require 'test_helper'
+
+class TestSimpleInteger < MiniTest::Unit::TestCase
+  CLASS = ::Semian::Simple::Integer
+
+  def setup
+    @integer = CLASS.new
+  end
+
+  def teardown
+    @integer.destroy
+  end
+
+  module IntegerTestCases
+    def test_access_value
+      assert_equal(0, @integer.value)
+      @integer.value = 99
+      assert_equal(99, @integer.value)
+      time_now = (Time.now).to_i
+      @integer.value = time_now
+      assert_equal(time_now, @integer.value)
+      @integer.value = 6
+      assert_equal(6, @integer.value)
+      @integer.value = 6
+      assert_equal(6, @integer.value)
+    end
+
+    def test_increment
+      @integer.increment(4)
+      assert_equal(4, @integer.value)
+      @integer.increment
+      assert_equal(5, @integer.value)
+      @integer.increment(-2)
+      assert_equal(3, @integer.value)
+    end
+
+    def test_reset_on_init
+      assert_equal(0, @integer.value)
+    end
+
+    def test_reset
+      @integer.increment(5)
+      @integer.reset
+      assert_equal(0, @integer.value)
+    end
+  end
+
+  include IntegerTestCases
+end

data/test/simple_sliding_window_test.rb

@@ -0,0 +1,65 @@
+require 'test_helper'
+
+class TestSimpleSlidingWindow < MiniTest::Unit::TestCase
+  CLASS = ::Semian::Simple::SlidingWindow
+
+  def setup
+    @sliding_window = CLASS.new(max_size: 6)
+    @sliding_window.clear
+  end
+
+  def teardown
+    @sliding_window.destroy
+  end
+
+  module SlidingWindowTestCases
+    def test_sliding_window_push
+      assert_equal(0, @sliding_window.size)
+      @sliding_window << 1
+      assert_sliding_window(@sliding_window, [1], 6)
+      @sliding_window << 5
+      assert_sliding_window(@sliding_window, [1, 5], 6)
+    end
+
+    def test_sliding_window_resize
+      assert_equal(0, @sliding_window.size)
+      @sliding_window << 1 << 2 << 3 << 4 << 5 << 6
+      assert_sliding_window(@sliding_window, [1, 2, 3, 4, 5, 6], 6)
+      @sliding_window.resize_to 6
+      assert_sliding_window(@sliding_window, [1, 2, 3, 4, 5, 6], 6)
+      @sliding_window.resize_to 5
+      assert_sliding_window(@sliding_window, [2, 3, 4, 5, 6], 5)
+      @sliding_window.resize_to 6
+      assert_sliding_window(@sliding_window, [2, 3, 4, 5, 6], 6)
+    end
+
+    def test_sliding_window_edge_falloff
+      assert_equal(0, @sliding_window.size)
+      @sliding_window << 0 << 1 << 2 << 3 << 4 << 5 << 6 << 7
+      assert_sliding_window(@sliding_window, [2, 3, 4, 5, 6, 7], 6)
+      @sliding_window.shift
+      assert_sliding_window(@sliding_window, [3, 4, 5, 6, 7], 6)
+    end
+
+    def resize_to_less_than_1_raises
+      assert_raises ArgumentError do
+        @sliding_window.resize_to 0
+      end
+    end
+  end
+
+  module SlidingWindowUtilityMethods
+    def assert_sliding_window(sliding_window, array, max_size)
+      # Get private member, the sliding_window doesn't expose the entire array
+      data = sliding_window.instance_variable_get("@window")
+      assert_equal(array, data)
+      assert_equal(max_size, sliding_window.max_size)
+    end
+  end
+
+  include SlidingWindowTestCases
+
+  private
+
+  include SlidingWindowUtilityMethods
+end

data/test/simple_state_test.rb

@@ -0,0 +1,45 @@
+require 'test_helper'
+
+class TestSimpleEnum < MiniTest::Unit::TestCase
+  CLASS = ::Semian::Simple::State
+
+  def setup
+    @state = CLASS.new
+  end
+
+  def teardown
+    @state.destroy
+  end
+
+  module StateTestCases
+    def test_start_closed?
+      assert @state.closed?
+    end
+
+    def test_open
+      @state.open
+      assert @state.open?
+      assert_equal @state.value, :open
+    end
+
+    def test_close
+      @state.close
+      assert @state.closed?
+      assert_equal @state.value, :closed
+    end
+
+    def test_half_open
+      @state.half_open
+      assert @state.half_open?
+      assert_equal @state.value, :half_open
+    end
+
+    def test_reset
+      @state.reset
+      assert @state.closed?
+      assert_equal @state.value, :closed
+    end
+  end
+
+  include StateTestCases
+end

data/test/test_helper.rb

@@ -21,6 +21,11 @@ Toxiproxy.populate([
     upstream: 'localhost:6379',
     listen: 'localhost:16379',
   },
+  {
+    name: 'semian_test_net_http',
+    upstream: 'localhost:31050',
+    listen: 'localhost:31051',
+  },
 ])
 
 class MiniTest::Unit::TestCase

data/test/unprotected_resource_test.rb

@@ -15,7 +15,7 @@ class UnprotectedResourceTest < MiniTest::Unit::TestCase
   end
 
   def test_resource_tickets
-    assert_equal -1, @resource.tickets
+    assert_equal(-1, @resource.tickets)
   end
 
   def test_resource_count

metadata

@@ -1,37 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: semian
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Scott Francis
 - Simon Eskildsen
 autorequire: 
 bindir: bin
-cert_chain:
-- |
-  -----BEGIN CERTIFICATE-----
-  MIIDcDCCAligAwIBAgIBATANBgkqhkiG9w0BAQUFADA/MQ8wDQYDVQQDDAZhZG1p
-  bnMxFzAVBgoJkiaJk/IsZAEZFgdzaG9waWZ5MRMwEQYKCZImiZPyLGQBGRYDY29t
-  MB4XDTE0MDUxNTIwMzM0OFoXDTE1MDUxNTIwMzM0OFowPzEPMA0GA1UEAwwGYWRt
-  aW5zMRcwFQYKCZImiZPyLGQBGRYHc2hvcGlmeTETMBEGCgmSJomT8ixkARkWA2Nv
-  bTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBAL0/81O3e1vh5smcwp2G
-  MpLQ6q0kejQLa65bPYPxdzWA1SYOKyGfw+yR9LdFzsuKpwWzKq6zX35lj1IckWS4
-  bNBEQzxmufUxU0XPM02haFB8fOfDJzdXsWte9Ge4IFwahwn68gpMqN+BvxL+KMYz
-  Iut9YmN44d4LZdsENEIO5vmybuG2vYDz7R56qB0PA+Q2P2CdhymsBad2DQs69FBo
-  uico9V6VMYYctL9lCYdzu9IXrOYNTt88suKIVzzAlHOKeN0Ng5qdztFoTR8sfxDr
-  Ydg3KHl5n47wlpgd8R0f/4b5gGxW+v9pyJCgQnLlRu7DedVSvv7+GMtj3g9r3nhJ
-  KqECAwEAAaN3MHUwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAwHQYDVR0OBBYEFI/o
-  maf34HXbUOQsdoLHacEKQgunMB0GA1UdEQQWMBSBEmFkbWluc0BzaG9waWZ5LmNv
-  bTAdBgNVHRIEFjAUgRJhZG1pbnNAc2hvcGlmeS5jb20wDQYJKoZIhvcNAQEFBQAD
-  ggEBADkK9aj5T0HPExsov4EoMWFnO+G7RQ28C30VAfKxnL2UxG6i4XMHVs6Xi94h
-  qXFw1ec9Y2eDUqaolT3bviOk9BB197+A8Vz/k7MC6ci2NE+yDDB7HAC8zU6LAx8Y
-  Iqvw7B/PSZ/pz4bUVFlTATif4mi1vO3lidRkdHRtM7UePSn2rUpOi0gtXBP3bLu5
-  YjHJN7wx5cugMEyroKITG5gL0Nxtu21qtOlHX4Hc4KdE2JqzCPOsS4zsZGhgwhPs
-  fl3hbtVFTqbOlwL9vy1fudXcolIE/ZTcxQ+er07ZFZdKCXayR9PPs64heamfn0fp
-  TConQSX2BnZdhIEYW+cKzEC/bLc=
-  -----END CERTIFICATE-----
-date: 2015-03-11 00:00:00.000000000 Z
+cert_chain: []
+date: 2015-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -89,6 +67,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: thin
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: toxiproxy
   requirement: !ruby/object:Gem::Requirement
@@ -113,7 +105,9 @@ extensions:
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rubocop.yml"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.md
 - README.md
@@ -125,24 +119,33 @@ files:
 - lib/semian/circuit_breaker.rb
 - lib/semian/instrumentable.rb
 - lib/semian/mysql2.rb
+- lib/semian/net_http.rb
 - lib/semian/platform.rb
 - lib/semian/protected_resource.rb
 - lib/semian/redis.rb
 - lib/semian/resource.rb
+- lib/semian/simple_integer.rb
+- lib/semian/simple_sliding_window.rb
+- lib/semian/simple_state.rb
 - lib/semian/unprotected_resource.rb
 - lib/semian/version.rb
+- repodb.yml
 - scripts/install_toxiproxy.sh
 - semian.gemspec
 - test/circuit_breaker_test.rb
 - test/helpers/background_helper.rb
 - test/instrumentation_test.rb
 - test/mysql2_test.rb
+- test/net_http_test.rb
 - test/redis_test.rb
 - test/resource_test.rb
 - test/semian_test.rb
+- test/simple_integer_test.rb
+- test/simple_sliding_window_test.rb
+- test/simple_state_test.rb
 - test/test_helper.rb
 - test/unprotected_resource_test.rb
-homepage: https://github.com/csfrancis/semian
+homepage: https://github.com/shopify/semian
 licenses:
 - MIT
 metadata: {}
@@ -162,7 +165,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.2.3
 signing_key: 
 specification_version: 4
 summary: Bulkheading for Ruby with SysV semaphores