On Sep 14, 2009, at 10:47 AM, Mat Schaffer wrote:


On Sep 14, 2009, at 1:14 AM, Alex Boisvert wrote:
'generated' refers to classes that may be generated from source (e.g. xmlbeans) prior to compilation.

checking for .jar (your prior patch) or checking for a file (your new file) seem to be equivalent and both are better, as far as I can tell.

if you want to clean this up and submit a patch, i'll be happy to apply it. i'll add a spec for the generated case at the same time.

thanks for your persistence... i should have picked up on this earlier.

alex

Thanks for getting back to me, Alex. I've attached the patch for what I have so far, but I'm not thrilled about the file system access. I'll see what I can come up with in terms of mocking but I thought I'd put this out there to see if you had any feedback on that front.
-Mat

<eclipse_local_lib.patch>

Spoke too soon. Here's a patch using rspec's mocking features. If you have any thoughts on how to make it cleaner, let me know, but I'm pretty happy with it.

Attachment: eclipse_local_lib.mocked.patch
Description: Binary data



Thanks,
Mat

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to