Ensure callback is called only once

[thecodrr]

Fixes #143

This PR fixes multiple issues with the queue/enqueue logic ensuring:

• onQueueEmpty is called only once
• The crawler aborts on error (before it kept crawling which was pointless)

Another change I had to make was how the crawler always returned results maxDepth + 1 deep. This might break some things (but not really sure since all tests pass).

cc @SuperchupuDev can you test this out with tinyglobby and let me know if something breaks?

[43081j]

does this change mean the root directory isn't pushed anymore? presumably directoryPath can be /, so this change would mean we push the child directories but not / itself.

that may be right, just checking i understand the logic and if this is a change in behaviour

[43081j]

looks good to me!

it may be worth adding a test for crawl("/") and maxDepth: 0 since thats what uncovered some of this in the first place

pretty sure we already test maxDepth elsewhere, just not at the root

[SuperchupuDev]

hi @thecodrr! thanks for fixing this so fast. i haven't been able to test this due to not having a windows machine nearby and the lack of preview builds. it'd be easy to do that if #124 happened (in which case let me know, i can easily open a pr to do it)

[SuperchupuDev]

@thecodrr tinyglobby@main tests seem to not be negatively affected by this fix, thank you @43081j for testing

[SuperchupuDev]

hey @thecodrr, it turns out this does break tests related to maxDepth, sorry for not finding out earlier 😅. thankfully, it's really easy to fix! i've opened a pr at #149