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

TestQueries fail and output lots of logs like [WARN ] Skipped low-level close of cursor (write, TX #x alive) #5

Closed
miraclesu opened this issue Dec 9, 2018 · 3 comments

Comments

@miraclesu
Copy link

miraclesu commented Dec 9, 2018

OS: MacOS 10.12.6
Go: 1.11.2 amd64
libobjectbox: 0.4
objectbox-go: v0.8.0

go test -run=TestQueries

--- FAIL: TestQueries (0.11s)
    query_test.go:225: Executing 128 test cases
    assert.go:73: case #27 expected {Int64 in [94|47]}, but got {Int64 in [47|94]}
         Call stack:
        query_test.go:241

FAIL
exit status 1
FAIL	github.com/objectbox/objectbox-go/test	0.123s

go test -run='TestConcurrentPut$'

[WARN ] Skipped low-level close of cursor (write, TX #4 alive)
[WARN ] Skipped low-level close of cursor (write, TX #4 alive)
[WARN ] Skipped low-level close of cursor (write, TX #5 alive)
[WARN ] Skipped low-level close of cursor (write, TX #13 alive)
[WARN ] Skipped low-level close of cursor (write, TX #9 alive)
[WARN ] Skipped low-level close of cursor (write, TX #14 alive)
...
[WARN ] Skipped low-level close of cursor (write, TX #103 alive)
[WARN ] Skipped low-level close of cursor (write, TX #104 alive)
PASS
ok  	github.com/objectbox/objectbox-go/test	0.038s
@miraclesu miraclesu changed the title TestQueries fail and output lots of log like [WARN ] Skipped low-level close of cursor (write, TX #x alive) TestQueries fail and output lots of logs like [WARN ] Skipped low-level close of cursor (write, TX #x alive) Dec 9, 2018
@vaind
Copy link
Collaborator

vaind commented Dec 10, 2018

The IN query set order isn't fixed so we will update the tests to accommodate that.

The other part, the warnings, shouldn't have any impact but we will investigate.

Thanks for the report.

@greenrobot
Copy link
Member

greenrobot commented Dec 14, 2018

@miraclesu I just pushed the fix for the tests; could you please pull (just git; no other upgrade required) and verify?

@miraclesu
Copy link
Author

@greenrobot Yes, the tests passed. Thank you!

vaind added a commit that referenced this issue Nov 5, 2019
Conventions

Closes #5

See merge request objectbox/objectbox-go!59
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

No branches or pull requests

3 participants