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
CATTY-333 Use StoreProjectUploader in UploadViewController #1405
Conversation
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.
Please also add a test, ideally by injecting a test double for the uploader and executing the uploadAction to verify whether the proper method of the uploader is being called (please do not actually upload the project).
fbdc78a
to
42398c9
Compare
42398c9
to
e743bac
Compare
Hi, @m-herold I have made the changes. Please review |
e743bac
to
9b22603
Compare
9b22603
to
0e78475
Compare
Hi, @m-herold I have made the changes. Please review |
Hi @m-herold Jenkin test fails due to |
Jenkins, retest this please. |
0e78475
to
733f635
Compare
Hi, @m-herold I have updated the PR. Please review |
Jenkins, retest this please. |
32fe1f7
to
f3becb6
Compare
f3becb6
to
a4c015a
Compare
StoreProjectUploader
inUploadViewController
UploadViewController
Your checklist for this pull request
Please review the contributing guidelines and wiki pages of this repository.