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
Feature/verifying target object has all needed dependencies #5
Feature/verifying target object has all needed dependencies #5
Conversation
|
||
// Verifying that target object has all needed dependencies | ||
for (NSString *propertyName in dependencies) { | ||
XCTAssertTrue([propertyNames containsObject:propertyName], @"У объекта %@ не было найдено свойство %@", targetObject, propertyName); |
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.
We use English language in this project
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.
@igrekde Do you mean this comment?
У объекта %@ не было найдено свойство %@
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.
Yes
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.
Hmm...however, project has some Russian comments, like this:
XCTAssertNotNil(dependencyObject, @"Свойство %@ объекта %@ не должно быть nil", propertyName, targetObject);
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.
@igrekde ok, I'm fix it
@spbvasilenko thanks for your contribution! One small fix and we are done! |
@igrekde In the last commit this is comment has been fixed |
Great, today I'll release a new version! |
…ct-has-all-needed-dependencies Feature/verifying target object has all needed dependencies
@igrekde Thank you! |
I'm added case for verifying that target object has all needed dependencies.
This is pull request for 4 issue