added a basic self-test framework...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>

ui (gen4)/features/core.js

@@ -855,5 +855,67 @@ module.Tasks = ImageGridFeatures.Feature({
 
 
 
+//---------------------------------------------------------------------
+// Self test framework...
+
+var selfTest =
+module.selfTest = function(func){
+	func.__self_test__ = true
+	return func
+}
+
+var SelfTestActions = actions.Actions({
+	config: {
+		'run-selftest-on-start': true,
+	},
+
+	runSelfTest: ['System/Run self test',
+		selfTest(function(mode){
+			var that = this
+			var logger = this.logger && this.logger.push('Self test')
+
+			var tests = this.actions
+				.filter(function(action){ 
+					return action != 'runSelfTest'
+			   			&& (that[action].func.__self_test__ 
+							|| that.getActionAttr(action, 'self_test'))})
+
+			logger 
+				&& tests.forEach(function(action){ 
+					logger.emit('found', action) })
+
+
+			tests.forEach(function(action){
+				that[action]()
+
+				logger.emit('done', action)
+			})
+		})],
+})
+
+var SelfTest = 
+module.SelfTest = ImageGridFeatures.Feature({
+	title: '',
+	doc: '',
+
+	tag: 'self-test',
+	depends: [
+		'lifecycle'	
+	],
+	priority: 'low',
+
+	actions: SelfTestActions, 
+
+	handlers: [
+		['start',
+			function(){ 
+				this.config['run-selftest-on-start'] 
+					&& this.runSelfTest() }]
+	],
+})
+
+
+
+
 /**********************************************************************
 * vim:set ts=4 sw=4 :                               */ return module })

ui (gen4)/features/keyboard.js

@@ -153,7 +153,7 @@ module.GLOBAL_KEYBOARD2 = {
 		meta_Q: 'close',
 
 		// XXX
-		F5: keyboard.doc('Full reload viewer', 
+		F5: keyboard.doc('Reload viewer (full)', 
 			function(){ 
 				//a.stop()
 				/*
@@ -437,6 +437,40 @@ var KeyboardActions = actions.Actions({
 	get keyboard(){
 		return this.__keyboard_object },
 
+	testKeyboardDoc: ['- Interface/',
+		{self_test: true},
+		function(){
+			var that = this
+			var keys = this.keyboard.keys()
+
+			var index = {}
+			Object.keys(keys).forEach(function(mode){
+				Object.keys(keys[mode]).forEach(function(code){
+					if(code == ''){
+						return
+					}
+
+					var a = keyboard.parseActionCall(code)
+					var doc = a.doc || that.getDocTitle(a.action) || null
+
+					// check if we have no doc...
+					if(doc == null || doc == ''){
+						console.warn('Action has no doc: "'
+							+ a.action +'" at: "'+ code +'"') 
+					}
+
+					// see if two actions have the same doc...
+					if(index[doc] && index[doc] != a.action){
+						console.warn('Actions have same title: "' 
+							+ index[doc] +'" and "'+ a.action
+							+'" at: "'+ code +'"')
+					}
+
+					index[doc] = a.action
+				})
+			})
+		}],
+
 	// XXX need a clean deep copy to restore...
 	resetKeyBindings: ['Interface/Restore default key bindings',
 		function(){ 
@@ -444,7 +478,12 @@ var KeyboardActions = actions.Actions({
 
 	keyHandler: ['- Interface/Get or set key handler',
 		function(mode, key, action){ 
-			return this.keyboard.handler(mode, key, action) }],
+			var res = this.keyboard.handler(mode, key, action) 
+			// return res only if we get a handler...
+			if(!action){
+				return res
+			}
+		}],
 
 	// Get normalized, flat set of actions and keys that trigger them...
 	//
@@ -575,9 +614,8 @@ var KeyboardActions = actions.Actions({
 				// NOTE: the target element must be focusable...
 				var target =
 				this.__keyboard_event_source =
-					(this.config['keyboard-event-source'] 
-							|| this.config['keyboard-event-source'] == 'window') ? 
-						$(window)
+					this.config['keyboard-event-source'] == null ? this.ribbons.viewer 
+					: this.config['keyboard-event-source'] == 'window' ? $(window)
 					: this.config['keyboard-event-source'] == 'viewer' ? this.ribbons.viewer
 					: this.config['keyboard-event-source'] == 'document' ? $(document)
 					: $(this.config['keyboard-event-source'])
@@ -646,7 +684,8 @@ var KeyboardActions = actions.Actions({
 	//		new key
 	// XXX BUG sections with doc do not show up in title...
 	// XXX sub-group by path (???)
-	browseKeyboardBindings: ['Interface/Keyboard bindings...',
+	// XXX place this in /Doc/.. (???)
+	browseKeyboardBindings: ['Interface|Help/Keyboard bindings...',
 		widgets.makeUIDialog(function(path, edit, get_text){
 			var actions = this
 			var keybindings = this.keybindings
@@ -836,7 +875,6 @@ var KeyboardActions = actions.Actions({
 
 			return dialog
 		})],
-	// XXX place this in /Doc/.. (???)
 	editKeyboardBindings: ['Interface/Keyboard bindings editor...',
 		widgets.uiDialog(function(path){ 
 			return this.browseKeyboardBindings(path, true) })],
@@ -860,6 +898,7 @@ var KeyboardActions = actions.Actions({
 				background: 'white',
 				focusable: true,
 			})],
+
 })
 
 var Keyboard = 
@@ -871,6 +910,9 @@ module.Keyboard = core.ImageGridFeatures.Feature({
 	depends: [
 		'ui'
 	],
+	suggested: [
+		'self-test',
+	],
 
 	actions: KeyboardActions, 
 
@@ -896,6 +938,10 @@ module.Keyboard = core.ImageGridFeatures.Feature({
 					}
 				}
 			}],
+
+		['keyHandler',
+			function(res, mode, key, action){
+				action && this.checkKeyboardDoc() }],
 	],
 })
 

ui (gen4)/features/ui-status.js

@@ -496,7 +496,7 @@ var StatusBarActions = actions.Actions({
 
 	// XXX should these be here???
 	// XXX should this show a dialog???
-	editStatusBarIndex: ['- Interface/',
+	editStatusBarIndex: ['- Interface/Edit image focus position in statusbar',
 		function(){
 			if((this.config['status-bar-index'] || {} )['editable']){
 				this.toggleStatusBar('?') == 'none' && this.toggleStatusBar()
@@ -505,7 +505,7 @@ var StatusBarActions = actions.Actions({
 				this.ribbons.viewer.find('.global-info .index .position').focus().click()
 			}
 		}],
-	editStatusBarRibbon: ['- Interface/',
+	editStatusBarRibbon: ['- Interface/Edit ribbon focus position in statusbar',
 		function(){
 			this.toggleStatusBar('?') == 'none' && this.toggleStatusBar()
 

ui (gen4)/features/ui-widgets.js

@@ -775,9 +775,10 @@ var BrowseActionsActions = actions.Actions({
 				'Crop/-81:Uncrop all',
 				'Crop/-82:Uncrop',
 
-			'-50:Interface',
-			'-60:Workspace',
-			'-70:System',
+			'-40:Interface',
+			'-50:Workspace',
+			'-60:System',
+			'-70:Help',
 			'-80:---',
 			'-90:Development',
 			'-90:Test',