summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorsnowleo <snowleo@e251c2fe-e539-e718-e476-b85c1f46cddb>2011-06-06 12:06:41 +0000
committersnowleo <snowleo@e251c2fe-e539-e718-e476-b85c1f46cddb>2011-06-06 12:06:41 +0000
commitb7791caa75893b1dac11e0e71f69a90fccd0f5ad (patch)
tree909c6f49ff52bb358b5900c5142f6557169f426d
parent6c862e72b5a0800770d45b33a52b7f613ef6a259 (diff)
downloadEssentials-b7791caa75893b1dac11e0e71f69a90fccd0f5ad.tar
Essentials-b7791caa75893b1dac11e0e71f69a90fccd0f5ad.tar.gz
Essentials-b7791caa75893b1dac11e0e71f69a90fccd0f5ad.tar.lz
Essentials-b7791caa75893b1dac11e0e71f69a90fccd0f5ad.tar.xz
Essentials-b7791caa75893b1dac11e0e71f69a90fccd0f5ad.zip
cleanup
git-svn-id: https://svn.java.net/svn/essentials~svn/trunk@1593 e251c2fe-e539-e718-e476-b85c1f46cddb
-rw-r--r--Essentials/test/com/earth2me/essentials/EconomyTest.java95
1 files changed, 46 insertions, 49 deletions
diff --git a/Essentials/test/com/earth2me/essentials/EconomyTest.java b/Essentials/test/com/earth2me/essentials/EconomyTest.java
index 74f5f11d2..9677c9dd5 100644
--- a/Essentials/test/com/earth2me/essentials/EconomyTest.java
+++ b/Essentials/test/com/earth2me/essentials/EconomyTest.java
@@ -4,22 +4,22 @@ import com.earth2me.essentials.api.Economy;
import com.earth2me.essentials.api.NoLoanPermittedException;
import com.earth2me.essentials.api.UserDoesNotExistException;
import java.io.IOException;
-import java.util.logging.Level;
-import java.util.logging.Logger;
import junit.framework.TestCase;
import org.bukkit.plugin.InvalidDescriptionException;
+import org.junit.Test;
public class EconomyTest extends TestCase
{
- private final OfflinePlayer base1;
- private final Essentials ess;
+ private final transient Essentials ess;
+ private final static String NPCNAME = "npc1";
+ private final static String PLAYERNAME = "TestPlayer1";
- public EconomyTest(String testName)
+ public EconomyTest(final String testName)
{
super(testName);
ess = new Essentials();
- FakeServer server = new FakeServer();
+ final FakeServer server = new FakeServer();
try
{
ess.setupForTesting(server);
@@ -32,46 +32,45 @@ public class EconomyTest extends TestCase
{
fail("IOException");
}
- base1 = new OfflinePlayer("TestPlayer1");
- server.addPlayer(base1);
+ server.addPlayer(new OfflinePlayer(PLAYERNAME));
}
-
+
// only one big test, since we use static instances
+ @Test
public void testEconomy()
{
// test NPC
- String npcName = "npc1";
- assertFalse(Economy.playerExists(npcName));
- assertTrue(Economy.createNPC(npcName));
- assertTrue(Economy.playerExists(npcName));
- assertNotNull(ess.getOfflineUser(npcName));
+ assertFalse("NPC does not exists", Economy.playerExists(NPCNAME));
+ assertTrue("Create NPC", Economy.createNPC(NPCNAME));
+ assertTrue("NPC exists", Economy.playerExists(NPCNAME));
+ assertNotNull("NPC can be accessed", ess.getOfflineUser(NPCNAME));
try
{
- Economy.removeNPC(npcName);
+ Economy.removeNPC(NPCNAME);
}
catch (UserDoesNotExistException ex)
{
fail(ex.getMessage());
}
- assertFalse(Economy.playerExists(npcName));
-
+ assertFalse("NPC can be removed", Economy.playerExists(NPCNAME));
+
//test Math
try
{
- String playerName = "TestPlayer1";
- assertTrue(Economy.playerExists(playerName));
- Economy.resetBalance(playerName);
- assertEquals(0.0, Economy.getMoney(playerName));
- Economy.add(playerName, 10.0);
- assertEquals(10.0, Economy.getMoney(playerName));
- Economy.subtract(playerName, 5.0);
- assertEquals(5.0, Economy.getMoney(playerName));
- Economy.multiply(playerName, 2.0);
- assertEquals(10.0, Economy.getMoney(playerName));
- Economy.divide(playerName, 2.0);
- assertEquals(5.0, Economy.getMoney(playerName));
- Economy.setMoney(playerName, 10.0);
- assertEquals(10.0, Economy.getMoney(playerName));
+
+ assertTrue("Player exists", Economy.playerExists(PLAYERNAME));
+ Economy.resetBalance(PLAYERNAME);
+ assertEquals("Player has no money", 0.0, Economy.getMoney(PLAYERNAME));
+ Economy.add(PLAYERNAME, 10.0);
+ assertEquals("Add money", 10.0, Economy.getMoney(PLAYERNAME));
+ Economy.subtract(PLAYERNAME, 5.0);
+ assertEquals("Subtract money", 5.0, Economy.getMoney(PLAYERNAME));
+ Economy.multiply(PLAYERNAME, 2.0);
+ assertEquals("Multiply money", 10.0, Economy.getMoney(PLAYERNAME));
+ Economy.divide(PLAYERNAME, 2.0);
+ assertEquals("Divide money", 5.0, Economy.getMoney(PLAYERNAME));
+ Economy.setMoney(PLAYERNAME, 10.0);
+ assertEquals("Set money", 10.0, Economy.getMoney(PLAYERNAME));
}
catch (NoLoanPermittedException ex)
{
@@ -81,24 +80,23 @@ public class EconomyTest extends TestCase
{
fail(ex.getMessage());
}
-
+
//test Format
- assertEquals("$1000", Economy.format(1000.0));
- assertEquals("$10", Economy.format(10.0));
- assertEquals("$10.10", Economy.format(10.10));
- assertEquals("$10.10", Economy.format(10.102));
- assertEquals("$10.11", Economy.format(10.109));
-
-
+ assertEquals("Format $1000", "$1000", Economy.format(1000.0));
+ assertEquals("Format $10", "$10", Economy.format(10.0));
+ assertEquals("Format $10.10", "$10.10", Economy.format(10.10));
+ assertEquals("Format $10.10", "$10.10", Economy.format(10.102));
+ assertEquals("Format $10.11", "$10.11", Economy.format(10.109));
+
+
//test Exceptions
try
{
- String playerName = "TestPlayer1";
- assertTrue(Economy.playerExists(playerName));
- Economy.resetBalance(playerName);
- assertEquals(0.0, Economy.getMoney(playerName));
- Economy.subtract(playerName, 5.0);
- fail();
+ assertTrue("Player exists", Economy.playerExists(PLAYERNAME));
+ Economy.resetBalance(PLAYERNAME);
+ assertEquals("Reset balance", 0.0, Economy.getMoney(PLAYERNAME));
+ Economy.subtract(PLAYERNAME, 5.0);
+ fail("Did not throw exception");
}
catch (NoLoanPermittedException ex)
{
@@ -107,12 +105,11 @@ public class EconomyTest extends TestCase
{
fail(ex.getMessage());
}
-
+
try
{
- String playerName = "UnknownPlayer";
- Economy.resetBalance(playerName);
- fail();
+ Economy.resetBalance("UnknownPlayer");
+ fail("Did not throw exception");
}
catch (NoLoanPermittedException ex)
{