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

tests are failing (selinux?) #9

Closed
ignatenkobrain opened this issue Feb 26, 2017 · 6 comments
Closed

tests are failing (selinux?) #9

ignatenkobrain opened this issue Feb 26, 2017 · 6 comments

Comments

@ignatenkobrain
Copy link
Contributor

test test_multi ... FAILED
test test_fd ... FAILED
test test_path ... FAILED

failures:

---- test_multi stdout ----
	thread 'test_multi' panicked at 'assertion failed: items.remove(&*it)', tests/main.rs:59

---- test_fd stdout ----
	thread 'test_fd' panicked at 'assertion failed: `(left == right)` (left: `Some("security.selinux")`, right: `None`)', tests/main.rs:16
note: Run with `RUST_BACKTRACE=1` for a backtrace.

---- test_path stdout ----
	thread 'test_path' panicked at 'assertion failed: `(left == right)` (left: `Some("security.selinux")`, right: `None`)', tests/main.rs:33
@Stebalien
Copy link
Owner

Does that fix it? Also, do you need me to cut a release (I usually don't for test case changes).

@ignatenkobrain
Copy link
Contributor Author

@Stebalien better, but one test still fails:

running 3 tests
test test_multi ... ok
test test_fd ... ok
test test_path ... FAILED

failures:

---- test_path stdout ----
	thread 'test_path' panicked at 'assertion failed: `(left == right)` (left: `1`, right: `0`)', tests/main.rs:44
note: Run with `RUST_BACKTRACE=1` for a backtrace.


failures:
    test_path

test result: FAILED. 2 passed; 1 failed; 0 ignored; 0 measured

@Stebalien
Copy link
Owner

Stebalien commented Feb 27, 2017 via email

@ignatenkobrain
Copy link
Contributor Author

@Stebalien thanks for fixing... About release -- up to you. I can just apply two patches in Fedora.

Stebalien added a commit that referenced this issue Feb 27, 2017
Ignore non-user xattrs in tests.

This should fix tests on systems with, e.g., selinux.

Fixes #9
@Stebalien
Copy link
Owner

Stebalien commented Feb 27, 2017 via email

@ignatenkobrain
Copy link
Contributor Author

Thanks a lot!

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

2 participants