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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/credentialmanager/federatedcredential-basics.html

Issue 1745963002: CREDENTIAL: Do type checks for credential constructors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/credentialmanager/passwordcredential-basics.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <title>Credential Manager: FederatedCredential basics.</title> 2 <title>Credential Manager: FederatedCredential basics.</title>
3 <script src="../resources/testharness.js"></script> 3 <script src="../resources/testharness.js"></script>
4 <script src="../resources/testharnessreport.js"></script> 4 <script src="../resources/testharnessreport.js"></script>
5 <script src="/serviceworker/resources/interfaces.js"></script> 5 <script src="/serviceworker/resources/interfaces.js"></script>
6 <script> 6 <script>
7 test(function() { 7 test(function() {
8 var credential = new FederatedCredential({ 8 var credential = new FederatedCredential({
9 'id': 'id', 9 'id': 'id',
10 'provider': 'https://federation.test/', 10 'provider': 'https://federation.test/',
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 test(function() { 84 test(function() {
85 assert_throws(new SyntaxError(), function () { 85 assert_throws(new SyntaxError(), function () {
86 var credential = new FederatedCredential({ 86 var credential = new FederatedCredential({
87 'id': 'id', 87 'id': 'id',
88 'provider': '-', 88 'provider': '-',
89 'name': 'name', 89 'name': 'name',
90 'iconURL': 'https://example.test/icon.png' 90 'iconURL': 'https://example.test/icon.png'
91 }); 91 });
92 }); 92 });
93 }, 'Construct a FederatedCredential with an invalid federation URL.'); 93 }, 'Construct a FederatedCredential with an invalid federation URL.');
94
95 test(function() {
96 assert_throws(new TypeError(),
97 () => { new FederatedCredential(); });
98 assert_throws(new TypeError(),
99 () => { new FederatedCredential({}); });
100 assert_throws(new TypeError(),
101 () => { new FederatedCredential({ 'id': undefined }); });
102 assert_throws(new TypeError(),
103 () => { new FederatedCredential({ 'id': '' }); });
104 assert_throws(new TypeError(),
105 () => { new FederatedCredential({ 'provider': undefined }); }) ;
106 assert_throws(new TypeError(),
107 () => { new FederatedCredential({ 'provider': '' }); });
108 assert_throws(new TypeError(),
109 () => { new FederatedCredential({ 'id': undefined, 'provider': undefined }); });
110 assert_throws(new TypeError(),
111 () => { new FederatedCredential({ 'id': undefined, 'provider': '' }); });
112 assert_throws(new TypeError(),
113 () => { new FederatedCredential({ 'id': undefined, 'provider': undefined }); });
114 }, 'FederatedCredential objects require IDs and Providers');
94 </script> 115 </script>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/credentialmanager/passwordcredential-basics.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698