Added a couple of unit tests for the URL Catcher plugin

Added an extra test file + tests for the Logging Plugin's reverse file reader
Issue 1106

git-svn-id: http://svn.dmdirc.com/trunk@3911 00569f92-eb28-0410-84fd-f71c24880f

test/com/dmdirc/addons/logging/ReverseFileReaderTest.java

@@ -30,95 +30,137 @@ import java.util.Stack;
 import org.junit.Test;
 import static org.junit.Assert.*;
 
-public class ReverseFileReaderTest extends junit.framework.TestCase {
+public class ReverseFileReaderTest {
 
     @Test
-    public void testIndividual() {
-        try {
-            final ReverseFileReader reader =
-                    new ReverseFileReader(new File(getClass().getClassLoader().
-                    getResource("com/dmdirc/addons/logging/test1.txt").
-                    toURI()));
-            assertEquals("Line 7", reader.getNextLine());
-            assertEquals("Line 6", reader.getNextLine());
-            assertEquals("Line 5", reader.getNextLine());
-            assertEquals("Line 4", reader.getNextLine());
-            assertEquals("Line 3", reader.getNextLine());
-            assertEquals("Line 2", reader.getNextLine());
-            assertEquals("Line 1", reader.getNextLine());
-            reader.close();
-        } catch (URISyntaxException ex) {
-            assertFalse(true);
-        } catch (IOException ex) {
-            assertFalse(true);
-        }
+    public void testIndividual() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));
+        assertEquals("Line 7", reader.getNextLine());
+        assertEquals("Line 6", reader.getNextLine());
+        assertEquals("Line 5", reader.getNextLine());
+        assertEquals("Line 4", reader.getNextLine());
+        assertEquals("Line 3", reader.getNextLine());
+        assertEquals("Line 2", reader.getNextLine());
+        assertEquals("Line 1", reader.getNextLine());
+        reader.close();
     }
     
     @Test
-    public void testStack() {
-        try {
-            final ReverseFileReader reader =
-                    new ReverseFileReader(new File(getClass().getClassLoader().
-                    getResource("com/dmdirc/addons/logging/test1.txt").
-                    toURI()));
-            final Stack<String> lines = reader.getLines(10);
-            
-            assertEquals(7, lines.size());
-            assertEquals("Line 1", lines.pop());
-            assertEquals("Line 2", lines.pop());
-            assertEquals("Line 3", lines.pop());
-            assertEquals("Line 4", lines.pop());
-            assertEquals("Line 5", lines.pop());
-            assertEquals("Line 6", lines.pop());
-            assertEquals("Line 7", lines.pop());
-            reader.close();
-        } catch (URISyntaxException ex) {
-            assertFalse(true);
-        } catch (IOException ex) {
-            assertFalse(true);
-        }
+    public void testCarriageReturn() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test2.txt").
+                toURI()));
+        reader.getNextLine();
+        assertEquals("Normal line", reader.getNextLine());
+        reader.close();        
     }
     
     @Test
-    public void testSmallStack() {
-        try {
-            final ReverseFileReader reader =
-                    new ReverseFileReader(new File(getClass().getClassLoader().
-                    getResource("com/dmdirc/addons/logging/test1.txt").
-                    toURI()));
-            final Stack<String> lines = reader.getLines(3);
-            
-            assertEquals(3, lines.size());
-            assertEquals("Line 5", lines.pop());
-            assertEquals("Line 6", lines.pop());
-            assertEquals("Line 7", lines.pop());
-            reader.close();
-        } catch (URISyntaxException ex) {
-            assertFalse(true);
-        } catch (IOException ex) {
-            assertFalse(true);
-        }
+    public void testLongLine() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test2.txt").
+                toURI()));
+        assertEquals("This is a line that is longer than 50 characters, so " +
+                "should cause the reader to have to scan back multiple times.",
+                reader.getNextLine());
+        reader.close();        
+    }    
+    
+    @Test
+    public void testStack() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));
+        final Stack<String> lines = reader.getLines(10);
+
+        assertEquals(7, lines.size());
+        assertEquals("Line 1", lines.pop());
+        assertEquals("Line 2", lines.pop());
+        assertEquals("Line 3", lines.pop());
+        assertEquals("Line 4", lines.pop());
+        assertEquals("Line 5", lines.pop());
+        assertEquals("Line 6", lines.pop());
+        assertEquals("Line 7", lines.pop());
+        reader.close();
     }
     
     @Test
-    public void testReset() {
-        try {
-            final ReverseFileReader reader =
-                    new ReverseFileReader(new File(getClass().getClassLoader().
-                    getResource("com/dmdirc/addons/logging/test1.txt").
-                    toURI()));
-            assertEquals("Line 7", reader.getNextLine());
-            assertEquals("Line 6", reader.getNextLine());
-            reader.reset();
-            assertEquals("Line 7", reader.getNextLine());
-            assertEquals("Line 6", reader.getNextLine());
-            assertEquals("Line 5", reader.getNextLine());
-            reader.close();
-        } catch (URISyntaxException ex) {
-            assertFalse(true);
-        } catch (IOException ex) {
-            assertFalse(true);
-        }
-    }    
+    public void testSmallStack() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));
+        final Stack<String> lines = reader.getLines(3);
+
+        assertEquals(3, lines.size());
+        assertEquals("Line 5", lines.pop());
+        assertEquals("Line 6", lines.pop());
+        assertEquals("Line 7", lines.pop());
+        reader.close();
+    }
+    
+    @Test
+    public void testReset() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));
+        assertEquals("Line 7", reader.getNextLine());
+        assertEquals("Line 6", reader.getNextLine());
+        reader.reset();
+        assertEquals("Line 7", reader.getNextLine());
+        assertEquals("Line 6", reader.getNextLine());
+        assertEquals("Line 5", reader.getNextLine());
+        reader.close();
+    }
+    
+    @Test(expected=IOException.class)
+    public void testIllegalClose() throws URISyntaxException, IOException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));        
+        reader.close();
+        reader.close();
+    }
+    
+    @Test(expected=IOException.class)
+    public void testIllegalReset() throws URISyntaxException, IOException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));        
+        reader.close();
+        reader.reset();
+    }
+    
+    @Test(expected=IOException.class)
+    public void testIllegalGetNextLine() throws URISyntaxException, IOException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));        
+        reader.close();
+        reader.getNextLine();
+    }
+    
+    @Test
+    public void testSeekLength() throws IOException, URISyntaxException {
+        final ReverseFileReader reader =
+                new ReverseFileReader(new File(getClass().getClassLoader().
+                getResource("com/dmdirc/addons/logging/test1.txt").
+                toURI()));
+        reader.setSeekLength((byte) 100);
+        assertEquals((byte) 100, reader.getSeekLength());
+    }
 
+    public static junit.framework.Test suite() {
+        return new junit.framework.JUnit4TestAdapter(ReverseFileReaderTest.class);
+    }
 }

test/com/dmdirc/addons/logging/test2.txt

@@ -0,0 +1,2 @@
+Normal line
+This is a line that is longer than 50 characters, so should cause the reader to have to scan back multiple times.

test/com/dmdirc/addons/urlcatcher/UrlCatcherPluginTest.java

@@ -0,0 +1,66 @@
+/*
+ * Copyright (c) 2006-2008 Chris Smith, Shane Mc Cormack, Gregory Holmes
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.addons.urlcatcher;
+
+import com.dmdirc.actions.CoreActionType;
+import com.dmdirc.config.IdentityManager;
+import org.junit.Test;
+import static org.junit.Assert.*;
+
+public class UrlCatcherPluginTest extends junit.framework.TestCase {
+
+    @Test
+    public void testURLCounting() {
+        IdentityManager.load();
+        
+        final UrlCatcherPlugin plugin = new UrlCatcherPlugin();
+
+        plugin.processEvent(CoreActionType.CHANNEL_MESSAGE, null,
+                null, "This is a message - http://www.google.com/ foo");
+        plugin.processEvent(CoreActionType.CHANNEL_MESSAGE, null,
+                null, "This is a message - http://www.google.com/ foo");
+        plugin.processEvent(CoreActionType.CHANNEL_MESSAGE, null,
+                null, "This is a message - http://www.google.com/ foo");
+        
+        assertEquals(1, plugin.getURLS().size());
+        assertEquals(3, (int) plugin.getURLS().get("http://www.google.com/"));
+    }
+    
+    @Test
+    public void testURLCatching() {
+        IdentityManager.load();
+        
+        final UrlCatcherPlugin plugin = new UrlCatcherPlugin();
+
+        plugin.processEvent(CoreActionType.CHANNEL_MESSAGE, null,
+                null, "http://www.google.com/ www.example.com foo://bar.baz");
+        plugin.processEvent(CoreActionType.CHANNEL_MESSAGE, null,
+                null, "No URLs here, no sir!");        
+        
+        assertEquals(3, plugin.getURLS().size());
+        assertTrue(plugin.getURLS().containsKey("http://www.google.com/"));
+        assertTrue(plugin.getURLS().containsKey("www.example.com"));
+        assertTrue(plugin.getURLS().containsKey("foo://bar.baz"));
+    }    
+
+}