fix: Fargate provider storage size setting doesn't work

[ToddMurphy92]

• Corrected the logical grouping of the nullish coalescing operator and ternary operator for the ephemeralStorageGiB property.
• Ensured that the ephemeralStorageGiB property is correctly set when provided in the props.
• This fix ensures that the correct storage value is reflected in the synthesized CloudFormation template.
• Prior to this fix the storage value would always be set to 25

[kichik]

Thanks!

Can you give me access to push to the branch to update the unit tests (I think I can't because main branch is locked?)? Or commit this change directly:

 test/providers.test.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/test/providers.test.ts b/test/providers.test.ts
index 5261e36..52c26bf 100644
--- a/test/providers.test.ts
+++ b/test/providers.test.ts
@@ -39,7 +39,7 @@ test('CodeBuild provider privileged', () => {
     Environment: {
       PrivilegedMode: true,
     },
-  }), 2/*runners*/+3/*image builders*/);
+  }), 2/*runners*/ + 3/*image builders*/);
 
   template.hasResourceProperties('AWS::CodeBuild::Project', Match.objectLike({
     Environment: {
@@ -73,6 +73,7 @@ test('Fargate provider', () => {
   new FargateRunnerProvider(stack, 'provider', {
     vpc: vpc,
     securityGroups: [sg],
+    ephemeralStorageGiB: 100,
   });
 
   const template = Template.fromStack(stack);
@@ -87,6 +88,7 @@ test('Fargate provider', () => {
         Name: 'runner',
       },
     ],
+    EphemeralStorage: { SizeInGiB: 100 },
   }));
 });

[ToddMurphy92]

No worries @kichik
I've made you a collaborator of the fork I created - did that fix the issue for you?

[kichik]

I still couldn't push from CodeSpaces but the UI worked 🤷