Skip to content

Commit ea742f6

Browse files
committed
refactor: error message spelling
1 parent 68bd811 commit ea742f6

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

hcloud/routes.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,26 +152,26 @@ func (r *routes) resolveRouteTarget(ctx context.Context, nodeName string) (*core
152152

153153
id, isCloudServer, err := providerid.ToServerID(node.Spec.ProviderID)
154154
if err != nil {
155-
return nil, nil, fmt.Errorf("error parsing providerID %q for node %s: %w", node.Spec.ProviderID, nodeName, err)
155+
return nil, nil, fmt.Errorf("error parsing provider id %q for node %s: %w", node.Spec.ProviderID, nodeName, err)
156156
}
157157
if !isCloudServer {
158-
return nil, nil, fmt.Errorf("node %s is not a Cloud server, routes are only supported for Cloud servers", node.Name)
158+
return nil, nil, fmt.Errorf("node %s is not a cloud server, routes are only supported for cloud servers", node.Name)
159159
}
160160

161161
server, err := r.serverCache.ByID(ctx, id)
162162
if errors.Is(err, hcops.ErrNotFound) {
163163
server, err = r.serverCache.ByName(ctx, node.Name)
164164
}
165165
if err != nil {
166-
return nil, nil, fmt.Errorf("error fetching node %s by ID: %w", nodeName, err)
166+
return nil, nil, fmt.Errorf("error fetching node %s by id: %w", nodeName, err)
167167
}
168168

169169
privNet, ok := findServerPrivateNetByID(server, r.network.ID)
170170
if !ok {
171171
r.serverCache.InvalidateCache()
172172
server, err = r.serverCache.ByID(ctx, server.ID)
173173
if err != nil {
174-
return nil, nil, fmt.Errorf("error fetching node %s by ID: %w", nodeName, err)
174+
return nil, nil, fmt.Errorf("error fetching node %s by id: %w", nodeName, err)
175175
}
176176
privNet, ok = findServerPrivateNetByID(server, r.network.ID)
177177
if !ok {

hcloud/routes_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ func TestRoutes_CreateRoute_RobotProviderID(t *testing.T) {
250250
DestinationCIDR: "10.5.0.0/24",
251251
})
252252
require.Error(t, err)
253-
assert.ErrorContains(t, err, "not a Cloud server")
253+
assert.ErrorContains(t, err, "not a cloud server")
254254
}
255255

256256
// TestRoutes_CreateRoute_NodeNameDrift proves the routes controller still works when the

0 commit comments

Comments
 (0)