From 85f91043a164bdacc1c6738ae0e7c50077d55c79 Mon Sep 17 00:00:00 2001 From: Jim Date: Fri, 6 Sep 2024 10:32:47 -0400 Subject: [PATCH] test (cache/search): fix NewTestServer(...) storage (#5081) Fix ensures that the TestServer returned uses the correct directory for writing its sqlite files, which was created via t.TempDir() --- internal/clientcache/cmd/search/search_test.go | 4 ++-- internal/clientcache/internal/daemon/testing.go | 3 +++ 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/internal/clientcache/cmd/search/search_test.go b/internal/clientcache/cmd/search/search_test.go index 3718e49479..1e5e134df2 100644 --- a/internal/clientcache/cmd/search/search_test.go +++ b/internal/clientcache/cmd/search/search_test.go @@ -140,9 +140,9 @@ func TestSearch(t *testing.T) { assert.Nil(t, apiErr) assert.NotNil(t, resp) assert.NotNil(t, r) - assert.EqualValues(t, r, &daemon.SearchResult{ + assert.EqualValues(t, &daemon.SearchResult{ RefreshStatus: daemon.NotRefreshing, - }) + }, r) }) t.Run("empty response from query", func(t *testing.T) { diff --git a/internal/clientcache/internal/daemon/testing.go b/internal/clientcache/internal/daemon/testing.go index b29ffc6b7d..7faa91a8c2 100644 --- a/internal/clientcache/internal/daemon/testing.go +++ b/internal/clientcache/internal/daemon/testing.go @@ -37,6 +37,9 @@ func NewTestServer(t *testing.T, cmd Commander, opt ...Option) *TestServer { RecheckSupportInterval: DefaultRecheckSupportInterval, LogWriter: io.Discard, DotDirectory: dotDir, + // we need to provide this, otherwise it will open a store in the user's + // home dir. See db.Open(...) + DatabaseUrl: dotDir + "cache.db?_pragma=foreign_keys(1)", } s, err := New(ctx, cfg)