Refactors

[tusharsadhwani]

• use hasattr instead of getattr(..., None) is None
• Rename TestEventLoopPolicy class so that pytest doesn't warn about not being able to test that class.

[alex-sherman]

Thanks for the PR!

Personally I prefer the gettatr method. It would be weird if, for whatever reason, we add _loop member to the class and that breaks it (even though the value is None). I'd request reverting that.

I would also suggest renaming to EventLoopTestPolicy instead to avoid the underscore.

[tusharsadhwani]

getattr would also break on an equally unlikely _loop = 42 definition. My humble suggestion would he to consider using more widely understood code instead of optimizing for the fringe None case.

EventLoopTestPolicy sounds good.

[alex-sherman]

42 and None are not equally likely, None would be a pretty obvious choice for something that is lazy initialized and was previously the way these fields worked. I don't really see the benefit of the mild cleanup vs. a less robust implementation. Please revert the hasattr change.