-
Notifications
You must be signed in to change notification settings - Fork 482
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: add non-debian based tests for gawk #3482
Conversation
Hey there @terriko some of the tests are failing and i'm not sure how to debug them, i'm getting errors when running the scanner tests locally as well. I have attached a copy of my local ouput. Will you be able to help me on this?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I'm not seeing any of the things you're seeing, and they're not occurring in github actions -- maybe try updating your data cache if they're still happening for you?
that said, I am getting a single error on the tests claiming that the condensed downloads weren't committed. I see them in this PR, so I'm not sure what went wrong, but you might need to re-generate them and re-commit?
=========================== short test summary info ============================
FAILED test/test_condensed_downloads.py::test_condensed_downloads - AssertionError: Condensed downloads are not commited
assert False
+ where False = all(<generator object test_condensed_downloads.<locals>.<genexpr> at 0x7f38474f7a70>)
===== 1 failed, 1853 passed, 21 skipped, 68 warnings in 1925.11s (0:32:05) =====
Thank you terriko for the reply, i checked on it agaion and it was the binary files that caused the issue, i had the put the .deb files instead of the binaries previously. Hope it's all good now :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad you got it sorted, looks like we're ready to merge then!
Add Fedora rpm and openwrt ipk files to gawk tests
fixes: #3409