Eclipse Community Forums: TMF (Xtext) » xtext-utils - integration test failing, whitespace mismatch?

Help

Home

Home » Modeling » TMF (Xtext) » xtext-utils - integration test failing, whitespace mismatch?

Show: Today's Messages :: Show Polls :: Message Navigator

xtext-utils - integration test failing, whitespace mismatch? [message #754604]

Thu, 03 November 2011 13:25

Scott Finnie

Messages: 94
Registered: October 2011

Member

Integration test failing and I'm not sure why. Error is:

org.junit.ComparisonFailure: expected:<Domain "Banking" {[]}> but was:<Domain "Banking" {[ ]}>

So only difference appears to be whitespace?

The source file was created using the generated xtext editor and shows no validation errors in the IDE. Here's the file:

Domain "Banking" {}

And the relevant grammar rule:

Domain:
	'Domain' name=String '{'
	(elements+=ModelElement)*
	'}';

The test passes if I change the input to:

Domain "Banking" { }

i.e. add space between braces...?

thx.

[Updated on: Thu, 03 November 2011 13:31]

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754608 is a reply to message #754604]

Thu, 03 November 2011 13:56

Alexander Nittka

Messages: 1193
Registered: July 2009

Senior Member

Hi,

if you want two things to be identical, they have to be. The test does not check semantic equivalence.

Alex

Need training, onsite consulting or any other kind of help for Xtext?
Go visit http://xtext.itemis.com or send a mail to xtext@itemis.de

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754627 is a reply to message #754608]

Thu, 03 November 2011 16:13

Scott Finnie

Messages: 94
Registered: October 2011

Member

@Alexander: point is that you don't specify in the test what to compare against.

Xtest-utils must generate the comparison string in the background. Net result is that a valid input file fails - because the auto-generated file includes different whitespace. So the test fails even though the input file is valid.

Easy enough to fix for the simple example above, but not practical for real examples.

Unless i'm missing something...?

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754635 is a reply to message #754627]

Thu, 03 November 2011 16:58

Christian Dietrich

Messages: 14665
Registered: July 2009

Senior Member

Did you use the formatter on your file in the editor?

~Christian

Twitter : @chrdietrich
Blog : https://www.dietrich-it.de

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754640 is a reply to message #754635]

Thu, 03 November 2011 17:42

Scott Finnie

Messages: 94
Registered: October 2011

Member

@Christian - thanks, that works. Although the 'formatter' seems to remove all formatting, resulting in a single string with no cr/lf etc. So OK for creating test input that works but not for readable test data... Sad

However I can work around that, just means keeping a 'formatted' and readable version of the file. Thanks for the suggestion.

- S.

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754642 is a reply to message #754640]

Thu, 03 November 2011 17:43

Christian Dietrich

Messages: 14665
Registered: July 2009

Senior Member

Hi,

you can "adjust" formatting Wink

see the generated yourdslFormatter class.

~Christian

Twitter : @chrdietrich
Blog : https://www.dietrich-it.de

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754654 is a reply to message #754642]

Thu, 03 November 2011 18:57

Scott Finnie

Messages: 94
Registered: October 2011

Member

Thanks Christian Smile

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754671 is a reply to message #754604]

Thu, 03 November 2011 19:54

Karsten Thoms

Messages: 762
Registered: July 2009
Location: Dortmund, Germany

Senior Member

And you can disable formatting comparison by calling
ignoreFormattingDifferences();

in a @Before method.

The integration test *does* test that everything is as expected, including every whitespace. Since this is the only way to test the formatter.

~Karsten

Need professional support for Xtext, EMF, Eclipse IDE?
Go to: http://devhub.karakun.com
Twitter : @kthoms
Blog : www.karsten-thoms.de

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #754672 is a reply to message #754671]

Thu, 03 November 2011 20:02

Karsten Thoms

Messages: 762
Registered: July 2009
Location: Dortmund, Germany

Senior Member

I have extended the documentation a bit.

http://code.google.com/a/eclipselabs.org/p/xtext-utils/wiki/Unit_Testing

Need professional support for Xtext, EMF, Eclipse IDE?
Go to: http://devhub.karakun.com
Twitter : @kthoms
Blog : www.karsten-thoms.de

Report message to a moderator

Re: xtext-utils - integration test failing, whitespace mismatch? [message #1003230 is a reply to message #754672]

Sun, 20 January 2013 17:50

Nathan M

Messages: 5
Registered: January 2013

Junior Member

The problem I am having is the when running the test, the formatter is using the default one (IIndentationInformation.Default) which always uses a literal tab character. But the actual default xtext implementation is PreferenceStoreIndentationInformation.java, which uses the eclipse preference. So the eclipse preference can be set to use spaces, and so the test file gets formatted as such, but then the serialization comparison fails because it's expecting a tab instead of spaces.
So is there a way to make the formatter used in the test use the PreferenceStoreIndentationInformation class instead?

Report message to a moderator

Previous Topic:	Limiting scoped elements, but allowing simple names to be used
Next Topic:	problem running a xtext file

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Fri Apr 19 07:38:27 GMT 2024

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter