op-deployer: Marshal tag-based ArtifactsLocator with correct scheme (e…

…thereum-optimism#12494)
samlaf · Nov 10, 2024 · 3dc3a74 · 3dc3a74
1 parent 4bac1d2
commit 3dc3a74
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/op-deployer/pkg/deployer/opcm/artifacts_locator.go b/op-deployer/pkg/deployer/opcm/artifacts_locator.go
@@ -45,7 +45,7 @@ func (a *ArtifactsLocator) MarshalText() ([]byte, error) {
 	}
 
 	if a.Tag != "" {
-		return []byte(a.Tag), nil
+		return []byte("tag://" + a.Tag), nil
 	}
 
 	return nil, fmt.Errorf("no URL, path or tag set")

diff --git a/op-deployer/pkg/deployer/opcm/artifacts_locator_test.go b/op-deployer/pkg/deployer/opcm/artifacts_locator_test.go
@@ -79,6 +79,10 @@ func TestArtifactsLocator_Marshaling(t *testing.T) {
 			}
 			require.NoError(t, err)
 			require.Equal(t, tt.out, &a)
+
+			marshalled, err := a.MarshalText()
+			require.NoError(t, err)
+			require.Equal(t, tt.in, string(marshalled))
 		})
 	}
 }