refactor: Move atKeys file exists check from the isWritable method to…

… the enroll method.

packages/at_onboarding_cli/CHANGELOG.md

@@ -1,5 +1,6 @@
 ## 1.7.1
-- When submitting an enrollment request, check for write permissions of AtKeys file path.
+
+- fix: When submitting an enrollment request, check for write permissions of AtKeys file path.
 ## 1.7.0
 - feat: add `auto` command to the activate CLI 
 ## 1.6.4

packages/at_onboarding_cli/lib/src/cli/auth_cli.dart

@@ -351,6 +351,11 @@ Future<void> enroll(ArgResults argResults, {AtOnboardingService? svc}) async {
 
   File f = File(argResults[AuthCliArgs.argNameAtKeys]);
 
+  if (f.existsSync()) {
+    stderr.writeln('Error: atKeys file ${f.path} already exists');
+    return;
+  }
+
   // "_isWritable" attempts to create the directories for the given [file] if they do not exist,
   // and then tries to open the file in write mode to verify write permissions.
   //
@@ -430,27 +435,21 @@ Future<void> enroll(ArgResults argResults, {AtOnboardingService? svc}) async {
 /// [file]: The [File] instance to check for write access.
 ///
 /// Exceptions:
-/// - If the file already exists, a [PathExistsException] is caught, and an error message is printed to stderr.
 /// - If the file does not have write permissions, a [PathAccessException] is caught, and an error message is printed to stderr.
 /// - Any other exceptions are caught and logged, indicating a failure to determine write access.
 @visibleForTesting
 bool isWritable(File file) {
   try {
     // If the directories do not exist, create them.
     // "recursive" is set to true to ensure that any missing parent directories are created.
-    // "exclusive" is set to true to check if the file already exists; if it does,
-    // an error will be logged and returned.
-    file.createSync(recursive: true, exclusive: true);
+    file.createSync(recursive: true);
     // Try opening the file in write mode, which requires write permissions
     RandomAccessFile raf = file.openSync(mode: FileMode.write);
     raf.closeSync();
     // In [AtOnboardingServiceImpl._generateAtKeysFile] method, there is a check which returns error
     // if the file already exists. Therefore, delete the file here after checking for write permissions.
     file.deleteSync();
     return true;
-  } on PathExistsException {
-    stderr.writeln('Error: atKeys file ${file.path} already exists');
-    return false;
   } on PathAccessException {
     stderr.writeln(
         'Error : atKeys file ${file.path} does not have write permissions');

packages/at_onboarding_cli/test/auth_cli_test.dart

@@ -19,15 +19,6 @@ void main() {
       expect(isWritable(File(dirPath)), false);
     });
 
-    test(
-        'A test to verify isWritable returns false if file already exists in the directory',
-        () async {
-      final directory = Directory(dirPath);
-      // Create the directory first to ensure it exists before calling isWritable.
-      await directory.create(recursive: true);
-      expect(isWritable(File(dirPath)), false);
-    });
-
     test(
         'A test verify isWritable returns true if directory does not have a file already',
         () {