ユニットテストでネストされた約束がタイムアウトで失敗する

Question

Axios XHRライブラリのラッパーがあり、私のコードの単体テストを書こうとしています。しかし、私がテストを実行すると、タイムアウトが超過したと私に伝えられなくなります。アサーションを実行するためにテストをどのように構造化する必要がありますか？

export const clientRequest = (xhrClient, endpoint, params = {}) => { 
    const method = params.method || 'get' 
    const {config, data, noOrg, unrestricted} = params 

    let reqParams = data 
    if (!isNil(data) && method === 'get') { 
    reqParams = {params: data} 
    } 

    const authConfig = unrestricted ? {...config, withCredentials: false} : 
    {...config, withCredentials: true} 

    const concatEndpoint = noOrg ? endpoint : 
    `${Cookies.get('organization') || 'default' }${endpoint}` 

    return new Promise((resolve, reject) => { 
    xhrClient[method](concatEndpoint, reqParams, authConfig) 
     .then(response => resolve(response)) 
     .catch(err => reject(err)) 
    }) 
} 

、問題のテスト：

describe('clientRequest()',() => { 
    const resolveSpy = sinon.spy() 
    const fakeClient = { 
     get:() => new Promise(resolveSpy), 
    } 

    it.only('should make a call to the supplied method', (done) => { 
     const result = xhr.clientRequest(fakeClient, '/foobar', {method: 'get'}) 
     result.then(() => { 
     expect(resolveSpy).to.have.beenCalledWith('/foobar', undefined, {withCredentials: true}) 
     done() 
     }) 
    }) 
    }) 

Answer 1

fakeClient.getは決して解決されていないようだ

はここでラッパーコードです。

それは時に見つけ出されるべき方法ですので、私はfakeClientをこのように変更することをお勧め：

const fakeClient = { 
    get:() => Promise.resolve() 
} 
sinon.spy(fakeClient, 'get'); 

約束チェーンでエラーを処理するためにcatch呼び出しを追加することも重要です。

result.then(() => { 
    expect(resolveSpy).to.have.beenCalledWith('/foobar', undefined, {withCredentials: true}) 
    done() 
}).catch(done); 

Answer 2

私は@ MarcoLの提案に続いて、期待を返しました。

it('should make a call to the supplied method',() => { 
    const result = xhr.clientRequest(fakeClient, '/foobar', {method: 'get'}) 
    return result.then(() => (
    expect(resolveSpy).to.have.been.calledWith('/foobar', undefined, {withCredentials: true}) 
)) 
})