diff options
| author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-10-10 13:01:02 +0300 |
|---|---|---|
| committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-10-14 13:44:54 +0300 |
| commit | 1881ab188e2789715ad1307fb1748d6c14e72474 (patch) | |
| tree | 5b89a52a44d9e7390208d2f233433dec8629fac4 /internal/gitaly/service/hook/testhelper_test.go | |
| parent | 7d88809fa9bac82129b76e472f80e917eb0e3806 (diff) | |
global: Reorder `testing.TB` before `context.Context` arguments
A few weeks ago we have decided that `testing.TB` arguments should be
first, `context.Context` second. This is rather a matter of perssnal
taste instead of correctness, but we should strive for consistency.
Reorder all arguments to match the agreed-upon style.
Diffstat (limited to 'internal/gitaly/service/hook/testhelper_test.go')
| -rw-r--r-- | internal/gitaly/service/hook/testhelper_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/gitaly/service/hook/testhelper_test.go b/internal/gitaly/service/hook/testhelper_test.go index 926d16ca4..98e307f07 100644 --- a/internal/gitaly/service/hook/testhelper_test.go +++ b/internal/gitaly/service/hook/testhelper_test.go @@ -23,7 +23,7 @@ func TestMain(m *testing.M) { testhelper.Run(m) } -func setupHookService(ctx context.Context, tb testing.TB) (config.Cfg, *gitalypb.Repository, string, gitalypb.HookServiceClient) { +func setupHookService(tb testing.TB, ctx context.Context) (config.Cfg, *gitalypb.Repository, string, gitalypb.HookServiceClient) { tb.Helper() cfg := testcfg.Build(tb) @@ -31,7 +31,7 @@ func setupHookService(ctx context.Context, tb testing.TB) (config.Cfg, *gitalypb client, conn := newHooksClient(tb, cfg.SocketPath) tb.Cleanup(func() { conn.Close() }) - repo, repoPath := gittest.CreateRepository(ctx, tb, cfg, gittest.CreateRepositoryConfig{ + repo, repoPath := gittest.CreateRepository(tb, ctx, cfg, gittest.CreateRepositoryConfig{ Seed: gittest.SeedGitLabTest, }) |