elastic / elastic/beats

tests: remove redundant Python tests that are already covered by existing Go tests

Open
#49,669 9 comments 0 reactions 0 assignees View on GitHub
refactoring Team:Elastic-Agent-Data-Plane
Dominant language
Go
Stars
12.7k
Forks
5k
Avg merge
2d 15m
Merged PRs (30d)
385

Description

Multiple python tests from filebeat/tests/system have Go equivalents in the tree, for example filebeat/tests/system/test_multiline.py and TestParsersMultilineMaxLines, TestParsersMultilineTimeout and friends are defined in filebeat/input/filestream/parsers_integration_test.go.

We need to investigate and confirm which python tests already exist in Go, write the missing ones using Go, and finally delete the python test file.

These python tests started flaking more recently, so we need to take measures to confirm the equivalent Go tests are stable with -count.

Contributor guide

Open the contributing guide

Research direction

Compare the Python tests in filebeat/tests/system/test_multiline.py with the Go coverage in filebeat/input/filestream/parsers_integration_test.go, including TestParsersMultilineMaxLines and TestParsersMultilineTimeout. Run the equivalent Go tests with -count to check stability, identify missing coverage, and confirm the Python file can be removed once all relevant cases are covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, python
Domain
testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.