Skip to content

Commit

Permalink
up
Browse files Browse the repository at this point in the history
  • Loading branch information
slicklash committed Oct 8, 2024
1 parent e5c8caf commit 22665e8
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions tests/test_java.py
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,6 @@ def test_java_safemode_version_check(
monkeypatch.setitem(JavaProfiler.MINIMAL_SUPPORTED_VERSIONS, 8, (Version("8.999"), 0))

with make_java_profiler(profiler_state) as profiler:
profiler._profiler_state.get_container_name(application_pid)
process = profiler._select_processes_to_profile()[0]
jvm_version_str = cast_away_optional(get_java_version(process, profiler._profiler_state.stop_event))
jvm_version = parse_jvm_version(jvm_version_str)
Expand All @@ -246,7 +245,6 @@ def test_java_safemode_build_number_check(
profiler_state: ProfilerState,
) -> None:
with make_java_profiler(profiler_state) as profiler:
profiler._profiler_state.get_container_name(application_pid)
process = profiler._select_processes_to_profile()[0]
jvm_version_str = cast_away_optional(get_java_version(process, profiler._profiler_state.stop_event))
jvm_version = parse_jvm_version(jvm_version_str)
Expand Down Expand Up @@ -419,7 +417,6 @@ def test_sanity_other_jvms(
frequency=99,
java_async_profiler_mode="cpu",
) as profiler:
profiler._profiler_state.get_container_name(application_pid)
process = psutil.Process(application_pid)
assert search_for in cast_away_optional(get_java_version(process, profiler._profiler_state.stop_event))
process_collapsed = snapshot_pid_collapsed(profiler, application_pid)
Expand All @@ -441,7 +438,6 @@ def test_sanity_latest_jvms(
"""

with make_java_profiler(profiler_state) as profiler:
profiler._profiler_state.get_container_name(application_pid)
# sanity check that this is the correct JVM we're targeting
assert search_for in cast_away_optional(
get_java_version(psutil.Process(application_pid), profiler._profiler_state.stop_event)
Expand Down

0 comments on commit 22665e8

Please sign in to comment.