Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: bad rx-state after cleanup #6503

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

enrico-teterra-maramalabs
Copy link

@enrico-teterra-maramalabs enrico-teterra-maramalabs commented Oct 15, 2024

This PR contains:

a failing test

Describe the problem you have without this PR

to reproduce the issue:

  • have an rx-state
  • make more than 5 writes to ensure _cleanup does its thing
  • the state has an additional "" property which causes problems further down the line

@pubkey
Copy link
Owner

pubkey commented Oct 16, 2024

Can you make the test by only using the "normal" RxDB api without accessing the internal storageInstance?
Atm this test does not seem to match you reproduction steps.

@enrico-teterra-maramalabs
Copy link
Author

yep no problem, please take a look at the test now

the test as it is now fails with:

+ expected - actual

       {
      -  "": {
      -    "foo": "bar6"
      -  }
         "foo": "bar6"
       }

@enrico-teterra-maramalabs enrico-teterra-maramalabs changed the title test: create state with cleaned up operation test: bad rx-state after cleanup Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants