Action unit test

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

src/com/dmdirc/actions/Action.java

@@ -70,7 +70,7 @@ public class Action extends ActionModel implements Serializable {
     private String location;
 
     /** The config file we're using. */
-    private ConfigFile config;
+    protected ConfigFile config;
 
     /**
      * Creates a new instance of Action. The group and name specified must
@@ -149,7 +149,7 @@ public class Action extends ActionModel implements Serializable {
     /**
      * Loads this action from the config instance.
      */
-    private void loadActionFromConfig() {
+    protected void loadActionFromConfig() {
         if (config.isFlatDomain(DOMAIN_TRIGGERS)) {
             if (!loadTriggers(config.getFlatDomain(DOMAIN_TRIGGERS))) {
                 return;

test/com/dmdirc/actions/ActionTest.java

@@ -23,9 +23,14 @@ package com.dmdirc.actions;
 
 import com.dmdirc.actions.interfaces.ActionType;
 
+import com.dmdirc.util.ConfigFile;
+import com.dmdirc.util.InvalidConfigFileException;
+import com.dmdirc.util.TextFile;
 import java.io.File;
+import java.io.IOException;
 import java.util.ArrayList;
 
+import java.util.Arrays;
 import org.junit.Test;
 import static org.junit.Assert.*;
 
@@ -53,17 +58,17 @@ public class ActionTest extends junit.framework.TestCase {
                 new File(ActionManager.getDirectory() + "unit-test-two"
                 + File.separator + "test1").isFile());
     }
-    
+
     @Test
     public void testSetName() {
         action.setName("test2");
-        
+
         assertFalse("setName must remove old file",
                 new File(ActionManager.getDirectory() + "unit-test-two"
                 + File.separator + "test1").isFile());
         assertTrue("setName must create new file",
                 new File(ActionManager.getDirectory() + "unit-test-two"
-                + File.separator + "test2").isFile());        
+                + File.separator + "test2").isFile());
     }
 
     @Test
@@ -75,4 +80,23 @@ public class ActionTest extends junit.framework.TestCase {
                 + File.separator + "test2").isFile());
     }
 
+    @Test
+    public void testRead() throws IOException, InvalidConfigFileException {
+        ActionManager.init();
+
+        final Action action = new Action("unit-test", "doesn't_exist");
+        action.config = new ConfigFile(new TextFile(getClass().getResourceAsStream("action1")));
+        action.config.read();
+        action.loadActionFromConfig();
+
+        assertTrue(Arrays.equals(action.getTriggers(),
+                new ActionType[]{CoreActionType.SERVER_AWAY}));
+        assertEquals("(0&1)", action.getConditionTree().toString());
+        assertTrue(Arrays.equals(action.getResponse(), new String[]{"/away"}));
+        assertEquals(new ActionCondition(1, CoreActionComponent.STRING_LENGTH,
+                CoreActionComparison.INT_EQUALS, "0"), action.getConditions().get(0));
+        assertEquals(new ActionCondition("foo", CoreActionComparison.STRING_CONTAINS,
+                "bar"), action.getConditions().get(1));
+    }
+
 }

test/com/dmdirc/actions/action1

@@ -0,0 +1,30 @@
+# This is a DMDirc configuration file.
+# Written on: Mon Jan 14 06:19:51 GMT 2008
+
+# This section indicates which sections below take key/value
+# pairs, rather than a simple list. It should be placed above
+# any sections that take key/values.
+keysections:
+  condition 0
+  condition 1
+
+triggers:
+  SERVER_AWAY
+
+response:
+  /away
+
+conditiontree:
+  (0&1)
+
+condition 0:
+  argument=1
+  component=STRING_LENGTH
+  target=0
+  comparison=INT_EQUALS
+
+condition 1:
+  argument=-1
+  starget=foo
+  target=bar
+  comparison=STRING_CONTAINS