testdata/expected/results-offline.csv is quite big
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 5
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
testdata/expected/results-offline.csv was already kind of big and you've 4x it, into 12k lines (1.3MB). Any way to break it into smaller files? If not in this MR, at least a plan on how we could achieve that
I have considered that.
The main problem here is, in the current implementation the file diff between expected and result is used as a strategy to analyze differences.
Breaking into per test files might help reducing the big file problem, but I am not sure this is a good solution here.
Do you have any suggestions?
Originally posted by @alanpeixinho in https://github.com/kernelci/regzbot/issues/33#issuecomment-4970279258
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with testdata/expected/results-offline.csv and trace how expected and result files are compared and how their diffs are used to analyze differences. Determine whether the data can be split without losing that comparison behavior; done means the expected data is smaller or partitioned and the existing difference analysis remains usable, with a documented plan if it cannot be changed directly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100