Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

Commit 6d0c780

Browse files
committed
Return pointer to struct to match interface signature
1 parent 4c4af0f commit 6d0c780

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

package_stemcell/factory/packager_factory.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ func (f *PackagerFactory) Packager(sourceConfig config.SourceConfig, outputConfi
3030
case config.VMDK:
3131
options := package_parameters.VmdkPackageParameters{}
3232
logger := colorlogger.ConstructLogger(logLevel, color, os.Stderr)
33-
vmdkPackager := packagers.VmdkPackager{
33+
vmdkPackager := &packagers.VmdkPackager{
3434
Stop: make(chan struct{}),
3535
Debugf: logger.Debugf,
3636
BuildOptions: options,

package_stemcell/factory/packager_factory_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ var _ = Describe("Factory", func() {
3333
actualPackager, err := packagerFactory.Packager(sourceConfig, outputConfig, 0, false)
3434
Expect(err).NotTo(HaveOccurred())
3535

36-
Expect(actualPackager).To(BeAssignableToTypeOf(packagers.VmdkPackager{}))
37-
Expect(actualPackager).NotTo(BeAssignableToTypeOf(packagers.VCenterPackager{}))
36+
Expect(actualPackager).To(BeAssignableToTypeOf(&packagers.VmdkPackager{}))
37+
Expect(actualPackager).NotTo(BeAssignableToTypeOf(&packagers.VCenterPackager{}))
3838
})
3939
})
4040

@@ -50,8 +50,8 @@ var _ = Describe("Factory", func() {
5050
actualPackager, err := packagerFactory.Packager(sourceConfig, outputConfig, 0, false)
5151
Expect(err).NotTo(HaveOccurred())
5252

53-
Expect(actualPackager).To(BeAssignableToTypeOf(packagers.VCenterPackager{}))
54-
Expect(actualPackager).NotTo(BeAssignableToTypeOf(packagers.VmdkPackager{}))
53+
Expect(actualPackager).To(BeAssignableToTypeOf(&packagers.VCenterPackager{}))
54+
Expect(actualPackager).NotTo(BeAssignableToTypeOf(&packagers.VmdkPackager{}))
5555
})
5656
})
5757

0 commit comments

Comments
 (0)