The tests for SceptrePlan are pretty terrible and could use being completely redone

[jfalkenstein]

In looking over our codebase recently, I discovered that our tests on SceptrePlan are downright bad. Test coverage is poor, it doesn't really test all the functionality, and they're not very easy to read or maintain.

Ideally, we would have tests that cover all "public" methods of the class and validate the expected outcomes against the actual outcomes.

This is a very useful, low-risk opportunity for those looking to contribute to Sceptre.

[ericabrauer]

You can add me as an assignee, not sure how much bandwidth I have vs how much workload there is but I can at least start with taking a stab at breaking this out into chunks. Any help/paired programming sessions would definitely be appreciated :)

[jfalkenstein]

@ericabrauer, I'd be happy to assist ; )

[alex-harvey-z3q]

@ericabrauer I would be inclined to close this issue unless you are still working on it? We all know that the tests aren't perfect, but without Jon here to help, it seems unlikely it could get prioritised.

[ericabrauer]

@alex-harvey-z3q - I am so sorry! I thought I responded. I’m happy to close or maybe even break this out into smaller chunks if it’s still needed and if I could get some help.