mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Add failing spec for Menu.buildFromTemplate

Browse source
This commit is contained in:
Kevin Sawicki 2015-12-07 09:12:48 -08:00
parent e1d7ef7e24
commit f178787737
2 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
spec/api-menu-spec.coffee
View file

@ -1,6 +1,6 @@
assert = require 'assert'
{remote} = require 'electron'
{remote, ipcRenderer} = require 'electron'
{Menu, MenuItem} = remote.require 'electron'
describe 'menu module', ->
@ -9,6 +9,11 @@ describe 'menu module', ->
menu = Menu.buildFromTemplate [label: 'text', extra: 'field']
assert.equal menu.items[0].extra, 'field'
it 'does not modify the specified template', ->
template = [label: 'text', submenu: [label: 'sub']]
builtTemplate = ipcRenderer.sendSync('menu-build-from-template', template)
assert.deepStrictEqual builtTemplate, template
describe 'Menu.buildFromTemplate should reorder based on item position specifiers', ->
it 'should position before existing item', ->
menu = Menu.buildFromTemplate [

7
spec/static/main.js
View file

@ -3,6 +3,7 @@ const app = electron.app;
const ipcMain = electron.ipcMain;
const dialog = electron.dialog;
const BrowserWindow = electron.BrowserWindow;
const Menu = electron.Menu;
const path = require('path');
@ -100,4 +101,10 @@ app.on('ready', function() {
});
event.returnValue = "done";
});
// Verify Menu.buildFromTemplate does not modify the specified template
ipcMain.on('menu-build-from-template', function(event, template) {
Menu.buildFromTemplate(template);
event.returnValue = template;
})
});