Fix valid_provider! raising ArgumentError instead of InvalidConfigError

lib/roast/cogs/agent/config.rb

@@ -51,7 +51,7 @@ def use_default_provider!
         def valid_provider!
           provider = @values[:provider] || VALID_PROVIDERS.first
           unless VALID_PROVIDERS.include?(provider)
-            raise ArgumentError, "'#{provider}' is not a valid provider. Available providers include: #{VALID_PROVIDERS.join(", ")}"
+            raise InvalidConfigError, "'#{provider}' is not a valid provider. Available providers include: #{VALID_PROVIDERS.join(", ")}"
           end
 
           provider

lib/roast/cogs/chat/config.rb

@@ -67,7 +67,7 @@ def use_default_provider!
         def valid_provider!
           provider = @values[:provider] || PROVIDERS.keys.first
           unless PROVIDERS.include?(provider)
-            raise ArgumentError, "'#{provider}' is not a valid provider. Available providers include: #{PROVIDERS.keys.join(", ")}"
+            raise InvalidConfigError, "'#{provider}' is not a valid provider. Available providers include: #{PROVIDERS.keys.join(", ")}"
           end
 
           provider

test/roast/cogs/agent/config_test.rb

@@ -31,7 +31,7 @@ def setup
       test "valid_provider! raises on invalid provider" do
         @config.provider(:invalid_provider)
 
-        error = assert_raises(ArgumentError) do
+        error = assert_raises(Cog::Config::InvalidConfigError) do
           @config.valid_provider!
         end
 

test/roast/cogs/chat/config_test.rb

@@ -50,7 +50,7 @@ def setup
       test "valid_provider! raises on invalid provider" do
         @config.provider(:invalid_provider)
 
-        error = assert_raises(ArgumentError) do
+        error = assert_raises(Cog::Config::InvalidConfigError) do
           @config.valid_provider!
         end