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

Unified Diff: remoting/webapp/unittests/menu_button_unittest.js

Issue 984203003: Move mocks and unittest JS files to sit alongside production code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/webapp/unittests/l10n_unittest.js ('k') | remoting/webapp/unittests/mock_signal_strategy.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/unittests/menu_button_unittest.js
diff --git a/remoting/webapp/unittests/menu_button_unittest.js b/remoting/webapp/unittests/menu_button_unittest.js
deleted file mode 100644
index 1367b0da40b42c62eeae3573814eed7f58b26fe7..0000000000000000000000000000000000000000
--- a/remoting/webapp/unittests/menu_button_unittest.js
+++ /dev/null
@@ -1,94 +0,0 @@
-// Copyright 2014 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.
-
-(function() {
-
-'use strict';
-
-/** @type {(sinon.Spy|function():void)} */
-var onShow = null;
-/** @type {(sinon.Spy|function():void)} */
-var onHide = null;
-/** @type {remoting.MenuButton} */
-var menuButton = null;
-
-module('MenuButton', {
- setup: function() {
- var fixture = document.getElementById('qunit-fixture');
- fixture.innerHTML =
- '<span class="menu-button" id="menu-button-container">' +
- '<button class="menu-button-activator">Click me</button>' +
- '<ul>' +
- '<li id="menu-option-1">Option 1</li>' +
- '</ul>' +
- '</span>';
- onShow = /** @type {(sinon.Spy|function():void)} */ (sinon.spy());
- onHide = /** @type {(sinon.Spy|function():void)} */ (sinon.spy());
- menuButton = new remoting.MenuButton(
- document.getElementById('menu-button-container'),
- /** @type {function():void} */ (onShow),
- /** @type {function():void} */ (onHide));
- },
- teardown: function() {
- onShow = null;
- onHide = null;
- menuButton = null;
- }
-});
-
-test('should display on click', function() {
- var menu = menuButton.menu();
- ok(menu.offsetWidth == 0 && menu.offsetHeight == 0);
- menuButton.button().click();
- ok(menu.offsetWidth != 0 && menu.offsetHeight != 0);
-});
-
-test('should dismiss when the menu is clicked', function() {
- var menu = menuButton.menu();
- menuButton.button().click();
- menu.click();
- ok(menu.offsetWidth == 0 && menu.offsetHeight == 0);
-});
-
-test('should dismiss when anything outside the menu is clicked', function() {
- var menu = menuButton.menu();
- menuButton.button().click();
- var x = menu.offsetRight + 1;
- var y = menu.offsetBottom + 1;
- var notMenu = document.elementFromPoint(x, y);
- base.debug.assert(notMenu != menu);
- notMenu.click();
- ok(menu.offsetWidth == 0 && menu.offsetHeight == 0);
-});
-
-test('should dismiss when menu item is clicked', function() {
- var menu = menuButton.menu();
- menuButton.button().click();
- var element = document.getElementById('menu-option-1');
- element.click();
- ok(menu.offsetWidth == 0 && menu.offsetHeight == 0);
-});
-
-test('should invoke callbacks', function() {
- ok(!onShow.called);
- menuButton.button().click();
- ok(onShow.called);
- ok(!onHide.called);
- menuButton.menu().click();
- ok(onHide.called);
-});
-
-test('select method should set/unset background image', function() {
- var element = document.getElementById('menu-option-1');
- var style = window.getComputedStyle(element);
- ok(style.backgroundImage == 'none');
- remoting.MenuButton.select(element, true);
- style = window.getComputedStyle(element);
- ok(style.backgroundImage != 'none');
- remoting.MenuButton.select(element, false);
- style = window.getComputedStyle(element);
- ok(style.backgroundImage == 'none');
-});
-
-}());
« no previous file with comments | « remoting/webapp/unittests/l10n_unittest.js ('k') | remoting/webapp/unittests/mock_signal_strategy.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698