games-board/rmahjong: Fix tests ResourceWarnings, remove unittest2 DEPEND

The inittest2 dependence is unneccessary as python3 internal unittest is used.

Update patches to fix passing float into int("") in other way.
Update test.py patch to fix numerous 'ResourceWarning's.

Signed-off-by: Sergey Torokhov <torokhov-s-a@yandex.ru>
This commit is contained in:
Sergey Torokhov
2020-10-27 07:25:27 +03:00
parent 0d278df7e4
commit 3a392ae491
3 changed files with 34 additions and 35 deletions

View File

@@ -1,35 +1,3 @@
diff --git a/client/client.py b/client/client.py
--- a/client/client.py
+++ b/client/client.py
@@ -138,10 +138,10 @@ class Mahjong:
def init_player_boxes(self, names, player_winds, score):
self.player_boxes = [
- PlayerBox((50, 700), names[0], player_winds[0], int(score[0]), direction_up, (0,-80)),
- PlayerBox((954, 50), names[1], player_winds[1], int(score[1]), direction_left, (-210, 0)),
- PlayerBox((700, 0), names[2], player_winds[2], int(score[2]), direction_up, (0,80)),
- PlayerBox((0, 50), names[3], player_winds[3], int(score[3]), direction_right, (80,0)) ]
+ PlayerBox((50, 700), names[0], player_winds[0], int(float(score[0])), direction_up, (0,-80)),
+ PlayerBox((954, 50), names[1], player_winds[1], int(float(score[1])), direction_left, (-210, 0)),
+ PlayerBox((700, 0), names[2], player_winds[2], int(float(score[2])), direction_up, (0,80)),
+ PlayerBox((0, 50), names[3], player_winds[3], int(float(score[3])), direction_right, (80,0)) ]
for widget in self.player_boxes:
self.gui.add_widget(widget)
diff --git a/client/states.py b/client/states.py
--- a/client/states.py
+++ b/client/states.py
@@ -555,8 +555,8 @@ class ScoreState(RoundPreparingState):
results = []
for wind in winds:
name = (self.mahjong.get_player_name(wind))
- score = (int(self.message[wind + "_score"]))
- payment = (int(self.message[wind + "_payment"]))
+ score = (int(float(self.message[wind + "_score"])))
+ payment = (int(float(self.message[wind + "_payment"])))
results.append((name, score, payment))
results.sort(key = lambda r: r[1], reverse = True)
return results
diff --git a/client/tilepainter.py b/client/tilepainter.py
--- a/client/tilepainter.py
+++ b/client/tilepainter.py
@@ -67,6 +35,24 @@ index 042ee54..dc7e463 100644
def is_hand_open(sets):
for set in sets:
@@ -97,14 +97,14 @@
if wintype == "Ron":
if player_wind.name == "WE":
- return (name, round_to_base(score / 2 * 3, 100))
+ return (name, round_to_base(score // 2 * 3, 100))
else:
return (name, score)
else:
if player_wind.name == "WE":
- return (name, (round_to_base(score / 2, 100), 0))
+ return (name, (round_to_base(score // 2, 100), 0))
else:
- return (name, (round_to_base(score / 4, 100), round_to_base(score / 2, 100)))
+ return (name, (round_to_base(score // 4, 100), round_to_base(score // 2, 100)))
def quick_pons_and_kans(hand):
d = {}
@@ -274,7 +275,7 @@ def eval_sets(pair, sets, round_wind, player_wind, last_tile, wintype):
# Other hands
for name, fn in score_functions:

View File

@@ -1,3 +1,17 @@
diff a/server/botengine.py b/server/botengine.py
--- a/server/botengine.py
+++ b/server/botengine.py
@@ -53,7 +53,10 @@
def shutdown(self):
self.thread.thread_quit = True
+ self.process.stdin.close()
+ self.process.stdout.close()
self.process.terminate()
+ self.process.wait()
#self._write("QUIT\n")
#self.join()
diff --git a/server/test.py b/server/test.py
--- a/server/test.py
+++ b/server/test.py

View File

@@ -38,7 +38,6 @@ RDEPEND="
dev-python/pyopengl[${PYTHON_MULTI_USEDEP}]
')
"
DEPEND="test? ( dev-python/unittest2 )"
PATCHES=(
"${FILESDIR}/${PN}-0.4_fix_python3_compat.patch"
@@ -62,7 +61,7 @@ src_compile() {
}
src_test() {
cd "${S}/server/" && python3 test.py
cd "${S}/server/" && python3 test.py -v
}
src_install() {
@@ -79,5 +78,5 @@ src_install() {
dobin "rmahjong"
doicon -s 48 "${DISTDIR}/kmahjongg_${PN}.png"
make_desktop_entry "${PN}" "RMahjong" "kmahjongg_${PN}.png" "Game;BoardGame;" || die "Failed making desktop entry!"
make_desktop_entry "${PN}" "RMahjong" "kmahjongg_${PN}" "Game;BoardGame;" || die "Failed making desktop entry!"
}