search

lipinggm / tlb

Automatically exported from code.google.com/p/tlb
0 stars 0 forks source link

RSpec/Test Unit Inflections should not be loaded if already loaded #75

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Since we do a remove_method inside the inflection, there is a corner case where 
in the fork flow, the file will be required twice. In this case we should not 
fail saying cannot remove_method :foo. We should do what we have done for the 
Cucumber implementation.

Original issue reported on code.google.com by itspa...@gmail.com on 10 May 2011 at 8:26

GoogleCodeExporter commented 9 years ago

Original comment by singh.janmejay on 13 May 2011 at 4:03

GoogleCodeExporter commented 9 years ago

Original comment by singh.janmejay on 13 May 2011 at 4:12

GoogleCodeExporter commented 9 years ago 
RSpec doesn't provide an option to explicitly fork. However, this bug may exist 
in spec_server mode. But, most people do not use spec_server mode to run on CI 
where they would be using TLB. So, I guess this wont be an issue.

Original comment by itspa...@gmail.com on 27 May 2011 at 1:26

GoogleCodeExporter commented 9 years ago 
TestUnit support doesn't have a fork mode. Have checked rake/testtask.rb.

Closing this, unless someone comes up with a really good usecase for supporting 
spec_server or spork.

Original comment by singh.janmejay on 27 May 2011 at 1:28