I disagree that the comments are excessive, but they could each use a
preceding newline.  Also, they don't really explain what the objects are.

Carlos, what does "An imported Translation Unit match source" mean?  I
can't parse that comment at all.  "match source"?

And "wrapped around a result item" - it doesn't seem to be wrapped
around anything.

On 2013-08-01 13:07, David Mason wrote:
> In
> zanata-war/src/test/java/org/zanata/webtrans/server/rpc/TransMemoryMergeHandlerTest.java:
> 
>> +   @Test
>> +   public void willIgnoreImportedTMResults() throws Exception
>> +   {
>> +      // Given:
>> +      // an action with threshold 80% and trans unit id
>> +      final long transUnitId = 1L;
>> +      prepareActionAndMockSecurityService(80, transUnitId);
>> +      TransMemoryMerge mergeAction = prepareActionAndMockSecurityService(80, false, transUnitId);
>> +      // A Text Flow to be translated
>> +      HTextFlow hTextFlow = TestFixture.makeHTextFlow(transUnitId, hLocale, ContentState.New, "pot/a.po");
>> +      // An imported Translation Unit match source ...
>> +      TransMemoryUnit tuResultSource = TestFixture.makeTransMemoryUnit(10L, hLocale);
>> +      // ... wrapped around a result item
>> +      TransMemoryResultItem mostSimilarTM = importedTmResult(11L, 100);
>> +      // A Translation memory query
>> +      TransMemoryQuery tmQuery = new TransMemoryQuery(hTextFlow.getContents(), FUZZY_PLURAL);
> 
> Excessive comments here as above.
> 
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zanata/zanata-server/pull/78/files#r5521535>.
> 


-- 
Sean Flanigan

Senior Software Engineer
Engineering - Internationalisation
Red Hat