Add update_connection_pool action

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2024-05-17 08:19:08 UTC using RuboCop version 1.63.5.
+# on 2024-10-05 16:27:38 UTC using RuboCop version 1.66.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -33,7 +33,7 @@ Lint/UnderscorePrefixedVariableName:
   Exclude:
     - 'spec/lib/droplet_kit/client_spec.rb'
 
-# Offense count: 35
+# Offense count: 36
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AutoCorrect, IgnoreEmptyBlocks, AllowUnusedKeywordArguments.
 Lint/UnusedBlockArgument:
@@ -55,12 +55,12 @@ Metrics/AbcSize:
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns, inherit_mode.
 # AllowedMethods: refine
 Metrics/BlockLength:
-  Max: 127
+  Max: 131
 
 # Offense count: 1
 # Configuration parameters: CountComments, CountAsOne.
 Metrics/ClassLength:
-  Max: 133
+  Max: 137
 
 # Offense count: 6
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
@@ -90,7 +90,7 @@ Naming/VariableNumber:
     - 'spec/lib/droplet_kit/resources/firewall_resource_spec.rb'
     - 'spec/lib/droplet_kit/resources/load_balancer_resource_spec.rb'
 
-# Offense count: 33
+# Offense count: 34
 # Configuration parameters: Prefixes, AllowedPatterns.
 # Prefixes: when, with, without
 RSpec/ContextWording:
@@ -110,17 +110,11 @@ RSpec/ContextWording:
     - 'spec/lib/droplet_kit/resources/vpc_resource_spec.rb'
     - 'spec/support/resource_context.rb'
 
-# Offense count: 120
+# Offense count: 121
 # Configuration parameters: CountAsOne.
 RSpec/ExampleLength:
   Max: 43
 
-# Offense count: 37
-# Configuration parameters: Include, CustomTransform, IgnoreMethods, SpecSuffixOnly.
-# Include: **/*_spec*rb*, **/spec/**/*
-RSpec/FilePath:
-  Enabled: false
-
 # Offense count: 2
 RSpec/IteratedExpectation:
   Exclude:
@@ -179,7 +173,7 @@ Style/SymbolProc:
     - 'lib/droplet_kit/resources/kubernetes_cluster_resource.rb'
     - 'lib/tasks/resource_doc.rake'
 
-# Offense count: 90
+# Offense count: 91
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.
 # URISchemes: http, https

README.md

@@ -194,6 +194,7 @@ Actions supported:
 * `client.databases.create_connection_pool(database_connection_pool, id: 'id')`
 * `client.databases.find_connection_pool(id: 'id', name: 'name')`
 * `client.databases.list_connection_pools(id: 'id')`
+* `client.databases.update_connection_pool(database_connection_pool, id: 'id', name: 'name')`
 * `client.databases.delete_connection_pool(id: 'id', name: 'name')`
 * `client.databases.set_eviction_policy(database_eviction_policy, id: 'id')`
 * `client.databases.get_eviction_policy(id: 'id')`

lib/droplet_kit/mappings/database_connection_pool_mapping.rb

@@ -9,10 +9,10 @@ class DatabaseConnectionPoolMapping
       root_key singular: 'pool', plural: 'pools', scopes: [:read]
 
       property :name, scopes: %i[read create]
-      property :mode, scopes: %i[read create]
-      property :size, scopes: %i[read create]
-      property :db, scopes: %i[read create]
-      property :user, scopes: %i[read create]
+      property :mode, scopes: %i[read create update]
+      property :size, scopes: %i[read create update]
+      property :db, scopes: %i[read create update]
+      property :user, scopes: %i[read create update]
       property :connection, scopes: [:read], include: DatabaseConnectionMapping
     end
   end

lib/droplet_kit/resources/database_resource.rb

@@ -142,6 +142,11 @@ class DatabaseResource < ResourceKit::Resource
         handler(200) { |response| DatabaseConnectionPoolMapping.extract_collection(response.body, :read) }
       end
 
+      action :update_connection_pool, 'PUT /v2/databases/:id/pools/:name' do
+        body { |object| DatabaseConnectionPoolMapping.representation_for(:update, object) }
+        handler(204) { |response| true }
+      end
+
       action :delete_connection_pool, 'DELETE /v2/databases/:id/pools/:name' do
         handler(204) { |response| true }
       end

spec/lib/droplet_kit/resources/database_resource_spec.rb

@@ -535,6 +535,27 @@
     end
   end
 
+  describe '#update_connection_pool' do
+    context 'for a successful update of a database connection pool' do
+      let(:connection_pool_name) { 'backend-pool' }
+
+      it 'updates the database connection pool' do
+        database_connection_pool = DropletKit::DatabaseConnectionPool.new(
+          mode: 'transaction',
+          size: 10,
+          db: 'defaultdb',
+          user: 'doadmin'
+        )
+
+        json_body = DropletKit::DatabaseConnectionPoolMapping.representation_for(:update, database_connection_pool)
+        request = stub_do_api("/v2/databases/#{database_cluster_id}/pools/#{connection_pool_name}", :put).with(body: json_body).to_return(status: 204)
+        resource.update_connection_pool(database_connection_pool, id: database_cluster_id, name: connection_pool_name)
+
+        expect(request).to have_been_made
+      end
+    end
+  end
+
   describe '#delete_connection_pool' do
     let(:connection_pool_name) { 'backend-pool' }