internal/resource/container.go

@@ -100,13 +100,11 @@ func (c *Container) URI() string {
 
 func (c *Container) SetURI(uri string) error {
 	resourceUri, e := url.Parse(uri)
-	if e == nil {
 	if resourceUri.Scheme == c.Type() {
 		c.Name, e = filepath.Abs(filepath.Join(resourceUri.Hostname(), resourceUri.RequestURI()))
 	} else {
 		e = fmt.Errorf("%w: %s is not a %s", ErrInvalidResourceURI, uri, c.Type())
 	}
-	}
 	return e
 }
 

internal/resource/container_test.go

@@ -34,7 +34,7 @@ func TestReadContainer(t *testing.T) {
   state: present
 `
 	m := &mocks.MockContainerClient{
-		InjectContainerList: func(ctx context.Context, options container.ListOptions) ([]types.Container, error) {
+		InjectContainerList: func(ctx context.Context, options types.ContainerListOptions) ([]types.Container, error) {
 			return []types.Container{
 				{ID: "123456789abc"},
 				{ID: "123456789def"},

tests/mocks/resource.go

@@ -42,6 +42,8 @@ func (m *MockResource) Type() string {
 }
 
 func (m *MockResource) UnmarshalJSON(data []byte) error {
+  fmt.Printf("UnmarshalJSON %#v\n", string(data))
+  panic(data)
   if err := json.Unmarshal(data, m); err != nil {
     return err
   }