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

Add new tests #626

Merged
merged 9 commits into from
Dec 17, 2022
Merged

Add new tests #626

merged 9 commits into from
Dec 17, 2022

Conversation

zzmasoud
Copy link
Contributor

Hey @kean 👋 I've added some new tests that will increase the code coverage from 91.3% to 92.3% .

@zzmasoud
Copy link
Contributor Author

The previous failing check was because of CoreImageFilter and GaussianBlur on a WatchOS target which is unavailable. I've added #if os(iOS) || os(tvOS) || os(macOS) to prevent running these tests on that target.

@zzmasoud
Copy link
Contributor Author

zzmasoud commented Nov 20, 2022

So, the error is:

testDefaultImageCost() encountered an error (Test runner never began executing tests after launching. If you believe this error represents a bug, please attach the result bundle at /Users/runner/Library/Developer/Xcode/DerivedData/Nuke-bycrsvaaztorfmesymoxfuvcgnfe/Logs/Test/Test-Nuke-2022.11.20_14-31-06-+0000.xcresult)

I've just checked and tested it again, and even with randomized order ON, nothing failed. I didn't modify that test at all. Do you have any idea what's going on?

@kean
Copy link
Owner

kean commented Dec 17, 2022

Thanks for the contribution, @zzmasoud. I apologize for the delay – I was on extended hiatus.

@kean kean merged commit 6a42c4c into kean:master Dec 17, 2022
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

Successfully merging this pull request may close these issues.

2 participants