Skip to content

Commit

Permalink
pass nil as the jsonBody for the dataset release function
Browse files Browse the repository at this point in the history
  • Loading branch information
nanjiangshu committed Sep 18, 2024
1 parent ae67160 commit 95c6bf9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
4 changes: 1 addition & 3 deletions sda-admin/dataset/dataset.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,7 @@ func Release(apiURI, token, datasetID string) error {
}
parsedURL.Path = path.Join(parsedURL.Path, "dataset/release") + "/" + datasetID

jsonBody := []byte("{}")
_, err = helpers.PostRequest(parsedURL.String(), token, jsonBody)

_, err = helpers.PostRequest(parsedURL.String(), token, nil)
if err != nil {
return err
}
Expand Down
6 changes: 2 additions & 4 deletions sda-admin/dataset/dataset_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,8 @@ func TestRelease_Success(t *testing.T) {

expectedURL := "http://example.com/dataset/release/dataset-123"
token := "test-token"
jsonBody := []byte(`{}`)

mockHelpers.On("PostRequest", expectedURL, token, jsonBody).Return([]byte(`{}`), nil)
mockHelpers.On("PostRequest", expectedURL, token, []byte(nil)).Return([]byte(`{}`), nil)

err := Release("http://example.com", token, "dataset-123")
assert.NoError(t, err)
Expand All @@ -84,9 +83,8 @@ func TestRelease_PostRequestFailure(t *testing.T) {

expectedURL := "http://example.com/dataset/release/dataset-123"
token := "test-token"
jsonBody := []byte(`{}`)

mockHelpers.On("PostRequest", expectedURL, token, jsonBody).Return([]byte(nil), errors.New("failed to send request"))
mockHelpers.On("PostRequest", expectedURL, token, []byte(nil)).Return([]byte(nil), errors.New("failed to send request"))

err := Release("http://example.com", token, "dataset-123")
assert.Error(t, err)
Expand Down

0 comments on commit 95c6bf9

Please sign in to comment.