123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699 |
- const assert = require('assert')
- const {ipcRenderer, remote} = require('electron')
- const {BrowserWindow, Menu, MenuItem} = remote
- const {sortMenuItems} = require('../lib/browser/api/menu-utils')
- const {closeWindow} = require('./window-helpers')
- describe('Menu module', () => {
- describe('Menu.buildFromTemplate', () => {
- it('should be able to attach extra fields', () => {
- const menu = Menu.buildFromTemplate([
- {
- label: 'text',
- extra: 'field'
- }
- ])
- assert.equal(menu.items[0].extra, 'field')
- })
- it('does not modify the specified template', () => {
- const template = [{label: 'text', submenu: [{label: 'sub'}]}]
- const result = ipcRenderer.sendSync('eval', `const template = [{label: 'text', submenu: [{label: 'sub'}]}]\nrequire('electron').Menu.buildFromTemplate(template)\ntemplate`)
- assert.deepStrictEqual(result, template)
- })
- it('does not throw exceptions for undefined/null values', () => {
- assert.doesNotThrow(() => {
- Menu.buildFromTemplate([
- {
- label: 'text',
- accelerator: undefined
- },
- {
- label: 'text again',
- accelerator: null
- }
- ])
- })
- })
- describe('Menu sorting and building', () => {
- describe('sorts groups', () => {
- it('does a simple sort', () => {
- const items = [
- {
- label: 'two',
- id: '2',
- afterGroupContaining: ['1'] },
- { type: 'separator' },
- {
- id: '1',
- label: 'one'
- }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two',
- afterGroupContaining: ['1']
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('resolves cycles by ignoring things that conflict', () => {
- const items = [
- {
- id: '2',
- label: 'two',
- afterGroupContaining: ['1']
- },
- { type: 'separator' },
- {
- id: '1',
- label: 'one',
- afterGroupContaining: ['2']
- }
- ]
- const expected = [
- {
- id: '1',
- label: 'one',
- afterGroupContaining: ['2']
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two',
- afterGroupContaining: ['1']
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('ignores references to commands that do not exist', () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two',
- afterGroupContaining: ['does-not-exist']
- }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two',
- afterGroupContaining: ['does-not-exist']
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('only respects the first matching [before|after]GroupContaining rule in a given group', () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- beforeGroupContaining: ['1']
- },
- {
- id: '4',
- label: 'four',
- afterGroupContaining: ['2']
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- }
- ]
- const expected = [
- {
- id: '3',
- label: 'three',
- beforeGroupContaining: ['1']
- },
- {
- id: '4',
- label: 'four',
- afterGroupContaining: ['2']
- },
- { type: 'separator' },
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- })
- describe('moves an item to a different group by merging groups', () => {
- it('can move a group of one item', () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- after: ['1']
- },
- { type: 'separator' }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- {
- id: '3',
- label: 'three',
- after: ['1']
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it("moves all items in the moving item's group", () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- after: ['1']
- },
- {
- id: '4',
- label: 'four'
- },
- { type: 'separator' }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- {
- id: '3',
- label: 'three',
- after: ['1']
- },
- {
- id: '4',
- label: 'four'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it("ignores positions relative to commands that don't exist", () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- after: ['does-not-exist']
- },
- {
- id: '4',
- label: 'four',
- after: ['1']
- },
- { type: 'separator' }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- {
- id: '3',
- label: 'three',
- after: ['does-not-exist']
- },
- {
- id: '4',
- label: 'four',
- after: ['1']
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('can handle recursive group merging', () => {
- const items = [
- {
- id: '1',
- label: 'one',
- after: ['3']
- },
- {
- id: '2',
- label: 'two',
- before: ['1']
- },
- {
- id: '3',
- label: 'three'
- }
- ]
- const expected = [
- {
- id: '3',
- label: 'three'
- },
- {
- id: '2',
- label: 'two',
- before: ['1']
- },
- {
- id: '1',
- label: 'one',
- after: ['3']
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('can merge multiple groups when given a list of before/after commands', () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- after: ['1', '2']
- }
- ]
- const expected = [
- {
- id: '2',
- label: 'two'
- },
- {
- id: '1',
- label: 'one'
- },
- {
- id: '3',
- label: 'three',
- after: ['1', '2']
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- it('can merge multiple groups based on both before/after commands', () => {
- const items = [
- {
- id: '1',
- label: 'one'
- },
- { type: 'separator' },
- {
- id: '2',
- label: 'two'
- },
- { type: 'separator' },
- {
- id: '3',
- label: 'three',
- after: ['1'],
- before: ['2']
- }
- ]
- const expected = [
- {
- id: '1',
- label: 'one'
- },
- {
- id: '3',
- label: 'three',
- after: ['1'],
- before: ['2']
- },
- {
- id: '2',
- label: 'two'
- }
- ]
- assert.deepEqual(sortMenuItems(items), expected)
- })
- })
- it('should position before existing item', () => {
- const menu = Menu.buildFromTemplate([
- {
- id: '2',
- label: 'two'
- }, {
- id: '3',
- label: 'three'
- }, {
- id: '1',
- label: 'one',
- before: ['2']
- }
- ])
- assert.equal(menu.items[0].label, 'one')
- assert.equal(menu.items[1].label, 'two')
- assert.equal(menu.items[2].label, 'three')
- })
- it('should position after existing item', () => {
- const menu = Menu.buildFromTemplate([
- {
- id: '2',
- label: 'two',
- after: ['1']
- },
- {
- id: '1',
- label: 'one'
- }, {
- id: '3',
- label: 'three'
- }
- ])
- assert.equal(menu.items[0].label, 'one')
- assert.equal(menu.items[1].label, 'two')
- assert.equal(menu.items[2].label, 'three')
- })
- it('should filter excess menu separators', () => {
- const menuOne = Menu.buildFromTemplate([
- {
- type: 'separator'
- }, {
- label: 'a'
- }, {
- label: 'b'
- }, {
- label: 'c'
- }, {
- type: 'separator'
- }
- ])
- assert.equal(menuOne.items.length, 3)
- assert.equal(menuOne.items[0].label, 'a')
- assert.equal(menuOne.items[1].label, 'b')
- assert.equal(menuOne.items[2].label, 'c')
- const menuTwo = Menu.buildFromTemplate([
- {
- type: 'separator'
- }, {
- type: 'separator'
- }, {
- label: 'a'
- }, {
- label: 'b'
- }, {
- label: 'c'
- }, {
- type: 'separator'
- }, {
- type: 'separator'
- }
- ])
- assert.equal(menuTwo.items.length, 3)
- assert.equal(menuTwo.items[0].label, 'a')
- assert.equal(menuTwo.items[1].label, 'b')
- assert.equal(menuTwo.items[2].label, 'c')
- })
- it('should continue inserting items at next index when no specifier is present', () => {
- const menu = Menu.buildFromTemplate([
- {
- id: '2',
- label: 'two'
- }, {
- id: '3',
- label: 'three'
- }, {
- id: '4',
- label: 'four'
- }, {
- id: '5',
- label: 'five'
- }, {
- id: '1',
- label: 'one',
- before: ['2']
- }
- ])
- assert.equal(menu.items[0].label, 'one')
- assert.equal(menu.items[1].label, 'two')
- assert.equal(menu.items[2].label, 'three')
- assert.equal(menu.items[3].label, 'four')
- assert.equal(menu.items[4].label, 'five')
- })
- })
- })
- describe('Menu.getMenuItemById', () => {
- it('should return the item with the given id', () => {
- const menu = Menu.buildFromTemplate([
- {
- label: 'View',
- submenu: [
- {
- label: 'Enter Fullscreen',
- accelerator: 'ControlCommandF',
- id: 'fullScreen'
- }
- ]
- }
- ])
- const fsc = menu.getMenuItemById('fullScreen')
- assert.equal(menu.items[0].submenu.items[0], fsc)
- })
- })
- describe('Menu.insert', () => {
- it('should store item in @items by its index', () => {
- const menu = Menu.buildFromTemplate([
- {label: '1'},
- {label: '2'},
- {label: '3'}
- ])
- const item = new MenuItem({ label: 'inserted' })
- menu.insert(1, item)
- assert.equal(menu.items[0].label, '1')
- assert.equal(menu.items[1].label, 'inserted')
- assert.equal(menu.items[2].label, '2')
- assert.equal(menu.items[3].label, '3')
- })
- })
- describe('Menu.append', () => {
- it('should add the item to the end of the menu', () => {
- const menu = Menu.buildFromTemplate([
- {label: '1'},
- {label: '2'},
- {label: '3'}
- ])
- const item = new MenuItem({ label: 'inserted' })
- menu.append(item)
- assert.equal(menu.items[0].label, '1')
- assert.equal(menu.items[1].label, '2')
- assert.equal(menu.items[2].label, '3')
- assert.equal(menu.items[3].label, 'inserted')
- })
- })
- describe('Menu.popup', () => {
- let w = null
- let menu
- beforeEach(() => {
- w = new BrowserWindow({show: false, width: 200, height: 200})
- menu = Menu.buildFromTemplate([
- {label: '1'},
- {label: '2'},
- {label: '3'}
- ])
- })
- afterEach(() => {
- menu.closePopup()
- menu.closePopup(w)
- return closeWindow(w).then(() => { w = null })
- })
- it('throws an error if options is not an object', () => {
- assert.throws(() => {
- menu.popup()
- }, /Options must be an object/)
- })
- it('should emit menu-will-show event', (done) => {
- menu.on('menu-will-show', () => { done() })
- menu.popup({window: w})
- })
- it('should emit menu-will-close event', (done) => {
- menu.on('menu-will-close', () => { done() })
- menu.popup({window: w})
- menu.closePopup()
- })
- it('returns immediately', () => {
- const input = {window: w, x: 100, y: 101}
- const output = menu.popup(input)
- assert.equal(output.x, input.x)
- assert.equal(output.y, input.y)
- assert.equal(output.browserWindow, input.window)
- })
- it('works without a given BrowserWindow and options', () => {
- const {browserWindow, x, y} = menu.popup({x: 100, y: 101})
- assert.equal(browserWindow.constructor.name, 'BrowserWindow')
- assert.equal(x, 100)
- assert.equal(y, 101)
- })
- it('works with a given BrowserWindow, options and callback', (done) => {
- const {x, y} = menu.popup({
- window: w,
- x: 100,
- y: 101,
- callback: () => done()
- })
- assert.equal(x, 100)
- assert.equal(y, 101)
- menu.closePopup()
- })
- it('works with a given BrowserWindow, no options, and a callback', (done) => {
- menu.popup({window: w, callback: () => done()})
- menu.closePopup()
- })
- })
- describe('Menu.setApplicationMenu', () => {
- it('sets a menu', () => {
- const menu = Menu.buildFromTemplate([
- {label: '1'},
- {label: '2'}
- ])
- Menu.setApplicationMenu(menu)
- assert.notEqual(Menu.getApplicationMenu(), null)
- })
- it('unsets a menu with null', () => {
- Menu.setApplicationMenu(null)
- assert.equal(Menu.getApplicationMenu(), null)
- })
- })
- })
|