Add unit test to fetchChucnkSizeKiB

Update existing unit test to have suffix GiB

Add comment to clarify cache size is always in GiB

Author: cemakd

pkg/gce-pd-csi-driver/cache.go

@@ -573,6 +573,7 @@ func isCachingSetup(mainLvName string) (error, bool) {
 	return nil, false
 }
 
+// cacheSize is always in GiB
 func fetchChunkSizeKiB(cacheSize string) (string, error) {
 	var chunkSize float64
 

pkg/gce-pd-csi-driver/cache_test.go

@@ -13,24 +13,29 @@ func TestFetchChunkSizeKiB(t *testing.T) {
 	}{
 		{
 			name:         "chunk size is in the allowed range",
-			cacheSize:    "500",
+			cacheSize:    "500GiB",
 			expChunkSize: "512KiB", //range defined in fetchChunkSizeKiB
 		},
 		{
 			name:         "chunk size is set to the range ceil",
-			cacheSize:    "30000000",
+			cacheSize:    "30000000GiB",
 			expChunkSize: "1048576KiB", //range defined in fetchChunkSizeKiB - max 1GiB
 		},
 		{
 			name:         "chunk size is set to the allowed range floor",
-			cacheSize:    "100",
+			cacheSize:    "100GiB",
 			expChunkSize: "160KiB", //range defined in fetchChunkSizeKiB - min 160 KiB
 		},
 		{
 			name:         "cacheSize set to KiB also sets the chunk size to range floor",
-			cacheSize:    "1",
+			cacheSize:    "1GiB",
 			expChunkSize: "160KiB", //range defined in fetchChunkSizeKiB - min 160 KiB
 		},
+		{
+			name:         "chunk size with GiB string parses correctly",
+			cacheSize:    "375GiB",
+			expChunkSize: "384KiB",
+		},
 		{
 			name:         "invalid cacheSize",
 			cacheSize:    "fdfsdKi",