nonnative 2.21.0 → 2.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 50e3259a9203deec0efaadd273f5dbb0af97707e7226399147e081c57c19e1a9
-  data.tar.gz: 3986e9250edbf08bd3b570610849cc6f6a998dc1773ef14eee7cb9da45771db6
+  metadata.gz: c6a28ed00f9a9f438eb6bb7e8d22d35db3fb782cd474e9fb0208ea0995fb2095
+  data.tar.gz: 1f0e7dca06e10e39b8c85d6a0e6e72173e42e59247b50b125f7b8ef2ecf059f0
 SHA512:
-  metadata.gz: b7d31913722e55932103ab8855d3a01b4fccc358dfec987e16ebc6978cc39f6a34eed25f46d788084a3fa665fe5af9165da4091f8d512064ef550e12aa26c817
-  data.tar.gz: 8f3aa8a30785c105735916ba45275ee3f72d1f1d76edec4399124a262995664e7866901ca18cb7d9b85a7108196b46bf8c76d2376e6505d628f15854c821de9a
+  metadata.gz: b0c6743f7ea67383c12d28d613485eed2297518e28e39212da0c246a9bf1ad368b15dbeacf93946763ff241e9a5bbc179ac2a0493e5ea00d93ab7c6b1c06eeb0
+  data.tar.gz: 1bc6deab711706f351ff546ba776b2952c38452ad05c29897820a08dfede80267673e3bc50048f6ed525edee3025c558b2fb7dcdb1cf7cf50c124cfa86e3a312

data/.circleci/config.yml

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

data/.gitignore

@@ -6,3 +6,4 @@
 test/reports/*
 /tmp/
 vendor
+ISSUES.md

data/AGENTS.md

@@ -25,6 +25,56 @@ of dependencies.
 - Benchmarks only: `make benchmarks`
 - Cleanup: `make clean-dep`, `make clean-reports`
 
+## Intentional Design Choices
+
+- Root `make` compatibility is GNU Make 4+/`gmake` oriented through the shared
+  `bin/build/make/*.mak` fragments. Do not flag GNU Make 3.81 parsing failures
+  as code issues unless the task is explicitly about supporting GNU Make 3.81.
+- The configured HTTP proxy feature intentionally uses the external
+  `www.afalkowski.com` host through `features/support/http_proxy_server.rb`.
+  Do not flag this external dependency as a code issue unless the task is
+  explicitly about making HTTP proxy fixtures fully hermetic.
+- `test/Makefile` intentionally consumes the shared Buf make fragment as-is.
+  Do not flag its inherited `breaking` target's `subdir=api` comparison as a
+  code issue unless the task is explicitly about changing test proto breaking
+  checks.
+- `nonnative` is a test-support gem used across many downstream projects.
+  Do not flag the absence of an isolated gem build/install smoke test as a
+  test gap unless the task is explicitly about release packaging or gem
+  publication validation.
+- The Cucumber `@startup` lifecycle tag is exercised by downstream projects
+  that use this gem. Do not flag missing in-repository `@startup` acceptance
+  coverage as a test gap unless the task is explicitly about changing
+  Cucumber startup hook behavior.
+- `Nonnative.go_argv` and `Nonnative.go_command` flag combinations are checked
+  by external/downstream usage. Do not flag missing in-repository exhaustive
+  `-test.*` flag or tool-filtering assertions as a test gap unless the task is
+  explicitly about changing Go command generation.
+- Generated gRPC test stubs under `test/grpc/` are updated on demand when the
+  test proto changes. Do not flag the absence of an automatic generated-stub
+  freshness check as a test gap unless the task is explicitly about changing
+  test proto generation or generated-file validation.
+- Buf linting for the test-only proto module is intentionally not part of the
+  required local validation surface. Do not flag missing root-level validation
+  for `test/buf.yaml` as a test gap unless the task is explicitly about test
+  proto linting or Buf validation.
+- The public proxy reset Cucumber steps and `@reset` hook are tested by
+  external/downstream suites. Do not flag missing in-repository direct proxy
+  reset step coverage as a test gap unless the task is explicitly about
+  changing proxy reset behavior.
+- `Nonnative.clear` configuration and pool reset behavior is covered by
+  external/downstream suites. Do not flag missing in-repository assertions for
+  configuration or pool clearing as a test gap unless the task is explicitly
+  about changing clear/reset lifecycle behavior.
+- Full repository validation is expected to run both regular features and
+  benchmark-tagged lifecycle scenarios. Do not flag rollback/timeout lifecycle
+  coverage living under `features/benchmark.feature` as a test gap unless the
+  task is explicitly about changing validation target composition.
+- Service proxy round-trip behavior is validated by downstream projects that
+  use `nonnative` against real dependencies. Do not flag the in-repository
+  service proxy success scenario's connection-only assertion as a test gap
+  unless the task is explicitly about changing service proxy forwarding.
+
 ## Runtime Model
 
 Public entry point: `lib/nonnative.rb`.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    nonnative (2.21.0)
+    nonnative (2.22.0)
       concurrent-ruby (>= 1, < 2)
       config (>= 5, < 6)
       cucumber (>= 7, < 12)
@@ -24,7 +24,7 @@ GEM
     benchmark-malloc (0.2.0)
     benchmark-perf (0.6.0)
     benchmark-trend (0.4.0)
-    bigdecimal (4.1.1)
+    bigdecimal (4.1.2)
     builder (3.3.0)
     concurrent-ruby (1.3.6)
     config (5.6.1)
@@ -49,7 +49,7 @@ GEM
       cucumber-tag-expressions (> 6, < 9)
     cucumber-cucumber-expressions (18.1.0)
       bigdecimal
-    cucumber-gherkin (39.0.0)
+    cucumber-gherkin (39.1.0)
       cucumber-messages (>= 31, < 33)
     cucumber-html-formatter (23.1.0)
       cucumber-messages (> 23, < 33)
@@ -79,9 +79,9 @@ GEM
       google-protobuf (>= 3.25, < 5.0)
       googleapis-common-protos-types (~> 1.0)
     http-accept (1.7.0)
-    http-cookie (1.1.4)
+    http-cookie (1.1.6)
       domain_name (~> 0.5)
-    json (2.19.3)
+    json (2.19.5)
     language_server-protocol (3.17.0.5)
     lint_roller (1.1.0)
     logger (1.7.0)
@@ -89,15 +89,14 @@ GEM
     mime-types (3.7.0)
       logger
       mime-types-data (~> 3.2025, >= 3.2025.0507)
-    mime-types-data (3.2026.0407)
+    mime-types-data (3.2026.0414)
     mini_mime (1.1.5)
     multi_test (1.1.0)
-    mustermann (3.0.4)
-      ruby2_keywords (~> 0.0.1)
+    mustermann (3.1.1)
     netrc (0.11.0)
     nio4r (2.7.5)
     ostruct (0.6.3)
-    parallel (2.0.1)
+    parallel (2.1.0)
     parser (3.3.11.1)
       ast (~> 2.4.1)
       racc
@@ -114,7 +113,7 @@ GEM
       base64 (>= 0.1.0)
       rack (>= 3.0.0)
     rainbow (3.1.1)
-    rake (13.3.1)
+    rake (13.4.2)
     rbs (4.0.2)
       logger
       prism (>= 1.6.0)
@@ -147,7 +146,7 @@ GEM
     rspec-support (3.13.7)
     rspec-wait (1.0.2)
       rspec (>= 3.4)
-    rubocop (1.86.1)
+    rubocop (1.86.2)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)
@@ -166,7 +165,6 @@ GEM
       prism (>= 1.2, < 2.0)
       rbs (>= 3, < 5)
     ruby-progressbar (1.13.0)
-    ruby2_keywords (0.0.5)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)

data/lib/nonnative/configuration.rb

@@ -151,7 +151,7 @@ module Nonnative
       servers = cfg.servers || []
       servers.each do |fd|
         server do |s|
-          s.klass = Object.const_get(fd.class)
+          s.klass = Object.const_get(server_class_name(fd))
           runner_attributes(s, fd)
 
           proxy s, fd.proxy
@@ -172,6 +172,11 @@ module Nonnative
       end
     end
 
+    def server_class_name(server)
+      values = server.to_h
+      values[:class] || values.fetch('class')
+    end
+
     def runner_attributes(runner, loaded)
       runner.name = loaded.name
       runner.timeout = loaded.timeout

data/lib/nonnative/process.rb

@@ -41,14 +41,19 @@ module Nonnative
     #
     # The process is signalled using the configured signal (defaults to `INT` when not set).
     #
-    # @return [Integer, nil] the pid that was stopped (or `nil` if the process was never started)
+    # @return [Array<(Integer, Boolean)>]
+    #   a tuple of:
+    #   - the pid that was stopped (or `nil` if the process was never started)
+    #   - whether the process exited before the configured timeout
     def stop
+      stopped = true
+
       if process_exists?
         process_kill
-        wait_stop
+        stopped = !wait_stop.nil?
       end
 
-      pid
+      [pid, stopped]
     ensure
       proxy.stop
     end
@@ -85,10 +90,6 @@ module Nonnative
       environment = service.environment.to_h
       environment = environment.transform_keys(&:to_s).transform_values(&:to_s)
 
-      environment.each_key do |k|
-        environment[k] = ENV.fetch(k, nil) || environment[k]
-      end
-
       command = Array(service.command.call)
 
       spawn(environment, *command, %i[out err] => [service.log, 'a']).tap do |pid|

data/lib/nonnative/version.rb

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

data/lib/nonnative.rb

@@ -238,8 +238,10 @@ module Nonnative
       errors = []
       return if @pool.nil?
 
-      errors.concat(@pool.stop do |name, id, result|
+      errors.concat(@pool.stop do |name, values, result|
+        id, stopped = Array(values).then { |v| [v.first, v.fetch(1, true)] }
         errors << "Stopped #{name} with id #{id}, though did not respond in time" unless result
+        errors << "Stopped #{name} with id #{id}, though the process did not exit in time" unless stopped
       end)
       nil
     rescue StandardError => e
@@ -302,8 +304,10 @@ module Nonnative
       errors = []
       return errors if @pool.nil?
 
-      errors.concat(@pool.rollback do |name, id, result|
+      errors.concat(@pool.rollback do |name, values, result|
+        id, stopped = Array(values).then { |v| [v.first, v.fetch(1, true)] }
         errors << "Rollback failed for #{name} with id #{id}, because it did not stop in time" unless result
+        errors << "Rollback failed for #{name} with id #{id}, because the process did not exit in time" unless stopped
       end)
     rescue StandardError => e
       errors << unexpected_lifecycle_error(:rollback, e)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nonnative
 version: !ruby/object:Gem::Version
-  version: 2.21.0
+  version: 2.22.0
 platform: ruby
 authors:
 - Alejandro Falkowski