Ticket #1810 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

Configure.pl too noisy about missing tests.

Reported by: coke Owned by: jkeenan
Priority: normal Milestone:
Component: configure Version: trunk
Severity: medium Keywords:
Cc: Language:
Patch status: applied Platform:

Description

r49429.

% perl Configure.pl

No tests exist for configure step auto::stat at Configure.pl line 17
No tests exist for configure step auto::timespec at Configure.pl line 17
No tests exist for configure step auto::infnan at Configure.pl line 17

...

Configure.pl shouldn't be carping about this. If we need a separate coverage test, that's fine (and I think we have one) but this shouldn't be shown to someone who isn't asking for it.

Change History

Changed 4 years ago by jkeenan

  • owner set to jkeenan
  • status changed from new to assigned

Changed 4 years ago by jkeenan

This revision is being worked on in the tt1810_missing_step_tests branch.

Coke, could you check this branch out and give it a spin? I have a few more tests to write, but I think it's in pretty good shape?

Thanks.

kid51

Changed 4 years ago by jkeenan

  • status changed from assigned to closed
  • resolution set to fixed
  • patch set to applied

The parrot-tickets mailing list has an entry from Coke for this ticket indicating that he was satisfied with the branch. I don't know why I don't yet see that comment in this ticket.

In any event, I have merged the branch into trunk at r49492. I have one corner case to cover in testing -- and I also have to write the two step tests files that gave rise to this ticket (those for auto::stat and auto::timespec).

But this is good enough for now and for the upcoming release. Closing ticket.

Thank you very much.

kid51

Note: See TracTickets for help on using tickets.