Istanbul: Else path not taken

Created on 10 Mar 2016  路  1Comment  路  Source: gotwarlost/istanbul

There are a number of functions which not have else branch. Considers covering but that is not covered else branch in which case

setEndDate: function(newDate) {
    if (this._setEndDate(newDate)) {
        this._notifyOnPropertyChanged('endDate');
        this._notify('onEndDateChange', this._options.endDate);
        this._notify('onDateRangeChange', this._options.startDate, this._options.endDate);
    }
}

Most helpful comment

@xaircore have you tested a scenario where this._setEndDate(newDate) returns false? That's likely the "else path" it's complaining about.

>All comments

@xaircore have you tested a scenario where this._setEndDate(newDate) returns false? That's likely the "else path" it's complaining about.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

blindenvy picture blindenvy  路  19Comments

asciidisco picture asciidisco  路  47Comments

graemej picture graemej  路  22Comments

NiGhTTraX picture NiGhTTraX  路  36Comments

ottes picture ottes  路  36Comments