-
Notifications
You must be signed in to change notification settings - Fork 144
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
Update tests for Themis iOS wrapper for new SecureMessage API #394
Conversation
@@ -37,55 +37,105 @@ - (void)testGeneratingKeys { | |||
} | |||
|
|||
- (void)testSecureMessageEC256 { |
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.
real code changes in this file
@@ -10,7 +10,6 @@ import XCTest | |||
class SecureMessageTestsSwift: XCTestCase { | |||
|
|||
func testExampleGeneratingKeys() { | |||
|
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.
real code changes in this file
tests/objcthemis/Podfile
Outdated
#pod 'themis', :git => "https://github.com/cossacklabs/themis.git" | ||
|
||
# updated secure message API | ||
pod 'themis', :git => "https://github.com/cossacklabs/themis.git", :branch => "vxtl/SM-ios" |
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.
will be changed to master on merge #393
@@ -5,31 +5,31 @@ PODS: | |||
- BoringSSL/Implementation (10.0.6): | |||
- BoringSSL/Interface (= 10.0.6) | |||
- BoringSSL/Interface (10.0.6) | |||
- GRKOpenSSLFramework (1.0.2.14.1) |
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.
skip
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.
Noice 👍
return [self wrapUnwrapWithPrivateKey:privateKeyData publicKey:publicKeyData]; | ||
} | ||
|
||
- (void)wrapUnwrapWithPrivateKey:(NSData *)privateKeyData publicKey:(NSData *)publicKeyData { | ||
TSMessage *encrypter = [[TSMessage alloc] initInEncryptModeWithPrivateKey:privateKeyData peerPublicKey:publicKeyData]; | ||
|
||
NSString *message = @"- Knock, knock.\n- Who’s there?\n*very long pause...*\n- Java."; |
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.
🤣 😂 ☕️
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.
Looks good, but there are some minor duplication mistakes in the new-new tests.
@ilammy thank you for pointing out copy-paste issues, fixed |
This is a second PR for updating Themis iOS wrapper for new SecureMessage API (#389).
It contains updating Secure Message test project to include new checks.
It must be merged after merging PR #393 – then I'll relink example podspec to use code from master.