Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Unified Diff: content/common/origin_trials/trial_token_validator_unittest.cc

Issue 1909633003: Collect UMA data for Origin Trials (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/origin_trials/trial_token_validator_unittest.cc
diff --git a/content/common/origin_trials/trial_token_validator_unittest.cc b/content/common/origin_trials/trial_token_validator_unittest.cc
index 59011e785f2e87c0f7963619cdbf23986721993c..8cce482bb260918b8bb934b78b77877725724b02 100644
--- a/content/common/origin_trials/trial_token_validator_unittest.cc
+++ b/content/common/origin_trials/trial_token_validator_unittest.cc
@@ -12,6 +12,7 @@
#include "base/time/time.h"
#include "content/public/common/content_client.h"
#include "testing/gtest/include/gtest/gtest.h"
+#include "third_party/WebKit/public/platform/WebOriginTrialTokenStatus.h"
#include "url/gurl.h"
namespace content {
@@ -120,41 +121,51 @@ class TrialTokenValidatorTest : public testing::Test {
};
TEST_F(TrialTokenValidatorTest, ValidateValidToken) {
- EXPECT_TRUE(TrialTokenValidator::ValidateToken(
- kSampleToken, appropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::Success,
+ TrialTokenValidator::ValidateToken(
+ kSampleToken, appropriate_origin_, kAppropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateInappropriateOrigin) {
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kSampleToken, inappropriate_origin_, kAppropriateFeatureName));
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kSampleToken, insecure_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::WrongOrigin,
+ TrialTokenValidator::ValidateToken(
+ kSampleToken, inappropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::WrongOrigin,
+ TrialTokenValidator::ValidateToken(kSampleToken, insecure_origin_,
+ kAppropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateInappropriateFeature) {
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kSampleToken, appropriate_origin_, kInappropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::WrongFeature,
+ TrialTokenValidator::ValidateToken(
+ kSampleToken, appropriate_origin_, kInappropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateInvalidSignature) {
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kInvalidSignatureToken, appropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::InvalidSignature,
+ TrialTokenValidator::ValidateToken(kInvalidSignatureToken,
+ appropriate_origin_,
+ kAppropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateUnparsableToken) {
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kUnparsableToken, appropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::Malformed,
+ TrialTokenValidator::ValidateToken(kUnparsableToken,
+ appropriate_origin_,
+ kAppropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateExpiredToken) {
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kExpiredToken, appropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::Expired,
+ TrialTokenValidator::ValidateToken(
+ kExpiredToken, appropriate_origin_, kAppropriateFeatureName));
}
TEST_F(TrialTokenValidatorTest, ValidateValidTokenWithIncorrectKey) {
SetPublicKey(kTestPublicKey2);
- EXPECT_FALSE(TrialTokenValidator::ValidateToken(
- kSampleToken, appropriate_origin_, kAppropriateFeatureName));
+ EXPECT_EQ(blink::WebOriginTrialTokenStatus::InvalidSignature,
+ TrialTokenValidator::ValidateToken(
+ kSampleToken, appropriate_origin_, kAppropriateFeatureName));
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698