style(azure database): satisfy godot, fieldalignment, and unused linters

Add trailing periods to four exported comments (godot), reorder the
mock pager struct fields so the pointer-scannable region shrinks to
24 bytes (govet fieldalignment), and drop the unused boolPtr test
helper (unused). No behavior change; 703 azure-module tests pass.

Author: cristim

providers/azure/services/database/client.go

@@ -62,13 +62,13 @@ type CapabilitiesClient interface {
 	ListByLocation(ctx context.Context, locationName string, options *armsql.CapabilitiesClientListByLocationOptions) (armsql.CapabilitiesClientListByLocationResponse, error)
 }
 
-// SQLServersPager interface for listing SQL servers (enables mocking)
+// SQLServersPager interface for listing SQL servers (enables mocking).
 type SQLServersPager interface {
 	More() bool
 	NextPage(ctx context.Context) (armsql.ServersClientListResponse, error)
 }
 
-// SQLManagedInstancesPager interface for listing SQL managed instances (enables mocking)
+// SQLManagedInstancesPager interface for listing SQL managed instances (enables mocking).
 type SQLManagedInstancesPager interface {
 	More() bool
 	NextPage(ctx context.Context) (armsql.ManagedInstancesClientListResponse, error)
@@ -138,12 +138,12 @@ func (c *DatabaseClient) SetCapabilitiesClient(client CapabilitiesClient) {
 	c.capabilitiesClient = client
 }
 
-// SetServersPager sets the SQL servers pager (for testing)
+// SetServersPager sets the SQL servers pager (for testing).
 func (c *DatabaseClient) SetServersPager(pager SQLServersPager) {
 	c.serversPager = pager
 }
 
-// SetManagedInstancesPager sets the SQL managed instances pager (for testing)
+// SetManagedInstancesPager sets the SQL managed instances pager (for testing).
 func (c *DatabaseClient) SetManagedInstancesPager(pager SQLManagedInstancesPager) {
 	c.managedInstancesPager = pager
 }

providers/azure/services/database/client_test.go

@@ -1250,9 +1250,9 @@ func TestDatabaseClient_PurchaseCommitment_DisplayNameConformsToAzureAllowlist(t
 // MockSQLServersPager mocks the SQLServersPager interface.
 // CallCount tracks how many times NextPage was invoked.
 type MockSQLServersPager struct {
+	err       error
 	pages     []armsql.ServersClientListResponse
 	index     int
-	err       error
 	CallCount int
 }
 
@@ -1276,9 +1276,9 @@ func (m *MockSQLServersPager) NextPage(_ context.Context) (armsql.ServersClientL
 // MockSQLManagedInstancesPager mocks the SQLManagedInstancesPager interface.
 // CallCount tracks how many times NextPage was invoked.
 type MockSQLManagedInstancesPager struct {
+	err       error
 	pages     []armsql.ManagedInstancesClientListResponse
 	index     int
-	err       error
 	CallCount int
 }
 
@@ -1299,8 +1299,6 @@ func (m *MockSQLManagedInstancesPager) NextPage(_ context.Context) (armsql.Manag
 	return page, nil
 }
 
-func boolPtr(b bool) *bool { return &b }
-
 // buildMIPage builds a single-page managed-instances response with the
 // given ZoneRedundant values.
 func buildMIPage(zoneRedundant ...bool) armsql.ManagedInstancesClientListResponse {