100% UT coverage for ParseTreeTablePModel.java #3606

Closed
romani opened this Issue Dec 2, 2016 · 2 comments

Comments

Projects
None yet
2 participants

@romani romani changed the title from we need 100% coverage to we need 100% coverage for ParseTreeTablePModel.java Dec 2, 2016

@romani romani changed the title from we need 100% coverage for ParseTreeTablePModel.java to 100% UT coverage for ParseTreeTablePModel.java Dec 2, 2016

@ssdeep

This comment has been minimized.

Show comment
Hide comment
@ssdeep

ssdeep Dec 7, 2016

Was working on it.
ParseTreeTablePModel.isCellEditable() will always be false because getColumnClass() never returns ParseTreeTablePModel, it only returns ParseTreeTableModel.

ssdeep commented Dec 7, 2016

Was working on it.
ParseTreeTablePModel.isCellEditable() will always be false because getColumnClass() never returns ParseTreeTablePModel, it only returns ParseTreeTableModel.

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 7, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 8, 2016

ssdeep pushed a commit to ssdeep/checkstyle that referenced this issue Dec 8, 2016

romani added a commit that referenced this issue Dec 9, 2016

romani added a commit that referenced this issue Dec 9, 2016

romani added a commit that referenced this issue Dec 9, 2016

@romani romani added this to the 7.4 milestone Dec 9, 2016

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Dec 9, 2016

Member

fixes are merged.

Member

romani commented Dec 9, 2016

fixes are merged.

@romani romani closed this Dec 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment