feat: Made the suggested changes

Author: KrishDave1

spec/ParseUser.spec.js

@@ -4431,30 +4431,30 @@ describe('allowClientClassCreation option', () => {
   });
 
   it('should respect custom log level for username already exists error', async () => {
-    const loggerController = {
-      error: jasmine.createSpy('error'),
-      warn: jasmine.createSpy('warn'),
-      info: jasmine.createSpy('info')
-    };
+    const logger = require('../lib/logger').logger;
+    const logSpy = spyOn(logger, 'warn').and.callThrough();
+
     await reconfigureServer({
       logEvents: {
-        usernameAlreadyExists: 'warn'
+        usernameAlreadyExists: 'warn',
       },
-      loggerController
     });
+
     const user = new Parse.User();
     user.setUsername('existingUser');
     user.setPassword('password');
     await user.signUp();
+
     const duplicateUser = new Parse.User();
     duplicateUser.setUsername('existingUser');
     duplicateUser.setPassword('password');
+
     try {
       await duplicateUser.signUp();
     } catch (error) {
       expect(error.code).toBe(Parse.Error.USERNAME_TAKEN);
-      expect(loggerController.warn).toHaveBeenCalled();
-      expect(loggerController.error).not.toHaveBeenCalled();
+      expect(logSpy).toHaveBeenCalled();
+      expect(logger.error).not.toHaveBeenCalled();
     }
   });
 });

src/Options/docs.js

@@ -89,9 +89,6 @@ export interface ParseServerOptions {
   logLevel: ?string;
   /* Options for customizing log levels for specific events
   :DEFAULT: {} */
-  logEvents: ?LogEventsOptions;
-  /* (Optional) Overrides the log levels used internally by Parse Server to log events.
-  :DEFAULT: {} */
   logLevels: ?LogLevels;
   /* Maximum number of logs to keep. If not set, no logs will be removed. This can be a number of files or number of days. If using days, add 'd' as the suffix. (default: null) */
   maxLogFiles: ?NumberOrString;
@@ -638,12 +635,10 @@ export interface LogLevels {
   :DEFAULT: error
   */
   cloudFunctionError: ?string;
-}
-
-export interface LogEventsOptions {
   /* Log level for the "username already exists" error when trying to sign up.
-     Possible values: 'info', 'error', 'verbose', or false to disable logging.
-     If not specified, the default behavior (logging as an error) will be used.
-  :DEFAULT: undefined */
-  usernameAlreadyExists: ?string | boolean;
+  Possible values: 'info', 'error', 'verbose', or false to disable logging.
+  If not specified, the default behavior (logging as an error) will be used.
+  :DEFAULT: undefined 
+  */
+  usernameAlreadyExists: 'silent' | 'error' | 'info' | 'verbose';
 }

src/Options/index.js

@@ -1563,16 +1563,9 @@ RestWrite.prototype.runDatabaseOperation = function () {
             'Account already exists for this username.'
           );
 
-          // Check for custom log level
-          const logLevel = this.config.logEvents && this.config.logEvents.usernameAlreadyExists;
-          if (logLevel) {
-            // Use custom log level
-            this.config.loggerController[logLevel](JSON.stringify(usernameError));
-          } else if (logLevel !== false) {
-            // Use default error logging if not explicitly disabled
-            this.config.loggerController.error(JSON.stringify(usernameError));
-          }
-          throw usernameError;
+          return this.config.loggerController[this.config.logLevels.usernameAlreadyExists](
+            JSON.stringify(usernameError)
+          );
         }
 
         if (error && error.userInfo && error.userInfo.duplicated_field === 'email') {