From 3d6865fafc394e888dea07abd7f2afcc524ecee7 Mon Sep 17 00:00:00 2001 From: Cheah Chu Yeow Date: Mon, 24 Nov 2014 08:14:05 +0000 Subject: [PATCH] builder/digitalocean: use DigitalOcean image ID for artifact Id(). --- builder/digitalocean/artifact.go | 4 ++-- builder/digitalocean/artifact_test.go | 12 +++++++++++- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/builder/digitalocean/artifact.go b/builder/digitalocean/artifact.go index 3f756a05d..d1d878193 100644 --- a/builder/digitalocean/artifact.go +++ b/builder/digitalocean/artifact.go @@ -3,6 +3,7 @@ package digitalocean import ( "fmt" "log" + "strconv" ) type Artifact struct { @@ -29,8 +30,7 @@ func (*Artifact) Files() []string { } func (a *Artifact) Id() string { - // mimicing the aws builder - return fmt.Sprintf("%s:%s", a.regionName, a.snapshotName) + return strconv.FormatUint(uint64(a.snapshotId), 10) } func (a *Artifact) String() string { diff --git a/builder/digitalocean/artifact_test.go b/builder/digitalocean/artifact_test.go index 83681b3fa..4492c7bf3 100644 --- a/builder/digitalocean/artifact_test.go +++ b/builder/digitalocean/artifact_test.go @@ -1,8 +1,9 @@ package digitalocean import ( - "github.com/mitchellh/packer/packer" "testing" + + "github.com/mitchellh/packer/packer" ) func TestArtifact_Impl(t *testing.T) { @@ -13,6 +14,15 @@ func TestArtifact_Impl(t *testing.T) { } } +func TestArtifactId(t *testing.T) { + a := &Artifact{"packer-foobar", 42, "San Francisco", nil} + expected := "42" + + if a.Id() != expected { + t.Fatalf("artifact ID should match: %v", expected) + } +} + func TestArtifactString(t *testing.T) { a := &Artifact{"packer-foobar", 42, "San Francisco", nil} expected := "A snapshot was created: 'packer-foobar' in region 'San Francisco'"