test: fix 4 testes drifted pós-Lovable reimport (substitui PR #125 ob…

src/components/layout/AppLogo.visual.test.tsx

@@ -17,7 +17,7 @@ describe('AppLogo Visual Consistency', () => {
     expect(iconContainer).toBeInTheDocument();
     const icon = iconContainer?.querySelector('svg');
     expect(icon).toHaveClass('text-primary-foreground');
-    expect(iconContainer).toHaveClass('h-9 w-9');
+    expect(iconContainer).toHaveClass('h-10 w-10');
   });
 
   it('renders light variant with primary background and primary foreground icon', () => {

src/components/quotes/__tests__/QuoteBuilderDiscountAdvanced.test.tsx

@@ -1,13 +1,14 @@
 import { describe, it, expect, vi } from 'vitest';
-import { render, screen, fireEvent, waitFor } from '@testing-library/react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import React from 'react';
 import { QuoteBuilderSummaryColumn } from '../QuoteBuilderSummaryColumn';
+import type { QuoteItem } from '@/hooks/quotes';
 
 describe('QuoteBuilderSummaryColumn Advanced Discount Scenarios', () => {
   const formatCurrency = (v: number) => `R$ ${v.toLocaleString('pt-BR', { minimumFractionDigits: 2 })}`;
 
   const defaultProps = {
-    items: [{ id: '1', product_name: 'Item 1', quantity: 1, unit_price: 1000, product_sku: 'SKU-1' } as any],
+    items: [{ id: '1', product_name: 'Item 1', quantity: 1, unit_price: 1000, product_sku: 'SKU-1' } as unknown as QuoteItem],
     activeItemIndex: null,
     setActiveItemIndex: vi.fn(),
     removeItem: vi.fn(),
@@ -34,7 +35,7 @@ describe('QuoteBuilderSummaryColumn Advanced Discount Scenarios', () => {
     const setDiscountValue = vi.fn();
     render(<QuoteBuilderSummaryColumn {...defaultProps} setDiscountValue={setDiscountValue} />);
 
-    const input = screen.getByPlaceholderText('0%');
+    const input = screen.getByTestId('quote-discount-input');
     fireEvent.change(input, { target: { value: '110' } });
 
     // CurrencyInput should show range error
@@ -51,7 +52,7 @@ describe('QuoteBuilderSummaryColumn Advanced Discount Scenarios', () => {
       />
     );
 
-    const input = screen.getByPlaceholderText('R$ 0,00');
+    const input = screen.getByTestId('quote-discount-input');
     fireEvent.change(input, { target: { value: '1500' } });
 
     expect(await screen.findByText(/Valor máximo é/)).toBeInTheDocument();
@@ -72,7 +73,7 @@ describe('QuoteBuilderSummaryColumn Advanced Discount Scenarios', () => {
       />
     );
 
-    const input = screen.getByPlaceholderText('R$ 0,00');
+    const input = screen.getByTestId('quote-discount-input');
 
     // Should allow 1100
     fireEvent.change(input, { target: { value: '1100' } });

src/pages/auth/AuthBranding.test.tsx

@@ -1,8 +1,8 @@
 import { render, act } from '@testing-library/react';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { ContinuousRockets } from "@/pages/auth/AuthBranding";
+import { SpaceScene } from "@/pages/auth/AuthBranding";
 
-// Mock lucide-react to avoid icon rendering issues in test
+// Mock lucide-react para evitar render real de ícones no teste.
 vi.mock('lucide-react', () => ({
   Rocket: () => <div data-testid="rocket-icon" />,
   Gift: () => <div />,
@@ -12,11 +12,14 @@ vi.mock('lucide-react', () => ({
   Brain: () => <div />,
 }));
 
-
-// Tests for the rocket animation in the branding panel.
-
-
-describe('ContinuousRockets Component', () => {
+/**
+ * A animação de foguetes foi inlinada de ContinuousRockets para dentro de
+ * SpaceScene (junto a planetas/astronautas/meteoros). O spawn agora é por
+ * setInterval(2000ms) — sem o array de delays fixo do componente antigo.
+ * Estes testes validam a mesma intenção: montar sem crash e spawnar/remover
+ * foguetes ao longo do tempo, sem fixar contagens frágeis de internals.
+ */
+describe('SpaceScene — animação de foguetes', () => {
   beforeEach(() => {
     vi.useFakeTimers();
   });
@@ -26,52 +29,39 @@ describe('ContinuousRockets Component', () => {
   });
 
   it('renders without crashing', () => {
-    const { container } = render(<ContinuousRockets />);
+    const { container } = render(<SpaceScene />);
     expect(container.firstChild).toBeInTheDocument();
   });
 
-  it('spawns initial rockets after delays', async () => {
-    const { getAllByTestId } = render(<ContinuousRockets />);
-
-    // The component has: const delays = [0, 200, 500, 900, 1400, 2000, 2800];
-
-    act(() => {
-      vi.advanceTimersByTime(2000);
-    });
+  it('spawns rockets over time (interval-based)', () => {
+    const { queryAllByTestId } = render(<SpaceScene />);
 
-    // At 2000ms, 6 rockets should have spawned (0, 200, 500, 900, 1400, 2000)
-    expect(getAllByTestId('rocket-icon').length).toBe(6);
+    // Sem foguetes imediatamente no mount (spawn é disparado pelo intervalo).
+    const initial = queryAllByTestId('rocket-icon').length;
 
     act(() => {
-      vi.advanceTimersByTime(1000);
+      vi.advanceTimersByTime(7000); // > 3 ciclos de 2000ms
     });
 
-    // After 3000ms, all 7 initial rockets should have spawned (at 0, 0.2, 0.5, 0.9, 1.4, 2.0, 2.8s)
-    expect(getAllByTestId('rocket-icon').length).toBeGreaterThanOrEqual(7);
+    expect(queryAllByTestId('rocket-icon').length).toBeGreaterThanOrEqual(initial);
+    expect(queryAllByTestId('rocket-icon').length).toBeGreaterThan(0);
   });
 
-  it('removes rockets after their duration', async () => {
-    const { getAllByTestId, queryAllByTestId } = render(<ContinuousRockets />);
+  it('removes rockets after their duration', () => {
+    const { queryAllByTestId } = render(<SpaceScene />);
 
     act(() => {
-      vi.advanceTimersByTime(3000);
+      vi.advanceTimersByTime(6000);
     });
+    const peak = queryAllByTestId('rocket-icon').length;
+    expect(peak).toBeGreaterThan(0);
 
-    const initialCount = getAllByTestId('rocket-icon').length;
-    expect(initialCount).toBe(7);
-
-    // Rocket duration is 1.5-3s (initial) + 0.5s removal delay
-    // Advancing 8 seconds should clear all initial rockets
+    // Cada foguete se auto-remove após sua duration; avançar bastante deve
+    // manter a contagem limitada (não cresce indefinidamente).
     act(() => {
-      vi.advanceTimersByTime(8000);
+      vi.advanceTimersByTime(20000);
     });
-
-    // They should be removed, but new ones spawn every 2.8s
-    // At 11s total:
-    // Sustained cycle starts after mount. 
-    // Spawns at: 2.8s, 5.6s, 8.4s.
-    // At 11s, those might still be there or removed depending on duration (2.2-5s)
-    const currentCount = queryAllByTestId('rocket-icon').length;
-    expect(currentCount).toBeLessThan(7);
+    const later = queryAllByTestId('rocket-icon').length;
+    expect(later).toBeLessThanOrEqual(peak + 10);
   });
 });

src/pages/auth/AuthBranding.visual.test.tsx

@@ -20,14 +20,6 @@ vi.mock('@/components/layout/AppLogo', () => ({
   AppLogo: () => <div data-testid="app-logo" />,
 }));
 
-vi.mock('./AuthBranding', async () => {
-  const actual = await vi.importActual('./AuthBranding') as any;
-  return {
-    ...actual,
-    ContinuousRockets: () => <div data-testid="rockets" />,
-  };
-});
-
 describe('AuthBrandingPanel Visual Classes', () => {
   it('has correct responsive width and margin classes on the grid container', () => {
     const { container } = render(
@@ -40,11 +32,8 @@ describe('AuthBrandingPanel Visual Classes', () => {
     expect(grid).toBeInTheDocument();
 
     const classes = grid?.className || '';
+    // Layout atual do painel: grid full-width, sem overflow lateral (-mx) do design antigo.
     expect(classes).toContain('w-full');
-    expect(classes).toContain('lg:w-[105%]');
-    expect(classes).toContain('xl:w-[110%]');
-    expect(classes).toContain('lg:-mx-[2.5%]');
-    expect(classes).toContain('xl:-mx-[5%]');
   });
 
   it('has correct padding and gap classes', () => {
@@ -58,11 +47,11 @@ describe('AuthBrandingPanel Visual Classes', () => {
     expect(grid?.className).toContain('gap-3');
     expect(grid?.className).toContain('sm:gap-5');
 
-    const cards = container.querySelectorAll('.rounded-2xl');
+    const cards = container.querySelectorAll('.rounded-3xl');
+    expect(cards.length).toBeGreaterThan(0);
     cards.forEach(card => {
-      expect(card.className).toContain('px-4');
-      expect(card.className).toContain('sm:px-6');
-      expect(card.className).toContain('h-[99px]');
+      expect(card.className).toContain('px-5');
+      expect(card.className).toContain('h-[88px]');
     });
   });