Skip to content

Commit e90d123

Browse files
committed
[website] Check if coverage method exists
1 parent 4ba8017 commit e90d123

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

website/_assets/js/tryFlow.js.es6.liquid

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,10 @@ function getAnnotations(text, callback, options, editor) {
114114
CodeMirror.signal(editor, 'flowCoverage', coverage);
115115
return coverage.expressions.uncovered_locs
116116
})
117-
.catch(() => []),
117+
.catch(err => {
118+
console.error(err)
119+
return []
120+
}),
118121
flow
119122
.then(flowProxy => flowProxy.checkContent('-', text))
120123
.then(errors => {
@@ -245,7 +248,10 @@ class AsyncLocalFlow {
245248
}
246249

247250
coverage(filename, body) {
248-
return Promise.resolve(this._flow.coverage(filename, body))
251+
if (this._flow.coverage) {
252+
return Promise.resolve(this._flow.coverage(filename, body))
253+
}
254+
return Promise.reject(Error('coverage method is missing'))
249255
}
250256

251257
dumpTypes(filename, body) {

website/_assets/js/tryFlowWorker.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,12 @@ this.onmessage = function(e) {
3434
return;
3535
case "coverage":
3636
getFlow(data.version).then(function(flow) {
37-
var result = flow.coverage(data.filename, data.body);
38-
postMessage({id: data.id, type: "coverage", result: result});
37+
if (flow.coverage) {
38+
var result = flow.coverage(data.filename, data.body);
39+
postMessage({id: data.id, type: "coverage", result: result});
40+
} else {
41+
postMessage({id: data.id, type: "coverage", err: Error('coverage method is missing')});
42+
}
3943
})["catch"](function (e) {
4044
postMessage({id: data.id, type: "coverage", err: e});
4145
})

0 commit comments

Comments
 (0)