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

Five broken tests (June 2024) #17

Open
KirkLandTrip opened this issue Jun 10, 2024 · 3 comments
Open

Five broken tests (June 2024) #17

KirkLandTrip opened this issue Jun 10, 2024 · 3 comments

Comments

@KirkLandTrip
Copy link

Hey
I tried building & running tests today. I used the last tests.zip from the release "Milestone 1" which is from August 2022.
There are currently 5 failed tests:

failures:

---- parser::tests::test_collect_shared_strings_system stdout ----
thread 'parser::tests::test_collect_shared_strings_system' panicked at src\parser.rs:397:70:
called `Result::unwrap()` on an `Err` value: Path

---- parser::tests::test_collect_strings_system stdout ----
thread 'parser::tests::test_collect_strings_system' panicked at src\parser.rs:363:57:
called `Result::unwrap()` on an `Err` value: Dir

---- parser::tests::test_collect_timesync_system stdout ----
thread 'parser::tests::test_collect_timesync_system' panicked at src\parser.rs:369:58:
called `Result::unwrap()` on an `Err` value: Path

---- parser::tests::test_collect_strings_archive stdout ----
thread 'parser::tests::test_collect_strings_archive' panicked at src\parser.rs:431:9:
assertion `left == right` failed
  left: "B6B65F4DC53ED38FEB0DDF61809853"
 right: "5283D7FC2531558F2C1ACE9AF26A0F"

---- parser::tests::test_shared_strings_archive stdout ----
thread 'parser::tests::test_shared_strings_archive' panicked at src\parser.rs:411:9:
assertion `left == right` failed
  left: 532
 right: 1976


failures:
    parser::tests::test_collect_shared_strings_system
    parser::tests::test_collect_strings_archive
    parser::tests::test_collect_strings_system
    parser::tests::test_collect_timesync_system
    parser::tests::test_shared_strings_archive

test result: FAILED. 186 passed; 5 failed; 0 ignored; 0 measured; 0 filtered out; finished in 13.72s

I'm guessing fixes to the code broke the tests over time. Do you guys have updated versions of those tests that you can release?
Thank you.

@puffyCid
Copy link
Collaborator

As mentioned in BUILDING.md.

Please make sure to download the tests.zip file from the release link and extract to your local repo if you want to run tests.

Let me know if u still get errors after running the tests

@KirkLandTrip
Copy link
Author

That's exactly what I did.
Other tests also use files from tests.zip and they passed.
It's just those 5 that even with the extracted files don't pass.

@puffyCid
Copy link
Collaborator

puffyCid commented Jul 7, 2024

Are you on a macOS laptop?
Unfortunately, some of these test expect a macOS system. The tests below require macOS, sorry.

---- parser::tests::test_collect_shared_strings_system stdout ----
thread 'parser::tests::test_collect_shared_strings_system' panicked at src\parser.rs:397:70:
called `Result::unwrap()` on an `Err` value: Path

---- parser::tests::test_collect_strings_system stdout ----
thread 'parser::tests::test_collect_strings_system' panicked at src\parser.rs:363:57:
called `Result::unwrap()` on an `Err` value: Dir

---- parser::tests::test_collect_timesync_system stdout ----
thread 'parser::tests::test_collect_timesync_system' panicked at src\parser.rs:369:58:
called `Result::unwrap()` on an `Err` value: Path

The 2 other tests should not require macOS. I downloaded the lastest code from the repo and the test.zip file. Both tests ran fine. Did you try redownloading the tests.zip file? Perhaps, a file was removed by mistake?

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