nonnative 1.16.0 → 1.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 06c156ebea189af501939979653772a2f87dcb6c262f373134d061bd0f07a48c
-  data.tar.gz: 6d41fd8ac177d6565517423c73b031c0c4e03512f2406e7be6f12e2611dfe87d
+  metadata.gz: f32a99fc33eb1c0fd62360648c9b1de943c500ae674ae713b7c2ec56693ff165
+  data.tar.gz: c46a33b6451b3f69c4ae226106eaf58d2f54683627cde12217859179a396bc0e
 SHA512:
-  metadata.gz: a34000c945bc7902645dd32a381311160353193efce3fcc9fbdc563a9e191b192b5e14b72bb4c1efa9d87a821558b801ba002d213e5aad7dceeef1f1ebf12e92
-  data.tar.gz: 9d31997089ebaf6e01b4394b4b13d916791e8d5366acb0fd78e09db3cc662609202d1ac0d6ed605dda35796eae84201f607e5255cf6a25e8e2dedd01ef87e43d
+  metadata.gz: 965c0c84a1be47fd5d9cf05cf887ec9572dbdddf5ec579a29519800891b68727152d0f4559bb90e20870beaa7b27f3052ee2ea3edf2e31e40db6d04b46c6ef98
+  data.tar.gz: 583c38fafc5eb0d6b2762b9a23f83319ade0d381c5ab262825e78b1cf9a3b777e1828baf1f1e696bc8f2b2564ce291328c0bce515b92368404c634f776f93d3a

data/.rubocop.yml

@@ -6,10 +6,10 @@ Layout/LineLength:
   Max: 120
 
 Metrics/MethodLength:
-  Max: 15
+  Max: 20
 
 Metrics/BlockLength:
-  Max: 35
+  Max: 60
 
 Metrics/AbcSize:
   Max: 20
@@ -58,3 +58,12 @@ Style/RedundantRegexpEscape:
 
 Style/RedundantFetchBlock:
   Enabled: true
+
+Style/AccessorGrouping:
+  Enabled: true
+
+Style/BisectedAttrAccessor:
+  Enabled: true
+
+Style/RedundantAssignment:
+  Enabled: true

data/Gemfile.lock

@@ -1,9 +1,10 @@
 PATH
   remote: .
   specs:
-    nonnative (1.16.0)
+    nonnative (1.21.0)
+      concurrent-ruby (~> 1.0, >= 1.0.5)
       cucumber (~> 3.1, >= 3.1.2)
-      grpc (~> 1.28)
+      grpc (>= 1, < 2)
       puma (~> 4.3, >= 4.3.3)
       rest-client (~> 2.1)
       rspec-benchmark (~> 0.6.0)
@@ -57,10 +58,10 @@ GEM
     google-protobuf (3.12.2)
     googleapis-common-protos-types (1.0.5)
       google-protobuf (~> 3.11)
-    grpc (1.28.0)
-      google-protobuf (~> 3.11)
+    grpc (1.30.1)
+      google-protobuf (~> 3.12)
       googleapis-common-protos-types (~> 1.0)
-    grpc-tools (1.28.0)
+    grpc-tools (1.30.1)
     http-accept (1.7.0)
     http-cookie (1.0.3)
       domain_name (~> 0.5)
@@ -81,7 +82,7 @@ GEM
       ruby2_keywords (~> 0.0.1)
     netrc (0.11.0)
     nio4r (2.5.2)
-    nokogiri (1.10.9)
+    nokogiri (1.10.10)
       mini_portile2 (~> 2.4.0)
     parallel (1.19.2)
     parser (2.7.1.4)
@@ -125,13 +126,13 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.9.0)
     rspec-support (3.9.3)
-    rubocop (0.86.0)
+    rubocop (0.87.1)
       parallel (~> 1.10)
-      parser (>= 2.7.0.1)
+      parser (>= 2.7.1.1)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.7)
       rexml
-      rubocop-ast (>= 0.0.3, < 1.0)
+      rubocop-ast (>= 0.1.0, < 1.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 2.0)
     rubocop-ast (0.1.0)
@@ -187,12 +188,12 @@ PLATFORMS
 DEPENDENCIES
   bundler (~> 2.1, >= 2.1.4)
   chutney (~> 2.0, >= 2.0.3.1)
-  grpc-tools (~> 1.28)
+  grpc-tools (>= 1, < 2)
   nonnative!
   rake (~> 13.0, >= 13.0.1)
-  rubocop (~> 0.86.0)
+  rubocop (~> 0.87.1)
   simplecov (~> 0.17.1)
-  solargraph (~> 0.39.7)
+  solargraph (~> 0.39.11)
 
 BUNDLED WITH
    2.1.4

data/Makefile

@@ -7,7 +7,7 @@ outdated-dependencies:
 	bundle outdated --only-explicit
 
 features: clean
-	bundle exec cucumber --profile report $(feature)
+	bundle exec cucumber --profile report --fail-fast $(feature)
 
 analysis:
 	bundle exec rubocop

data/README.md

@@ -249,8 +249,8 @@ module Nonnative
     end
 
     class GRPCServer < Nonnative::GRPCServer
-      def configure(grpc)
-        grpc.handle(GreeterService.new)
+      def svc
+        GreeterService.new
       end
     end
   end
@@ -309,8 +309,10 @@ Nonnative.configure do |config|
   config.strategy = :manual
 
   config.server do |d|
-    d.proxy.type = 'chaos'
-    d.proxy.port = 20_000
+    d.proxy = {
+      type: 'chaos',
+      port: 20_000
+    }
   end
 end
 ```

data/lib/nonnative.rb

@@ -10,6 +10,7 @@ require 'sinatra'
 require 'rest-client'
 require 'puma'
 require 'puma/server'
+require 'concurrent'
 
 require 'nonnative/version'
 require 'nonnative/error'
@@ -35,10 +36,10 @@ require 'nonnative/proxy'
 require 'nonnative/no_proxy'
 require 'nonnative/chaos_proxy'
 
-Thread.abort_on_exception = true
-
 module Nonnative
   class << self
+    attr_reader :pool
+
     def load_configuration(path)
       @configuration ||= Nonnative::Configuration.load_file(path) # rubocop:disable Naming/MemoizedInstanceVariableName
     end

data/lib/nonnative/chaos_proxy.rb

@@ -2,6 +2,14 @@
 
 module Nonnative
   class ChaosProxy < Nonnative::Proxy
+    def initialize(service)
+      @connections = Concurrent::Hash.new
+      @mutex = Mutex.new
+      @state = :none
+
+      super service
+    end
+
     def start
       @tcp_server = ::TCPServer.new('0.0.0.0', service.port)
       @thread = Thread.new { perform_start }
@@ -12,21 +20,32 @@ module Nonnative
       tcp_server.close
     end
 
+    def close_all
+      apply_state :close_all
+    end
+
+    def reset
+      apply_state :none
+    end
+
     def port
       service.proxy.port
     end
 
     private
 
-    attr_reader :tcp_server, :thread
+    attr_reader :tcp_server, :thread, :connections, :mutex, :state
 
     def perform_start
       loop do
-        Thread.start(tcp_server.accept) { |local_socket| connect(local_socket) }
+        thread = Thread.start(tcp_server.accept) { |local_socket| connect(local_socket) }
+        connections[thread.object_id] = thread
       end
     end
 
     def connect(local_socket)
+      return local_socket.close if state?(:close_all)
+
       remote_socket = create_remote_socket
       return unless remote_socket
 
@@ -36,9 +55,12 @@ module Nonnative
         break if write(ready, local_socket, remote_socket)
         break if write(ready, remote_socket, local_socket)
       end
+    rescue Errno::ECONNRESET
+      # Just ignore it.
     ensure
       local_socket.close
       remote_socket&.close
+      connections.delete(Thread.current.object_id)
     end
 
     def create_remote_socket
@@ -60,5 +82,13 @@ module Nonnative
 
       false
     end
+
+    def apply_state(state)
+      mutex.synchronize { @state = state }
+    end
+
+    def state?(state)
+      mutex.synchronize { @state == state }
+    end
   end
 end

data/lib/nonnative/configuration.rb

@@ -42,8 +42,10 @@ module Nonnative
             proxy = fd['proxy']
 
             if proxy
-              s.proxy.type = proxy['type']
-              s.proxy.port = proxy['port']
+              s.proxy = {
+                type: proxy['type'],
+                port: proxy['port']
+              }
             end
           end
         end
@@ -56,9 +58,7 @@ module Nonnative
       self.servers = []
     end
 
-    attr_accessor :strategy
-    attr_accessor :processes
-    attr_accessor :servers
+    attr_accessor :strategy, :processes, :servers
 
     def process
       process = Nonnative::ConfigurationProcess.new

data/lib/nonnative/configuration_process.rb

@@ -2,11 +2,6 @@
 
 module Nonnative
   class ConfigurationProcess
-    attr_accessor :name
-    attr_accessor :command
-    attr_accessor :timeout
-    attr_accessor :port
-    attr_accessor :file
-    attr_accessor :signal
+    attr_accessor :name, :command, :timeout, :port, :file, :signal
   end
 end

data/lib/nonnative/configuration_proxy.rb

@@ -2,8 +2,7 @@
 
 module Nonnative
   class ConfigurationProxy
-    attr_accessor :type
-    attr_accessor :port
+    attr_accessor :type, :port
 
     def initialize
       self.type = 'none'

data/lib/nonnative/configuration_server.rb

@@ -2,14 +2,17 @@
 
 module Nonnative
   class ConfigurationServer
-    attr_accessor :name
-    attr_accessor :klass
-    attr_accessor :timeout
-    attr_accessor :port
-    attr_accessor :proxy
+    attr_accessor :name, :klass, :timeout, :port
+
+    attr_reader :proxy
 
     def initialize
-      self.proxy = Nonnative::ConfigurationProxy.new
+      @proxy = Nonnative::ConfigurationProxy.new
+    end
+
+    def proxy=(value)
+      proxy.type = value[:type]
+      proxy.port = value[:port]
     end
   end
 end

data/lib/nonnative/grpc_server.rb

@@ -4,20 +4,15 @@ module Nonnative
   class GRPCServer < Nonnative::Server
     def initialize(service)
       @server = GRPC::RpcServer.new
+      server.handle(svc)
 
       super service
     end
 
-    def configure(server)
-      # Classes will add configuration
-    end
-
     protected
 
     def perform_start
       server.add_http2_port("0.0.0.0:#{proxy.port}", :this_port_is_insecure)
-      configure server
-
       server.run
     end
 

data/lib/nonnative/pool.rb

@@ -7,13 +7,16 @@ module Nonnative
     end
 
     def start(&block)
-      all = processes + servers
-      process_all(all, :start, :open?, &block)
+      [servers, processes].each { |t| process(t, :start, :open?, &block) }
     end
 
     def stop(&block)
-      all = processes + servers
-      process_all(all, :stop, :closed?, &block)
+      [processes, servers].each { |t| process(t, :stop, :closed?, &block) }
+    end
+
+    def server_by_name(name)
+      index = configuration.servers.find_index { |s| s.name == name }
+      servers[index].first
     end
 
     private
@@ -32,7 +35,7 @@ module Nonnative
       end
     end
 
-    def process_all(all, type_method, port_method, &block)
+    def process(all, type_method, port_method, &block)
       types = []
       pids = []
       threads = []

data/lib/nonnative/server.rb

@@ -2,8 +2,9 @@
 
 module Nonnative
   class Server < Nonnative::Service
+    attr_reader :proxy
+
     def initialize(service)
-      @id = SecureRandom.hex(5)
       @proxy = Nonnative::ProxyFactory.create(service)
 
       super service
@@ -17,7 +18,7 @@ module Nonnative
         wait_start
       end
 
-      id
+      object_id
     end
 
     def stop
@@ -30,15 +31,11 @@ module Nonnative
         wait_stop
       end
 
-      id
+      object_id
     end
 
-    protected
-
-    attr_reader :id
-
     private
 
-    attr_reader :proxy, :thread
+    attr_reader :thread
   end
 end

data/lib/nonnative/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Nonnative
-  VERSION = '1.16.0'
+  VERSION = '1.21.0'
 end

data/nonnative.gemspec

@@ -24,8 +24,9 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
+  spec.add_dependency 'concurrent-ruby', '~> 1.0', '>= 1.0.5'
   spec.add_dependency 'cucumber', '~> 3.1', '>= 3.1.2'
-  spec.add_dependency 'grpc', '~> 1.28'
+  spec.add_dependency 'grpc', ['>= 1', '< 2']
   spec.add_dependency 'puma', '~> 4.3', '>= 4.3.3'
   spec.add_dependency 'rest-client', '~> 2.1'
   spec.add_dependency 'rspec-benchmark', '~> 0.6.0'
@@ -34,9 +35,9 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'bundler', '~> 2.1', '>= 2.1.4'
   spec.add_development_dependency 'chutney', '~> 2.0', '>= 2.0.3.1'
-  spec.add_development_dependency 'grpc-tools', '~> 1.28'
+  spec.add_development_dependency 'grpc-tools', ['>= 1', '< 2']
   spec.add_development_dependency 'rake', '~> 13.0', '>= 13.0.1'
-  spec.add_development_dependency 'rubocop', '~> 0.86.0'
+  spec.add_development_dependency 'rubocop', '~> 0.87.1'
   spec.add_development_dependency 'simplecov', '~> 0.17.1'
-  spec.add_development_dependency 'solargraph', '~> 0.39.7'
+  spec.add_development_dependency 'solargraph', '~> 0.39.11'
 end

metadata

@@ -1,15 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: nonnative
 version: !ruby/object:Gem::Version
-  version: 1.16.0
+  version: 1.21.0
 platform: ruby
 authors:
 - Alex Falkowski
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-07-03 00:00:00.000000000 Z
+date: 2020-07-09 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.5
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.5
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
@@ -34,16 +54,22 @@ dependencies:
   name: grpc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '1.28'
+        version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '1.28'
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: puma
   requirement: !ruby/object:Gem::Requirement
@@ -176,16 +202,22 @@ dependencies:
   name: grpc-tools
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '1.28'
+        version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '1.28'
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -212,14 +244,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.86.0
+        version: 0.87.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.86.0
+        version: 0.87.1
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -240,14 +272,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.39.7
+        version: 0.39.11
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.39.7
+        version: 0.39.11
 description: Allows you to keep using the power of ruby to test other systems
 email:
 - alexrfalkowski@gmail.com
@@ -301,7 +333,7 @@ homepage: https://github.com/alexfalkowski/nonnative
 licenses:
 - Unlicense
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -316,8 +348,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Allows you to keep using the power of ruby to test other systems
 test_files: []