Problems with ElmTextSearch.remove and later calls to ElmTextSearch.search

[mthadley]

Hey there! For starters, thanks for making this awesome library!

I was having some problems today with ElmTextSearch.remove. After calling it to remove a document from an index, I noticed that later search results seemed to come up empty. Perhaps I have misinterpreted what remove is supposed to do. Either way, I think I was able to reproduce it what I was seeing with a failing test.

For example, try removing the call to remove in the test, and you'll see that the test will start passing:

removeDoesNotBreakSearchResults : Test
removeDoesNotBreakSearchResults =
    test "Remove does not break searching" <|
        \() ->
            getIndexDoc1Doc2 ()
                -- Commenting these lines causes the test to pass
                -- |> Index.remove doc2_
                -- |> TestUtils.getResultIgnoreError
                |> Index.search "Sally"
                |> TestUtils.getResultIgnoreError
                |> Tuple.second
                |> List.map Tuple.first
                |> Expect.equal [doc1_.cid]

I'm not familiar with the internals of the library, so I didn't attempt a fix. However, even before that, does this seem like an actual bug?

[rluiten]

On the surface it looks to me like a bug.
I hope to allocate some time to this in the next week or so.
Thank you for reporting it with a unit test.