Skip to content

Migrate genio module from avocado to autils#102

Merged
richtja merged 1 commit into
avocado-framework:mainfrom
harvey0100:genio
Mar 18, 2026
Merged

Migrate genio module from avocado to autils#102
richtja merged 1 commit into
avocado-framework:mainfrom
harvey0100:genio

Conversation

@harvey0100
Copy link
Copy Markdown
Contributor

Add genio module to autils/file/ category with comprehensive unit and functional tests. The module provides generic IO operations for file reading, writing, appending, and pattern matching.

Files added:

  • autils/file/genio.py - Main module (line-for-line from avocado)
  • metadata/file/genio.yml - Module metadata
  • tests/unit/modules/file/genio.py - 21 unit tests
  • tests/functional/modules/file/genio.py - 13 functional tests
  • docs/source/utils.rst - Documentation entry

Note: Requires crypto module to be merged first (dependency).

Assisted-By: Cursor-Claude-Opus-4.5

@harvey0100 harvey0100 self-assigned this Feb 2, 2026
@mr-avocado mr-avocado Bot moved this to Review Requested in Default project Feb 2, 2026
@harvey0100 harvey0100 force-pushed the genio branch 3 times, most recently from c88bfef to 540d0c4 Compare February 2, 2026 14:07
@harvey0100
Copy link
Copy Markdown
Contributor Author

harvey0100 commented Feb 2, 2026

Waiting on Crypto dependency to be merged.

@harvey0100 harvey0100 marked this pull request as draft February 3, 2026 15:05
@harvey0100 harvey0100 marked this pull request as ready for review March 16, 2026 13:26
@harvey0100 harvey0100 force-pushed the genio branch 2 times, most recently from 7d0c138 to cab9674 Compare March 16, 2026 13:45
Add genio module to autils/file/ category with comprehensive unit and
functional tests. The module provides generic IO operations for file
reading, writing, appending, and pattern matching.

Files added:
- autils/file/genio.py - Main module (line-for-line from avocado)
- metadata/file/genio.yml - Module metadata
- tests/unit/modules/file/genio.py - 21 unit tests
- tests/functional/modules/file/genio.py - 13 functional tests
- docs/source/utils.rst - Documentation entry

Note: Requires crypto module to be merged first (dependency).

Assisted-By: Cursor-Claude-Opus-4.5
Signed-off-by: Harvey Lynden <hlynden@redhat.com>
Made-with: Cursor
Copy link
Copy Markdown
Contributor

@richtja richtja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @harvey0100, thanks for the migration it LGTM.

@richtja richtja merged commit 645c9d1 into avocado-framework:main Mar 18, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Review Requested to Done 114 in Default project Mar 18, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (6488869) to head (81428c5).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff     @@
##   main   #102   +/-   ##
===========================
===========================

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done 114

Development

Successfully merging this pull request may close these issues.

2 participants