@onlineapps/conn-base-storage 1.0.4 → 1.0.5

package/tests/unit/internal-url-adapter.test.js

@@ -9,8 +9,6 @@ describe('InternalUrlAdapter @unit', () => {
 
   beforeEach(() => {
     adapter = new InternalUrlAdapter();
-    // Tests run inside a Docker container; force local-mode behavior unless explicitly enabled.
-    adapter.isRunningInDocker = jest.fn(() => false);
   });
 
   describe('URL Resolution', () => {
@@ -27,7 +25,7 @@ describe('InternalUrlAdapter @unit', () => {
       const internalUrl = 'internal://storage/bucket/path/to/file.json';
       const resolved = adapter.resolve(internalUrl);
 
-      expect(resolved).toBe('http://localhost:9000/bucket/path/to/file.json');
+      expect(resolved).toBe('http://127.0.0.1:9000/bucket/path/to/file.json');
     });
 
     test('should resolve internal://registry URL', () => {
@@ -35,7 +33,7 @@ describe('InternalUrlAdapter @unit', () => {
       const internalUrl = 'internal://registry/services/list';
       const resolved = adapter.resolve(internalUrl);
 
-      expect(resolved).toBe('http://localhost:8080/services/list');
+      expect(resolved).toBe('http://127.0.0.1:8080/services/list');
     });
 
     test('should resolve internal://monitoring URL', () => {
@@ -58,7 +56,7 @@ describe('InternalUrlAdapter @unit', () => {
       const internalUrl = 'internal://storage/bucket/file?version=2&download=true';
       const resolved = adapter.resolve(internalUrl);
 
-      expect(resolved).toBe('http://localhost:9000/bucket/file?version=2&download=true');
+      expect(resolved).toBe('http://127.0.0.1:9000/bucket/file?version=2&download=true');
     });
 
     test('should throw error for unknown service', () => {
@@ -77,7 +75,7 @@ describe('InternalUrlAdapter @unit', () => {
     });
 
     test('should preserve query parameters in conversion', () => {
-      const actualUrl = 'http://localhost:9000/bucket/file?version=2';
+      const actualUrl = 'http://127.0.0.1:9000/bucket/file?version=2';
       const internal = adapter.toInternal(actualUrl, 'storage');
 
       expect(internal).toBe('internal://storage/bucket/file?version=2');
@@ -86,7 +84,7 @@ describe('InternalUrlAdapter @unit', () => {
 
   describe('Service Registration', () => {
     test('should register new service', () => {
-      adapter.registerService('custom', 'custom-container:8080', 'localhost:8080');
+      adapter.registerService('custom', 'custom-container:8080', '127.0.0.1:8080');
 
       const internalUrl = 'internal://custom/api/endpoint';
       adapter.dockerNetwork = true;
@@ -96,13 +94,13 @@ describe('InternalUrlAdapter @unit', () => {
     });
 
     test('should override existing service', () => {
-      adapter.registerService('storage', 'new-storage:9001', 'localhost:9001');
+      adapter.registerService('storage', 'new-storage:9001', '127.0.0.1:9001');
 
       const internalUrl = 'internal://storage/bucket/file';
       adapter.dockerNetwork = false;
       const resolved = adapter.resolve(internalUrl);
 
-      expect(resolved).toBe('http://localhost:9001/bucket/file');
+      expect(resolved).toBe('http://127.0.0.1:9001/bucket/file');
     });
   });
 
@@ -120,9 +118,9 @@ describe('InternalUrlAdapter @unit', () => {
 
       const transformed = adapter.transformResponse(response);
 
-      expect(transformed.url).toBe('http://localhost:9000/bucket/file.json');
-      expect(transformed.backup).toBe('http://localhost:9000/backup/file.json');
-      expect(transformed.metadata.thumbnail).toBe('http://localhost:9000/thumbnails/thumb.jpg');
+      expect(transformed.url).toBe('http://127.0.0.1:9000/bucket/file.json');
+      expect(transformed.backup).toBe('http://127.0.0.1:9000/backup/file.json');
+      expect(transformed.metadata.thumbnail).toBe('http://127.0.0.1:9000/thumbnails/thumb.jpg');
     });
 
     test('should handle arrays in response', () => {
@@ -138,8 +136,8 @@ describe('InternalUrlAdapter @unit', () => {
 
       const transformed = adapter.transformResponse(response);
 
-      expect(transformed.files[0]).toBe('http://localhost:9000/file1.json');
-      expect(transformed.files[1]).toBe('http://localhost:9000/file2.json');
+      expect(transformed.files[0]).toBe('http://127.0.0.1:9000/file1.json');
+      expect(transformed.files[1]).toBe('http://127.0.0.1:9000/file2.json');
       expect(transformed.files[2]).toBe('http://external.com/file3.json');
     });
 
@@ -165,19 +163,18 @@ describe('InternalUrlAdapter @unit', () => {
   });
 
   describe('Environment Detection', () => {
-    test('should detect Docker environment from env variable', () => {
-      const originalEnv = process.env.DOCKER_ENV;
+    test('should respect DOCKER_NETWORK env hint via runtime config', () => {
+      const original = process.env.DOCKER_NETWORK;
+      process.env.DOCKER_NETWORK = 'true';
 
-      process.env.DOCKER_ENV = 'true';
       const adapter1 = new InternalUrlAdapter();
-      expect(adapter1.isRunningInDocker()).toBe(true);
+      expect(adapter1.dockerNetwork).toBe(true);
 
-      delete process.env.DOCKER_ENV;
-      const adapter2 = new InternalUrlAdapter();
-      // Může být stále true pokud běžíme v Dockeru
-
-      // Obnovit původní hodnotu
-      if (originalEnv) process.env.DOCKER_ENV = originalEnv;
+      if (original === undefined) {
+        delete process.env.DOCKER_NETWORK;
+      } else {
+        process.env.DOCKER_NETWORK = original;
+      }
     });
   });
 
@@ -199,17 +196,15 @@ describe('InternalUrlAdapter @unit', () => {
         services: {
           'my-service': {
             docker: 'my-container:3000',
-            local: 'localhost:3000'
+            local: '127.0.0.1:3000'
           }
         }
       });
 
-      // Tests run inside Docker; force local-mode behavior for this adapter instance.
-      customAdapter.isRunningInDocker = jest.fn(() => false);
       customAdapter.dockerNetwork = false;
       const resolved = customAdapter.resolve('internal://my-service/api');
 
-      expect(resolved).toBe('http://localhost:3000/api');
+      expect(resolved).toBe('http://127.0.0.1:3000/api');
     });
   });
 });