Good call.  As written, this would actually never user topLabelFormatter.  topUnit is defaulted to '', so the condition in the ternary would always be true.