nonnative 2.15.0 → 2.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f556739cd561141b772bdeaa9f7cdc8f9934d734bf7517d4e17ef16f2f809f15
-  data.tar.gz: fbba3f9c3f5902d2c385a8e509c27f681f6e44876d02ee7d63b00a66edb19a1d
+  metadata.gz: 9b79bc3b75303cd4d605a4ef81c7b3c1eedd188204d088864fefb65dc73f0922
+  data.tar.gz: c8e4b5ef1bb12213e80d7084d5b3809206279a44bca610947114fd41dd188745
 SHA512:
-  metadata.gz: 65e74315f3bb6e81b1c2797fce743c2f0741a52e2edd40ba608a30e4d582e08b1408008249dd76e81972a438054810390b709f785e6ef455b719ef89658a572c
-  data.tar.gz: 0e43f244c7867e12a512a71aea9f70f2fa0323c62f97f5c417512e131450a7fc714419cb2af0798a692848d5fe95111417ea263b80c1ce5a6f853663a0c03fce
+  metadata.gz: a58c635d40b6eeb8b6aa7e9560900bad247fe768f9839b7e141f6be4e500b5fa2ec11267524621db66989f6eaf24601b790716a4508e28203e4845862ad5a289
+  data.tar.gz: 111995ab26c566d24df0fb037c094883ac07adf2142254fde6822fb9e4fcea59c855196c650bac382e5ab02fe2610181611040a96f3d13515aef586495ffaf72

data/.circleci/config.yml

@@ -3,7 +3,7 @@ version: 2.1
 jobs:
   build:
     docker:
-      - image: alexfalkowski/ruby:2.12
+      - image: alexfalkowski/ruby:3.8
     working_directory: ~/nonnative
     steps:
       - checkout:
@@ -35,7 +35,7 @@ jobs:
     resource_class: arm.large
   sync:
     docker:
-      - image: alexfalkowski/release:7.13
+      - image: alexfalkowski/release:8.8
     working_directory: ~/nonnative
     steps:
       - checkout:
@@ -46,7 +46,7 @@ jobs:
     resource_class: arm.large
   version:
     docker:
-      - image: alexfalkowski/release:7.13
+      - image: alexfalkowski/release:8.8
     working_directory: ~/nonnative
     steps:
       - checkout:
@@ -58,7 +58,7 @@ jobs:
     resource_class: arm.large
   wait-all:
     docker:
-      - image: alexfalkowski/ruby:2.12
+      - image: alexfalkowski/ruby:3.8
     steps:
       - run: echo "all applicable jobs finished"
     resource_class: arm.large

data/AGENTS.md

@@ -1,154 +1,110 @@
 # AGENTS.md
 
-This repo is the `nonnative` Ruby gem: a Ruby-first harness for end-to-end testing of systems implemented in other languages by starting processes/servers/services, waiting on TCP port readiness, and optionally putting fault-injection proxies in front of them.
+## Shared skills
 
-## Shared skill
+This repository uses the shared skills from `bin/skills/`. Read
+`bin/AGENTS.md` for the canonical shared skill list and use the smallest
+matching skill for the task.
 
-Use the shared `coding-standards` skill from `./bin/skills/coding-standards`
-for cross-repository coding, review, testing, documentation, and PR
-conventions. Treat this `AGENTS.md` as the repo-specific companion to that
-skill.
+`nonnative` is a Ruby gem for end-to-end testing systems implemented in other
+languages. It starts processes, in-process servers, and proxy-only services,
+waits on TCP readiness/shutdown, and can place fault-injection proxies in front
+of dependencies.
 
-## Quick map
+## Map And Commands
 
-- Library code: `lib/nonnative/**/*.rb`
-- Acceptance tests: `features/**/*.feature`, `features/support/**/*.rb`, `features/step_definitions/**/*.rb`
-- Generated gRPC Ruby stubs for tests: `test/grpc/**/*`
-- Test proto files: `test/nonnative/v1/*.proto`
+- Library: `lib/nonnative/**/*.rb`
+- Cucumber features/support: `features/**/*.feature`, `features/support/**/*.rb`, `features/step_definitions/**/*.rb`
+- Generated gRPC test stubs: `test/grpc/**/*`
+- Test protos: `test/nonnative/v1/*.proto`
 - Build wiring: root `Makefile` includes `bin/build/make/*.mak`
-
-## Key repo dependency
-
-This repo depends on the `bin/` git submodule.
-
-- `.gitmodules` points to `git@github.com:alexfalkowski/bin.git`
-- CI runs `git submodule sync && git submodule update --init`
-- If `bin/` is missing or you do not have SSH access, `make` targets will fail
-
-## Core commands
-
+- Required submodule: `bin/` from `git@github.com:alexfalkowski/bin.git`; missing SSH/submodule setup breaks `make`
 - Install deps: `make dep`
 - Lint: `make lint`
-- Run features: `make features`
-- Run benchmarks only: `make benchmarks`
-- Clean deps: `make clean-dep`
-- Clean reports: `make clean-reports`
-
-## Runtime model
-
-Public entry point is `lib/nonnative.rb`.
-
-Main module API:
-
-- `configure`
-- `start`
-- `stop`
-- `clear`
-- `reset`
-- `pool`
+- Features: `make features`
+- Benchmarks only: `make benchmarks`
+- Cleanup: `make clean-dep`, `make clean-reports`
 
-Configuration lives in `Nonnative::Configuration` and is built either:
+## Runtime Model
 
-- programmatically with `config.process`, `config.server`, `config.service`
-- from YAML with `config.load_file(...)`
+Public entry point: `lib/nonnative.rb`.
 
-Runtime runners:
+Main API: `configure`, `start`, `stop`, `clear`, `reset`, `pool`.
 
-- `Nonnative::Process`: manages an OS process
-- `Nonnative::Server`: manages an in-process Ruby server thread
-- `Nonnative::Service`: manages only proxy lifecycle for an externally managed dependency
+Configuration is `Nonnative::Configuration`, built with
+`config.process`, `config.server`, `config.service`, or
+`config.load_file(...)`.
 
-`Nonnative::Pool` starts services first, then servers/processes, and stops in the reverse direction.
+Runners:
 
-Readiness and shutdown checks are TCP-only via `Nonnative::Port#open?` and `#closed?`.
+- `Nonnative::Process`: OS process
+- `Nonnative::Server`: in-process Ruby server thread
+- `Nonnative::Service`: proxy lifecycle for an externally managed dependency
 
-## Cucumber integration
+`Nonnative::Pool` starts services first, then servers/processes, and stops in
+reverse. Readiness and shutdown checks are TCP-only via
+`Nonnative::Port#open?` and `#closed?`.
 
-Cucumber integration lives in `lib/nonnative/cucumber.rb`.
+## Cucumber Surface
 
-Treat `lib/nonnative/cucumber.rb` as a public compatibility surface for library consumers.
-Existing hooks and step text should not be removed or renamed unless the user explicitly wants a breaking change.
+`lib/nonnative/cucumber.rb` is public compatibility surface. Do not remove or
+rename hooks/step text unless the user explicitly requests a breaking change.
 
-Supported tags:
+Lifecycle tags:
 
 - `@startup`: start before scenario, stop after scenario
 - `@manual`: scenario starts manually, stop after scenario
 - `@clear`: call `Nonnative.clear` before scenario
 - `@reset`: reset proxies after scenario
 
-Repo-owned feature files also use suite taxonomy tags:
+Suite taxonomy tags: `@acceptance`, `@contract`, `@proxy`, `@config`,
+`@service`, `@benchmark`, `@slow`. `make features` excludes `@benchmark`;
+`make benchmarks` runs only `@benchmark`.
 
-- `@acceptance`: end-to-end runner and client flows
-- `@contract`: lower-level lifecycle / command coverage
-- `@proxy`: proxy-specific coverage
-- `@config`: scenarios or example sets that load YAML/configuration
-- `@service`: coverage centered on external services
-- `@benchmark`: benchmark-only scenarios
-- `@slow`: slower-running scenarios, currently benchmarks
+`Nonnative.clear` clears configuration, logger, observability client, and pool.
+`require 'nonnative'` loads Cucumber integration lazily and is safe outside a
+booted Cucumber runtime. For start-once-per-test-run, use
+`require 'nonnative/startup'`.
 
-`make features` excludes `@benchmark`; `make benchmarks` runs only `@benchmark`.
+## Proxy And Config Gotchas
 
-`Nonnative.clear` now clears:
+Proxy wiring is the easiest mistake:
 
-- configuration
-- logger
-- observability client
-- pool
+- Runner `host` / `port` are client-facing and used for readiness/shutdown
+- For `fault_injection`, nested `proxy.host` / `proxy.port` are the upstream target
+- Clients connect to runner `host` / `port` when a proxy is enabled
 
-`require 'nonnative'` still loads the Cucumber integration, but hook/step registration is lazy, so plain `require 'nonnative'` is safe outside a booted Cucumber runtime.
+Proxy kinds: `none`, `fault_injection`.
+Fault-injection states: `none`, `close_all`, `delay`, `invalid_data`.
 
-For “start once per test run”, use `require 'nonnative/startup'`.
+Config rules:
 
-## Proxy wiring
-
-This is the easiest thing to get wrong.
-
-- Runner `host` / `port` are the client-facing endpoint and the values used by readiness/shutdown checks
-- For `fault_injection`, nested `proxy.host` / `proxy.port` are the upstream target behind the proxy
-- Clients should connect to the runner `host` / `port` when a proxy is enabled
-
-Available proxy kinds:
-
-- `none`
-- `fault_injection`
-
-Fault injection states:
-
-- `none`
-- `close_all`
-- `delay`
-- `invalid_data`
-
-## Config gotchas
-
-- Services must be declared under `services:` in YAML, not `processes:`
+- YAML services belong under `services:`, not `processes:`
 - There is no top-level `config.wait`; `wait` is per runner
-- Service configs use `config.service do |s| ... end`, so use `s.host` / `s.port`
-- Proxy examples need both sides of the split:
-  - runner `host` / `port` = proxy endpoint
-  - nested `proxy.host` / `proxy.port` = upstream target
+- Programmatic service config uses `config.service do |s| ... end`, so use `s.host` / `s.port`
+- Proxy examples need both endpoint sides: runner `host` / `port` for the proxy, nested `proxy.host` / `proxy.port` for upstream
+
+## Fixtures And Limitations
 
-## Test fixtures worth knowing
+Useful fixtures:
 
-- Local process fixture: `features/support/bin/start`
-- HTTP fixtures: `features/support/http_server.rb`, `features/support/http_proxy_server.rb`
-- TCP fixture: `features/support/tcp_server.rb`
-- gRPC fixtures: `features/support/grpc_server.rb`, plus generated stubs under `test/grpc/`
+- Process: `features/support/bin/start`
+- HTTP: `features/support/http_server.rb`, `features/support/http_proxy_server.rb`
+- TCP: `features/support/tcp_server.rb`
+- gRPC: `features/support/grpc_server.rb`, generated stubs in `test/grpc/`
 
-## Important limitations / gotchas
+Limitations:
 
-- Ruby version is constrained by `nonnative.gemspec` to `>= 4.0.0` and `< 5.0.0`
 - The `grpc` Ruby library uses a global logger; per-server gRPC loggers are not really supported
-- `make` depends on the `bin/` submodule being present
 - Local Ruby/Bundler mismatches can break native extensions on macOS
-- Coverage output and Cucumber reports are written under `test/reports`
+- Coverage and Cucumber reports go under `test/reports`
 - Port checks can be flaky if tests reuse ports unexpectedly
 
-## Where to look first
+## Look First
 
-- Lifecycle orchestration: `lib/nonnative.rb`, `lib/nonnative/pool.rb`
-- Readiness / timeouts: `lib/nonnative/port.rb`, `lib/nonnative/timeout.rb`
+- Lifecycle: `lib/nonnative.rb`, `lib/nonnative/pool.rb`
+- Readiness/timeouts: `lib/nonnative/port.rb`, `lib/nonnative/timeout.rb`
 - Process lifecycle: `lib/nonnative/process.rb`
-- Proxies / fault injection: `lib/nonnative/fault_injection_proxy.rb`, `lib/nonnative/socket_pair_factory.rb`
-- Cucumber integration: `lib/nonnative/cucumber.rb`, `lib/nonnative/startup.rb`, `features/support/env.rb`
+- Proxies: `lib/nonnative/fault_injection_proxy.rb`, `lib/nonnative/socket_pair_factory.rb`
+- Cucumber: `lib/nonnative/cucumber.rb`, `lib/nonnative/startup.rb`, `features/support/env.rb`
 - Config loading: `lib/nonnative/configuration.rb`, `lib/nonnative/configuration_runner.rb`, `lib/nonnative/configuration_proxy.rb`

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    nonnative (2.15.0)
+    nonnative (2.16.0)
       concurrent-ruby (>= 1, < 2)
       config (>= 5, < 6)
       cucumber (>= 7, < 12)
@@ -206,4 +206,4 @@ DEPENDENCIES
   simplecov-cobertura
 
 BUNDLED WITH
-  4.0.10
+  4.0.11

data/README.md

@@ -56,6 +56,8 @@ Runner fields (process/server/service):
 
 For `fault_injection`, the nested `proxy.host`/`proxy.port` describe the upstream target behind the proxy. In-process server implementations typically bind there via `proxy.host` / `proxy.port`.
 
+Nonnative readiness and shutdown checks are TCP-only. Configure ports that are dedicated to the test run; if another process is already listening on the same `host`/`port`, results are undefined.
+
 ### Lifecycle strategies (Cucumber integration)
 
 Nonnative ships Cucumber hooks (when loaded) that support these tags/strategies:
@@ -547,9 +549,13 @@ Nonnative.configure do |config|
   config.log = 'nonnative.log'
 
   config.process do |p|
+    p.host = '127.0.0.1'
+    p.port = 20_000
+
     p.proxy = {
       kind: 'fault_injection',
-      port: 20_000,
+      host: '127.0.0.1',
+      port: 12_321,
       log: 'proxy_server.log',
       wait: 1,
       options: {
@@ -569,9 +575,12 @@ url: http://localhost:4567
 log: nonnative.log
 processes:
   -
+    host: 127.0.0.1
+    port: 20000
     proxy:
       kind: fault_injection
-      port: 20000
+      host: 127.0.0.1
+      port: 12321
       log: proxy_server.log
       wait: 1
       options:
@@ -592,9 +601,13 @@ Nonnative.configure do |config|
   config.log = 'nonnative.log'
 
   config.server do |s|
+    s.host = '127.0.0.1'
+    s.port = 20_000
+
     s.proxy = {
       kind: 'fault_injection',
-      port: 20_000,
+      host: '127.0.0.1',
+      port: 12_321,
       log: 'proxy_server.log',
       wait: 1,
       options: {
@@ -614,9 +627,12 @@ url: http://localhost:4567
 log: nonnative.log
 servers:
   -
+    host: 127.0.0.1
+    port: 20000
     proxy:
       kind: fault_injection
-      port: 20000
+      host: 127.0.0.1
+      port: 12321
       log: proxy_server.log
       wait: 1
       options:

data/lib/nonnative/configuration_proxy.rb

@@ -15,8 +15,8 @@ module Nonnative
   # @see Nonnative.proxies
   class ConfigurationProxy
     # @return [String] proxy kind name (for example `"none"` or `"fault_injection"`)
-    # @return [String] proxy bind host (defaults to `"0.0.0.0"`)
-    # @return [Integer] proxy bind port (defaults to `0`)
+    # @return [String] upstream host used by proxy implementations (defaults to `"0.0.0.0"`)
+    # @return [Integer] upstream port used by proxy implementations (defaults to `0`)
     # @return [String, nil] path to proxy log file (implementation-dependent)
     # @return [Numeric] wait interval (seconds) after proxy state changes (defaults to `0.1`)
     # @return [Hash] proxy implementation options (implementation-dependent)

data/lib/nonnative/configuration_runner.rb

@@ -51,8 +51,8 @@ module Nonnative
     #
     # @param value [Hash] proxy attributes
     # @option value [String] :kind proxy kind name (for example `"fault_injection"`)
-    # @option value [String] :host proxy bind host (optional)
-    # @option value [Integer] :port proxy bind port
+    # @option value [String] :host upstream host behind the proxy (optional)
+    # @option value [Integer] :port upstream port behind the proxy
     # @option value [String] :log proxy log file path
     # @option value [Numeric] :wait wait interval (seconds) after state changes (optional)
     # @option value [Hash] :options proxy implementation specific options

data/lib/nonnative/cucumber.rb

@@ -32,6 +32,13 @@ module Nonnative
     end
 
     module ProxySteps
+      PROXY_OPERATIONS = {
+        'close_all' => :close_all,
+        'delay' => :delay,
+        'invalid_data' => :invalid_data,
+        'reset' => :reset
+      }.freeze
+
       def install_proxy_steps
         install_proxy_mutation_steps
         install_proxy_reset_steps
@@ -40,17 +47,17 @@ module Nonnative
       def install_proxy_mutation_steps
         Given('I set the proxy for process {string} to {string}') do |name, operation|
           process = Nonnative.pool.process_by_name(name)
-          process.proxy.send(operation)
+          Nonnative::Cucumber::Registration.apply_proxy_operation(process.proxy, operation)
         end
 
         Given('I set the proxy for server {string} to {string}') do |name, operation|
           server = Nonnative.pool.server_by_name(name)
-          server.proxy.send(operation)
+          Nonnative::Cucumber::Registration.apply_proxy_operation(server.proxy, operation)
         end
 
         Given('I set the proxy for service {string} to {string}') do |name, operation|
           service = Nonnative.pool.service_by_name(name)
-          service.proxy.send(operation)
+          Nonnative::Cucumber::Registration.apply_proxy_operation(service.proxy, operation)
         end
       end
 
@@ -70,9 +77,19 @@ module Nonnative
           service.proxy.reset
         end
       end
+
+      def apply_proxy_operation(proxy, operation)
+        method = PROXY_OPERATIONS.fetch(operation) do
+          raise ArgumentError, "Unsupported proxy operation '#{operation}'"
+        end
+
+        proxy.public_send(method)
+      end
     end
 
     module LifecycleSteps
+      SERVICE_UNAVAILABLE = 'Service Unavailable'
+
       def install_state_steps
         install_start_step
         install_attempt_start_step
@@ -110,7 +127,11 @@ module Nonnative
 
         Then('I should see {string} as unhealthy') do |service|
           wait_for { Nonnative.observability.health(opts).code }.to eq(503)
-          wait_for { Nonnative.observability.health(opts).body }.to include(service)
+          wait_for { Nonnative.observability.health(opts).body }.to satisfy do |body|
+            body = body.to_s.strip
+
+            body == SERVICE_UNAVAILABLE || body.include?(service)
+          end
         end
       end
 
@@ -119,7 +140,11 @@ module Nonnative
 
         Then('I should see {string} as healthy') do |service|
           wait_for { Nonnative.observability.health(opts).code }.to eq(200)
-          wait_for { Nonnative.observability.health(opts).body }.to_not include(service)
+          wait_for { Nonnative.observability.health(opts).body }.to satisfy do |body|
+            body = body.to_s.strip
+
+            body != SERVICE_UNAVAILABLE && !body.include?(service)
+          end
         end
       end
 

data/lib/nonnative/version.rb

@@ -4,5 +4,5 @@ module Nonnative
   # The current gem version.
   #
   # @return [String]
-  VERSION = '2.15.0'
+  VERSION = '2.16.0'
 end

data/lib/nonnative.rb

@@ -189,7 +189,7 @@ module Nonnative
     #
     # @return [Hash{String=>Class}]
     def proxies
-      @proxies ||= { 'fault_injection' => Nonnative::FaultInjectionProxy }.freeze
+      @proxies ||= { 'fault_injection' => Nonnative::FaultInjectionProxy }
     end
 
     # Resolves a proxy implementation for a configured kind.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nonnative
 version: !ruby/object:Gem::Version
-  version: 2.15.0
+  version: 2.16.0
 platform: ruby
 authors:
 - Alejandro Falkowski
@@ -345,7 +345,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.11
 specification_version: 4
 summary: Allows you to keep using the power of ruby to test other systems
 test_files: []