Refatora configuração do roteador para usar array de origens CORS e simplifica a lógica de configuração

Author: Joaolpridolficarvalho

fontes/infraestrutura/centro-configuracoes/configuracao-roteador.ts

@@ -21,19 +21,13 @@ export class ConfiguracaoRoteador extends ConfiguracaoComum {
         const roteador = componentes['roteador'] as RoteadorInterface;
         roteador.ativarDesativarBodyParser(this.bodyParser);
         roteador.ativarDesativarCors(this.cors);
-        if (typeof roteador.configurarOrigensCors === 'function') {
-            roteador.configurarOrigensCors(this.origensCors);
-        } else if (typeof (roteador as any).configurarCors === 'function') {
-            (roteador as any).configurarCors(this.origensCors);
-        }
+        roteador.configurarOrigensCors(this.origensCors);
         roteador.ativarDesativarCookieParser(this.cookieParser);
         roteador.ativarDesativarExpressJson(this.json);
         roteador.ativarDesativarHelmet(this.helmet);
         roteador.ativarDesativarMorgan(this.morgan);
         roteador.ativarDesativarPassport(this.passport);
-        if (typeof roteador.configurarPorta === 'function') {
-            roteador.configurarPorta(this.porta);
-        }
+        roteador.configurarPorta(this.porta);
         // Nota: configurarArquivosEstaticos é chamado em liquido.ts com caminho absoluto
     }
 }

fontes/infraestrutura/roteador/roteador.ts

@@ -32,7 +32,7 @@ export class Roteador implements RoteadorInterface {
     bodyParser = false;
 
     cors = false;
-    origensCors: string | string[] = '*';
+    origensCors: string[] = ['*'];
     passport = false;
 
     constructor(autoDocumentador: AutoDocumentador) {
@@ -163,7 +163,8 @@ export class Roteador implements RoteadorInterface {
             return;
         }
 
-        this.origensCors = origem && origem.trim().length > 0 ? origem : '*';
+        this.origensCors = origem.split(',').
+        map(origem => origem.trim()).filter(origem => origem.length > 0);
     }
 
     /**
@@ -173,22 +174,17 @@ export class Roteador implements RoteadorInterface {
      *          lista de origens permitidas.
      */
     resolverOpcoesCors(): { origin: string[] } | undefined {
-        if (this.origensCors === '*') {
+        
+        if (Array.isArray(this.origensCors)
+            && this.origensCors.length === 1
+            && this.origensCors[0] === '*') {
             return undefined;
         }
-
-        const origens = Array.isArray(this.origensCors)
-            ? this.origensCors
-            : this.origensCors
-                .split(',')
-                .map(origem => origem.trim())
-                .filter(origem => origem.length > 0);
-
-        if (origens.length === 0) {
+        if (this.origensCors.length === 0) {
             return undefined;
         }
 
-        return { origin: origens };
+        return { origin: this.origensCors };
     }
 
     ativarDesativarPassport(valor: boolean): void {

testes/infraestrutura/centro-configuracoes.test.ts

@@ -198,21 +198,7 @@ describe('Testes das classes de configuração', () => {
             expect(roteador.configurarArquivosEstaticos).not.toHaveBeenCalled();
         });
 
-        it('deve repassar origensCors ao roteador sem depender de configurarPorta', () => {
-            const config = new ConfiguracaoRoteador({ origensCors: ['https://exemplo.com'] });
-            const roteador = {
-                ativarDesativarBodyParser: jest.fn(),
-                ativarDesativarCors: jest.fn(),
-                ativarDesativarCookieParser: jest.fn(),
-                ativarDesativarExpressJson: jest.fn(),
-                ativarDesativarHelmet: jest.fn(),
-                ativarDesativarMorgan: jest.fn(),
-                ativarDesativarPassport: jest.fn(),
-                configurarOrigensCors: jest.fn()
-            };
-            config.configurar({ roteador });
-            expect(roteador.configurarOrigensCors).toHaveBeenCalledWith(['https://exemplo.com']);
-        });
+
 
         it('não deve chamar configurarArquivosEstaticos quando diretorioEstatico for vazio', () => {
             const config = new ConfiguracaoRoteador({ diretorioEstatico: '' });

testes/infraestrutura/cors-origem.test.ts

@@ -103,7 +103,8 @@ describe('CORS com origem configurável', () => {
                 ativarDesativarExpressJson: jest.fn(),
                 ativarDesativarHelmet: jest.fn(),
                 ativarDesativarMorgan: jest.fn(),
-                ativarDesativarPassport: jest.fn()
+                ativarDesativarPassport: jest.fn(),
+                configurarPorta: jest.fn(),
             };
 
             configuracao.configurar({ roteador: roteadorSimulado });