Address first set of comments from @ormsbee. All changes in test_shib.py

* Groups imports
* Converts variables to python convention
* self.store = modulestore() always (no try-catch)
* helper function -> identity generator
* don't use is for string compare
* assertTrue(isinstance(...)) -> assertIsInstance
* Use unittest.skipUnless to explicitly skip tests