Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Simple CPAN distribution.

Report information
The Basics
Id: 35735
Status: resolved
Priority: 0/
Queue: Test-Simple
People
Owner: Nobody in particular
Requestors: pjs [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu May 08 20:27:30 2008 pjs [...] cpan.org - Ticket created
Subject: Remove dash from description to match TAP specification
The TAP specification doesn't include a dash as part of the description, so TAP::Parser thinks every test line from Test::Builder contains a description starting with a dash, that I then have to remove. Seems that strict TAP compliance would mean not outputting that dash-space.
  Thu May 08 23:42:49 2008 schwern [...] pobox.com - Correspondence added
Subject: Re: [rt.cpan.org #35735] Remove dash from description to match TAP specification
Date: Thu, 08 May 2008 20:42:24 -0700
To: bug-Test-Simple [...] rt.cpan.org
From: Michael G Schwern <schwern [...] pobox.com>
Peter Scott via RT wrote: Show quoted text
> The TAP specification doesn't include a dash as part of the description, so TAP::Parser thinks > every test line from Test::Builder contains a description starting with a dash, that I then have to > remove. Seems that strict TAP compliance would mean not outputting that dash-space.
Strict TAP compliance doesn't say a description cannot contain a dash. :) But I understand how it causes problems in interpretation. I am inclined to leave this for Test::Builder2 because this is exactly one of those innocent changes in formatting that have brought CPAN down in the past. -- 'All anyone gets in a mirror is themselves,' she said. 'But what you gets in a good gumbo is everything.' -- "Witches Abroad" by Terry Prachett
  Thu May 08 23:42:50 2008 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 03 20:53:52 2009 mschwern [...] cpan.org - Correspondence added
I tried this out in TB2 and it made the output hard to read. ok 1 some stuff ok 2 another test description vs ok 1 - some stuff ok 2 - another test description Without the dash, there's not enough visual separation to make the description easy to read, which is why the dash was added in the first place. So, instead, I'm getting TAP changed to allow for an optional dash which the TAP parser will strip. This also resolves some grammatical ambiguities. See this thread for details. http://www.ietf.org/mail-archive/web/tap/current/msg00378.html
  Tue Feb 03 20:53:53 2009 mschwern [...] cpan.org - Status changed from 'open' to 'resolved'