Changeset 267 in josm


Ignore:
Timestamp:
Jun 30, 2007 10:01:02 PM (6 years ago)
Author:
imi
Message:
  • added and fixed some tests
Files:
5 added
11 edited

Legend:

Unmodified
Added
Removed

  • src/org/openstreetmap/josm/Main.java

    r266 r267  
    234234                // iterate all plugins and collect all libraries of all plugins: 
    235235                List<URL> allPluginLibraries = new ArrayList<URL>(); 
    236                 for (Collection<PluginInformation> c : p.values()) { 
    237                         for (PluginInformation info : c) { 
    238                                 allPluginLibraries.addAll(info.getLibraries()); 
    239                         } 
    240                 } 
     236                for (Collection<PluginInformation> c : p.values()) 
     237                        for (PluginInformation info : c) 
     238                                allPluginLibraries.addAll(info.libraries); 
    241239                // create a classloader for all plugins: 
    242240                URL[] jarUrls = new URL[allPluginLibraries.size()]; 
    243241                jarUrls = allPluginLibraries.toArray(jarUrls); 
    244242                URLClassLoader pluginClassLoader = new URLClassLoader(jarUrls, Main.class.getClassLoader()); 
    245                  
    246                  
     243 
    247244                for (Collection<PluginInformation> c : p.values()) { 
    248245                        for (PluginInformation info : c) { 
    249246                                try { 
    250                                         info.setClassLoader(pluginClassLoader); // set the common classloader 
    251                                         Class<?> klass = info.loadClass(); 
     247                                        Class<?> klass = info.loadClass(pluginClassLoader); 
    252248                                        ImageProvider.sources.add(0, klass); 
    253249                                        System.out.println("loading "+info.name); 

  • src/org/openstreetmap/josm/gui/dialogs/HistoryDialog.java

    r203 r267  
    7272                } 
    7373        }; 
     74 
    7475        /** 
    7576         * Main table. 3 columns: 
    … …  
    8081 
    8182        private Map<OsmPrimitive, List<HistoryItem>> cache = new HashMap<OsmPrimitive, List<HistoryItem>>(); 
    82         private JLabel notLoaded = new JLabel("<html><i><p align=\"center\">"+tr("Click Reload to refresh list")+"</p></i></html>"); 
     83        private JLabel notLoaded = new JLabel("<html><i>"+tr("Click Reload to refresh list")+"</i></html>"); 
    8384        private JButton reloadButton = new JButton(tr("Reload"), ImageProvider.get("dialogs/refresh")); 
    8485        private JButton revertButton = new JButton(tr("Revert"), ImageProvider.get("dialogs/revert")); 
    … …  
    8889                historyPane.setVisible(false); 
    8990                notLoaded.setVisible(true); 
     91                notLoaded.setHorizontalAlignment(JLabel.CENTER); 
    9092 
    9193                history.setDefaultRenderer(Object.class, new DefaultTableCellRenderer(){ 
    … …  
    131133                reloadButton.setToolTipText(tr("Reload all currently selected objects and refresh the list.")); 
    132134                reloadButton.putClientProperty("help", "Dialog/History/Reload"); 
     135                 
    133136                revertButton.addActionListener(new ActionListener(){ 
    134137                        public void actionPerformed(ActionEvent e) { 

  • src/org/openstreetmap/josm/plugins/Plugin.java

    r237 r267  
    4040public abstract class Plugin { 
    4141 
    42         private String name; 
     42        String name; 
    4343 
    4444        public Plugin() { 
    45                 URL[] urls = ((URLClassLoader)getClass().getClassLoader()).getURLs(); 
    46                 name = urls[urls.length-1].toString(); 
    47                 if (name.toLowerCase().endsWith(".jar")) { 
    48                         int lastSlash = name.lastIndexOf('/'); 
    49                         name = name.substring(lastSlash+1, name.length()-4); 
    50                 } 
     45                try { 
     46                URL[] urls = ((URLClassLoader)getClass().getClassLoader()).getURLs(); 
     47                name = urls[urls.length-1].toString(); 
     48                if (name.toLowerCase().endsWith(".jar")) { 
     49                        int lastSlash = name.lastIndexOf('/'); 
     50                        name = name.substring(lastSlash+1, name.length()-4); 
     51                } 
     52        } catch (RuntimeException e) { 
     53                name = "unknown"; 
     54        } 
    5155    } 
    5256 
    5357        /** 
    5458         * @return The name of this plugin. This is the name of the .jar file. 
     59         * @deprecated Plugins have to know their name by themself. 
    5560         */ 
    56         public final String getName() { 
     61        @Deprecated public final String getName() { 
    5762                return name; 
    5863        } 
    5964        /** 
    6065         * @return The directory for the plugin to store all kind of stuff. 
     66         * @deprecated Use <code>Main.pref.getPreferencesDir()+"plugins/"+name+"/";</code> instead. 
    6167         */ 
    62         public final String getPluginDir() { 
     68        @Deprecated public final String getPluginDir() { 
    6369                return Main.pref.getPreferencesDir()+"plugins/"+name+"/"; 
    6470        } 

  • src/org/openstreetmap/josm/plugins/PluginException.java

    r149 r267  
    1111 */ 
    1212public class PluginException extends RuntimeException { 
    13         private final PluginProxy plugin; 
    14         private final String name; 
     13        public final PluginProxy plugin; 
     14        public final String name; 
    1515 
    1616        public PluginException(PluginProxy plugin, String name, Throwable cause) { 
    … …  
    1919                this.name = name; 
    2020    } 
    21          
    22         public PluginProxy getPlugin() { 
    23                 return plugin; 
    24         } 
    25         public String getName() { 
    26                 return name; 
    27         } 
    2821} 

  • src/org/openstreetmap/josm/plugins/PluginInformation.java

    r250 r267  
    66import java.io.InputStream; 
    77import java.net.URL; 
    8 import java.net.URLClassLoader; 
    98import java.util.ArrayList; 
    109import java.util.Collection; 
    … …  
    3332        public final String author; 
    3433        public final int stage; 
    35         protected final List<URL> libraries = new ArrayList<URL>(); 
    36         protected ClassLoader classLoader; 
     34        public final List<URL> libraries = new ArrayList<URL>(); 
    3735 
    3836        public final Map<String, String> attr = new TreeMap<String, String>(); 
    … …  
    8280                                        s = entry.toString(); 
    8381                                        entry = new StringBuilder(); 
    84                                         if (!s.startsWith("/") && !s.startsWith("\\") && !s.matches("^.\\:")) 
     82                                        if (!s.startsWith("/") && !s.startsWith("\\") && !s.matches("^.\\:") && file != null) 
    8583                                                s = file.getParent() + File.separator + s; 
    8684                                        libraries.add(new URL(getURLString(s))); 
    … …  
    111109         * Load the class of the plugin 
    112110         */ 
    113         public Class<?> loadClass() { 
     111        public Class<?> loadClass(ClassLoader classLoader) { 
    114112                try { 
    115                         Class<?> realClass = Class.forName(className, true, getClassLoader()); 
     113                        Class<?> realClass = Class.forName(className, true, classLoader); 
    116114                        return realClass; 
    117115                } catch (Exception e) { 
    … …  
    126124        } 
    127125 
    128         /** 
    129          * Returns all libraries the plugin needs (the plugin's jar file itself and all jars declared 
    130          * in the manifest's Class-Path section). 
    131          * @return the libraries a list of URLs to the libraries. 
    132          */ 
    133         public List<URL> getLibraries() { 
    134                 return libraries; 
    135         } 
    136  
    137         /** 
    138      * @return the classLoader 
    139      */ 
    140     public ClassLoader getClassLoader() { 
    141                 // if I have no classloader set, create one for me and my libraries: 
    142                 if(classLoader == null) { 
    143                         URL[] urls = new URL[libraries.size()]; 
    144                         urls = libraries.toArray(urls); 
    145                         classLoader = URLClassLoader.newInstance(urls, getClass().getClassLoader()); 
    146                 } 
    147         return classLoader; 
    148     } 
    149  
    150         /** 
    151      * @param classLoader the classLoader to set 
    152      */ 
    153     public void setClassLoader(ClassLoader classLoader) { 
    154         this.classLoader = classLoader; 
    155     } 
    156          
    157126        /** 
    158127         * Try to find a plugin after some criterias. Extract the plugin-information 

  • src/org/openstreetmap/josm/plugins/PluginProxy.java

    r238 r267  
    2424                this.plugin = plugin; 
    2525                this.info = info; 
     26 
     27                // setting name of the plugin by reflection 
     28                if (plugin instanceof Plugin) { 
     29                        try { 
     30                        Plugin.class.getDeclaredField("name").set(plugin, info.name); 
     31                } catch (Exception e) { 
     32                        if (e instanceof RuntimeException) 
     33                                throw (RuntimeException)e; 
     34                        throw new RuntimeException(e); 
     35                } 
     36                } 
    2637    } 
    2738 

  • src/org/openstreetmap/josm/tools/BugReportExceptionHandler.java

    r243 r267  
    4343                        // Check for an explicit problem when calling a plugin function 
    4444                        if (e instanceof PluginException) { 
    45                                 PluginProxy plugin = ((PluginException)e).getPlugin(); 
     45                                PluginProxy plugin = ((PluginException)e).plugin; 
    4646                                if (plugin != null && !plugin.misbehaving) { 
    4747                                        JOptionPane.showMessageDialog(Main.parent, tr("The plugin {0} throwed an exception: {1}\nIt may be outdated. Please contact the plugin's autor.\nThis message will not shown again until JOSM is restarted.", plugin.info.name, e.getMessage())); 

  • test/org/openstreetmap/josm/data/osm/OsmPrimitiveTest.java

    r146 r267  
    4444        } 
    4545 
    46         public void testHashCodeReturnsIdIfNotZeroAndNotZeroIfIdIsZero() { 
    47                 osm.id = 23; 
    48                 assertEquals(23, osm.hashCode()); 
    49                 osm.id = 0; 
    50                 assertTrue(0 != osm.hashCode()); 
    51         } 
    52  
    5346        public void testVisit() { 
    5447                osm.visit(new Visitor(){ 

  • test/org/openstreetmap/josm/plugins/PluginExceptionTest.java

    r153 r267  
    99                PluginException e = new PluginException(new PluginProxy(new String(), null), "42", barEx); 
    1010                assertEquals(barEx, e.getCause()); 
    11                 assertEquals("42", e.getName()); 
     11                assertEquals("42", e.name); 
    1212        } 
    1313 

  • test/org/openstreetmap/josm/plugins/PluginInformationTest.java

    r169 r267  
    11package org.openstreetmap.josm.plugins; 
    22 
     3import java.io.ByteArrayInputStream; 
     4import java.io.ByteArrayOutputStream; 
    35import java.io.File; 
     6import java.net.URL; 
     7import java.net.URLClassLoader; 
     8import java.util.Arrays; 
     9import java.util.Collection; 
     10import java.util.jar.JarInputStream; 
    411 
    512import junit.framework.TestCase; 
    613 
     14import org.openstreetmap.josm.Main; 
     15import org.openstreetmap.josm.data.Preferences; 
     16 
    717public class PluginInformationTest extends TestCase { 
     18 
     19        @Override protected void setUp() throws Exception { 
     20            super.setUp(); 
     21            Main.pref = new Preferences(){ 
     22                @Override public Collection<String> getAllPossiblePreferenceDirs() { 
     23                        return Arrays.asList(new String[]{getClass().getResource("..").getFile()}); 
     24            } 
     25        }; 
     26        } 
    827 
    928        public void testConstructorExtractsAttributesFromManifest() throws Exception { 
    1029                PluginInformation info = new PluginInformation(new File(getClass().getResource("simple.jar").getFile())); 
    1130                String s = getClass().getResource(".").getFile(); 
    12                  
    13                 assertEquals(4, info.libraries.size()); 
    14                 assertEquals(s+"foo", info.libraries.get(1).getFile()); 
    15                 assertEquals(s+"bar", info.libraries.get(2).getFile()); 
    16                 assertEquals(s+"C:/Foo%20and%20Bar", info.libraries.get(3).getFile()); 
    17                  
    18                 assertEquals("imi", info.author); 
    19                 assertEquals("Simple", info.className); 
    20                 assertEquals("Simpler", info.description); 
    21                 assertEquals(true, info.early); 
     31        assertEquals(4, info.libraries.size()); 
     32        assertEquals(s+"foo", info.libraries.get(1).getFile()); 
     33        assertEquals(s+"bar", info.libraries.get(2).getFile()); 
     34        assertEquals(s+"C:/Foo%20and%20Bar", info.libraries.get(3).getFile()); 
     35         
     36        assertEquals("imi", info.author); 
     37        assertEquals("Simple", info.className); 
     38        assertEquals("Simpler", info.description); 
     39        assertEquals(true, info.early); 
     40    } 
     41 
     42        public void testConstructorRequiresJarWithManifest() throws Exception { 
     43                try { 
     44                new PluginInformation(new File(getClass().getResource("no_manifest.jar").getFile())); 
     45                fail("Exception because missing manifest excpected"); 
     46        } catch (PluginException e) { 
     47        } 
     48    } 
     49         
     50        public void testConstructorWithInputStream() throws Exception { 
     51                JarInputStream f = new JarInputStream(getClass().getResourceAsStream("simple.jar")); 
     52                ByteArrayOutputStream out = new ByteArrayOutputStream(); 
     53                f.getManifest().write(out); 
     54 
     55                PluginInformation info = new PluginInformation(null, "simple", new ByteArrayInputStream(out.toByteArray())); 
     56        assertEquals("Only the 3 external classpaths are added (as we are using bootstrap classpath for plugin", 
     57                        3, info.libraries.size()); 
     58    } 
     59         
     60        public void testLoadClassInstantiatePlugin() throws Exception { 
     61                PluginInformation info = new PluginInformation(new File(getClass().getResource("working.jar").getFile())); 
     62                ClassLoader cl = new URLClassLoader(new URL[]{getClass().getResource("working.jar")}); 
     63                assertNotNull(info.load(info.loadClass(cl))); 
     64    } 
     65         
     66        // This is so the bugtracker always detect coding problems as "plugin problems" 
     67        public void testLoadThrowsPluginExceptionOnRuntimeException() throws Exception { 
     68                PluginInformation info = new PluginInformation(new File(getClass().getResource("working.jar").getFile())); 
     69                try { 
     70                info.load(null); 
     71                fail("Exception excpected because null-Class"); 
     72        } catch (PluginException e) { 
     73        } 
     74        try { 
     75                info.loadClass(null); 
     76                fail("Exception excpected because null-ClassLoader"); 
     77        } catch (PluginException e) { 
     78        } 
     79    } 
     80         
     81        public void testFindPluginReturnsInformationFromBootstrapClasspath() throws Exception { 
     82            PluginInformation info = PluginInformation.findPlugin("test_simple"); 
     83            assertEquals("Simpler", info.description); 
     84    } 
     85         
     86        public void testFindPluginReturnsFromPreferencesDirs() throws Exception { 
     87            PluginInformation info = PluginInformation.findPlugin("simple"); 
     88            assertEquals("Simpler", info.description); 
     89    } 
     90         
     91        public void testFindPluginForUnknownReturnsNull() throws Exception { 
     92                assertNull(PluginInformation.findPlugin("asdf")); 
     93        } 
     94 
     95        public void testPluginLocationsReturnModifiedPreferenceLocations() throws Exception { 
     96            setUp(); 
     97            Collection<String> locations = PluginInformation.getPluginLocations(); 
     98            assertEquals(1, locations.size()); 
     99            assertTrue(locations.iterator().next().endsWith("/plugins")); 
    22100    } 
    23101} 

  • test/org/openstreetmap/josm/tools/XmlObjectParserTest.java

    r193 r267  
    5252        } 
    5353 
    54         public void testManyTags() throws Exception { 
    55                 StringBuilder b = new StringBuilder("<all>"); 
    56                 for (int i = 0; i < 50000; ++i) { 
    57                         if (Math.random() > 0.5) { 
    58                                 b.append("<foo bar='blob"); 
    59                                 b.append(i); 
    60                                 b.append("'/>"); 
    61                         } else { 
    62                                 b.append("<foo><bar>yuppel"); 
    63                                 b.append(i); 
    64                                 b.append("</bar></foo>"); 
    65                         } 
    66                 } 
    67                 b.append("</all>"); 
    68  
    69                 System.gc(); 
    70                 long memBefore = Runtime.getRuntime().freeMemory(); 
    71                 parser = createParser(b.toString()); 
    72                 Thread.sleep(300); 
    73                 System.gc(); 
    74                 long memAfter = Runtime.getRuntime().freeMemory(); 
    75                 assertTrue("2MB should be more than enough. "+(memAfter-memBefore), memAfter-memBefore < 2*1024*1024); 
    76  
    77                 for (int i = 0; i < 50000; ++i) { 
    78                         Foo f = (Foo)parser.next(); 
    79                         assertTrue(f.bar.equals("blob"+i) || f.bar.equals("yuppel"+i)); 
    80                 } 
    81                 assertFalse(parser.hasNext()); 
    82         } 
    83  
    8454        public void testIterable() throws Exception { 
    8555                parser = createParser("<xml><foo bar='yo'/><foo bar='yo'/><foo bar='yo'/></xml>"); 
Note: See TracChangeset for help on using the changeset viewer.