[IMPALA-4750] Future proof use of pytest in Impala environment

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Impala 2.8.0
• Fix Version/s: Impala 2.9.0
• Component/s: Infrastructure
• Labels: test-infra
• Assignee: David Knupp
• Reporter: David Knupp
• Created: 10/Jan/17 03:34
• Updated: 16/Jun/17 19:01
• Resolved: 03/Feb/17 00:56

Description

We currently do a couple of things that will soon be deprecated in upcoming versions of pytest:

1. In run-tests.py, we pass args to pytest.main() as a string, rather than a list. The preferred way to pass args is to use a list.
2. Even though we --ignore certain directories that might contain classes or functions that confuse pytest's auto-discovery, we also import those classes (like TestDimension, and TestMatrix) into other modules, and later versions of pytest will see these imported classes as possible test classes. The fix is just to rename the classes by prepending them with Impala.

Issue Links

• is duplicated by: IMPALA-4824 Rename common infra classes that look like test classes to pytest