review fix

Author: raju-opti

lib/error/error_notifier.spec.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { describe, it, expect, vi } from 'vitest';
 
 import { DefaultErrorNotifier } from './error_notifier';
@@ -21,7 +36,7 @@ describe('DefaultErrorNotifier', () => {
     expect(errorHandler.handleError).toHaveBeenCalledWith(error);
   });
 
-  it.only('should resolve the message of an OptimizelyError before calling the error handler', () => {
+  it('should resolve the message of an OptimizelyError before calling the error handler', () => {
     const errorHandler = { handleError: vi.fn() };
     const messageResolver = mockMessageResolver('err');
     const errorNotifier = new DefaultErrorNotifier(errorHandler, messageResolver);

lib/error/error_notifier.ts

@@ -1,5 +1,19 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { MessageResolver } from "../message/message_resolver";
-import { sprintf } from "../utils/fns";
 import { ErrorHandler } from "./error_handler";
 import { OptimizelyError } from "./optimizly_error";
 

lib/error/error_notifier_factory.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { errorResolver } from "../message/message_resolver";
 import { ErrorHandler } from "./error_handler";
 import { DefaultErrorNotifier } from "./error_notifier";

lib/error/error_reporter.spec.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { describe, it, expect, vi } from 'vitest';
 
 import { ErrorReporter } from './error_reporter';

lib/error/error_reporter.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { LoggerFacade } from "../logging/logger";
 import { ErrorNotifier } from "./error_notifier";
 import { OptimizelyError } from "./optimizly_error";

lib/error/optimizly_error.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { MessageResolver } from "../message/message_resolver";
 import { sprintf } from "../utils/fns";
 

lib/logging/logger.spec.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { describe, beforeEach, afterEach, it, expect, vi, afterAll } from 'vitest';
 
 import { ConsoleLogHandler, LogLevel, OptimizelyLogger } from './logger';
@@ -15,7 +30,7 @@ describe('ConsoleLogHandler', () => {
     debugSpy.mockClear();
     infoSpy.mockClear();
     warnSpy.mockClear();
-    vi.useFakeTimers().setSystemTime((0));
+    vi.useFakeTimers().setSystemTime(0);
   });
 
   afterEach(() => {
@@ -69,7 +84,7 @@ describe('ConsoleLogHandler', () => {
     logger.log(LogLevel.Error, 'error message');
 
     expect(infoSpy).toHaveBeenCalledWith('[OPTIMIZELY] - INFO 1970-01-01T00:00:00.000Z info message');
-    expect(debugSpy).toHaveBeenCalledWith('[OPTIMIZELY] - DEGUB 1970-01-01T00:00:00.000Z debug message');
+    expect(debugSpy).toHaveBeenCalledWith('[OPTIMIZELY] - DEBUG 1970-01-01T00:00:00.000Z debug message');
     expect(warnSpy).toHaveBeenCalledWith('[OPTIMIZELY] - WARN 1970-01-01T00:00:00.000Z warn message');
     expect(errorSpy).toHaveBeenCalledWith('[OPTIMIZELY] - ERROR 1970-01-01T00:00:00.000Z error message');
   });
@@ -82,7 +97,7 @@ describe('ConsoleLogHandler', () => {
     logger.log(LogLevel.Error, 'error message');
 
     expect(infoSpy).toHaveBeenCalledWith('PREFIX - INFO 1970-01-01T00:00:00.000Z info message');
-    expect(debugSpy).toHaveBeenCalledWith('PREFIX - DEGUB 1970-01-01T00:00:00.000Z debug message');
+    expect(debugSpy).toHaveBeenCalledWith('PREFIX - DEBUG 1970-01-01T00:00:00.000Z debug message');
     expect(warnSpy).toHaveBeenCalledWith('PREFIX - WARN 1970-01-01T00:00:00.000Z warn message');
     expect(errorSpy).toHaveBeenCalledWith('PREFIX - ERROR 1970-01-01T00:00:00.000Z error message');
   });

lib/logging/logger.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright 2019, 2024, Optimizely
+ * Copyright 2019, 2024, 2025, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the License");
  * you may not use this file except in compliance with the License.
@@ -25,7 +25,7 @@ export enum LogLevel {
 }
 
 export const LogLevelToUpper: Record<LogLevel, string> = {
-  [LogLevel.Debug]: 'DEGUB',
+  [LogLevel.Debug]: 'DEBUG',
   [LogLevel.Info]: 'INFO',
   [LogLevel.Warn]: 'WARN',
   [LogLevel.Error]: 'ERROR',
@@ -133,8 +133,6 @@ export class OptimizelyLogger implements LoggerFacade {
   }
 
   private log(level: LogLevel, message: string | Error, args: any[]): void {
-    console.log('level', level, 'this.level', this.level, message)
-
     if (level < this.level) {
       return;
     }

lib/logging/logger_factory.spec.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { describe, it, expect, beforeEach, vi } from 'vitest';
 
 vi.mock('./logger', async (importOriginal) => {

lib/logging/logger_factory.ts

@@ -1,3 +1,18 @@
+/**
+ * Copyright 2025, Optimizely
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 import { ConsoleLogHandler, LogHandler, LogLevel, OptimizelyLogger } from './logger';
 import { errorResolver, infoResolver, MessageResolver } from '../message/message_resolver';