Skip to content

Commit 05b45ac

Browse files
Merge pull request #141 from smithlabcode/removing-redundant-tests
Removing redundant tests
2 parents 0bf3a77 + e57c295 commit 05b45ac

File tree

3 files changed

+9
-25
lines changed

3 files changed

+9
-25
lines changed

Makefile.am

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ EXTRA_DIST = \
6767
data/two_epialleles.states \
6868
data/araTha1_simulated.counts.gz \
6969
test_scripts/test_abismalidx.test \
70-
test_scripts/test_simreads.test \
7170
test_scripts/test_abismal.test \
7271
test_scripts/test_format.test \
7372
test_scripts/test_uniq.test \
@@ -96,8 +95,15 @@ AM_CPPFLAGS = -I $(top_srcdir)/src/common -I $(top_srcdir)/src/smithlab_cpp -I $
9695

9796
CXXFLAGS = -Wall -O3 $(OPENMP_CXXFLAGS) # default has optimization on
9897

99-
TESTS = test_scripts/test_simreads.test \
100-
test_scripts/test_abismalidx.test \
98+
### ADS: Testing with `make check` can be a long process. Testing with
99+
### `make check` is designed to work recursively. Modifying this
100+
### behavior requires overriding build-in rules which generates
101+
### warnings, probably for good reason. Defining a check-local only
102+
### adds to functionality and does not remove the dependency that
103+
### causes the recursion. The preferred alternative is to do `make`
104+
### first and then `make check-TESTS` after. This will not do tests
105+
### recursively.
106+
TESTS = test_scripts/test_abismalidx.test \
101107
test_scripts/test_abismal.test \
102108
test_scripts/test_format.test \
103109
test_scripts/test_uniq.test \
@@ -230,8 +236,6 @@ CLEANFILES = \
230236
tests/reads.mstats \
231237
tests/reads.sam \
232238
tests/reads.ustats \
233-
tests/simreads_1.fq \
234-
tests/simreads_2.fq \
235239
tests/tRex1.idx \
236240
tests/tRex1_promoters.roi.bed \
237241
tests/reads.counts.select \

data/md5sum.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@ af8b97ef25af8a16ced1537ca4a74d07 tests/reads.hmr
1414
916fe07a1d03449b65dfc1458d7c1c40 tests/reads.mstats
1515
65816bee34b977adf1dd798fbf85d758 tests/reads.sam
1616
1aecbb2b12bd867405118eae42de6743 tests/reads.ustats
17-
ae8e662c3ab23ba742a45a9784390e63 tests/simreads_1.fq
18-
d2723d2e99d2a3492af508633d61b30a tests/simreads_2.fq
1917
bcbf01be810cbf4051292813eb6b9225 tests/tRex1.idx
2018
7c84dafb85252a04a45704da71eba7c4 tests/tRex1_promoters.roi.bed
2119
e7e9590475a7f9b1ae41701d81892e57 tests/two_epialleles.amr

test_scripts/test_simreads.test

Lines changed: 0 additions & 18 deletions
This file was deleted.

0 commit comments

Comments
 (0)