Return pointer to struct to match interface signature

cloudfoundry · Jul 19, 2024 · 6d0c780 · 6d0c780
1 parent 4c4af0f
commit 6d0c780
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/package_stemcell/factory/packager_factory.go b/package_stemcell/factory/packager_factory.go
@@ -30,7 +30,7 @@ func (f *PackagerFactory) Packager(sourceConfig config.SourceConfig, outputConfi
 	case config.VMDK:
 		options := package_parameters.VmdkPackageParameters{}
 		logger := colorlogger.ConstructLogger(logLevel, color, os.Stderr)
-		vmdkPackager := packagers.VmdkPackager{
+		vmdkPackager := &packagers.VmdkPackager{
 			Stop:         make(chan struct{}),
 			Debugf:       logger.Debugf,
 			BuildOptions: options,

diff --git a/package_stemcell/factory/packager_factory_test.go b/package_stemcell/factory/packager_factory_test.go
@@ -33,8 +33,8 @@ var _ = Describe("Factory", func() {
 				actualPackager, err := packagerFactory.Packager(sourceConfig, outputConfig, 0, false)
 				Expect(err).NotTo(HaveOccurred())
 
-				Expect(actualPackager).To(BeAssignableToTypeOf(packagers.VmdkPackager{}))
-				Expect(actualPackager).NotTo(BeAssignableToTypeOf(packagers.VCenterPackager{}))
+				Expect(actualPackager).To(BeAssignableToTypeOf(&packagers.VmdkPackager{}))
+				Expect(actualPackager).NotTo(BeAssignableToTypeOf(&packagers.VCenterPackager{}))
 			})
 		})
 
@@ -50,8 +50,8 @@ var _ = Describe("Factory", func() {
 				actualPackager, err := packagerFactory.Packager(sourceConfig, outputConfig, 0, false)
 				Expect(err).NotTo(HaveOccurred())
 
-				Expect(actualPackager).To(BeAssignableToTypeOf(packagers.VCenterPackager{}))
-				Expect(actualPackager).NotTo(BeAssignableToTypeOf(packagers.VmdkPackager{}))
+				Expect(actualPackager).To(BeAssignableToTypeOf(&packagers.VCenterPackager{}))
+				Expect(actualPackager).NotTo(BeAssignableToTypeOf(&packagers.VmdkPackager{}))
 			})
 		})