Skip to content
Permalink
Browse files

Skip broken test_pantsd_memory_usage for python2. (#7765)

As discussed in #7761, although the issue is real, we'll be ditching
the python2 runtime shortly; so skip the test for python2.

Fixes #7761
  • Loading branch information...
jsirois committed May 19, 2019
1 parent 13f4baf commit 8d82e4be6a4b2ea20388117f96577971ffa8a005
Showing with 4 additions and 0 deletions.
  1. +4 −0 tests/python/pants_test/pantsd/test_pantsd_integration.py
@@ -9,6 +9,7 @@
import os
import re
import signal
import sys
import threading
import time
import unittest
@@ -400,6 +401,9 @@ def test_pantsd_pid_change(self):
# Remove the pidfile so that the teardown script doesn't try to kill process 9.
os.unlink(pidpath)

@unittest.skipIf(sys.version_info[0] == 2,
reason='Increases by ~0.52 under python 2 as described in '
'https://github.com/pantsbuild/pants/issues/7761.')
def test_pantsd_memory_usage(self):
"""Validates that after N runs, memory usage has increased by no more than X percent."""
number_of_runs = 10

0 comments on commit 8d82e4b

Please sign in to comment.
You can’t perform that action at this time.