extra print in test summary

[jangorecki]

Output I am getting after first commit in this PR.

All 8259 tests (last 1913.24) in tests/tests.Rraw completed ok in 57.3s elapsed (00:01:12 cpu)

and after fixing the problem, in second commit.

All 9993 tests (last 2161.2) in inst/tests/tests.Rraw completed ok in 00:01:32 elapsed (00:01:49 cpu)

[codecov]

Codecov Report

Merging #4850 (1084041) into master (87b8c15) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4850   +/-   ##
=======================================
  Coverage   99.47%   99.47%           
=======================================
  Files          73       73           
  Lines       14592    14592           
=======================================
  Hits        14515    14515           
  Misses         77       77           

Impacted Files	Coverage Δ
R/test.data.table.R	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87b8c15...1084041. Read the comment docs.

[mattdowle]

Output I am getting after first commit in this PR.

The first commit, which appears to be 1084041, appears to just change the type of quotes within a comment. Because that change is in a comment, I don't see why that would change anything.

and after fixing the problem

what problem exactly? I'm looking at the change to a comment and wondering why that changes anything.

[mattdowle]

I don't see any reason why not to merge this PR, though. In this case i don't need to understand the reason. Including env$prevtest in the last line is fine, whatever the reason.
Will merge after version bump.

[jangorecki]

@mattdowle first commit was cb6cae9. Second commit was 1084041 that changed quotes in comment making test script to not quit early during parsing.