Commit 16f7d9b2 authored by Robert Knight's avatar Robert Knight

Document the purpose of an `if` test and skip coverage

There is not a lot of point in testing this at the moment because the
expected behavior in this case isn't really defined, other than "doesn't
generate an error", and since we have no means of displaying that error,
there is nothing a test can readily check for.
parent 9690ba1c
...@@ -37,6 +37,12 @@ function AnnotationViewerContent({ ...@@ -37,6 +37,12 @@ function AnnotationViewerContent({
)[0]; )[0];
if (!topLevelAnnot) { if (!topLevelAnnot) {
// `annotationId` refers to a reply. We were able to fetch the reply
// but not the top-level annotation at the top of the thread.
//
// We need to decide what what be the most useful behavior in this case
// and implement it.
/* istanbul ignore next */
return; return;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment