Bug 534533 - 5 pde.genericeditor.extension.tests test failures on windows machine
Summary: 5 pde.genericeditor.extension.tests test failures on windows machine
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.8   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.8 RC1   Edit
Assignee: Vikas Chandra CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2018-05-10 06:59 EDT by Vikas Chandra CLA
Modified: 2018-05-14 06:24 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vikas Chandra CLA 2018-05-10 06:59:12 EDT 
See http://download.eclipse.org/eclipse/downloads/drops4/I20180508-2000/testResults.php

testMultipleContainersWithSameRepoPersist	Error	expected:<...sion="3.8"?><target>[ <locations> <location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit"> <unit id="unit1" version="1.0.0"/> <unit id="unit2" version="2.0.0"/> <repository location="TESTURI"/> </location> <location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit"> <repository location="TESTURI"/> </location> </locations>] </target>> but was:<...sion="3.8"?><target>[ <locations> <location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit"> <unit id="unit1" version="1.0.0"/> <unit id="unit2" version="2.0.0"/> <repository location="TESTURI"/> </location> <location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit"> <repository location="TESTURI"/> </location> </locations> ] </target>>

org.junit.ComparisonFailure: expected:<...sion="3.8"?><target>[
<locations>
<location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit">
<unit id="unit1" version="1.0.0"/>
<unit id="unit2" version="2.0.0"/>
<repository location="TESTURI"/>
</location>
<location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit">
<repository location="TESTURI"/>
</location>
</locations>]
</target>> but was:<...sion="3.8"?><target>[
<locations>
<location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit">
<unit id="unit1" version="1.0.0"/>
<unit id="unit2" version="2.0.0"/>
<repository location="TESTURI"/>
</location>
<location includeAllPlatforms="true" includeConfigurePhase="false" includeMode="slicer" includeSource="false" type="InstallableUnit">
<repository location="TESTURI"/>
</location>
</locations>
]
</target>>
at org.eclipse.pde.genericeditor.extension.tests.Bug531602FormattingTests.confirmMatch(Bug531602FormattingTests.java:120)
at org.eclipse.pde.genericeditor.extension.tests.Bug531602FormattingTests.testMultipleContainersWithSameRepoPersist(Bug531602FormattingTests.java:93)
at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:678)
at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:344)
Comment 1 Vikas Chandra CLA 2018-05-10 07:14:17 EDT 
Alex, any idea about this failure?
Comment 2 Alexander Kurtakov CLA 2018-05-10 07:49:42 EDT 
No idea. Can you reproduce locally ?
Comment 3 Alexander Kurtakov CLA 2018-05-10 07:50:22 EDT 
If yes would you please verify that https://git.eclipse.org/r/#/c/119426/ fixes them on windows?
Comment 4 Vikas Chandra CLA 2018-05-10 08:54:48 EDT 
>> Can you reproduce locally ?

Yes


>>yes would you please verify that https://git.eclipse.org/r/#/c/119426/ fixes them on windows?


No it doesnt fix.

I will look at this issue.

Thanks for the update, Alex
Comment 5 Eclipse Genie CLA 2018-05-10 09:06:54 EDT 
New Gerrit change created: https://git.eclipse.org/r/122421
Comment 6 Vikas Chandra CLA 2018-05-10 09:08:34 EDT 
comment#5 fixes this issue.
Comment 7 Vikas Chandra CLA 2018-05-10 09:17:24 EDT 
Without https://git.eclipse.org/r/122421 : 5 failures

With https://git.eclipse.org/r/122421 : 0 failure