optimizely
diff --git a/‎lib/modules/datafile-manager/httpPollingDatafileManager.ts
Lines changed: 2 additions & 1 deletion b/‎lib/modules/datafile-manager/httpPollingDatafileManager.ts
Lines changed: 2 additions & 1 deletion
diff --git a/‎tests/httpPollingDatafileManager.spec.ts
Lines changed: 31 additions & 1 deletion b/‎tests/httpPollingDatafileManager.spec.ts
Lines changed: 31 additions & 1 deletion
@@ -1,5 +1,5 @@
 /**
- * Copyright 2022-2023, Optimizely
+ * Copyright 2022-2024, Optimizely
  *
  * you may not use this file except in compliance with the License.
@@ -325,6 +325,7 @@ export default abstract class HttpPollingDatafileManager implements DatafileMana
     if (isSuccessStatusCode(response.statusCode)) {
       return response.body;
     }
+    logger.error(`Datafile fetch request failed with status: ${response.statusCode}`);
     return '';
   }
 
 
@@ -1,5 +1,5 @@
 /**
- * Copyright 2022, Optimizely
+ * Copyright 2022, 2024, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,6 +20,7 @@ import { DatafileManagerConfig } from '../lib/modules/datafile-manager/datafileM
 import { advanceTimersByTime, getTimerCount } from './testUtils';
 import PersistentKeyValueCache from '../lib/modules/datafile-manager/persistentKeyValueCache';
 
+
 jest.mock('../lib/modules/datafile-manager/backoffController', () => {
   return jest.fn().mockImplementation(() => {
     const getDelayMock = jest.fn().mockImplementation(() => 0);
@@ -32,6 +33,8 @@ jest.mock('../lib/modules/datafile-manager/backoffController', () => {
 });
 
 import BackoffController from '../lib/modules/datafile-manager/backoffController';
+import { LoggerFacade, getLogger } from '../lib/modules/logging';
+import { resetCalls, spy, verify } from 'ts-mockito';
 
 // Test implementation:
 //   - Does not make any real requests: just resolves with queued responses (tests push onto queuedResponses)
@@ -93,8 +96,19 @@ const testCache: PersistentKeyValueCache = {
 };
 
 describe('httpPollingDatafileManager', () => {
+
+  let spiedLogger: LoggerFacade;
+
+  const loggerName = 'DatafileManager';
+
+  beforeAll(() => {
+      const actualLogger = getLogger(loggerName);
+      spiedLogger = spy(actualLogger);
+  });
+    
   beforeEach(() => {
     jest.useFakeTimers();
+    resetCalls(spiedLogger);
   });
 
   let manager: TestDatafileManager;
@@ -179,6 +193,22 @@ describe('httpPollingDatafileManager', () => {
       manager = new TestDatafileManager({ sdkKey: '123', updateInterval: 1000, autoUpdate: true });
     });
 
+    it('logs an error if fetching datafile fails', async () => {
+      manager.queuedResponses.push(
+        {
+          statusCode: 500,
+          body: '',
+          headers: {},
+        }
+      );
+
+      manager.start();
+      await advanceTimersByTime(1000);
+      await manager.responsePromises[0];
+
+      verify(spiedLogger.error('Datafile fetch request failed with status: 500')).once();
+    });
+
     describe('initial state', () => {
       it('returns null from get before becoming ready', () => {
         expect(manager.get()).toEqual('');
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/**`
`2`		`- * Copyright 2022-2023, Optimizely`
	`2`	`+ * Copyright 2022-2024, Optimizely`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -325,6 +325,7 @@ export default abstract class HttpPollingDatafileManager implements DatafileMana`
`325`	`325`	`if (isSuccessStatusCode(response.statusCode)) {`
`326`	`326`	`return response.body;`
`327`	`327`	`}`
	`328`	+ logger.error(`Datafile fetch request failed with status: ${response.statusCode}`);
`328`	`329`	`return '';`
`329`	`330`	`}`
`330`	`331`