update FileSystem Size to GB

Author: Laure-di

internal/services/file/filesystem.go

@@ -91,6 +91,12 @@ func ResourceFileSystemCreate(ctx context.Context, d *schema.ResourceData, m any
 		Tags:      types.ExpandStrings(d.Get("tags")),
 	}
 
+	if size, ok := d.GetOk("size_in_gb"); ok {
+		sizeInGB := size.(int)
+		sizeInBytes := uint64(sizeInGB) * uint64(scw.GB)
+		req.Size = sizeInBytes
+	}
+
 	file, err := api.CreateFileSystem(req, scw.WithContext(ctx))
 	if err != nil {
 		return diag.FromErr(err)
@@ -128,7 +134,8 @@ func ResourceFileSystemRead(ctx context.Context, d *schema.ResourceData, m any)
 	_ = d.Set("region", fileSystem.Region)
 	_ = d.Set("organization_id", fileSystem.OrganizationID)
 	_ = d.Set("status", fileSystem.Status)
-	_ = d.Set("size_in_gb", int64(fileSystem.Size))
+	_ = d.Set("size_in_gb", int(fileSystem.Size/scw.GB))
+	)
 	_ = d.Set("tags", fileSystem.Tags)
 	_ = d.Set("created_at", fileSystem.CreatedAt.Format(time.RFC3339))
 	_ = d.Set("updated_at", fileSystem.UpdatedAt.Format(time.RFC3339))
@@ -164,7 +171,8 @@ func ResourceFileSystemUpdate(ctx context.Context, d *schema.ResourceData, m any
 	}
 
 	if d.HasChange("size_in_gb") {
-		req.Size = types.ExpandUint64Ptr(d.Get("size_in_gb"))
+		sizeInGB := uint64(d.Get("size_in_gb").(int)) * uint64(scw.GB)
+		req.Size = types.ExpandUint64Ptr(sizeInGB)
 	}
 
 	if d.HasChange("tags") {

internal/services/file/filesystem_test.go

@@ -20,7 +20,7 @@ func TestAccFileSystem_Basic(t *testing.T) {
 
 	fileSystemName := "TestAccFileSystem_Basic"
 	fileSystemNameUpdated := "TestAccFileSystem_BasicUpdate"
-	sizeInGB := int64(100_000_000_000)
+	sizeInGB := 100
 
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
@@ -37,7 +37,7 @@ func TestAccFileSystem_Basic(t *testing.T) {
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckFileSystemExists(tt, "scaleway_file_filesystem.fs"),
 					resource.TestCheckResourceAttr("scaleway_file_filesystem.fs", "name", fileSystemName),
-					resource.TestCheckResourceAttr("scaleway_file_filesystem.fs", "size_in_gb", strconv.FormatInt(sizeInGB, 10)),
+					resource.TestCheckResourceAttr("scaleway_file_filesystem.fs", "size_in_gb", strconv.Itoa(sizeInGB)),
 				),
 			},
 			{
@@ -49,7 +49,7 @@ func TestAccFileSystem_Basic(t *testing.T) {
 				`, fileSystemNameUpdated, sizeInGB),
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckFileSystemExists(tt, "scaleway_file_filesystem.fs"),
-					resource.TestCheckResourceAttr("scaleway_file_filesystem.fs", "size_in_gb", strconv.FormatInt(sizeInGB, 10)),
+					resource.TestCheckResourceAttr("scaleway_file_filesystem.fs", "size_in_gb", strconv.Itoa(sizeInGB)),
 				),
 			},
 		},
@@ -61,7 +61,7 @@ func TestAccFileSystem_SizeTooSmallFails(t *testing.T) {
 	defer tt.Cleanup()
 
 	fileSystemName := "TestAccFileSystem_SizeTooSmallFails"
-	sizeInGB := int64(10_000_000_000)
+	sizeInGB := 10
 
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
@@ -86,7 +86,7 @@ func TestAccFileSystem_InvalidSizeGranularityFails(t *testing.T) {
 	defer tt.Cleanup()
 
 	fileSystemName := "TestAccFileSystem_InvalidSizeGranularityFails"
-	sizeInGB := int64(25_000_000_000)
+	sizeInGB := 250
 
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:          func() { acctest.PreCheck(t) },
@@ -100,7 +100,7 @@ func TestAccFileSystem_InvalidSizeGranularityFails(t *testing.T) {
 						size_in_gb = %d
 					}
 				`, fileSystemName, sizeInGB),
-				ExpectError: regexp.MustCompile("size_in_gb must be greater or equal to 100000000000"),
+				ExpectError: regexp.MustCompile("size does not respect constraint, size must be a multiple of 100000000000"),
 			},
 		},
 	})