Skipping p4 and svn requiring tests in test_assembla.py if those modules are not present

 






 

 


  Review Request #7867
  —
  Created Jan. 15, 2016 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  smith
 









 
 

  Review Board
 









 
 

  master
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  reviewboard
 









 
 

  
 






 








 





  


 
 

  
Certain tests in test_assembla.py which involve importing P4 or svn integration fail if those modules aren't installed.

I think we should skip them instead!


Now we respond to an ImportError being raised by skipping that test.

 








  


 
 

  
Ran unit tests in test_assembla.py - four tests that previously failed due to ImportError are now skipped.