From 16cb74694aa572aaa75cce3bacc781388778d8de Mon Sep 17 00:00:00 2001
From: joonatan <joonatan.a.nikkila@student.jyu.fi>
Date: Thu, 9 May 2024 13:09:45 +0300
Subject: [PATCH] =?UTF-8?q?Testej=C3=A4=20lis=C3=A4tty.=20Lis=C3=A4tty=20m?=
=?UTF-8?q?y=C3=B6s=20metodeita=20pelaajan=20j=C3=A4rjest=C3=A4miseen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
---
src/HT/fx/Karkelot.java | 4 ++
src/HT/fx/PadelMainGUIController.java | 24 +----------
src/HT/fx/Pelaajat.java | 24 +++++++++++
.../fx/test/PadelMainGUIControllerTest.java | 33 +++++++++++++++
src/HT/fx/test/PelaajatTest.java | 42 +++++++++++++++++++
5 files changed, 104 insertions(+), 23 deletions(-)
create mode 100644 src/HT/fx/test/PadelMainGUIControllerTest.java
create mode 100644 src/HT/fx/test/PelaajatTest.java
diff --git a/src/HT/fx/Karkelot.java b/src/HT/fx/Karkelot.java
index 5f2bea5..a86bb53 100644
--- a/src/HT/fx/Karkelot.java
+++ b/src/HT/fx/Karkelot.java
@@ -158,6 +158,10 @@ public class Karkelot {
pelaajat.poistaPelaaja(pel);
}
+ public int etsiNimella(String nimi) {
+ return pelaajat.etsiNimella(nimi);
+ }
+
/**
* Testiohjelma kerhosta
diff --git a/src/HT/fx/PadelMainGUIController.java b/src/HT/fx/PadelMainGUIController.java
index 77b21fc..9a40033 100644
--- a/src/HT/fx/PadelMainGUIController.java
+++ b/src/HT/fx/PadelMainGUIController.java
@@ -61,7 +61,7 @@ public class PadelMainGUIController {
TextField haluttu =(TextField) NimiVbox.getChildren().get(m);
String muokattavannimi = haluttu.getText();
- muokattava = etsiNimella(muokattavannimi);
+ muokattava = karkelo.etsiNimella(muokattavannimi);
muokkaaPelaaja(null, "kanat");
@@ -264,28 +264,6 @@ public void poistaKaikkiKentat() {
}
- /**
- * Etsii tietyn nimisen pelaajan id:
- * @param nimi ketä etsitään
- * @return löydetyn pelaajan id
- */
- public int etsiNimella(String nimi) {
- Pelaaja pelaajat[] = getKarkelo().getPelaajat();
- int j=0;
-
- while(j<getKarkelo().getPelaajia()) {
-
- if(pelaajat[j].getNimi().equals(nimi)){
-
- return pelaajat[j].getTunnusNro();
- }
-
- j++;
- }
-
- return 0;
- }
-
/**
* järjestää pelaaja taulukon pelaajien ratingin mukaan
diff --git a/src/HT/fx/Pelaajat.java b/src/HT/fx/Pelaajat.java
index 5afa87e..c881979 100644
--- a/src/HT/fx/Pelaajat.java
+++ b/src/HT/fx/Pelaajat.java
@@ -50,6 +50,8 @@ public class Pelaajat {
* pelaajat.anna(2) == aku1 === false;
* pelaajat.anna(2) == aku2 === true;
* pelaajat.poistaPelaaja(aku3); pelaajat.getLkm() === 2;
+ * aku1.setNimi("Teppo");
+ * pelaajat.etsiNimella("Teppo") === 1;
* </pre>
*/
public void lisaa(Pelaaja pelaaja) throws SailoException {
@@ -116,6 +118,28 @@ public class Pelaajat {
}
}
+ /**
+ * Etsii tietyn nimisen pelaajan id:
+ * @param nimi ketä etsitään
+ * @return löydetyn pelaajan id
+ */
+ public int etsiNimella(String nimi) {
+ Pelaaja pelaajat[] = alkiot;
+ int j=0;
+
+ while(j<lkm) {
+
+ if(pelaajat[j].getNimi().equals(nimi)){
+
+ return pelaajat[j].getTunnusNro();
+ }
+
+ j++;
+ }
+
+ return 0;
+ }
+
/**
* @return pelaajat listana
*/
diff --git a/src/HT/fx/test/PadelMainGUIControllerTest.java b/src/HT/fx/test/PadelMainGUIControllerTest.java
new file mode 100644
index 0000000..e74b2cd
--- /dev/null
+++ b/src/HT/fx/test/PadelMainGUIControllerTest.java
@@ -0,0 +1,33 @@
+package HT.fx.test;
+// Generated by ComTest BEGIN
+import static org.junit.Assert.*;
+import org.junit.*;
+import HT.fx.*;
+// Generated by ComTest END
+
+/**
+ * Test class made by ComTest
+ * @version 2024.05.09 13:01:00 // Generated by ComTest
+ *
+ */
+@SuppressWarnings({ "all" })
+public class PadelMainGUIControllerTest {
+
+
+
+ // Generated by ComTest BEGIN
+ /** testEtsiNimella272 */
+ @Test
+ public void testEtsiNimella272() { // PadelMainGUIController: 272
+ Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja(), aku4 = new Pelaaja();
+ aku1.rekisteroi();
+ aku2.rekisteroi();
+ aku3.rekisteroi();
+ aku4.rekisteroi();
+ aku1.setNimi("Teppo1");
+ aku2.setNimi("Teppo2");
+ aku3.setNimi("Teppo3");
+ aku4.setNimi("Teppo4");
+ assertEquals("From: PadelMainGUIController line: 282", 2, etsiNimella("Teppo2"));
+ } // Generated by ComTest END
+}
\ No newline at end of file
diff --git a/src/HT/fx/test/PelaajatTest.java b/src/HT/fx/test/PelaajatTest.java
new file mode 100644
index 0000000..a284e73
--- /dev/null
+++ b/src/HT/fx/test/PelaajatTest.java
@@ -0,0 +1,42 @@
+package HT.fx.test;
+// Generated by ComTest BEGIN
+import static org.junit.Assert.*;
+import org.junit.*;
+import HT.fx.*;
+// Generated by ComTest END
+
+/**
+ * Test class made by ComTest
+ * @version 2024.05.09 13:09:09 // Generated by ComTest
+ *
+ */
+@SuppressWarnings({ "all" })
+public class PelaajatTest {
+
+
+ // Generated by ComTest BEGIN
+ /**
+ * testLisaa36
+ * @throws SailoException when error
+ */
+ @Test
+ public void testLisaa36() throws SailoException { // Pelaajat: 36
+ Pelaajat pelaajat = new Pelaajat();
+ Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja();
+ aku1.rekisteroi();
+ aku2.rekisteroi();
+ aku3.rekisteroi();
+ assertEquals("From: Pelaajat line: 43", 0, pelaajat.getLkm());
+ pelaajat.lisaa(aku1); assertEquals("From: Pelaajat line: 44", 1, pelaajat.getLkm());
+ pelaajat.lisaa(aku2); assertEquals("From: Pelaajat line: 45", 2, pelaajat.getLkm());
+ pelaajat.lisaa(aku3); assertEquals("From: Pelaajat line: 46", 3, pelaajat.getLkm());
+ assertEquals("From: Pelaajat line: 47", aku1, pelaajat.anna(1));
+ assertEquals("From: Pelaajat line: 48", aku2, pelaajat.anna(2));
+ assertEquals("From: Pelaajat line: 49", aku3, pelaajat.anna(3));
+ assertEquals("From: Pelaajat line: 50", false, pelaajat.anna(2) == aku1);
+ assertEquals("From: Pelaajat line: 51", true, pelaajat.anna(2) == aku2);
+ pelaajat.poistaPelaaja(aku3); assertEquals("From: Pelaajat line: 52", 2, pelaajat.getLkm());
+ aku1.setNimi("Teppo");
+ assertEquals("From: Pelaajat line: 54", 1, pelaajat.etsiNimella("Teppo"));
+ } // Generated by ComTest END
+}
\ No newline at end of file
--
GitLab