Skip to content

Commit

Permalink
Merge pull request #35 from vansante/upgrade-actions
Browse files Browse the repository at this point in the history
Remove some golangci settings defaults
  • Loading branch information
vansante authored Sep 6, 2023
2 parents 12f94c2 + b8a5fd6 commit e8eb9ff
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 15 deletions.
10 changes: 0 additions & 10 deletions .golangci.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
run:
# timeout for analysis, e.g. 30s, 5m, default is 1m
deadline: 30s

# exit code when at least one issue was found, default is 1
issues-exit-code: 1

# include test files or not, default is true
tests: true

# all available settings of specific linters
linters-settings:
gocyclo:
Expand Down
11 changes: 6 additions & 5 deletions reader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,12 @@ func TestFullGetRead(t *testing.T) {
src, err := randomFile(11325)
assert.NoError(t, err)

srcBuf, err := io.ReadAll(src)
assert.NoError(t, err)

_, err = src.Seek(0, io.SeekStart)
assert.NoError(t, err)

err = srv.ServeFileReader(context.Background(), fileID, src)
assert.NoError(t, err)

Expand All @@ -196,11 +202,6 @@ func TestFullGetRead(t *testing.T) {
resp, err := http.DefaultClient.Do(req)
assert.NoError(t, err)

_, err = src.Seek(0, io.SeekStart)
assert.NoError(t, err)
srcBuf, err := io.ReadAll(src)
assert.NoError(t, err)

file, err := io.ReadAll(resp.Body)
assert.NoError(t, err)
_ = resp.Body.Close()
Expand Down

0 comments on commit e8eb9ff

Please sign in to comment.