Fixes an old TODO in a test.
We use MessageDecrypter.DecryptMessage() to "decrypt" a message
that uses the "NONE" encryption scheme. This means that
DecryptMessage() does nothing but deserialize a protobuf.
We replace a TODO with this.
Change-Id: Ic90fabbc7a255a826183080e8f79a49ca239315e
diff --git a/encoder/envelope_maker_test.cc b/encoder/envelope_maker_test.cc
index 600ed39..06cb16f 100644
--- a/encoder/envelope_maker_test.cc
+++ b/encoder/envelope_maker_test.cc
@@ -22,6 +22,7 @@
#include "encoder/encoder.h"
#include "encoder/project_context.h"
#include "third_party/googletest/googletest/include/gtest/gtest.h"
+#include "util/encrypted_message_util.h"
namespace cobalt {
namespace encoder {
@@ -228,11 +229,12 @@
EncryptedMessage encrypted_message;
EXPECT_TRUE(envelope_maker_.MakeEncryptedEnvelope(&encrypted_message));
- // Decrypt and check the encrypted Envelope.
- // TODO(rudominer) Do a decryption here when encryption is enabled.
- std::string serialized_envelope = encrypted_message.ciphertext();
+ // Decrypt encrypted_message. (No actual decryption is involved since
+ // we used the NONE encryption scheme.)
+ util::MessageDecrypter decrypter("");
Envelope recovered_envelope;
- EXPECT_TRUE(recovered_envelope.ParseFromString(serialized_envelope));
+ EXPECT_TRUE(
+ decrypter.DecryptMessage(encrypted_message, &recovered_envelope));
// Check that it looks right.
EXPECT_EQ(2, recovered_envelope.batch_size());
