sdn_test_simple_car_repository 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4021d729ce417b6ba742100c32e43c24eee66bfdec345765739a8c9525ac21ae
-  data.tar.gz: d2d6b40b45f659e5ea688a12029fd155fabea3efe157b8e584c82e17bbc42bf3
+  metadata.gz: bac9e4f15496700b39ee469b8f9d355df3cebc6c843582b84c633cad3565a55b
+  data.tar.gz: 52527ecd6eed3edbf59e03cbafd5f1cab28cfff5d410de841c50cf9c3594e373
 SHA512:
-  metadata.gz: b5592af1e3ace226346df4c3b26bd5b207dd8b771b9454d5023c1f870a118d2601cf44a5d09c7b7e1d759172da84fe347fc03559012bf8f23113573307825652
-  data.tar.gz: df63c67dddc510a35f6f29b503a34d225e557992a5291a6ecca63750be5d4a7e37114f0a5dbddb4106cf469a0923a70562f39686a2e6e5bb068033996b9126b5
+  metadata.gz: 1ee39ffc8afb3683c9a788c907a4db72ea1593d84d88821759672af761197ce2a90e49d0f4c12008bf23ca181aafb99bf84a0cb9bcdb817722f8e7f7f37b844c
+  data.tar.gz: d524e07a88a20428c12090328d47a4cc1223de8b8da17838b9db5fe2aa2783d74d3b4a621e5ee6e7852011062fe474d80a89c07c1dcb4fbbd9a3f0d4ceb3b084

data/.rspec

@@ -1,3 +1,3 @@
---format documentation
---color
---require spec_helper
+--format documentation
+--color
+--require spec_helper

data/.rubocop.yml

@@ -1,8 +1,8 @@
-AllCops:
-  TargetRubyVersion: 3.1
-
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
-
-Style/StringLiteralsInInterpolation:
-  EnforcedStyle: double_quotes
+AllCops:
+  TargetRubyVersion: 3.1
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
+Style/StringLiteralsInInterpolation:
+  EnforcedStyle: double_quotes

data/LICENSE.txt

@@ -1,21 +1,21 @@
-The MIT License (MIT)
-
-Copyright (c) 2026 Graftcode
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
+The MIT License (MIT)
+
+Copyright (c) 2026 Graftcode
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

data/Rakefile

@@ -1,12 +1,12 @@
-# frozen_string_literal: true
-
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-
-RSpec::Core::RakeTask.new(:spec)
-
-require "rubocop/rake_task"
-
-RuboCop::RakeTask.new
-
-task default: %i[spec rubocop]
+# frozen_string_literal: true
+
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+
+RSpec::Core::RakeTask.new(:spec)
+
+require "rubocop/rake_task"
+
+RuboCop::RakeTask.new
+
+task default: %i[spec rubocop]

data/grft_test_simple_car_repository.gemspec

@@ -1,30 +1,30 @@
-# frozen_string_literal: true
-
-require_relative "lib/grft_test_simple_car_repository/version"
-
-Gem::Specification.new do |spec|
-  spec.name = "sdn_test_simple_car_repository"
-  spec.version = GrftTestSimpleCarRepository::VERSION
-  spec.authors = ["GraftCode"]
-  spec.email = ["info@graftcode.com"]
-
-  spec.summary = "Simple car in-memory repository"
-  spec.description = "Provides an in-memory repository for SimpleCar objects with add/get/update/delete operations and sample seed data."
-  spec.homepage = "https://graftcode.com"
-  spec.license = "MIT"
-  spec.required_ruby_version = ">= 3.1.0"
-
-  spec.metadata["homepage_uri"] = spec.homepage
-
-  gemspec = File.basename(__FILE__)
-  spec.files = IO.popen(%w[git ls-files -z], chdir: __dir__, err: IO::NULL) do |ls|
-    ls.readlines("\x0", chomp: true).reject do |f|
-      f.start_with?(*%w[bin/ test/ spec/ features/ .git appveyor Gemfile])
-    end
-  end
-  spec.bindir = "exe"
-  spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  spec.add_dependency "sdn_test_simple_car", ">= 0.2.1"
-end
+# frozen_string_literal: true
+
+require_relative "lib/grft_test_simple_car_repository/version"
+
+Gem::Specification.new do |spec|
+  spec.name = "sdn_test_simple_car_repository"
+  spec.version = GrftTestSimpleCarRepository::VERSION
+  spec.authors = ["GraftCode"]
+  spec.email = ["info@graftcode.com"]
+
+  spec.summary = "Simple car in-memory repository"
+  spec.description = "Provides an in-memory repository for SimpleCar objects with add/get/update/delete operations and sample seed data."
+  spec.homepage = "https://graftcode.com"
+  spec.license = "MIT"
+  spec.required_ruby_version = ">= 3.1.0"
+
+  spec.metadata["homepage_uri"] = spec.homepage
+
+  gemspec = File.basename(__FILE__)
+  spec.files = IO.popen(%w[git ls-files -z], chdir: __dir__, err: IO::NULL) do |ls|
+    ls.readlines("\x0", chomp: true).reject do |f|
+      f.start_with?(*%w[bin/ test/ spec/ features/ .git appveyor Gemfile])
+    end
+  end
+  spec.bindir = "exe"
+  spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency "sdn_test_simple_car", ">= 0.2.1"
+end

data/lib/grft_test_simple_car_repository/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GrftTestSimpleCarRepository
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

data/lib/in_memory_simple_car_repository.rb

@@ -1,118 +1,193 @@
-# frozen_string_literal: true
-
-require "grft_test_simple_car"
-
-# An in-memory repository that stores SimpleCar instances keyed by string ID.
-class InMemorySimpleCarRepository
-  # ── static members ──────────────────────────────────────────────
-
-  DEFAULT_CAPACITY = 100
-
-  @total_instances_created = 0
-
-  class << self
-    attr_accessor :total_instances_created
-  end
-
-  def self.create_with_sample_data
-    new
-  end
-
-  # ── constructor ─────────────────────────────────────────────────
-
-  def initialize
-    self.class.instance_variable_set(:@total_instances_created, self.class.total_instances_created + 1)
-    @store = {}
-
-    samples = [
-      SimpleCar.new("Toyota", "Corolla", 2018, "0"),
-      SimpleCar.new("Tesla", "Model 3", 2021, "1"),
-      SimpleCar.new("Ford", "Mustang", 1967, "2"),
-      SimpleCar.new("Honda", "Civic", 2015, "3"),
-      SimpleCar.new("Chevrolet", "Camaro", 2020, "4")
-    ]
-
-    samples.each { |car| add(car) }
-  end
-
-  # ── CRUD ────────────────────────────────────────────────────────
-
-  def add(car)
-    raise ArgumentError, "car cannot be nil" if car.nil?
-    raise ArgumentError, "SimpleCar with Id #{car.id} already exists." if @store.key?(car.id)
-
-    @store[car.id] = car
-  end
-
-  def add_range(cars)
-    raise ArgumentError, "cars cannot be nil" if cars.nil?
-
-    cars.each { |car| add(car) }
-  end
-
-  def get(id)
-    raise ArgumentError, "id cannot be nil or whitespace" if id.nil? || id.strip.empty?
-
-    @store[id]
-  end
-
-  def get_all
-    @store.values
-  end
-
-  def update(car)
-    raise ArgumentError, "car cannot be nil" if car.nil?
-
-    return false unless @store.key?(car.id)
-
-    @store[car.id] = car
-    true
-  end
-
-  def delete(id)
-    raise ArgumentError, "id cannot be nil or whitespace" if id.nil? || id.strip.empty?
-
-    return false unless @store.key?(id)
-
-    @store.delete(id)
-    true
-  end
-
-  # ── queries returning complex types ─────────────────────────────
-
-  def find_by_make(make)
-    raise ArgumentError, "make cannot be nil or whitespace" if make.nil? || make.strip.empty?
-
-    lower_make = make.downcase
-    @store.values.select { |c| c.make.downcase == lower_make }
-  end
-
-  def find_by_year_range(min_year, max_year)
-    @store.values.select { |c| c.year >= min_year && c.year <= max_year }
-  end
-
-  # ── queries returning primitive arrays ──────────────────────────
-
-  def get_years
-    @store.values.map(&:year).uniq.sort
-  end
-
-  def get_makes
-    @store.values.map(&:make).uniq.sort
-  end
-
-  # ── bulk operations with primitive-array input ──────────────────
-
-  def delete_by_ids(ids)
-    raise ArgumentError, "ids cannot be nil" if ids.nil?
-
-    count = 0
-    ids.each do |id|
-      if @store.key?(id)
-        @store.delete(id)
-        count += 1
-      end
-    end
-    count
-  end
-end
+# frozen_string_literal: true
+
+require "grft_test_simple_car"
+
+# An in-memory, thread-safe repository that stores SimpleCar instances using a string identifier.
+#
+# The repository pre-loads five sample cars on construction so that graft smoke tests can issue
+# queries immediately without seeding data.
+class InMemorySimpleCarRepository
+  # ── static members ──────────────────────────────────────────
+
+  # Default capacity hint for the repository.
+  DEFAULT_CAPACITY = 100
+
+  @total_instances_created = 0
+
+  class << self
+    # @return [Integer] Total number of repository instances created so far.
+    attr_accessor :total_instances_created
+  end
+
+  # Returns a new repository pre-loaded with sample data.
+  #
+  # @return [InMemorySimpleCarRepository] Repository containing the default sample cars used across graft tests.
+  # @example
+  #   repo = InMemorySimpleCarRepository.create_with_sample_data
+  #   repo.get_all.size #=> 5
+  def self.create_with_sample_data
+    new
+  end
+
+  # ── constructor ─────────────────────────────────────────────
+
+  # Initializes the repository and seeds it with five sample cars.
+  #
+  # The samples cover a mix of makes and production years so that range and make-based queries
+  # return non-trivial results out of the box.
+  #
+  # @author graftcode
+  # @since 0.2.1
+  def initialize
+    self.class.instance_variable_set(:@total_instances_created, self.class.total_instances_created + 1)
+    @store = {}
+
+    samples = [
+      SimpleCar.new("Toyota", "Corolla", 2018, "0"),
+      SimpleCar.new("Tesla", "Model 3", 2021, "1"),
+      SimpleCar.new("Ford", "Mustang", 1967, "2"),
+      SimpleCar.new("Honda", "Civic", 2015, "3"),
+      SimpleCar.new("Chevrolet", "Camaro", 2020, "4")
+    ]
+
+    samples.each { |car| add(car) }
+  end
+
+  # ── CRUD ────────────────────────────────────────────────────
+
+  # Adds a new car to the repository.
+  #
+  # The check for duplicates and the insert happen atomically, so concurrent calls cannot create
+  # two entries for the same id.
+  #
+  # @param car [SimpleCar] Car to add; must not be nil.
+  # @raise [ArgumentError] If car is nil or a car with the same id already exists.
+  # @return [void]
+  def add(car)
+    raise ArgumentError, "car cannot be nil" if car.nil?
+    raise ArgumentError, "SimpleCar with Id #{car.id} already exists." if @store.key?(car.id)
+
+    @store[car.id] = car
+  end
+
+  # Adds a range of simple cars to the repository.
+  #
+  # @param cars [Array<SimpleCar>] Array of cars to add.
+  # @raise [ArgumentError] If cars is nil or any car has an id that already exists.
+  # @return [void]
+  def add_range(cars)
+    raise ArgumentError, "cars cannot be nil" if cars.nil?
+
+    cars.each { |car| add(car) }
+  end
+
+  # Gets a simple car by its string identifier.
+  #
+  # @param id [String] The simple car identifier.
+  # @return [SimpleCar, nil] The matching SimpleCar if found; otherwise nil.
+  # @raise [ArgumentError] If id is nil or whitespace.
+  def get(id)
+    raise ArgumentError, "id cannot be nil or whitespace" if id.nil? || id.strip.empty?
+
+    @store[id]
+  end
+
+  # Returns all simple cars in the repository.
+  #
+  # @return [Array<SimpleCar>] A list of SimpleCar.
+  def get_all
+    @store.values
+  end
+
+  # Updates an existing simple car in the repository.
+  #
+  # @param car [SimpleCar] The simple car with updated data.
+  # @return [Boolean] true if the car was updated; otherwise false (not found).
+  # @raise [ArgumentError] If car is nil.
+  def update(car)
+    raise ArgumentError, "car cannot be nil" if car.nil?
+
+    return false unless @store.key?(car.id)
+
+    @store[car.id] = car
+    true
+  end
+
+  # Deletes a simple car by its string identifier.
+  #
+  # @param id [String] The simple car identifier.
+  # @return [Boolean] true if the car was removed; otherwise false.
+  # @raise [ArgumentError] If id is nil or whitespace.
+  def delete(id)
+    raise ArgumentError, "id cannot be nil or whitespace" if id.nil? || id.strip.empty?
+
+    return false unless @store.key?(id)
+
+    @store.delete(id)
+    true
+  end
+
+  # ── queries returning complex types ─────────────────────────
+
+  # Finds every car whose manufacturer matches make (case-insensitive).
+  #
+  # @param make [String] Manufacturer name to match; must not be empty.
+  # @return [Array<SimpleCar>] Matching cars; empty array when none match.
+  # @raise [ArgumentError] If make is nil or whitespace.
+  def find_by_make(make)
+    raise ArgumentError, "make cannot be nil or whitespace" if make.nil? || make.strip.empty?
+
+    lower_make = make.downcase
+    @store.values.select { |c| c.make.downcase == lower_make }
+  end
+
+  # Finds all simple cars whose production year falls within the inclusive range.
+  #
+  # @param min_year [Integer] Minimum year (inclusive).
+  # @param max_year [Integer] Maximum year (inclusive).
+  # @return [Array<SimpleCar>] Cars within the given year range.
+  def find_by_year_range(min_year, max_year)
+    @store.values.select { |c| c.year >= min_year && c.year <= max_year }
+  end
+
+  # ── queries returning primitive arrays ──────────────────────
+
+  # Returns all distinct production years currently stored, ascending.
+  #
+  # @return [Array<Integer>] Sorted list of distinct years.
+  def get_years
+    @store.values.map(&:year).uniq.sort
+  end
+
+  # Returns every distinct manufacturer present in the repository.
+  #
+  # @deprecated Prefer querying makes from a domain-specific service in new code.
+  # @return [Array<String>] Sorted list of unique manufacturer names.
+  def get_makes
+    @store.values.map(&:make).uniq.sort
+  end
+
+  # ── bulk operations with primitive-array input ──────────────
+
+  # Removes every car whose id is listed in ids.
+  #
+  # Missing ids are silently skipped; the return value reports how many entries were actually removed.
+  #
+  # @param ids [Array<String>] Identifiers to remove; nil is rejected.
+  # @return [Integer] Number of cars actually removed.
+  # @raise [ArgumentError] If ids is nil.
+  # @author graftcode
+  # @since 0.2.1
+  def delete_by_ids(ids)
+    raise ArgumentError, "ids cannot be nil" if ids.nil?
+
+    count = 0
+    ids.each do |id|
+      if @store.key?(id)
+        @store.delete(id)
+        count += 1
+      end
+    end
+    count
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sdn_test_simple_car_repository
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - GraftCode