我目前有一个带有构造函数和两个方法的ES6类。我有点困惑为什么在Promise.reject(ex)
中使用.then()
解析undefined。如果有人不介意解释我做错了什么,那将非常感激。
我有一个名为getYaml()
的方法,其中包含以下内容:
_getYaml(recordId) {
return new Promise((resolve, reject) => {
fs.readFile(this.workingDir + '/' + recordId + '.yaml', 'utf8', function(err, data) {
if (err) reject(err)
resolve(data)
})
})
}
然后,我有另一个名为getCompDoc
的方法,它使用其他方法,如下所示:
getCompDoc(recordId) {
return this._getYaml(recordId).then(data => {
let yaml = data
let yamlObj
try {
yamlObj = YAML.safeLoad(yaml)
} catch (ex) {
ex.message = `Failure to parse yaml. Error: ${ex.message}`
logger.error(ex.message, {}, ex)
return Promise.reject(ex)
}
let compDoc = {
// ...
}
return compDoc
}).catch(err => {
logger.error(err, {}, err)
})
}
然后我有一个测试来检查YAML解析错误是否被捕获,然后拒绝承诺,如下所示:
describe('error cases', () => {
const fakeRecordId = 'SomeYaml'
beforeEach(() => {
sinon.stub(myClass, '_getYaml').returns(Promise.resolve('{{&^%}egrinv&alidgj%^%^&$£@£@£}'))
})
afterEach(() => {
myClass._getYaml.restore()
})
it('Error parsing yaml, rejects with error', () => {
return expect(myClass.getCompDoc(fakeRecordId)).to.be.rejected.then(response => {
expect(response.message).to.match(/Failure to parse yaml. Error: /)
})
})
})
测试输出:
AssertionError: expected promise to be rejected but it was fulfilled with undefined
如果我只是返回在getCompDoc
方法中抛出的异常,我会按预期收到错误,但是只要我使用Promise.reject
它就会解析为undefined。
我想在getCompDoc
中包装return new Promise()
但是我不确定这是否是Promise构造函数反模式的一个例子。理想情况下,我想拒绝这一点,而不是直接返回错误,因为我可以断言该方法被拒绝而未实现。
你在getCompDoc
条款中“吞下”catch
中的错误。具体来说,这是一个代表您的代码的简化代码段:
let getYamlPromise = Promise.reject('REJECTED!');
let getCompDocPromise = getYamlPromise
.then(data => console.log('getYamlPromise', data))
.catch(error => console.error('getYamlPromise', error));
getCompDocPromise
.then(a => console.log('getCompDocPromise RESOLVED', a))
.catch(a => console.log('getCompDocPromise REJECTED', a));
正如你所看到的,getCompDocPromise
是用undefined解决的。如果您想传播错误,您的catch
子句将不得不抛出新错误或返回被拒绝的承诺:
let getYamlPromise = Promise.reject('REJECTED!');
let getCompDocPromise = getYamlPromise
.then(data => console.log('getYamlPromise', data))
.catch(error => {
console.error('getYamlPromise', error);
return Promise.reject(error);
});
getCompDocPromise
.then(a => console.log('getCompDocPromise RESOLVED', a))
.catch(a => console.log('getCompDocPromise REJECTED', a));