impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Brown (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-6049: breakpad tests: skip all tests with local filesystem
Date Fri, 13 Oct 2017 16:29:29 GMT
Michael Brown has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8272


Change subject: IMPALA-6049: breakpad tests: skip all tests with local filesystem
......................................................................

IMPALA-6049: breakpad tests: skip all tests with local filesystem

The breakpad tests were recently refactored to support inclusion of one
of them as a core test. In this refactor, we neglected to ensure
setup_class() called its parent. This means the skipping called in said
parent doesn't occur, and the test is executed in an unsupported
environment (local filesystem).

This patch fixes that by ensuring we call the parent setup_class() via
super().

Testing:

$ TARGET_FILESYSTEM="local" impala-py.test tests/custom_cluster/test_breakpad.py \
      -k test_abort_writes_minidump

tests/custom_cluster/test_breakpad.py::TestBreakpadCore::test_abort_writes_minidump
SKIPPED

Change-Id: Ib4a3ff29dd85c79c4c3b3e3afb699861e408aa95
---
M tests/custom_cluster/test_breakpad.py
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/72/8272/1
-- 
To view, visit http://gerrit.cloudera.org:8080/8272
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib4a3ff29dd85c79c4c3b3e3afb699861e408aa95
Gerrit-Change-Number: 8272
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Brown <mikeb@cloudera.com>

Mime
  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message