Ticket #1204 (new bug)

Opened 5 years ago

Last modified 3 years ago

Implement all TODO comments in t/pmc/parrotio.t

Reported by: dukeleto Owned by:
Priority: normal Milestone:
Component: testing Version: 1.7.0
Severity: medium Keywords:
Cc: Language:
Patch status: Platform:

Description

There are many things like:

$P1 = # TODO create a callback here

in the code. Write proper tests for these TODO comments.

If you close this ticket, coke++ will buy you beer.

Change History

  Changed 5 years ago by jkeenan

  • component changed from none to testing

follow-up: ↓ 3   Changed 3 years ago by jkeenan

Many? I only see these 4 (as of now):

[parrot] 507 $ grep -in TODO t/pmc/parrotio.t 
35:pir_output_is( <<'CODE', <<'OUT', 
  'open and close - synchronous', 
  todo => 'not yet implemented' );
117:    <<'CODE', <<'OUT', 'print, read, and readline - synchronous', 
  todo => 'not yet implemented' );
170:# TT #1204 pir_output_is( <<'CODE', <<'OUT', 
  'print, read, and readline - asynchronous', 
  todo => 'not yet implemented' );
173:pir_output_is( <<'CODE', <<'OUT', 'buffer_type', 
  todo => 'not yet implemented' );

Anyone want to take a stab at this?

Thank you very much.

kid51

in reply to: ↑ 2   Changed 3 years ago by jkeenan

Replying to jkeenan:

Many? I only see these 4 (as of now):

On closer look, I see that while there are only 4 tests that are TODOed, there are many items in inline comments that don't match m/todo/i but do indicate things that need to be done.

See, inter alia, TT #1177 and TT #1178.

Note: See TracTickets for help on using tickets.