trailing newline fails text/plain

[jsongHBO]

Is there a solution for
apiaryio/snowcrash#56
?

due to the inserted \n, comparing text/plain will always fail the dredd test.

Here's my transaction in the after hook

{ name: 'GET animals > Via id > Retuning text',
  id: 'GET /animals/G1234',
  host: 'localhost',
  port: '3000',
  request: 
   { body: '{}\n',
     headers: 
      { Accept: 'text/plain',
        'content-type': 'text/plain',
        'User-Agent': 'Dredd/0.3.1 (Darwin 13.1.0; x64)',
        'Content-Length': 3 },
     uri: '/animals/G1234',
     method: 'GET' },
  expected: 
   { headers: { 'Content-Type': 'text/plain' },
   **body: 'G1234\n',**
     statusCode: '200' },
  fullPath: '/animals/G1234',
  real: 
   { statusCode: 200,
     headers: 
      {
        'content-type': 'text/plain',
        'content-length': '5',
        connection: 'keep-alive' },
     **body: 'G1234'** } }

and my markdown is simply

    + Response 200 (text/plain)

        ```
        G1234
        ```

[netmilk]

Thank you for the report, but this is still a Snowcrash issue as you mentioned.

[JanBednarik]

Because it can't be solved in snowcrash, what do you think about command line option for dredd --ignore-newline which will ignore trailing newline in body?

[honzajavorek]

Related to #343.

[honzajavorek]

This is still an issue. Workaround can be done by hooks.

[honzajavorek]

This is specific to API Blueprint and quite dependent on how Drafter (the API Blueprint parser) handles the situation. The nature of the format makes it difficult for the parser to decide whether the body payload ends with a newline or not. Hence the parser always adds the newline. In JSON, this is not a problem, but in plain text formats, this is a serious issue.

Dredd could implement a strategy around this, but I'm not sure what would be the best way to approach this. In most cases, the newline is unwanted, but what if user expects the newline? Dredd cannot just always remove the newline.