[
https://issues.apache.org/jira/browse/BEAM-13910?focusedWorklogId=732697&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-732697
]
ASF GitHub Bot logged work on BEAM-13910:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 24/Feb/22 21:23
Start Date: 24/Feb/22 21:23
Worklog Time Spent: 10m
Work Description: jrmccluskey commented on a change in pull request
#16942:
URL: https://github.com/apache/beam/pull/16942#discussion_r814279141
##########
File path: sdks/go/pkg/beam/util/gcsx/gcs_test.go
##########
@@ -13,35 +13,62 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-package gcsx_test
+package gcsx
import (
- "context"
- "time"
+ "testing"
- "cloud.google.com/go/storage"
- "github.com/apache/beam/sdks/v2/go/pkg/beam/util/gcsx"
+ "github.com/apache/beam/sdks/v2/go/pkg/beam/internal/errors"
)
-func Example() {
- ctx := context.Background()
- c, err := gcsx.NewClient(ctx, storage.ScopeReadOnly)
- if err != nil {
- // do something
+func TestMakeObject(t *testing.T) {
+ if got, want := MakeObject("some-bucket", "some/path"),
"gs://some-bucket/some/path"; got != want {
+ t.Fatalf("MakeObject() Got: %v Want: %v", got, want)
}
+}
- buckets, object, err := gcsx.ParseObject("gs://some-bucket/some-object")
- if err != nil {
- // do something
+func TestParseObject(t *testing.T) {
+ tests := []struct {
+ object string
+ bucket string
+ path string
+ err error
+ }{
+ {
+ object: "gs://some-bucket/some-object",
+ bucket: "some-bucket",
+ path: "some-object",
+ err: nil,
+ },
+ {
+ object: "gs://some-bucket",
+ bucket: "some-bucket",
+ path: "",
+ err: nil,
+ },
+ {
+ object: "gs://",
+ bucket: "",
+ path: "",
+ err: errors.Errorf("object gs:// must have bucket"),
+ },
+ {
+ object: "other://some-bucket/some-object",
+ bucket: "",
+ path: "",
+ err: errors.Errorf("object
other://some-bucket/some-object must have 'gs' scheme"),
+ },
}
- ctx, cancel := context.WithTimeout(ctx, 30*time.Second)
- defer cancel()
-
- bytes, err := gcsx.ReadObject(ctx, c, buckets, object)
- if err != nil {
- // do something
+ for _, test := range tests {
Review comment:
Nit: The error message seems to encapsulate what the test is doing well
enough so it's easy to debug, but you could also consider adding a "name" field
to the test struct and using that as an identifier with t.Run()
(https://pkg.go.dev/testing#T.Run). Totally up to you in this case, but a
really helpful testing design pattern to keep in mind
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 732697)
Time Spent: 50m (was: 40m)
> Add unit testing to the gcsx package
> ------------------------------------
>
> Key: BEAM-13910
> URL: https://issues.apache.org/jira/browse/BEAM-13910
> Project: Beam
> Issue Type: Sub-task
> Components: sdk-go
> Reporter: Jack McCluskey
> Assignee: Danny McCormick
> Priority: P2
> Labels: starter
> Time Spent: 50m
> Remaining Estimate: 0h
>
> Add unit testing to the [gcsx
> package|https://github.com/apache/beam/tree/release-2.36.0/sdks/go/pkg/beam/util/gcsx]
> We want code coverage at or above 50%, it is currently at 0%.
> Helper functions can be tested easily, main functionality requires mocking
> GCS interactions.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
