#816 added Encryption status to file volume-detail

Author: allmightyspiff

plugin/commands/block/volume_detail.go

@@ -135,9 +135,9 @@ func (cmd *VolumeDetailCommand) Run(args []string) error {
 		return slErr.NewAPIError(T("Failed to decoded the note.\n"), err.Error(), 2)
 	}
 	table.Add(T("Notes"), decodedValue)
-	hasEncryption := T("false")
-	if (blockVolume.HasEncryptionAtRest != nil) {
-		hasEncryption = T(fmt.Sprintf("%v", *blockVolume.HasEncryptionAtRest))
+	hasEncryption := T("False")
+	if (blockVolume.HasEncryptionAtRest != nil && *blockVolume.HasEncryptionAtRest == true) {
+		hasEncryption := T("True")
 	}	
 	table.Add(T("Encrypted"), hasEncryption)
 	table.Print()

plugin/commands/block/volume_detail_test.go

@@ -66,7 +66,7 @@ var _ = Describe("Block Volume-Detail Tests", func() {
 				Expect(fakeUI.Outputs()).To(ContainSubstring("User name                  IBM01SEL278444-16"))
 				Expect(fakeUI.Outputs()).To(ContainSubstring("Endurance Tier             WRITEHEAVY_TIER"))
 				Expect(fakeUI.Outputs()).To(ContainSubstring("Notes                      -"))
-				Expect(fakeUI.Outputs()).To(ContainSubstring("Encrypted                  false"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("Encrypted                  False"))
 			})
 		})
 	})

plugin/commands/file/volume_detail.go

@@ -136,6 +136,11 @@ func (cmd *VolumeDetailCommand) Run(args []string) error {
 		return slErr.NewAPIError(T("Failed to decoded the note.\n"), err.Error(), 2)
 	}
 	table.Add(T("Notes"), decodedValue)
+	hasEncryption := T("False")
+	if (blockVolume.HasEncryptionAtRest != nil && *blockVolume.HasEncryptionAtRest == true) {
+		hasEncryption := T("True")
+	}	
+	table.Add(T("Encrypted"), hasEncryption)
 	table.Print()
 	return nil
 }

plugin/commands/file/volume_detail_test.go

@@ -1,15 +1,11 @@
 package file_test
 
 import (
-	"errors"
-
 	"github.com/IBM-Cloud/ibm-cloud-cli-sdk/testhelpers/terminal"
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
 
-	"github.com/softlayer/softlayer-go/datatypes"
 	"github.com/softlayer/softlayer-go/session"
-	"github.com/softlayer/softlayer-go/sl"
 	"github.ibm.com/SoftLayer/softlayer-cli/plugin/commands/file"
 	"github.ibm.com/SoftLayer/softlayer-cli/plugin/metadata"
 	"github.ibm.com/SoftLayer/softlayer-cli/plugin/testhelpers"
@@ -19,18 +15,18 @@ var _ = Describe("Volume detail", func() {
 	var (
 		fakeUI             *terminal.FakeUI
 		cliCommand         *file.VolumeDetailCommand
-		fakeSession        *session.Session
+		fakeSession *session.Session
+		fakeHandler *testhelpers.FakeTransportHandler
 		slCommand          *metadata.SoftlayerStorageCommand
-		FakeStorageManager *testhelpers.FakeStorageManager
+
 	)
 	BeforeEach(func() {
 		fakeUI = terminal.NewFakeUI()
-		fakeSession = testhelpers.NewFakeSoftlayerSession([]string{})
-		FakeStorageManager = new(testhelpers.FakeStorageManager)
+		fakeSession = testhelpers.NewFakeSoftlayerSession(nil)
+		fakeHandler = testhelpers.GetSessionHandler(fakeSession)
 		slCommand = metadata.NewSoftlayerStorageCommand(fakeUI, fakeSession, "file")
 		cliCommand = file.NewVolumeDetailCommand(slCommand)
 		cliCommand.Command.PersistentFlags().Var(cliCommand.OutputFlag, "output", "--output=JSON for json output.")
-		cliCommand.StorageManager = FakeStorageManager
 	})
 
 	Describe("Volume detail", func() {
@@ -51,7 +47,7 @@ var _ = Describe("Volume detail", func() {
 
 		Context("Volume detail with correct volume id but server API call fails", func() {
 			BeforeEach(func() {
-				FakeStorageManager.GetVolumeDetailsReturns(datatypes.Network_Storage{}, errors.New("Internal Server Error"))
+				fakeHandler.AddApiError("SoftLayer_Network_Storage", "getObject", 500, "Internal Server Error")
 			})
 			It("return error", func() {
 				err := testhelpers.RunCobraCommand(cliCommand.Command, "1234")
@@ -62,63 +58,14 @@ var _ = Describe("Volume detail", func() {
 		})
 
 		Context("Volume detail with correct volume id", func() {
-			BeforeEach(func() {
-				FakeStorageManager.GetVolumeDetailsReturns(datatypes.Network_Storage{
-					Id:               sl.Int(1234),
-					Username:         sl.String("myvolume"),
-					CapacityGb:       sl.Int(1000),
-					Iops:             sl.String("400"),
-					StorageTierLevel: sl.String("HEAVY_WRITE"),
-					StorageType: &datatypes.Network_Storage_Type{
-						KeyName: sl.String("performance"),
-					},
-					ServiceResource: &datatypes.Network_Service_Resource{
-						Datacenter: &datatypes.Location{
-							Name: sl.String("tok02"),
-						},
-					},
-					ServiceResourceBackendIpAddress: sl.String("9.9.9.9"),
-					SnapshotCapacityGb:              sl.String("500"),
-					ParentVolume: &datatypes.Network_Storage{
-						SnapshotSizeBytes: sl.String("10000"),
-					},
-					ActiveTransactionCount: sl.Uint(uint(1)),
-					ActiveTransactions: []datatypes.Provisioning_Version1_Transaction{
-						datatypes.Provisioning_Version1_Transaction{
-							TransactionStatus: &datatypes.Provisioning_Version1_Transaction_Status{
-								FriendlyName: sl.String("Restarting"),
-							},
-						},
-					},
-					ReplicationPartnerCount: sl.Uint(uint(1)),
-					ReplicationStatus:       sl.String("replication finished"),
-					ReplicationPartners: []datatypes.Network_Storage{
-						datatypes.Network_Storage{
-							Id:                              sl.Int(5678),
-							Username:                        sl.String("myreplicant"),
-							ServiceResourceBackendIpAddress: sl.String("9.9.9.8"),
-							ServiceResource: &datatypes.Network_Service_Resource{
-								Datacenter: &datatypes.Location{
-									Name: sl.String("dal10"),
-								},
-							},
-							ReplicationSchedule: &datatypes.Network_Storage_Schedule{
-								Type: &datatypes.Network_Storage_Schedule_Type{
-									Keyname: sl.String("DAILY"),
-								},
-							},
-						},
-					},
-				}, nil)
-			})
 			It("return no error", func() {
 				err := testhelpers.RunCobraCommand(cliCommand.Command, "1234")
 				Expect(err).NotTo(HaveOccurred())
-				Expect(fakeUI.Outputs()).To(ContainSubstring("1234"))
-				Expect(fakeUI.Outputs()).To(ContainSubstring("myvolume"))
-				Expect(fakeUI.Outputs()).To(ContainSubstring("tok02"))
-				Expect(fakeUI.Outputs()).To(ContainSubstring("9.9.9.9"))
-				Expect(fakeUI.Outputs()).To(ContainSubstring("Restarting"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("ID                         17336531"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("User name                  IBM01SEL278444-16"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("Endurance Tier             WRITEHEAVY_TIER"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("Notes                      -"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("Encrypted                  False"))
 			})
 		})
 	})

plugin/i18n/resources/en_US.all.json

@@ -2715,6 +2715,10 @@
     "id": "Enables/Disables snapshot space usage threshold warning for a given volume.",
     "translation": "Enables/Disables snapshot space usage threshold warning for a given volume."
   },
+  {
+    "id": "Encrypted",
+    "translation": "Encrypted"
+  },
   {
     "id": "End",
     "translation": "End"