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

Unit test failure in 'tests::blns' #4

Closed
luke-titley opened this issue Nov 8, 2019 · 9 comments

Comments

@luke-titley
Copy link

@luke-titley luke-titley commented Nov 8, 2019

Hey,
I'm getting a unit test failure in tests::blns.
I'm on a mac (mojave 10.14.10).

This is on the master and serde branches.

Release and debug builds.

running 6 tests
test hash::test_hashing ... ok
test tests::c_str_works ... ok
test tests::it_works ... ok
test tests::empty_string ... ok
test tests::blns ... FAILED
test tests::raft ... ok

failures:

---- tests::blns stdout ----
thread 'tests::blns' panicked at 'assertion failed: `(left == right)`
  left: `0`,
 right: `1315`', src/lib.rs:467:9
stack backtrace:
   0: std::panicking::default_hook::{{closure}}
   1: std::panicking::default_hook
   2: std::panicking::rust_panic_with_hook
   3: std::panicking::continue_panic_fmt
   4: std::panicking::begin_panic_fmt
   5: ustr::tests::blns
   6: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
   7: __rust_maybe_catch_panic
   8: test::run_test::run_test_inner::{{closure}}
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
@luke-titley

This comment has been minimized.

Copy link
Author

@luke-titley luke-titley commented Nov 8, 2019

 // check that the number of entries is the same
assert_eq!(super::num_entries(), hs.len());
@anderslanglands

This comment has been minimized.

Copy link
Owner

@anderslanglands anderslanglands commented Nov 8, 2019

@luke-titley

This comment has been minimized.

Copy link
Author

@luke-titley luke-titley commented Nov 9, 2019

Everything passes when it's all on one thread.

@anderslanglands

This comment has been minimized.

Copy link
Owner

@anderslanglands anderslanglands commented Nov 9, 2019

@luke-titley

This comment has been minimized.

Copy link
Author

@luke-titley luke-titley commented Nov 10, 2019

Is it intended to be thread safe ?

@anderslanglands

This comment has been minimized.

Copy link
Owner

@anderslanglands anderslanglands commented Nov 10, 2019

@luke-titley

This comment has been minimized.

Copy link
Author

@luke-titley luke-titley commented Nov 10, 2019

Aaah, right. Great!

@luke-titley

This comment has been minimized.

Copy link
Author

@luke-titley luke-titley commented Nov 12, 2019

Ah, I'm a fool. You already wrote this in the README.md

@anderslanglands

This comment has been minimized.

Copy link
Owner

@anderslanglands anderslanglands commented Nov 12, 2019

No worries :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.