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
[SR-5040] Convert "exclude" related functional tests to unit test #5001
Comments
Comment by Shintaro Katafuchi (JIRA) Can I work on the issue? And before getting started do you have the exact output picture in your mind?😃 @aciidb0mb3r |
Comment by Berta Devant (JIRA) hotchemi (JIRA User) are you still working on this? |
Comment by Shintaro Katafuchi (JIRA) bertadevant (JIRA User) Nope I haven't done anything so far. Actually I don't imagine entire picture as for how to fix that so it'd be better to ask @aciidb0mb3r, and now it is I guess. |
This should be fairly straight forward to do. There are several examples in `PackageBuilderV4Tests.swift` but feel free to ask any doubts! |
Comment by Tiago Martinho (JIRA) Sorry @aciidb0mb3r, I don't find the following functional tests to migrate them into unit tests: "testManifestExcludes1, testManifestExcludes2, testManifestExcludes3, testManifestExcludes4, testManifestExcludes5" |
Ah, I resolved this while improving our testing infrastructure. Sorry I forgot to update this bug. |
Additional Detail from JIRA
md5: c5a6f7f0bb05cb7ab9788c5412465b6a
Issue Description:
We have five functional tests to test "exclude" property. They are from the era when unit tests were not possible but now we should be able to remove them and merge with the existing unit tests.
testManifestExcludes1, testManifestExcludes2, testManifestExcludes3, testManifestExcludes4, testManifestExcludes5
The text was updated successfully, but these errors were encountered: