I like the part:

  the main
  test suite should only skip tests when external (Foo.pm) or
  internal-configurable (mpm, threads) dependencies are not met.

which we already ensure for the releases.

Though there is one catch: If nobody else but you can run the tests what's the point of committing them? Why committing incomplete tests in first place? How do we know whether the tests in opt_t are supposed to work or not? And therefore trying to report a problem to you just annoying you?

well, assuming we're all not working in a vacuum (which we are currently :) having tests available for incomplete features means that anybody can activate the tests and know exactly what we want from the (missing) featureset. so, theoretically you could commit some tests and somebody else could use them to hack away at the functionality you have in mind. seeing as how you're already overworked and all :)


So what is your favorite choice for the new directory name?

t_opt? (as in t/ but optional) - the drawback is that now the file completion for t/ won't now work nicely, so I'd rather go with opt_t/



I'd be in favor of just opt, or maybe dev or in_process - the t_ is redundant :)


but it doesn't imply that this is a test dir. Any other ideas?

I'd like to see it in t/ someplace - if t/response and some of the other t/ directories can be special, I'd think we could also make t/opt (differently) special as well, no?



> maybe dev or in_process


You missed the point of my idea - my proposal is not about having a place to dump under-construction tests, it's about having solid working tests, which for some reason don't fit into the main test suite, please see my original message which includes examples.

ah, yes. opt specifies optional, and the ones you are talking about really are not optional, are they, in the sense that if they fail it's ok. I'd rather see 'extended' or something like that - something that indicates additional tests that shouldn't fail (and it's problematic if they do).


--Geoff


--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to