Just happened to be browsing AbstractDataset and noticed that though setGroup() doesn't allow null groups, the constructor doesn't set the group. getGroup() doesn't do any sort of checking for nulls either, so it would be possible to accidentally get this undefined case.
Now, in my actual test cases I get a group, so this smells like a forgotten edge case. I don't have time to write a full test case to see for sure, but I thought I'd report it anyway.
Log in to post a comment.