Ticket #1185 (closed todo: done)

Opened 5 years ago

Last modified 5 years ago

[PATCH] attempted to rewrite t/op/00ff-dos.t in PIR

Reported by: bubaflub Owned by:
Priority: trivial Milestone:
Component: testing Version: 1.7.0
Severity: low Keywords:
Cc: Language:
Patch status: applied Platform: all

Description

I rewrote t/op/00ff-dos.t to cut my teeth on some PIR. Unfortunately the second test fails. As I understand the original, we just want to make sure that we can handle printing out DOS strings (stuff that has \r\n and stuff that has a CTRL+Z in it). Printing out the CTRL+Z fails. Any ideas?

Attachments

dos.patch Download (1.2 KB) - added by bubaflub 5 years ago.

Change History

Changed 5 years ago by bubaflub

Changed 5 years ago by bubaflub

Failure output:

Icarus:parrot bob$ prove -v t/op/00ff-dos.t 
t/op/00ff-dos....1..2
42
ok 1 - fileformat dos
42
ok 2 - fileformat dos w ctrl-z
FAILED test 2
	Failed 1/2 tests, 50.00% okay
Failed Test     Stat Wstat Total Fail  List of Failed
-------------------------------------------------------------------------------
t/op/00ff-dos.t                2    1  2
Failed 1/1 test scripts. 1/2 subtests failed.
Files=1, Tests=2,  0 wallclock secs ( 0.02 cusr +  0.02 csys =  0.04 CPU)
Failed 1/1 test programs. 1/2 subtests failed.

Changed 5 years ago by dukeleto

  • status changed from new to closed
  • resolution set to done
  • component changed from none to testing
  • patch set to applied
  • platform set to all
  • type changed from bug to todo

The original patch was close, but some escaping rules were escaping you :)

Tweaked patch applied as r42304 .

Thanks for your contribution!

Note: See TracTickets for help on using tickets.