Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port more integration tests to V1 chroot and some to V2 #8365

Merged
merged 10 commits into from Oct 1, 2019
11 changes: 0 additions & 11 deletions build-support/ci_lists/integration_chroot_blacklist.txt
@@ -1,24 +1,13 @@
tests/python/pants_test/backend/jvm/subsystems:jar_dependency_management_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/java:cache_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/java:java_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/rsc:rsc_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/zinc:zinc_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/zinc:zinc_compile_integration_with_zjars
tests/python/pants_test/backend/jvm/tasks:checkstyle_integration
tests/python/pants_test/backend/jvm/tasks:ivy_resolve_integration
tests/python/pants_test/backend/jvm/tasks:jar_publish_integration
tests/python/pants_test/backend/jvm/tasks:junit_run_integration
tests/python/pants_test/backend/jvm/tasks:junit_tests_concurrency_integration
tests/python/pants_test/backend/jvm/tasks:jvm_run_integration
tests/python/pants_test/backend/python/tasks/native:integration
tests/python/pants_test/backend/python/tasks:build_local_python_distributions_integration
tests/python/pants_test/backend/python/tasks:pytest_run_integration
tests/python/pants_test/backend/python/tasks:python_binary_integration
tests/python/pants_test/backend/python/tasks:setup_py_integration
tests/python/pants_test/backend/python:integration
tests/python/pants_test/invalidation:strict_deps_invalidation_integration
tests/python/pants_test/logging:test_native_engine_logging
tests/python/pants_test/pantsd:pantsd_integration
tests/python/pants_test/projects:testprojects_integration
tests/python/pants_test/targets:jvm_app_integration
tests/python/pants_test/tasks:bootstrap_jvm_tools_integration
10 changes: 10 additions & 0 deletions build-support/ci_lists/integration_v2_blacklist.txt
@@ -1,2 +1,12 @@
tests/python/pants_test/backend/jvm/tasks/jvm_compile/java:java_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/java:zinc_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/rsc:rsc_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/zinc:zinc_compile_integration
tests/python/pants_test/backend/jvm/tasks/jvm_compile/zinc:zinc_compile_integration_with_zjars
tests/python/pants_test/backend/jvm/tasks:junit_run_integration
tests/python/pants_test/backend/jvm/tasks:junit_tests_concurrency_integration
tests/python/pants_test/backend/jvm/tasks:junit_tests_integration
tests/python/pants_test/backend/jvm/tasks:jvm_run_integration
tests/python/pants_test/base:exception_sink_integration
tests/python/pants_test/reporting:reporting_integration
tests/python/pants_test/tasks:bootstrap_jvm_tools_integration
10 changes: 10 additions & 0 deletions testprojects/src/python/BUILD
Expand Up @@ -9,6 +9,11 @@ files(
],
)

files(
name = 'bad_requirements_directory',
sources = rglobs('bad_requirements/*'),
)

files(
name = 'build_file_imports_function_directory',
sources = rglobs('build_file_imports_function/*'),
Expand All @@ -29,6 +34,11 @@ files(
sources = rglobs('interpreter_selection/*'),
)

files(
name = 'nested_runs_directory',
sources = rglobs('nested_runs/*'),
)

files(
name = 'no_build_file_directory',
sources = rglobs('no_build_file/*'),
Expand Down
2 changes: 1 addition & 1 deletion tests/python/pants_test/backend/jvm/tasks/BUILD
Expand Up @@ -319,7 +319,7 @@ python_tests(
'tests/python/pants_test:int-test',
'testprojects/src/java/org/pantsbuild/testproject:unicode_directory',
],
timeout = 120,
timeout = 240,
tags = {'integration'},
)

Expand Down
1 change: 1 addition & 0 deletions tests/python/pants_test/backend/python/tasks/native/BUILD
Expand Up @@ -44,6 +44,7 @@ python_tests(
'tests/python/pants_test:int-test',
'tests/python/pants_test/backend/python/tasks/util',
'tests/python/pants_test/engine:scheduler_test_base',
'testprojects/src/python:python_distribution_directory',
],
tags={'platform_specific_behavior', 'integration'},
timeout=2400,
Expand Down
Expand Up @@ -5,9 +5,8 @@
import os
import re
import subprocess
import sys
from functools import wraps
from unittest import skip, skipIf
from unittest import skip
from zipfile import ZipFile

from pants.backend.native.config.environment import Platform
Expand All @@ -32,10 +31,6 @@ def wrapper(*args, **kwargs):
return wrapper


_IS_OSX = Platform.current == Platform.darwin
_PYTHON_MAJOR_MINOR = sys.version_info[:2]


class CTypesIntegrationTest(PantsRunIntegrationTest):

@classmethod
Expand All @@ -59,8 +54,6 @@ def use_pantsd_env_var(cls):
'testprojects/src/python/python_distribution/ctypes_with_extra_compiler_flags:bin'
)

@skipIf(_IS_OSX and _PYTHON_MAJOR_MINOR == (2, 7),
reason='Broken as described in https://github.com/pantsbuild/pants/issues/7763.')
@_toolchain_variants
def test_ctypes_binary_creation(self, toolchain_variant):
"""Create a python_binary() with all native toolchain variants, and test the result."""
Expand Down
4 changes: 2 additions & 2 deletions tests/python/pants_test/logging/test_native_engine_logging.py
Expand Up @@ -19,12 +19,12 @@ def use_pantsd_env_var(cls):
def test_native_logging(self):
expected_msg = "\[DEBUG\] engine::scheduler: Launching \d+ root"
pants_run = self.run_pants([
"-linfo", "list", "src/scala::"
"-linfo", "list", "3rdparty::"
])
self.assertNotRegex(pants_run.stderr_data, expected_msg)

pants_run = self.run_pants([
"-ldebug", "list", "src/scala::"
"-ldebug", "list", "3rdparty::"
])
self.assertRegex(pants_run.stderr_data, expected_msg)

Expand Down
5 changes: 5 additions & 0 deletions tests/python/pants_test/pantsd/BUILD
Expand Up @@ -81,6 +81,11 @@ python_tests(
'examples/src/java/org/pantsbuild/example:hello_directory',
'examples/src/python/example:hello_directory',
'examples/src/scala/org/pantsbuild/example:hello_directory',
'testprojects/src/java/org/pantsbuild/testproject:bundle_directory',
'testprojects/src/python:bad_requirements_directory',
'testprojects/src/python:coordinated_runs_directory',
'testprojects/src/python:nested_runs_directory',
'testprojects/src/python:print_env_directory',
],
tags = {'integration'},
timeout = 900
Expand Down
4 changes: 3 additions & 1 deletion tests/python/pants_test/pantsd/test_pantsd_integration.py
Expand Up @@ -400,7 +400,9 @@ def test_pantsd_memory_usage(self):
number_of_runs = 10
max_memory_increase_fraction = 0.40 # TODO https://github.com/pantsbuild/pants/issues/7647
with self.pantsd_successful_run_context() as (pantsd_run, checker, workdir, config):
cmd = ['filter', 'testprojects::']
# NB: This doesn't actually run against all testprojects, only those that are in the chroot,
# i.e. explicitly declared in this test file's BUILD.
cmd = ['list', 'testprojects::']
self.assert_success(pantsd_run(cmd))
initial_memory_usage = checker.current_memory_usage()
for _ in range(number_of_runs):
Expand Down
1 change: 1 addition & 0 deletions tests/python/pants_test/targets/BUILD
Expand Up @@ -43,6 +43,7 @@ python_tests(
sources = ['test_jvm_app_integration.py'],
dependencies = [
'tests/python/pants_test:int-test',
'testprojects/src/java/org/pantsbuild/testproject:bundle_directory',
],
tags = {'integration'},
timeout = 180,
Expand Down