Skip to content

Commit 62aa426

Browse files
Removing transient comments
1 parent b121bca commit 62aa426

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

pkg/resolver/cell_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -331,10 +331,6 @@ func TestMergeCellConfig(t *testing.T) {
331331
for name, tc := range tests {
332332
t.Run(name, func(t *testing.T) {
333333
t.Parallel()
334-
// FIXED: Calling the unexported function mergeCellConfig via correct capitalization for test access?
335-
// Since we are in package resolver, we can call mergeCellConfig directly.
336-
// However, in Go, unexported names are only visible within the SAME package.
337-
// The file header says 'package resolver', so we should use 'mergeCellConfig'.
338334
gw, topo := mergeCellConfig(tc.tpl, tc.overrides, tc.inline)
339335

340336
if diff := cmp.Diff(tc.wantGw, gw, cmpopts.IgnoreUnexported(resource.Quantity{}), cmpopts.EquateEmpty()); diff != "" {

pkg/resolver/shard_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,6 @@ func TestMergeShardConfig(t *testing.T) {
365365
for name, tc := range tests {
366366
t.Run(name, func(t *testing.T) {
367367
t.Parallel()
368-
// FIXED: Use correct internal function name
369368
orch, pools := mergeShardConfig(tc.tpl, tc.overrides, tc.inline)
370369

371370
if diff := cmp.Diff(tc.wantOrch, orch, cmpopts.IgnoreUnexported(resource.Quantity{})); diff != "" {

0 commit comments

Comments
 (0)