Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

unit tests #12

Open
jennschiffer opened this issue Nov 1, 2015 · 3 comments
Open

unit tests #12

jennschiffer opened this issue Nov 1, 2015 · 3 comments
Assignees

Comments

@jennschiffer
Copy link
Owner

I just did a big refactor of the very few tests that are on this project, which lead to finding some pretty major bugs. I was able to fix them, so I'm feeling p good about myself lol. Anyway, if anyone can come up with different ways to more strenuously test tables with sorting and pagination, that would be 💯

@edelooff
Copy link
Collaborator

edelooff commented Jan 7, 2016

Where we stand now, I think the actual sorting is tested pretty exhaustively. All of the text/value transforms are tested, loading is tested and the behavior of numbers in data-value attributes (natural sort instead of lex.).

What could do with better testing is the behavior of paginated tables. The first page of the paginated table is tested, but that's only a small portion of the things that a paginated table does. Navigation (with buttons or methods) isn't tested, nor are things like the page numbering.

When I find myself with more spare time I might have a look at this and ensure we test pagination more thoroughly. We can probably get away with a simpler test table as well, focusing on just testing the actual paginating (for this new suite).

@jennschiffer
Copy link
Owner Author

i'd be happy to take this on over the next 2 weeks. i'll have some downtime between projects then!

@edelooff
Copy link
Collaborator

I think this is resolved (for now anyway). I can't really think of any parts that seriously lack testing. There are some minor bits like the texts and classes that are set, but I'm not sure they really need the tests.

Mostly a matter of maintaining coverage at this point.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants