Skip to content

Add test file cleanup and fix .gitignore for Python tests#787

Merged
jt2594838 merged 1 commit into
developfrom
ty/update
Apr 20, 2026
Merged

Add test file cleanup and fix .gitignore for Python tests#787
jt2594838 merged 1 commit into
developfrom
ty/update

Conversation

@JackieTien97
Copy link
Copy Markdown
Contributor

Summary

  • Wrap test_lower_case_name and test_tsfile_config in try/finally blocks to ensure generated .tsfile files are cleaned up after tests
  • Fix test_tree_all_datatype_query_to_dataframe_variants cleanup bug: was deleting tablet_write_and_read.tsfile instead of record_write_and_read.tsfile
  • Add python/tsfile/dataset/__pycache__ to .gitignore

Test plan

  • Verify tests still pass after adding try/finally cleanup
  • Verify no leftover .tsfile files remain after running tests
  • Code formatted with Black (no changes needed)

- Wrap test_lower_case_name and test_tsfile_config in try/finally
  blocks to ensure generated .tsfile files are cleaned up
- Fix test_tree_all_datatype_query_to_dataframe_variants to clean up
  the correct file (record_write_and_read.tsfile instead of
  tablet_write_and_read.tsfile)
- Add python/tsfile/dataset/__pycache__ to .gitignore

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.75%. Comparing base (2ab087b) to head (654b1ec).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #787   +/-   ##
========================================
  Coverage    62.75%   62.75%           
========================================
  Files          706      706           
  Lines        42836    42836           
  Branches      6324     6324           
========================================
+ Hits         26881    26882    +1     
  Misses       14963    14963           
+ Partials       992      991    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jt2594838 jt2594838 merged commit eaf9030 into develop Apr 20, 2026
25 checks passed
@jt2594838 jt2594838 deleted the ty/update branch April 20, 2026 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants