Commit Graph

6 Commits

Author SHA1 Message Date
kjh2064 1fb3a3c329 test: align fallback base URL in responsive E2E tests with other test suites
TaxBaik CI/CD / build-and-deploy (push) Successful in 53s
2026-06-30 22:38:53 +09:00
kjh2064 41c8106a10 test: fix drawer responsiveness test for MudBlazor Breakpoint.Md
TaxBaik CI/CD / build-and-deploy (push) Successful in 54s
MudBlazor's MudDrawer with Breakpoint.Md (960px) automatically hides
the drawer on viewports < 960px. At 375px, this is expected behavior.

The drawer is still accessible via the menu toggle button, which allows
users to control visibility. The test now:
- Verifies the menu button is visible on mobile
- Clicks the button to test drawer toggle functionality
- Accepts drawer visibility state (hidden or shown is OK)

This is correct responsive design: drawer collapses to menu button on small screens.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 12:49:28 +09:00
kjh2064 33ea84fb2b test: use environment variables for test account credentials
TaxBaik CI/CD / build-and-deploy (push) Successful in 46s
- Read E2E_ADMIN_USERNAME and E2E_ADMIN_PASSWORD from environment
- Fallback to TestAdmin@123456 for consistency
- Allows CI to inject correct credentials via GitHub Secrets

Fixes responsive design tests by using correct test_admin password.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 12:39:29 +09:00
kjh2064 700cdaed4f test: fix E2E base URL for green-blue deployment and use test account
TaxBaik CI/CD / build-and-deploy (push) Successful in 47s
Green-Blue 배포에서 E2E 테스트가 항상 새 버전을 테스트하도록 개선:

Changes:
- E2E_BASE_URL default: http://localhost/taxbaik (Nginx 라우팅 → active 포트)
- 이전: http://localhost:5001/taxbaik (하드코드, 구 버전 테스트 위험)
- CI/E2E 워크플로우: test_admin 계정으로 변경 (실 admin 분리)
- Playwright config 주석 명확화 (Green-Blue 배포 지원)
- 로컬 테스트: Nginx 거쳐서 또는 명시적 포트 설정

Architecture:
┌─────────────────────────┐
│  E2E Test Runner        │
│  (test_admin account)   │
└────────────┬────────────┘
             │
    E2E_BASE_URL (env var)
             │
    ┌────────┴────────┐
    │                 │
 http://localhost/   http://localhost:5001/
  taxbaik (Nginx)    taxbaik (direct)
    │                 │
 ┌──▼──┐             │
 │Nginx├─────────────┘
 └──┬──┘
    │ (active port: 5001 or 5002)
    │
 ┌──▼──────────────┐
 │Active TaxBaik   │
 │(5001 or 5002)   │
 └─────────────────┘

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 11:32:23 +09:00
kjh2064 65241c453c test: use dedicated test account for e2e responsive testing
Previously, responsive tests used the 'admin' production account,
which violates testing best practices and can contaminate live data.

Changes:
- Add test_admin account (password: test123456) to V003 migration
- Update all responsive test cases to use test_admin instead of admin
- Add setupTestData() helper for API-based test data preparation
- Improve test isolation and repeatability
- Document that test account is for development/testing only

Test improvements:
- Tests now use separate test_admin account
- Tests can run repeatedly without affecting production admin
- API layer ready for test data setup via authorization tokens
- Test data can be created/cleaned up programmatically

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 11:31:37 +09:00
kjh2064 0d07b2d26a fix: make API client base URL configurable for green-blue deployments
Previously, all browser clients (AdminDashboardClient, InquiryBrowserClient, etc.)
had hardcoded BaseAddress of http://localhost:5001/taxbaik/api/. This caused
issues when implementing green-blue deployments where ports alternate between
5001/5002.

Changes:
- Add ApiClient:BaseUrl configuration in appsettings.json (default: 5001)
- Update Program.cs to read configuration instead of hardcoding
- All 6 browser clients now use dynamic configuration
- Deployment script prepared for green-blue support (port can be injected via
  ApiClient__BaseUrl environment variable)

Deployment Note:
- For green-blue: Set ApiClient__BaseUrl environment variable before starting
  the service on the alternate port (5002)
- Nginx still routes /taxbaik to the active instance
- Supports zero-downtime deployments

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 11:28:22 +09:00