[DRILL-7412] Minor unit test improvements

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.17.0
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Paul Rogers
• Reviewer: Arina Ielchiieva
• Created: 20/Oct/19 06:48
• Updated: 21/Oct/19 13:07
• Resolved: 21/Oct/19 13:07

Description

Many tests intentionally trigger errors. A debug-only log setting sent those errors to stdout. The resulting stack dumps simply cluttered the test output, so disabled error output to the console.

Drill can apply bounds checks to vectors. Tests run via Maven enable bounds checking. Now, bounds checking is also enabled in "debug mode" (when assertions are enabled, as in an IDE.)

Drill contains two test frameworks. The older BaseTestQuery was marked as deprecated, but many tests still use it and are unlikely to be changed soon. So, removed the deprecated marker to reduce the number of spurious warnings.

Issue Links

• links to: GitHub Pull Request #1876