6f19890a84
- Add waitForLoginResult() method using Promise.race to detect: - Success: .nc-workbench-icon element visible - Failure: '名称或密码错误' error text visible - Force login: click confirm button and re-detect - Extract timeout constants (PAGE_LOAD_TIMEOUT, LOGIN_RESULT_TIMEOUT, FORCE_LOGIN_TIMEOUT) - Improve error handling with clear error messages - Add unit tests for class structure verification - Fix test setup for Electron app mock Fixes: ERP login success/failure detection was ambiguous
94 lines
2.6 KiB
TypeScript
94 lines
2.6 KiB
TypeScript
import { describe, it, expect, beforeEach } from 'vitest'
|
|
import { ErpAuthService } from '../../src/main/services/erp/erp-auth'
|
|
import type { ErpConfig } from '../../src/main/types/erp.types'
|
|
|
|
describe('ERP Authentication Service (Unit)', () => {
|
|
describe('Session Management', () => {
|
|
it('should create service instance with config', () => {
|
|
const config: ErpConfig = {
|
|
url: 'https://test.example.com',
|
|
username: 'testuser',
|
|
password: 'testpass'
|
|
}
|
|
|
|
const service = new ErpAuthService(config)
|
|
|
|
expect(service).toBeDefined()
|
|
expect(service.isActive()).toBe(false)
|
|
})
|
|
|
|
it('should throw error when getting session before login', () => {
|
|
const config: ErpConfig = {
|
|
url: 'https://test.example.com',
|
|
username: 'testuser',
|
|
password: 'testpass'
|
|
}
|
|
|
|
const service = new ErpAuthService(config)
|
|
|
|
expect(() => service.getSession()).toThrow('Not logged in. Call login() first.')
|
|
})
|
|
|
|
it('should report inactive status before login', () => {
|
|
const config: ErpConfig = {
|
|
url: 'https://test.example.com',
|
|
username: 'testuser',
|
|
password: 'testpass'
|
|
}
|
|
|
|
const service = new ErpAuthService(config)
|
|
|
|
expect(service.isActive()).toBe(false)
|
|
})
|
|
})
|
|
|
|
describe('Close Method', () => {
|
|
it('should handle close when no session exists', async () => {
|
|
const config: ErpConfig = {
|
|
url: 'https://test.example.com',
|
|
username: 'testuser',
|
|
password: 'testpass'
|
|
}
|
|
|
|
const service = new ErpAuthService(config)
|
|
|
|
// Should not throw when closing without session
|
|
await expect(service.close()).resolves.toBeUndefined()
|
|
})
|
|
})
|
|
|
|
describe('Class Structure', () => {
|
|
let service: ErpAuthService
|
|
|
|
beforeEach(() => {
|
|
const config: ErpConfig = {
|
|
url: 'https://test.example.com',
|
|
username: 'testuser',
|
|
password: 'testpass'
|
|
}
|
|
service = new ErpAuthService(config)
|
|
})
|
|
|
|
it('should have login method that returns a Promise', () => {
|
|
expect(service.login).toBeDefined()
|
|
expect(typeof service.login).toBe('function')
|
|
expect(service.login()).toBeInstanceOf(Promise)
|
|
})
|
|
|
|
it('should have close method', () => {
|
|
expect(service.close).toBeDefined()
|
|
expect(typeof service.close).toBe('function')
|
|
})
|
|
|
|
it('should have getSession method', () => {
|
|
expect(service.getSession).toBeDefined()
|
|
expect(typeof service.getSession).toBe('function')
|
|
})
|
|
|
|
it('should have isActive method', () => {
|
|
expect(service.isActive).toBeDefined()
|
|
expect(typeof service.isActive).toBe('function')
|
|
})
|
|
})
|
|
})
|