Changeset 32686 for main/trunk/gli/src/org/greenstone/gatherer/Gatherer.java

Timestamp:

2018-12-14T22:23:47+13:00 (5 years ago)

Author:

ak19

Message:

For GUI testing, it's far and away easier if only GUI components had names. Else it's a real pain to access components from the AssertJ Swing API. Introducing setNamesRecursively, a new method (run on the Event Dispatch Thread EDT as is required by the code) that uses reflection to automate setting the names of hopefully all if not all the most important for testing GUI components of GLI. It's called from Gatherer's openGUI which is performed on the EDT, so no need to call invokeLater() to force this method to take place on the Event Dispatch Thread. There are 2 caveats: not sure all the components are named and not all of them appear to have a unique name, so I may expand the code hereafter to add numbering after checking against a hash map for uniqueness if I find this to be necessary

File:

• main/trunk/gli/src/org/greenstone/gatherer/Gatherer.java (modified) (1 diff)

main/trunk/gli/src/org/greenstone/gatherer/Gatherer.java

@@ -755 +755 @@
         }
     }*/
-
+        g_man.setName("GUIManager"); // g_man has a name "frame0" assigned automatically
+        // setNamesRecursively won't assign a name if one is already set
+        // Setting explicitly since GUIManager is preferred for its name
+        GUIManager.setNamesRecursively("", g_man);
+        GUIManager.printComponentNames(g_man, "");
+        
         // If there was a collection left open last time, reopen it 
         if (open_collection_file_path == null || new File(Gatherer.open_collection_file_path).isDirectory()) {