Revision: 1779
          http://svn.sourceforge.net/spring-rich-c/?rev=1779&view=rev
Author:   jhoskens
Date:     2007-07-10 03:44:37 -0700 (Tue, 10 Jul 2007)

Log Message:
-----------
Testing a mnemonic should use KeyEvent.VK_XXX keycodes instead of characters. 
Lowercase letters are not mapped to keycodes and thus cannot be used to compare 
to a mnemonic.
LabelInfo is already fixed to use uppercase letters only.

Modified Paths:
--------------
    
trunk/spring-richclient/core/src/test/java/org/springframework/richclient/core/LabelInfoTests.java
    
trunk/spring-richclient/support/src/test/java/org/springframework/richclient/command/config/CommandButtonLabelInfoTests.java

Modified: 
trunk/spring-richclient/core/src/test/java/org/springframework/richclient/core/LabelInfoTests.java
===================================================================
--- 
trunk/spring-richclient/core/src/test/java/org/springframework/richclient/core/LabelInfoTests.java
  2007-07-10 07:18:13 UTC (rev 1778)
+++ 
trunk/spring-richclient/core/src/test/java/org/springframework/richclient/core/LabelInfoTests.java
  2007-07-10 10:44:37 UTC (rev 1779)
@@ -15,6 +15,7 @@
  */
 package org.springframework.richclient.core;
 
+import java.awt.event.KeyEvent;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
@@ -86,7 +87,7 @@
         
         info = LabelInfo.valueOf("S\\&ave @&as");
         Assert.assertEquals("S&ave @as", info.getText());
-        Assert.assertEquals('a', info.getMnemonic());
+        Assert.assertEquals(KeyEvent.VK_A, info.getMnemonic());
         Assert.assertEquals(7, info.getMnemonicIndex());
         
     }
@@ -126,7 +127,7 @@
         LabelInfo info = LabelInfo.valueOf("&Save \\& Run");
         
         Assert.assertEquals(0, info.getMnemonicIndex());
-        Assert.assertEquals('S', info.getMnemonic());
+        Assert.assertEquals(KeyEvent.VK_S, info.getMnemonic());
         Assert.assertEquals("Save & Run", info.getText());
         
     }
@@ -152,7 +153,7 @@
         Assert.assertEquals("Something with an @ in it", info.getText());
         info = LabelInfo.valueOf("S\\&ave with an @ &as");
         Assert.assertEquals("S&ave with an @ as", info.getText());
-        Assert.assertEquals('a', info.getMnemonic());
+        Assert.assertEquals(KeyEvent.VK_A, info.getMnemonic());
         Assert.assertEquals(16, info.getMnemonicIndex());
         
     }
@@ -163,14 +164,14 @@
         assertEquals(0, info.getMnemonic());
         assertEquals(DEFAULT_MNEMONIC_INDEX, info.getMnemonicIndex());
 
-        info = new LabelInfo("test", 't');
+        info = new LabelInfo("test", KeyEvent.VK_T);
         assertEquals("test", info.getText());
-        assertEquals('t', info.getMnemonic());
+        assertEquals(KeyEvent.VK_T, info.getMnemonic());
         assertEquals(DEFAULT_MNEMONIC_INDEX, info.getMnemonicIndex());
 
-        info = new LabelInfo("test", 't', 3);
+        info = new LabelInfo("test", KeyEvent.VK_T, 3);
         assertEquals("test", info.getText());
-        assertEquals('t', info.getMnemonic());
+        assertEquals(KeyEvent.VK_T, info.getMnemonic());
         assertEquals(3, info.getMnemonicIndex());
     }
     
@@ -201,9 +202,9 @@
     }
 
     public void testConstructorEmptyText() {
-        LabelInfo info = new LabelInfo("", 'a', -1);
+        LabelInfo info = new LabelInfo("", KeyEvent.VK_A, -1);
         assertEquals("", info.getText());
-        assertEquals('a', info.getMnemonic());
+        assertEquals(KeyEvent.VK_A, info.getMnemonic());
         assertEquals(DEFAULT_MNEMONIC_INDEX, info.getMnemonicIndex());
     }
 
@@ -264,28 +265,28 @@
     public void testConfigureLabelForWithColon() {
         JTextField field = new JTextField();
         JLabel label = new JLabel();
-        LabelInfo info = new LabelInfo("Name:", 'N');
+        LabelInfo info = new LabelInfo("Name:", KeyEvent.VK_N);
         info.configureLabelFor(label, field);
 
         assertEquals("Name:", label.getText());
-        assertEquals('N', label.getDisplayedMnemonic());
+        assertEquals(KeyEvent.VK_N, label.getDisplayedMnemonic());
         assertEquals(field, label.getLabelFor());
     }
 
     public void testConfigureLabelForJPanel() {
         JPanel panel = new JPanel();
         JLabel label = new JLabel();
-        LabelInfo info = new LabelInfo("Name", 'N');
+        LabelInfo info = new LabelInfo("Name", KeyEvent.VK_N);
         info.configureLabelFor(label, panel);
 
         assertEquals("No colon for panel", "Name", label.getText());
-        assertEquals('N', label.getDisplayedMnemonic());
+        assertEquals(KeyEvent.VK_N, label.getDisplayedMnemonic());
         assertEquals(panel, label.getLabelFor());
     }
 
     public void testConstructorMnemonicIndexGreaterThanLength() {
         try {
-            new LabelInfo("test", 't', 4);
+            new LabelInfo("test", KeyEvent.VK_T, 4);
             fail("Mnemonic index must be < text.length()");
         }
         catch (IllegalArgumentException e) {
@@ -295,7 +296,7 @@
 
     public void testConstructorNegativeMnemonicIndex() {
         try {
-            new LabelInfo("test", 't', -2);
+            new LabelInfo("test", KeyEvent.VK_T, -2);
             fail("index must be >= -1");
         }
         catch (IllegalArgumentException e) {

Modified: 
trunk/spring-richclient/support/src/test/java/org/springframework/richclient/command/config/CommandButtonLabelInfoTests.java
===================================================================
--- 
trunk/spring-richclient/support/src/test/java/org/springframework/richclient/command/config/CommandButtonLabelInfoTests.java
        2007-07-10 07:18:13 UTC (rev 1778)
+++ 
trunk/spring-richclient/support/src/test/java/org/springframework/richclient/command/config/CommandButtonLabelInfoTests.java
        2007-07-10 10:44:37 UTC (rev 1779)
@@ -15,6 +15,7 @@
  */
 package org.springframework.richclient.command.config;
 
+import java.awt.event.KeyEvent;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
@@ -144,7 +145,7 @@
         System.out.println("XXXXXXXXXXXXXXX " + info.getText());
         System.out.println("XXXXXXXXXXX 16th char = " + 
info.getText().charAt(16));
         assertEquals("S&ave with an @ as", info.getText());
-        assertEquals('a', info.getMnemonic());
+        assertEquals(KeyEvent.VK_A, info.getMnemonic());
         assertEquals(16, info.getMnemonicIndex());
         assertNull(info.getAccelerator());
     }
@@ -153,7 +154,7 @@
         CommandButtonLabelInfo info = CommandButtonLabelInfo.valueOf("[EMAIL 
PROTECTED] &[EMAIL PROTECTED] A");
 
         assertEquals("[EMAIL PROTECTED] as", info.getText());
-        assertEquals('a', info.getMnemonic());
+        assertEquals(KeyEvent.VK_A, info.getMnemonic());
         assertEquals(5, info.getMnemonicIndex());
         assertNotNull("ctrl A is invalid keystroke", info.getAccelerator());
         assertEquals(KeyStroke.getKeyStroke("ctrl A"), info.getAccelerator());


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
spring-rich-c-cvs mailing list
spring-rich-c-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spring-rich-c-cvs

Reply via email to