Skip to content

Commit ff2115f

Browse files
luanweslley77luanweslley77
committed
test: add 104 unit tests across 6 files with QWEN_TEST_CREDS_PATH isolation
- Comprehensive test suite: 104 unit tests, 197 assertions - Test files: errors.test.ts, oauth.test.ts, file-lock.test.ts, token-manager.test.ts, request-queue.test.ts, platform.test.ts - Integration tests and multi-process stress tests - Test isolation via QWEN_TEST_CREDS_PATH (prevents modifying user credentials) - Complete documentation in tests/README.md - Test reorganization and safety improvements
1 parent d64b8f5 commit ff2115f

17 files changed

Lines changed: 1675 additions & 173 deletions

‎.gitignore‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,6 @@ dist/
33
*.log
44
.DS_Store
55
package-lock.json
6+
opencode.json
7+
reference/
8+
bunfig.toml

‎package.json‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,12 @@
77
"scripts": {
88
"build": "bun build ./src/index.ts --outdir ./dist --target node --format esm && bun build ./src/cli.ts --outdir ./dist --target node --format esm",
99
"dev": "bun run --watch src/index.ts",
10-
"typecheck": "tsc --noEmit"
10+
"typecheck": "tsc --noEmit",
11+
"test": "bun test",
12+
"test:watch": "bun test --watch",
13+
"test:integration": "bun run tests/integration/debug.ts full",
14+
"test:race": "bun run tests/integration/race-condition.ts",
15+
"test:robust": "bun run tests/robust/runner.ts"
1116
},
1217
"keywords": [
1318
"opencode",
@@ -39,7 +44,8 @@
3944
"@opencode-ai/plugin": "^1.1.48",
4045
"@types/node": "^22.0.0",
4146
"bun-types": "^1.1.0",
42-
"typescript": "^5.6.0"
47+
"typescript": "^5.6.0",
48+
"vitest": "^1.0.0"
4349
},
4450
"files": [
4551
"index.ts",

‎src/plugin/auth.ts‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,13 @@ import { QWEN_API_CONFIG } from '../constants.js';
1414

1515
/**
1616
* Get the path to the credentials file
17+
* Supports test override via QWEN_TEST_CREDS_PATH environment variable
1718
*/
1819
export function getCredentialsPath(): string {
20+
// Check for test override (prevents tests from modifying user credentials)
21+
if (process.env.QWEN_TEST_CREDS_PATH) {
22+
return process.env.QWEN_TEST_CREDS_PATH;
23+
}
1924
const homeDir = homedir();
2025
return join(homeDir, '.qwen', 'oauth_creds.json');
2126
}

‎src/plugin/token-manager.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -384,5 +384,6 @@ class TokenManager {
384384
}
385385
}
386386

387+
export { TokenManager };
387388
// Singleton instance
388389
export const tokenManager = new TokenManager();

‎src/qwen/oauth.ts‎

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -160,18 +160,14 @@ export async function pollDeviceToken(
160160
throw new SlowDownError();
161161
}
162162

163-
const error = new Error(
163+
throw new Error(
164164
`Token poll failed: ${errorData.error || 'Unknown error'} - ${errorData.error_description || responseText}`
165165
);
166-
(error as Error & { status?: number }).status = response.status;
167-
throw error;
168166
} catch (parseError) {
169167
if (parseError instanceof SyntaxError) {
170-
const error = new Error(
168+
throw new Error(
171169
`Token poll failed: ${response.status} ${response.statusText}. Response: ${responseText}`
172170
);
173-
(error as Error & { status?: number }).status = response.status;
174-
throw error;
175171
}
176172
throw parseError;
177173
}
@@ -321,8 +317,6 @@ export async function performDeviceAuthFlow(
321317
// Check if we should slow down
322318
if (error instanceof SlowDownError) {
323319
interval = Math.min(interval * 1.5, 10000); // Increase interval, max 10s
324-
} else if ((error as Error & { status?: number }).status === 401) {
325-
throw new Error('Device code expired or invalid. Please restart authentication.');
326320
} else {
327321
throw error;
328322
}

‎tests/README.md‎

Lines changed: 159 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,159 @@
1+
# Testes - opencode-qwencode-auth
2+
3+
Este diretório contém todos os testes do plugin, organizados por categoria.
4+
5+
## 📁 Estrutura
6+
7+
```
8+
tests/
9+
├── unit/ # Testes unitários formais (bun test)
10+
│ ├── auth-integration.test.ts
11+
│ ├── errors.test.ts
12+
│ ├── file-lock.test.ts
13+
│ ├── oauth.test.ts
14+
│ ├── request-queue.test.ts
15+
│ └── token-manager.test.ts
16+
│
17+
├── integration/ # Testes de integração manuais
18+
│ ├── debug.ts # End-to-end com API Qwen real
19+
│ └── race-condition.ts # Concorrência entre processos
20+
│
21+
└── robust/ # Stress tests
22+
├── runner.ts # Orquestrador de testes robustos
23+
└── worker.ts # Worker para testes multi-processo
24+
```
25+
26+
## 🧪 Testes Unitários
27+
28+
**Execução:**
29+
```bash
30+
bun test # Todos os testes
31+
bun test --watch # Watch mode
32+
bun test unit/ # Apenas testes unitários
33+
bun test <arquivo> # Teste específico
34+
```
35+
36+
**Cobertura:**
37+
- `errors.test.ts` - Sistema de erros e classificação (30+ testes)
38+
- `oauth.test.ts` - PKCE, OAuth helpers, constants (20+ testes)
39+
- `request-queue.test.ts` - Throttling e rate limiting (15+ testes)
40+
- `token-manager.test.ts` - Gerenciamento de tokens (10+ testes)
41+
- `file-lock.test.ts` - File locking mechanism (20+ testes)
42+
- `auth-integration.test.ts` - Integração de componentes (15+ testes)
43+
44+
**Total:** 100+ testes automatizados
45+
46+
## 🔬 Testes de Integração (Manuais)
47+
48+
### Debug (End-to-End)
49+
50+
Testa o sistema completo com a API Qwen real.
51+
52+
**Pré-requisitos:**
53+
- Login realizado (`opencode auth login`)
54+
- Credenciais válidas
55+
56+
**Execução:**
57+
```bash
58+
bun run test:integration
59+
# OU
60+
bun run tests/integration/debug.ts full
61+
```
62+
63+
**Testes incluídos:**
64+
- PKCE generation
65+
- Base URL resolution
66+
- Credentials persistence
67+
- Token expiry check
68+
- Token refresh
69+
- Retry mechanism
70+
- Throttling
71+
- TokenManager
72+
- 401 recovery
73+
- **Real Chat API call** (requer login)
74+
75+
### Race Condition
76+
77+
Testa concorrência entre múltiplos processos do plugin.
78+
79+
**Execução:**
80+
```bash
81+
bun run test:race
82+
# OU
83+
bun run tests/integration/race-condition.ts
84+
```
85+
86+
**O que testa:**
87+
- Dois processos tentando refresh simultâneo
88+
- File locking previne race conditions
89+
- Recuperação de locks stale
90+
91+
## 💪 Stress Tests (Robust)
92+
93+
Testes de alta concorrência e cenários extremos.
94+
95+
**Execução:**
96+
```bash
97+
bun run test:robust
98+
# OU
99+
bun run tests/robust/runner.ts
100+
```
101+
102+
**Testes incluídos:**
103+
1. **Race Condition (2 processos)** - Concorrência básica
104+
2. **Stress Concurrency (10 processos)** - Alta concorrência
105+
3. **Stale Lock Recovery** - Recuperação de locks abandonados
106+
4. **Corrupted File Recovery** - Arquivo de credenciais corrompido
107+
108+
**Duração:** ~30-60 segundos
109+
110+
## 📊 Scripts package.json
111+
112+
```json
113+
{
114+
"scripts": {
115+
"test": "bun test",
116+
"test:watch": "bun test --watch",
117+
"test:integration": "bun run tests/integration/debug.ts full",
118+
"test:race": "bun run tests/integration/race-condition.ts",
119+
"test:robust": "bun run tests/robust/runner.ts"
120+
}
121+
}
122+
```
123+
124+
## 🎯 Quando usar cada tipo
125+
126+
| Tipo | Quando usar | Requer login? | Automatizado? |
127+
|------|-------------|---------------|---------------|
128+
| **Unitários** | CI/CD, desenvolvimento diário | ❌ Não | ✅ Sim |
129+
| **Integration (debug)** | Validação manual, troubleshooting | ✅ Sim | ❌ Não |
130+
| **Race Condition** | Desenvolvimento de features novas | ❌ Não | ❌ Não |
131+
| **Robust** | Validação pré-release | ❌ Não | ❌ Não |
132+
133+
## 🔍 Debug de Testes
134+
135+
**Habilitar logs detalhados:**
136+
```bash
137+
OPENCODE_QWEN_DEBUG=1 bun test
138+
```
139+
140+
**Verbose mode no debug.ts:**
141+
```bash
142+
OPENCODE_QWEN_DEBUG=1 bun run tests/integration/debug.ts full
143+
```
144+
145+
## 📝 Adicionando Novos Testes
146+
147+
1. **Testes unitários:** Crie `tests/unit/<nome>.test.ts`
148+
2. **Testes de integração:** Crie `tests/integration/<nome>.ts`
149+
3. **Use `bun:test`:**
150+
```typescript
151+
import { describe, it, expect, mock } from 'bun:test';
152+
```
153+
154+
## ⚠️ Notas Importantes
155+
156+
1. **Testes unitários** não modificam credenciais reais
157+
2. **Testes de integração** podem modificar credenciais (usam cópias de teste)
158+
3. **Stress tests** criam locks temporários e os limpam automaticamente
159+
4. **Sempre rode** `bun test` antes de commitar

tests/debug.ts renamed to tests/integration/debug.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,18 +15,18 @@ import {
1515
refreshAccessToken,
1616
isCredentialsExpired,
1717
SlowDownError,
18-
} from '../src/qwen/oauth.js';
18+
} from '../../src/qwen/oauth.js';
1919
import {
2020
loadCredentials,
2121
saveCredentials,
2222
resolveBaseUrl,
2323
getCredentialsPath,
24-
} from '../src/plugin/auth.js';
25-
import { QWEN_API_CONFIG, QWEN_OAUTH_CONFIG, QWEN_OFFICIAL_HEADERS } from '../src/constants.js';
26-
import { retryWithBackoff, getErrorStatus } from '../src/utils/retry.js';
27-
import { RequestQueue } from '../src/plugin/request-queue.js';
28-
import { tokenManager } from '../src/plugin/token-manager.js';
29-
import type { QwenCredentials } from '../src/types.js';
24+
} from '../../src/plugin/auth.js';
25+
import { QWEN_API_CONFIG, QWEN_OAUTH_CONFIG, QWEN_OFFICIAL_HEADERS } from '../../src/constants.js';
26+
import { retryWithBackoff, getErrorStatus } from '../../src/utils/retry.js';
27+
import { RequestQueue } from '../../src/plugin/request-queue.js';
28+
import { tokenManager } from '../../src/plugin/token-manager.js';
29+
import type { QwenCredentials } from '../../src/types.js';
3030

3131
// ============================================
3232
// Logging Utilities

0 commit comments

Comments
 (0)