Add an unit test about pext HOT 7 CLOSED

Hi, I am table to take a look at this. Is there anything I should know before dealing with this issue?

from pext.

The main issue is that everything in the main file, https://github.com/Pext/Pext/blob/master/pext/pext.py, is very tangled up with the rather huge Window class at https://github.com/Pext/Pext/blob/master/pext/pext.py#L1689-L2248. You will see functions in every class starting with the word "bind" which is generally where things are coupled to the Window class (either there or the constructor already). So it is likely you'll have to figure out how to decouple some things with the Window class so they can be tested without actually opening a window. This issue is just as much about cleaning the code so it becomes more testable than writing an actual issue test and probably one of the hardest issues (unless you're good at refactoring).

from pext.

I see, I think the task is clear for now. Please assign me to this issue

from pext.

Unfortunately, I can't seem to assign you because you haven't collaborated before and aren't in a team: https://help.github.com/articles/assigning-issues-and-pull-requests-to-other-github-users/. I'm inviting you to an "Hacktoberfest" team to try to work around this weird design, will assign you then.

from pext.

Okay, assigned :)

from pext.

Hey, it's been 2 weeks, just poking to ask if you managed to make any progress and what your plan for this is :)

from pext.

With @ata2001's recent contributions, I consider this issue good to close. Of course, more tests are a great idea, but this is a great start, and we have a test now :)

from pext.