nonnative 2.8.0 → 2.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7370ea5a582e82e738d71cbffc4a3ad847e8050165b02d33f1c18301212ace2
-  data.tar.gz: d3c139dd5cb598c013196f8362b38929783910e38fd4c45227e0093e28626eb8
+  metadata.gz: 304dcea40b84eda206fe9964fe1479a54e0188f52f2a5300dbcf992101cab2b3
+  data.tar.gz: a6adb1b816c424709ab337d1b845dd106ff5141ddb6f26bfe73318c4f51bed23
 SHA512:
-  metadata.gz: 171e964df36ea5434a138fa0f01bfede21b7ad4b7708246b395e22eb3b373c9dda61299d72a663a759cd200572432d4ee7787f50de224808fd8a01c32315aa4e
-  data.tar.gz: 780c202b553579a078bdbcc0370d3d8641ea2c7a6268c7ab53182db970f1a299b8853fa8cf5ed6fd59ccd15bb46d63826c2adc5ef69bb010b661e4aba10a6971
+  metadata.gz: a7d8afad5ad7ff9b550c67fe73a683da85ae1bcfa611e0b25b0d570f44ebd5beba47ab50287d58206a51d6d61722bf42ae893929d2c4a99d5ea53b2e033fae91
+  data.tar.gz: 506907d924a16c49d583a91e78a7c6e17bb03e3e5d879f00f43e8d4313fb57957bf545538d2953d056ab0c3eecb94ff29ab9a1af22547c890c1a1e56e3f0335c

data/.circleci/config.yml

@@ -25,7 +25,8 @@ jobs:
             - vendor
       - run: make lint
       - run: make sec
-      - run: make features
+      - run: COVERAGE_NAME="Features" make features
+      - run: COVERAGE_NAME="Benchmarks" make benchmarks
       - store_test_results:
           path: test/reports
       - store_artifacts:

data/AGENTS.md

@@ -59,6 +59,9 @@ Readiness and shutdown checks are TCP-only via `Nonnative::Port#open?` and `#clo
 
 Cucumber integration lives in `lib/nonnative/cucumber.rb`.
 
+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.
+
 Supported tags:
 
 - `@startup`: start before scenario, stop after scenario
@@ -66,6 +69,18 @@ Supported tags:
 - `@clear`: call `Nonnative.clear` before scenario
 - `@reset`: reset proxies after scenario
 
+Repo-owned feature files also use suite taxonomy tags:
+
+- `@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
+
+`make features` excludes `@benchmark`; `make benchmarks` runs only `@benchmark`.
+
 `Nonnative.clear` now clears:
 
 - configuration
@@ -115,7 +130,7 @@ Fault injection states:
 
 ## Important limitations / gotchas
 
-- Ruby version is constrained by `nonnative.gemspec` to `>= 3.4.0` and `< 4.0.0`
+- 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

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    nonnative (2.8.0)
+    nonnative (2.11.0)
       concurrent-ruby (>= 1, < 2)
       config (>= 5, < 6)
       cucumber (>= 7, < 11)

data/README.md

@@ -64,6 +64,17 @@ Nonnative ships Cucumber hooks (when loaded) that support these tags/strategies:
 - `@clear`: clears memoized configuration, logger, observability client, and pool before scenario
 - `@reset`: resets proxies after scenario
 
+The repo’s own Cucumber suite also uses taxonomy tags to classify coverage:
+- `@acceptance`: end-to-end behavior across configured runners and clients
+- `@contract`: lower-level contract and lifecycle behavior
+- `@proxy`: proxy-specific behavior and failure injection
+- `@config`: coverage that exercises YAML/config loading
+- `@service`: scenarios centered on externally managed dependencies
+- `@benchmark`: benchmark-only scenarios run by `make benchmarks`
+- `@slow`: slower scenarios, currently used by benchmark coverage
+
+`make features` excludes `@benchmark`, while `make benchmarks` runs only `@benchmark`.
+
 Requiring `nonnative` is enough; the Cucumber hooks and step definitions are installed lazily once Cucumber’s Ruby DSL is ready.
 
 If you want “start once per test run”, require:

data/lib/nonnative/cucumber.rb

@@ -75,20 +75,40 @@ module Nonnative
     module LifecycleSteps
       def install_state_steps
         install_start_step
+        install_attempt_start_step
+        install_attempt_stop_step
         install_unhealthy_step
         install_healthy_step
       end
 
       def install_start_step
-        Given('I start the system') do
+        When('I start the system') do
           Nonnative.start
         end
       end
 
+      def install_attempt_start_step
+        When('I attempt to start the system') do
+          @start_error = nil
+          Nonnative.start
+        rescue StandardError => e
+          @start_error = e
+        end
+      end
+
+      def install_attempt_stop_step
+        When('I attempt to stop the system') do
+          @stop_error = nil
+          Nonnative.stop
+        rescue StandardError => e
+          @stop_error = e
+        end
+      end
+
       def install_unhealthy_step
         opts = observability_options
 
-        Given('I should see {string} as unhealthy') do |service|
+        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)
         end
@@ -132,11 +152,11 @@ module Nonnative
 
       def install_error_assertion_steps
         Then('starting the system should raise an error') do
-          expect { Nonnative.start }.to raise_error(Nonnative::StartError)
+          expect(@start_error).to be_a(Nonnative::StartError)
         end
 
         Then('stopping the system should raise an error') do
-          expect { Nonnative.stop }.to raise_error(Nonnative::StopError)
+          expect(@stop_error).to be_a(Nonnative::StopError)
         end
       end
 

data/lib/nonnative/invalid_data_socket_pair.rb

@@ -3,8 +3,8 @@
 module Nonnative
   # Socket-pair variant used by the fault-injection proxy to simulate corrupted/incoherent traffic.
   #
-  # When active, data written to the upstream socket is corrupted by shuffling the characters in the
-  # payload before forwarding.
+  # When active, data written to the upstream socket is corrupted by shuffling the payload bytes
+  # before forwarding.
   #
   # This behavior is enabled by calling {Nonnative::FaultInjectionProxy#invalid_data}.
   #
@@ -12,7 +12,10 @@ module Nonnative
   # @see Nonnative::SocketPairFactory
   # @see Nonnative::SocketPair
   class InvalidDataSocketPair < SocketPair
-    # Writes corrupted data to the socket by shuffling characters.
+    # Writes corrupted data to the socket by shuffling bytes.
+    #
+    # The payload must always change, otherwise short or repetitive inputs such as "test" can
+    # occasionally pass through unchanged and make fault-injection scenarios flaky.
     #
     # @param socket [IO] the socket to write to
     # @param data [String] the original payload
@@ -20,9 +23,46 @@ module Nonnative
     def write(socket, data)
       Nonnative.logger.info "shuffling socket data '#{socket.inspect}' for 'invalid_data' pair"
 
-      data = data.chars.shuffle.join
+      super(socket, corrupt(data))
+    end
+
+    private
+
+    def corrupt(data)
+      payload, delimiter = split_trailing_delimiter(data)
+      return "\0#{delimiter}" if payload.empty?
+
+      "#{corrupt_payload(payload)}#{delimiter}"
+    end
+
+    def split_trailing_delimiter(data)
+      index = trailing_delimiter_start(data)
+      payload = data.byteslice(0, index)
+      delimiter = data.byteslice(index, data.bytesize - index) || ''
+
+      [payload, delimiter]
+    end
+
+    def trailing_delimiter_start(data)
+      index = data.bytesize
+
+      while index.positive?
+        byte = data.getbyte(index - 1)
+        break unless [10, 13].include?(byte)
+
+        index -= 1
+      end
+
+      index
+    end
+
+    def corrupt_payload(payload)
+      bytes = payload.bytes
+      corrupted = bytes.shuffle
+      return corrupted.pack('C*') unless corrupted == bytes
 
-      super
+      corrupted[0] = (corrupted[0] + 1) % 256
+      corrupted.pack('C*')
     end
   end
 end

data/lib/nonnative/version.rb

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

metadata

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