refactor(test): avoid panic in tests with goroutines

Oleksandr Redkorequested to merge
alexandear/client-go:refactor/test-go-require into main

This MR enables testifylint.go-require rule and fixes these lint issues:

$ golangci-lint run
bulk_imports_test.go:45:3: go-require: do not use require in http handlers (testifylint)
bulk_imports_test.go:47:3: go-require: do not use require in http handlers (testifylint)
commits_test.go:142:3: go-require: do not use require in http handlers (testifylint)
commits_test.go:146:3: go-require: do not use require in http handlers (testifylint)
dependency_list_export_test.go:22:3: go-require: do not use require in http handlers (testifylint)
dependency_list_export_test.go:26:3: go-require: do not use require in http handlers (testifylint)
gitlab_test.go:985:3: go-require: do not use require in http handlers (testifylint)
pagination_test.go:203:4: go-require: do not use require in http handlers (testifylint)
pagination_test.go:218:4: go-require: do not use require in http handlers (testifylint)
pagination_test.go:233:4: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:19:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:22:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:26:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:28:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:55:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:71:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:91:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:115:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:138:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:157:3: go-require: do not use require in http handlers (testifylint)
project_aliases_test.go:180:3: go-require: do not use require in http handlers (testifylint)
request_handler_test.go:61:3: go-require: do not use require in http handlers (testifylint)
request_handler_test.go:62:3: go-require: do not use require in http handlers (testifylint)
request_handler_test.go:242:3: go-require: do not use require in http handlers (testifylint)
request_handler_test.go:243:3: go-require: do not use require in http handlers (testifylint)
25 issues:
* testifylint: 25

We can't use require.* in a goroutines because this leads to a panic when tests failed. See https://github.com/Antonboom/testifylint#go-require for details.

Merge request reports