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

Unified Diff: chrome/test/data/webui/certificate_viewer_dialog_test.js

Issue 7861024: Adds testing infrastructure to web_ui_browsertest to support testing HtmlDialogs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 3 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: chrome/test/data/webui/certificate_viewer_dialog_test.js
diff --git a/chrome/test/data/webui/certificate_viewer_dialog_test.js b/chrome/test/data/webui/certificate_viewer_dialog_test.js
new file mode 100644
index 0000000000000000000000000000000000000000..5b7804459e93c98c18efaea88ce3d72d6c441480
--- /dev/null
+++ b/chrome/test/data/webui/certificate_viewer_dialog_test.js
@@ -0,0 +1,89 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * Tests that the dialog opened to the correct URL.
+ * @param {string} url The URL to verify against, passed in from C++ to use the
+ * value in url_constants.cc.
+ */
+function testDialogURL(url) {
+ assertEquals(url, window.location.href);
+}
+
+/**
+ * Tests for the correct common name in the test certificate.
+ */
+function testCN() {
+ assertEquals('www.google.com', $('issued-cn').textContent);
+}
+
+/**
+ * Test the details pane of the certificate viewer. This verifies that a
+ * certificate in the chain can be selected to view the fields. And that fields
+ * can be selected to view their values.
+ */
+function testDetails() {
+ var certHierarchy = $('hierarchy');
+ var certFields = $('cert-fields');
+ var certFieldVal = $('cert-field-value');
+
+ // There must be at least one certificate in the hierarchy.
+ assertLT(0, certHierarchy.childNodes.length);
+ // No certificate fields should be currently loaded.
+ assertEquals(0, certFields.childNodes.length);
+
+ // Select the first certificate on the chain and ensure the details show up.
+ // Override the receive certificate function to catch when fields are loaded.
+ var getCertificateFields = cert_viewer.getCertificateFields;
+ cert_viewer.getCertificateFields = function(certFieldDetails) {
+ getCertificateFields(certFieldDetails);
+ cert_viewer.getCertificateFields = getCertificateFields;
+ runAssertions('testDetails', function() {
+ assertLT(0, certFields.childNodes.length);
+
+ // Test that a field can be selected to see the details for that field.
+ var item = getElementWithValue(certFields);
+ assertNotEquals(null, item);
+ certFields.selectedItem = item;
+ assertEquals(item.detail.payload.val, certFieldVal.textContent);
+
+ // Test that selecting an item without a value empties the field.
+ certFields.selectedItem = certFields.childNodes[0];
+ assertEquals('', certFieldVal.textContent);
+
+ testDone();
+ });
+ }
+ certHierarchy.selectedItem = certHierarchy.childNodes[0];
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// Support functions
Sheridan Rawlins 2011/09/09 15:47:50 I don't think you should need these - I would hope
flackr 2011/09/15 18:41:52 The issue I'm having is that the callback function
+
+/**
+ * Run a block of code failing the asynchronous test on any failed assertions.
+ * @param {string} testName The name of the test being run (for error messages).
+ * @param {function()} f A function to run which may throw an exception.
+ */
+function runAssertions(testName, f) {
+ var result = runTestFunction(testName, f, []);
+ if (!result[0])
+ testDone();
+}
+
+/**
+ * Find the first tree item (in the certificate fields tree) with a value.
+ * @param {!Element} tree Certificate fields subtree to search.
+ * @return {Element} The first found element with a value, or null if not found.
+ */
+function getElementWithValue(tree) {
+ for (var i = 0; i < tree.childNodes.length; i++) {
+ var element = tree.childNodes[i];
+ if (element.detail && element.detail.payload && element.detail.payload.val)
+ return element;
+ if (element = getElementWithValue(element))
+ return element;
+ }
+ return null;
+}
« chrome/browser/ui/webui/web_ui_browsertest.cc ('K') | « chrome/common/chrome_notification_types.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698