search

autogram-is / spidergram

Structural analysis tools for complex web sites
GNU General Public License v3.0
111 stars 4 forks source link

Improve test coverage #35

Open eaton opened 1 year ago

eaton commented 1 year ago

The 0.5.0 release had acceptable test coverage via Ava, but the restructuring of the repository came with a culling of the old test framework and linter settings. We need to get those back in place; jest may be a better choice given the number of setup-dependent integration tests we'll need: Ava defaults to "everything in parallel" which makes them a bit squirrelly.

In addition, we may want to stick with vanilla eslint rather than an opinionated wrapper like xo/unicorn; untangling the opinionatedness took us as much time as we gained from the initial setup.

eaton commented 1 year ago

Ava has been rolled back in, but we still need actual test coverage; current priorities are query generation and parsing tools, as that's where we have the most hiccups with unexpected input.

eaton commented 1 year ago

This is improving slowly but surely; tests for a number of the parsing operations have now been added, which makes refactoring them quite a bit less harrowing. We're bumping this one forward as it's an ongoing project, not a single checkbox to tick off.