Skip to content

Commit 92afc17

Browse files
committed
Don't hardcode module names in tmoduletests.nim
1 parent d894b15 commit 92afc17

File tree

2 files changed

+10
-18
lines changed

2 files changed

+10
-18
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,6 @@ jobs:
3131
run: |
3232
cd tests
3333
nim c -r tester
34+
# there's no need to add nimblepkg unit tests --
35+
# they are run by tmoduletests.nim
3436
- run: ./src/nimble install -y

tests/tmoduletests.nim

Lines changed: 8 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -3,25 +3,15 @@
33

44
{.used.}
55

6-
import unittest, os, osproc
7-
from nimblepkg/common import cd
6+
import unittest, os, osproc, strutils
87

98
suite "Module tests":
10-
template moduleTest(moduleName: string) =
9+
template moduleTest(modulePath: string) =
10+
let moduleName = splitFile(modulePath).name
1111
test moduleName:
12-
cd "..":
13-
check execCmdEx("nim c -r src/nimblepkg/" & moduleName).
14-
exitCode == QuitSuccess
12+
check execCmdEx("nim c -r " & modulePath).
13+
exitCode == QuitSuccess
1514

16-
moduleTest "common"
17-
moduleTest "download"
18-
moduleTest "jsonhelpers"
19-
moduleTest "packageinfo"
20-
moduleTest "packageparser"
21-
moduleTest "paths"
22-
moduleTest "reversedeps"
23-
moduleTest "sha1hashes"
24-
moduleTest "tools"
25-
moduleTest "topologicalsort"
26-
moduleTest "vcstools"
27-
moduleTest "version"
15+
for module in walkDir("../src/nimblepkg"):
16+
if readFile(module.path).contains("unittest"):
17+
moduleTest module.path

0 commit comments

Comments
 (0)