From 645fa4f1ab548bc3ebf65a119a45e780375f63b9 Mon Sep 17 00:00:00 2001 From: tiennm99 Date: Thu, 11 Jun 2026 21:49:03 +0700 Subject: [PATCH] fix(gold): drop redundant embedded selectors in portfolio tests --- internal/modules/gold/portfolio_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/modules/gold/portfolio_test.go b/internal/modules/gold/portfolio_test.go index a521e35..c3fd871 100644 --- a/internal/modules/gold/portfolio_test.go +++ b/internal/modules/gold/portfolio_test.go @@ -81,7 +81,7 @@ func (s *conflictOnceStore) CompareAndSwap(ctx context.Context, key string, expe s.conflicted = true competing := NewPortfolio(1) competing.AddVND(10) - if err := s.KVStore.PutJSON(ctx, key, competing); err != nil { + if err := s.PutJSON(ctx, key, competing); err != nil { return err } return storage.ErrConflict @@ -134,7 +134,7 @@ func TestUpdatePortfolioReturnsConflictAfterExhaustingRetries(t *testing.T) { if kv.attempts != portfolioUpdateAttempts { t.Errorf("CAS attempts = %d, want %d", kv.attempts, portfolioUpdateAttempts) } - if _, err := kv.KVStore.Get(ctx, "user:7"); !errors.Is(err, storage.ErrNotFound) { + if _, err := kv.Get(ctx, "user:7"); !errors.Is(err, storage.ErrNotFound) { t.Errorf("exhausted update must not persist anything, Get = %v", err) } }