diff --git a/.gitattributes b/.gitattributes index 0baef23afda..bf7d9b4e892 100644 --- a/.gitattributes +++ b/.gitattributes @@ -9765,7 +9765,6 @@ src/test/java/forge/GuiProgressBarWindowTest.java svneol=native#text/plain src/test/java/forge/GuiQuestFilterTest.java svneol=native#text/plain src/test/java/forge/GuiQuestOptionsTest.java svneol=native#text/plain src/test/java/forge/GuiWinLoseTest.java svneol=native#text/plain -src/test/java/forge/MoveTest.java svneol=native#text/plain src/test/java/forge/PanelTest.java svneol=native#text/plain src/test/java/forge/PhaseTest.java svneol=native#text/plain src/test/java/forge/ReadBoosterPackTest.java svneol=native#text/plain diff --git a/pom.xml b/pom.xml index 388331fa402..8488a137876 100644 --- a/pom.xml +++ b/pom.xml @@ -173,6 +173,7 @@ maven-surefire-plugin 2.9 + fast true diff --git a/src/site/apt/index.apt b/src/site/apt/index.apt index eea0fadecab..b9735ce47ca 100644 --- a/src/site/apt/index.apt +++ b/src/site/apt/index.apt @@ -2,7 +2,7 @@ Maven Project Site ----- ----- - 2011-06-30 + 2011-08-10 ----- Forge @@ -31,7 +31,7 @@ Forge defender, reach, and haste. Forge now also has a mana pool and supports creature enchantments. Forge runs on Windows, Linux, and Macs. You can download Forge from - {{{http://code.google.com/p/cardforge/downloads/list}here}}. + {{{http://cardforge.org/releases/releases/}here}}. <> @@ -47,7 +47,7 @@ Forge * {{{http://cardforge.org/}Main Website}} * {{{http://svn.slightlymagic.net/forge/trunk}Source Code Repository}} - * {{{http://svn.slightlymagic.net/websvn/listing.php?repname=forge}Web View of Source}} + * {{{http://svn.slightlymagic.net/websvn/listing.php?repname=forge}Web View of Source Code Repository}} * {{{http://mtgrares.blogspot.com/}Original Author's Blog}} diff --git a/src/test/java/forge/BoosterDraftAITest.java b/src/test/java/forge/BoosterDraftAITest.java index f46b3786ff5..35313546087 100644 --- a/src/test/java/forge/BoosterDraftAITest.java +++ b/src/test/java/forge/BoosterDraftAITest.java @@ -31,7 +31,7 @@ public class BoosterDraftAITest { }//for outer }//runTestPrint() - @Test + @Test(timeOut = 5000) public void runTest(BoosterDraftAI ai) { ReadDraftBoosterPack booster = new ReadDraftBoosterPack(); for (int outer = 0; outer < 1; outer++) { diff --git a/src/test/java/forge/BoosterDraft_1Test.java b/src/test/java/forge/BoosterDraft_1Test.java index 8e2d880d88a..8ebfe28f8cf 100644 --- a/src/test/java/forge/BoosterDraft_1Test.java +++ b/src/test/java/forge/BoosterDraft_1Test.java @@ -12,7 +12,7 @@ public class BoosterDraft_1Test { /** * */ - @Test(timeOut = 1000) + @Test(groups = {"UnitTest", "fast"}, timeOut = 1000) public void BoosterDraft_1Test1() throws Exception { BoosterDraft_1 draft = new BoosterDraft_1(); while (draft.hasNextChoice()) { diff --git a/src/test/java/forge/CardColorTest.java b/src/test/java/forge/CardColorTest.java index e501e4a99cd..e425a443655 100644 --- a/src/test/java/forge/CardColorTest.java +++ b/src/test/java/forge/CardColorTest.java @@ -13,7 +13,7 @@ public class CardColorTest { /** * */ - @Test(timeOut = 1000) + @Test(groups = {"UnitTest", "fast"}, timeOut = 1000) public void CardColorTest1() { ManaCost mc = new ManaCost("R W U"); EnumSet col = Color.ConvertManaCostToColor(mc); diff --git a/src/test/java/forge/CardReaderTest.java b/src/test/java/forge/CardReaderTest.java index f0b2fc700f6..0c83403356c 100644 --- a/src/test/java/forge/CardReaderTest.java +++ b/src/test/java/forge/CardReaderTest.java @@ -1,19 +1,18 @@ package forge; +import forge.properties.ForgeProps; +import forge.properties.NewConstants; +import net.slightlymagic.braids.util.ClumsyRunnable; +import org.testng.Assert; +import org.testng.annotations.Test; + import java.io.File; import java.io.IOException; import java.util.HashMap; import java.util.Map; -import forge.properties.ForgeProps; -import forge.properties.NewConstants; -import net.slightlymagic.braids.util.ClumsyRunnable; - import static net.slightlymagic.braids.util.testng.BraidsAssertFunctions.assertThrowsException; -import org.testng.Assert; -import org.testng.annotations.Test; - /** * Created by hand to test the CardReader class. */ @@ -27,7 +26,7 @@ public class CardReaderTest implements NewConstants { public static final int ESTIMATED_CARDS_IN_FOLDER = 9001; - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_nullMap() { final ClumsyRunnable withScissors = new ClumsyRunnable() { public void run() throws Exception { @@ -38,7 +37,7 @@ public class CardReaderTest implements NewConstants { assertThrowsException(NullPointerException.class, withScissors); } - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_nullCardsFolder() { final ClumsyRunnable withScissors = new ClumsyRunnable() { public void run() throws Exception { @@ -50,7 +49,7 @@ public class CardReaderTest implements NewConstants { assertThrowsException(NullPointerException.class, withScissors); } - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_nonexistentCardsFolder() { final ClumsyRunnable withScissors = new ClumsyRunnable() { public void run() throws Exception { @@ -64,7 +63,7 @@ public class CardReaderTest implements NewConstants { assertThrowsException(RuntimeException.class, withScissors); } - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_fileNotFolder() throws IOException { final File tmpFile = File.createTempFile("just-a-file", ".testng.tmp"); @@ -80,7 +79,7 @@ public class CardReaderTest implements NewConstants { assertThrowsException(RuntimeException.class, withScissors); } - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_findCard_zip() { final Map map = new HashMap(); final File cardsfolder = ForgeProps.getFile(CARDSFOLDER); @@ -96,7 +95,7 @@ public class CardReaderTest implements NewConstants { Assert.assertEquals(elvishWarrior.getName(), "Elvish Warrior", "name is correct"); } - @Test(timeOut = TEST_TIMEOUT) + @Test(groups = {"UnitTest", "fast"}, timeOut = TEST_TIMEOUT) public final void test_ReadCard_findCard_nonzip() { final Map map = new HashMap(); final File cardsfolder = ForgeProps.getFile(CARDSFOLDER); diff --git a/src/test/java/forge/GameActionTest.java b/src/test/java/forge/GameActionTest.java index 8d2496ffdff..88f4cf40c74 100644 --- a/src/test/java/forge/GameActionTest.java +++ b/src/test/java/forge/GameActionTest.java @@ -12,7 +12,7 @@ public class GameActionTest { /** *

main.

*/ - @Test(timeOut = 1000) + @Test(groups = {"UnitTest", "fast"}, timeOut = 5000) public void GameActionTest1() throws Exception { System.out.println("GameActionTest"); GameAction gameAction = new GameAction(); diff --git a/src/test/java/forge/GuiBoosterDraftTest.java b/src/test/java/forge/GuiBoosterDraftTest.java index 59e6d4d9074..ca690a40a12 100644 --- a/src/test/java/forge/GuiBoosterDraftTest.java +++ b/src/test/java/forge/GuiBoosterDraftTest.java @@ -16,7 +16,7 @@ public class GuiBoosterDraftTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void GuiBoosterDraftTest1() { Constant.Runtime.GameType[0] = Constant.GameType.Draft; Constant.Runtime.HumanDeck[0] = new Deck(Constant.GameType.Sealed); diff --git a/src/test/java/forge/GuiFilterTest.java b/src/test/java/forge/GuiFilterTest.java index cad480c45b9..304190c693c 100644 --- a/src/test/java/forge/GuiFilterTest.java +++ b/src/test/java/forge/GuiFilterTest.java @@ -3,8 +3,6 @@ package forge; import org.testng.Assert; import org.testng.annotations.Test; -import javax.swing.*; - /** * Created by IntelliJ IDEA. * User: dhudson @@ -15,11 +13,10 @@ public class GuiFilterTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void guiFilterTest1() { try { GuiFilter dialog = new GuiFilter(null, null); - dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); dialog.setVisible(true); Assert.assertNotNull(dialog); dialog.dispose(); diff --git a/src/test/java/forge/GuiProgressBarWindowTest.java b/src/test/java/forge/GuiProgressBarWindowTest.java index 83825ace4cf..9a43806bf79 100644 --- a/src/test/java/forge/GuiProgressBarWindowTest.java +++ b/src/test/java/forge/GuiProgressBarWindowTest.java @@ -15,7 +15,7 @@ public class GuiProgressBarWindowTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void GuiProgressBarWindowTest1() { try { Gui_ProgressBarWindow dialog = new Gui_ProgressBarWindow(); diff --git a/src/test/java/forge/GuiQuestFilterTest.java b/src/test/java/forge/GuiQuestFilterTest.java index 522b625b841..b33856bc8da 100644 --- a/src/test/java/forge/GuiQuestFilterTest.java +++ b/src/test/java/forge/GuiQuestFilterTest.java @@ -15,7 +15,7 @@ public class GuiQuestFilterTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void guiQuestFilterTest1() { try { GuiQuestFilter dialog = new GuiQuestFilter(null, null); diff --git a/src/test/java/forge/GuiQuestOptionsTest.java b/src/test/java/forge/GuiQuestOptionsTest.java index e971f092528..571d2533481 100644 --- a/src/test/java/forge/GuiQuestOptionsTest.java +++ b/src/test/java/forge/GuiQuestOptionsTest.java @@ -13,7 +13,7 @@ public class GuiQuestOptionsTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void GuiQuestOptionsTest1() { Gui_QuestOptions dialog = new Gui_QuestOptions(); dialog.setVisible(true); diff --git a/src/test/java/forge/GuiWinLoseTest.java b/src/test/java/forge/GuiWinLoseTest.java index e336fcec3ee..113bcfd4d69 100644 --- a/src/test/java/forge/GuiWinLoseTest.java +++ b/src/test/java/forge/GuiWinLoseTest.java @@ -1,7 +1,5 @@ package forge; -import forge.deck.Deck; - import org.testng.Assert; import org.testng.annotations.Test; @@ -9,13 +7,13 @@ import org.testng.annotations.Test; * Created by IntelliJ IDEA. * User: dhudson */ -@Test(groups = {"UnitTest"}) +@Test(groups = {"UnitTest", "fast"}) public class GuiWinLoseTest { /** * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void GuiWinLoseTest1() { Gui_WinLose dialog = new Gui_WinLose(); dialog.setVisible(true); diff --git a/src/test/java/forge/MoveTest.java b/src/test/java/forge/MoveTest.java deleted file mode 100644 index 69305343de7..00000000000 --- a/src/test/java/forge/MoveTest.java +++ /dev/null @@ -1,147 +0,0 @@ -package forge; - - -import org.testng.annotations.Test; - -import java.util.ArrayList; - - -/** - *

MoveTest class.

- * - * @author Forge - * @version $Id$ - */ -@Test(groups = {"UnitTest"}, timeOut = 1000) -public class MoveTest { - - class MoveConcrete extends Move { - - private final int myNumber; - - /** - * Constant classNumber=-1 - */ - private int classNumber = -1; - /** - * Constant array= - */ - private int[] array; - - /** - * Constant classIndex= - */ - private int classIndex; - private int myIndex = -1; - - /** - *

Constructor for MoveTest.

- * - * @param i_array an array of int. - */ - public MoveConcrete(int i_array[]) { - this(); - - classIndex = 0; - array = i_array; - } - - /** - *

Constructor for MoveTest.

- */ - public MoveConcrete() { - myNumber = classNumber; - classNumber++; - } - - /** - *

Getter for the field classNumber.

- * - * @return a int. - */ - public int getClassNumber() { - return classNumber; - } - - /** - *

Getter for the field myNumber.

- * - * @return a int. - */ - public int getMyNumber() { - return myNumber; - } - - /** - * {@inheritDoc} - */ - @Override - public Move[] generateMoves() { - ArrayList list = new ArrayList(); - - for (int i = 0; i < 2; i++) - list.add(new MoveTest()); - - Move m[] = new Move[list.size()]; - list.toArray(m); - return m; - } - - /** - * {@inheritDoc} - */ - @Override - public int getScore() { - if (myIndex == -1) { - myIndex = classIndex; - classIndex++; - } - return array[myIndex]; - }//getScore() - } - - //branching 2 - //fully test depths 1 and 2, one test of depth 3 - - /** - *

test.

- */ - @Test(timeOut = 1000) - public void MoveTest1() { - MoveConcrete t; - - t = new MoveConcrete(new int[]{4, 1, 6, 3, 2, 7, 6, 9}); - test("1", t.max(t, 3, true) == 7); - - t = new MoveConcrete(new int[]{1, 2}); - test("2", t.max(t, 1, true) == 2); - - t = new MoveConcrete(new int[]{2, 1}); - test("3", t.max(t, 1, true) == 2); - - - t = new MoveConcrete(new int[]{1, 2, 3, 4}); - test("4", t.max(t, 2, true) == 3); - - t = new MoveConcrete(new int[]{2, 1, 4, 3}); - test("5", t.max(t, 2, true) == 3); - - t = new MoveConcrete(new int[]{4, 3, 1, 2}); - test("6", t.max(t, 2, true) == 3); - - t = new MoveConcrete(new int[]{3, 4, 2, 1}); - test("7", t.max(t, 2, true) == 3); - } - - /** - *

test.

- * - * @param message a {@link java.lang.String} object. - * @param shouldBeTrue a boolean. - */ - public void test(String message, boolean shouldBeTrue) { - if (!shouldBeTrue) { - throw new RuntimeException(message); - } - } -} diff --git a/src/test/java/forge/PhaseTest.java b/src/test/java/forge/PhaseTest.java index 4695db27cd5..55329d50851 100644 --- a/src/test/java/forge/PhaseTest.java +++ b/src/test/java/forge/PhaseTest.java @@ -12,7 +12,7 @@ public class PhaseTest { * * */ - @Test(timeOut = 5000, enabled = false) + @Test(groups = {"UnitTest", "fast"}, timeOut = 5000, enabled = false) public void PhaseTest1() { Phase phase = new Phase(); for (int i = 0; i < phase.phaseOrder.length; i++) { diff --git a/src/test/java/forge/ReadQuestAssignmentTest.java b/src/test/java/forge/ReadQuestAssignmentTest.java index ed0d49d414d..9e426d40bbd 100644 --- a/src/test/java/forge/ReadQuestAssignmentTest.java +++ b/src/test/java/forge/ReadQuestAssignmentTest.java @@ -15,7 +15,7 @@ public class ReadQuestAssignmentTest implements NewConstants { * * */ - @Test(timeOut = 1000) + @Test(groups = {"UnitTest", "fast"}, timeOut = 1000) public void ReadQuestAssignmentTest1() { try { ReadQuest_Assignment read = new ReadQuest_Assignment(ForgeProps.getFile(QUEST.QUESTS), null); diff --git a/src/test/java/forge/TinyTest.java b/src/test/java/forge/TinyTest.java index f113bc31470..f128693fe0a 100644 --- a/src/test/java/forge/TinyTest.java +++ b/src/test/java/forge/TinyTest.java @@ -14,7 +14,7 @@ public class TinyTest { /** * Just a quick test to see if TestNG and Assert are working. */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void test_true() { Assert.assertTrue(true); } diff --git a/src/test/java/forge/card/cardFactory/CardFactoryTest.java b/src/test/java/forge/card/cardFactory/CardFactoryTest.java index a76005dc562..33b379a8793 100644 --- a/src/test/java/forge/card/cardFactory/CardFactoryTest.java +++ b/src/test/java/forge/card/cardFactory/CardFactoryTest.java @@ -21,7 +21,7 @@ import java.util.TreeSet; * @author Forge * @version $Id$ */ -@Test(groups = {"UnitTest"}) +@Test(groups = {"UnitTest"}, timeOut = 5000) public class CardFactoryTest implements NewConstants { private static CardFactoryInterface factory; @@ -35,7 +35,7 @@ public class CardFactoryTest implements NewConstants { * Just a quick test to see if Arc-Slogger is in the database, and if it * has the correct owner. */ - @Test(enabled = true) + @Test(enabled = true, timeOut = 5000) public final void test_getCard_1() { final Card card = factory.getCard("Arc-Slogger", null); Assert.assertNotNull(card, "card is not null"); @@ -47,7 +47,7 @@ public class CardFactoryTest implements NewConstants { * * This doesn't work with LazyCardFactory, so it is too slow to enable by default. */ - @Test(enabled = false) + @Test(enabled = false, timeOut = 5000) public final void test_getRandomCombinationWithoutRepetition_tooLarge() { BraidsAssertFunctions.assertThrowsException(IllegalArgumentException.class, new ClumsyRunnable() { @@ -70,7 +70,7 @@ public class CardFactoryTest implements NewConstants { * * This doesn't work with LazyCardFactory, so it is too slow to enable by default. */ - @Test(enabled = false) + @Test(enabled = false, timeOut = 5000) public final void test_getRandomCombinationWithoutRepetition_oneTenth() { int divisor = 10; final CardList actual = factory.getRandomCombinationWithoutRepetition(factory.size() / divisor); diff --git a/src/test/java/forge/card/mana/Mana_PartTest.java b/src/test/java/forge/card/mana/Mana_PartTest.java index fa2a41de061..b0a696e6a0e 100644 --- a/src/test/java/forge/card/mana/Mana_PartTest.java +++ b/src/test/java/forge/card/mana/Mana_PartTest.java @@ -15,7 +15,7 @@ public class Mana_PartTest { /** *

testPayManaCost.

*/ - @Test + @Test(groups = {"UnitTest", "fast"}, timeOut = 5000) public void testPayManaCost() { { //test constructor diff --git a/src/test/java/forge/gui/ListChooserTest.java b/src/test/java/forge/gui/ListChooserTest.java index b64f7cb856a..b2e69ae3bae 100644 --- a/src/test/java/forge/gui/ListChooserTest.java +++ b/src/test/java/forge/gui/ListChooserTest.java @@ -12,7 +12,7 @@ public class ListChooserTest { * * */ - @Test(timeOut = 1000) + @Test(groups = {"UnitTest", "fast"}, timeOut = 1000) public void ListChooserTest1() { ListChooser c = new ListChooser("test", "choose a or b", 0, 2, "a", "b"); System.out.println(c.show()); diff --git a/src/test/java/forge/gui/game/CardDetailPanelTest.java b/src/test/java/forge/gui/game/CardDetailPanelTest.java index ba5bc495430..f93f1658a6b 100644 --- a/src/test/java/forge/gui/game/CardDetailPanelTest.java +++ b/src/test/java/forge/gui/game/CardDetailPanelTest.java @@ -3,8 +3,6 @@ package forge.gui.game; import org.testng.Assert; import org.testng.annotations.Test; -import javax.swing.*; - /** * Created by IntelliJ IDEA. * User: dhudson @@ -15,7 +13,7 @@ public class CardDetailPanelTest { * * */ - @Test() + @Test(groups = {"UnitTest", "fast"}) public void cardDetailPanelTest1() { try { CardDetailPanel dialog = new CardDetailPanel(null);