Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Commit

Permalink
Fix test
Browse files Browse the repository at this point in the history
  • Loading branch information
Alex Eng committed Sep 26, 2013
1 parent 2d4d605 commit dac3f53
Showing 1 changed file with 4 additions and 0 deletions.
Expand Up @@ -214,6 +214,10 @@ private static HTextFlowTarget createTarget(Date lastChanged, String lastModifie
person.setName(lastModifiedPerson);
target.setLastModifiedBy(person);
target.setVersionNum(versionNum);
if(historyMap == null)
{
historyMap = Maps.newHashMap();
}
target.setHistory(historyMap);
target.setContents("a", "b");
return target;
Expand Down

1 comment on commit dac3f53

@seanf
Copy link
Contributor

@seanf seanf commented on dac3f53 Sep 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of dealing with null here, we could mark the historyMap argument as @ NotNull and then fix the methods which were trying to pass in null here so that they pass an empty map instead.

Please sign in to comment.