Update documentation and tests

Author: grepfruit19

README.md

@@ -12,12 +12,11 @@ We support multiple data types for flags (numbers, strings, booleans, objects) a
 
 ## Support Matrix
 
-| Ruby Version | OS                          |
-| -----------  | -----------                 |
-| Ruby 3.1.4   | Windows, MacOS, Linux       |
-| Ruby 3.2.3   | Windows, MacOS, Linux       |
-| Ruby 3.3.0   | Windows, MacOS, Linux       |
-
+| Ruby Version | OS                    |
+| ------------ | --------------------- |
+| Ruby 3.1.4   | Windows, MacOS, Linux |
+| Ruby 3.2.3   | Windows, MacOS, Linux |
+| Ruby 3.3.0   | Windows, MacOS, Linux |
 
 ## Installation
 
@@ -55,6 +54,8 @@ end
 
 # Create a client
 client = OpenFeature::SDK.build_client(name: "my-app")
+# Create a client for a different domain, this will use the provider assigned to that domain
+legacy_flag_client = OpenFeature::SDK.build_client(name: "Legacy", domain: "legacy_flags")
 
 # fetching boolean value feature flag
 bool_value = client.fetch_boolean_value(flag_key: 'boolean_flag', default_value: false)
@@ -102,7 +103,6 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for details on how to contribute to the O
 
 Our community meetings are held regularly and open to everyone. Check the [OpenFeature community calendar](https://calendar.google.com/calendar/u/0?cid=MHVhN2kxaGl2NWRoMThiMjd0b2FoNjM2NDRAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ) for specific dates and for the Zoom meeting links.
 
-
 ## License
 
 [Apache License 2.0](LICENSE)

lib/open_feature/sdk/api.rb

@@ -43,9 +43,11 @@ def configure(&block)
       end
 
       def build_client(name: nil, version: nil, domain: nil)
-        client_options = Metadata.new(name: name, version: version).freeze
+        client_options = Metadata.new(name: name, version: version, domain: domain).freeze
 
+        pp domain
         active_provider = provider(domain:).nil? ? Provider::NoOpProvider.new : provider(domain:)
+        pp active_provider
 
         Client.new(provider: active_provider, client_options: client_options, context: context)
       end

lib/open_feature/sdk/client.rb

@@ -8,7 +8,7 @@ class Client
       RESULT_TYPE = %i[boolean string number object].freeze
       SUFFIXES = %i[value details].freeze
 
-      attr_reader :metadata
+      attr_reader :metadata, :provider
 
       attr_accessor :hooks
 

lib/open_feature/sdk/metadata.rb

@@ -10,19 +10,22 @@ module SDK
     #
     # * <tt>version</tt> - Allows you to specify version of the Metadata structure
     #
+    # * <tt>domain</tt> - Allows you to specify the domain of the Metadata structure
+    #
     # Usage:
     #
-    #   metadata = Metadata.new(name: 'name-for-metadata', version: 'v1.1.3')
+    #   metadata = Metadata.new(name: 'name-for-metadata', version: 'v1.1.3', domain: 'test')
     #   metadata.name # 'name-for-metadata'
     #   metadata.version # version
     #   metadata_two = Metadata.new(name: 'name-for-metadata')
     #   metadata_two == metadata # true - equality based on values
     class Metadata
-      attr_reader :name, :version
+      attr_reader :name, :version, :domain
 
-      def initialize(name:, version: nil)
+      def initialize(name:, version: nil, domain: nil)
         @name = name
         @version = version
+        @domain = domain
       end
 
       def ==(other)

spec/open_feature/sdk/api_spec.rb

@@ -70,4 +70,44 @@
   context "with Requirement 1.1.6" do
     pending
   end
+
+  context "when domain is given" do
+    it "can generate a client both with and without that domain" do
+      provider = OpenFeature::SDK::Provider::InMemoryProvider.new
+
+      api.configure do |config|
+        config.set_provider(provider, domain: 'testing1')
+      end
+
+      client = api.build_client(name: "with-domain", domain: "testing1")
+      no_domain_client = api.build_client(name: "without-domain")
+
+      expect(client.provider).to be(provider)
+      expect(no_domain_client.provider).to be_an_instance_of(OpenFeature::SDK::Provider::NoOpProvider)
+    end
+  end
+
+  context "when domain is not provided" do
+    it "can generate a client without a domain properly" do
+      provider = OpenFeature::SDK::Provider::InMemoryProvider.new
+
+      api.configure do |config|
+        config.set_provider(provider)
+      end
+
+      no_domain_client = api.build_client(name: "without-domain")
+
+      expect(no_domain_client.provider).to be(provider)
+    end
+
+    it "can generate a client with a domain properly" do
+      api.configure do |config|
+        config.set_provider(OpenFeature::SDK::Provider::InMemoryProvider.new)
+      end
+
+      domain_client = api.build_client(name: "with-domain", domain: "testing2")
+      # This domain was never given a provider, so it should default to the NoOpProvider
+      expect(domain_client.provider).to be_an_instance_of(OpenFeature::SDK::Provider::NoOpProvider)
+    end
+  end
 end