Skip to content

Commit

Permalink
dev-python/tlsh: actually wire up testsuite
Browse files Browse the repository at this point in the history
Upstream has a testsuite, it's just in a different directory compared to
the python module itself. It internally uses the script we used to
crudely run, but it also actually checks the results against fixtures.

As a result, it checks more than just "smoketest: does the module import
and execute its functions without literally crashing".

It also doesn't redirect the testsuite stdout/stderr to devnull, since
it now actually does something constructive and we'd like to see the
status from it.

Signed-off-by: Eli Schwartz <eschwartz93@gmail.com>
Signed-off-by: Sam James <sam@gentoo.org>
  • Loading branch information
eli-schwartz authored and thesamesam committed Jun 17, 2024
1 parent d8bdd74 commit f218f5a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion dev-python/tlsh/tlsh-4.8.2-r1.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -26,5 +26,5 @@ PATCHES=(
)

python_test() {
"${EPYTHON}" test.py test.py test.py &> /dev/null || die
../Testing/python_test.sh "${EPYTHON}" || die
}
4 changes: 2 additions & 2 deletions dev-python/tlsh/tlsh-4.8.2.ebuild
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright 2022-2023 Gentoo Authors
# Copyright 2022-2024 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2

EAPI=8
Expand Down Expand Up @@ -30,5 +30,5 @@ src_prepare() {
}

python_test() {
"${EPYTHON}" test.py test.py test.py &> /dev/null || die
../Testing/python_test.sh "${EPYTHON}" || die
}

0 comments on commit f218f5a

Please sign in to comment.