New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Citrus validates text content although I did not tell it #197

Closed
Ernesto-Maserati-CH opened this Issue Feb 1, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@Ernesto-Maserati-CH

Ernesto-Maserati-CH commented Feb 1, 2017

I have a plain-text inbound message and want only to verify the HTTP code. But that does not seem to work.

       http(action -> action.client(CLIENT_ID)
                .response(HttpStatus.OK)
        );

This results in:

com.consol.citrus.exceptions.CitrusRuntimeException: org.w3c.dom.ls.LSException: Content is not allowed in prolog.

What is the matter?

@christophd

This comment has been minimized.

Member

christophd commented Feb 1, 2017

XML is the default message type so Citrus tries to parse the payload with XML parser in order to perform XML validation. You need to explicitly set the message type to PLAINTEXT.

@Ernesto-Maserati-CH

This comment has been minimized.

Ernesto-Maserati-CH commented Feb 3, 2017

I assumed the HTTP status code is not related to the message type.
If I don't care about the message type when I want to validate only the status code, why does Citrus have to care about it?

@christophd

This comment has been minimized.

Member

christophd commented Feb 9, 2017

You are right, message payload validation should be skipped here automatically.

@christophd christophd added this to the v2.7.1 milestone Feb 9, 2017

@christophd christophd added IN PROGRESS and removed READY labels Mar 1, 2017

@christophd christophd self-assigned this Mar 1, 2017

@christophd christophd closed this Mar 9, 2017

@christophd christophd removed the IN PROGRESS label Mar 9, 2017

@christophd

This comment has been minimized.

Member

christophd commented Mar 9, 2017

Fixed with #218

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment