Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Snap CommandRunner Migration (Issue #20 Part 2)

Đang mở
  • Có gì không ổn?
#28 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
go
Lĩnh vực
operating-systems, tooling

Hướng nghiên cứu

Bắt đầu bằng cách so sánh triển khai Snap với các lần migration CommandRunner của APT đã hoàn tất và của YUM hiện có, sau đó đọc interface CommandRunner và MockCommandRunner. Kiểm tra các test Snap hiện tại và các đường dẫn lệnh trước khi thực hiện thay đổi. Công việc được xem là hoàn tất khi đáp ứng các tiêu chí chấp nhận, bao gồm dependency injection, thực thi tương tác, xử lý LC_ALL=C, duy trì hành vi hiện có và độ bao phủ kiểm thử đáng kể.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Summary

Migrate Snap package manager from direct exec.Command calls to unified CommandRunner interface for consistent, testable command execution architecture.

This is Part 2 of Issue #20 - CommandRunner Pattern Implementation

Background

Following successful APT CommandRunner migration (Issue #27), implement the same architectural patterns for Snap package manager to achieve full consistency across the codebase.

Current State

  • Snap: Uses direct exec.Command calls
  • APT: ✅ Uses CommandRunner (Issue #27 completed)
  • YUM: ✅ Uses CommandRunner (already implemented)
  • Flatpak: Uses direct exec.Command calls (Issue #20-3)

Scope

  • Snap package manager migration from exec.Command to CommandRunner
  • Constructor standardization following APT/YUM patterns
  • Dependency injection for comprehensive testing capabilities
  • Test coverage expansion (currently 0% coverage for Snap)

Technical Implementation

CommandRunner Integration
  • Replace all exec.Command calls with CommandRunner interface
  • Implement automatic LC_ALL=C handling for consistent English output
  • Add built-in interactive mode support via RunInteractive()
  • Enable dependency injection for testing
Constructor Pattern

Following APT/YUM standardization:

  • NewPackageManager() - Production use with default CommandRunner
  • NewPackageManagerWithCustomRunner() - Testing use with mock CommandRunner
Testing Infrastructure
  • Implement comprehensive mock-based testing
  • Add environment variable tracking capabilities
  • Create cross-platform test scenarios
  • Achieve significant test coverage improvement (currently 0%)

Benefits

Architecture Benefits
  • ✅ Consistent interface - Same CommandRunner pattern across all package managers
  • ✅ Automatic LC_ALL=C - No manual environment setup needed
  • ✅ Built-in interactive support - Proper stdin/stdout/stderr handling
  • ✅ Unified testing approach - Same mocking patterns across codebase
Testing Benefits
  • ✅ Simple mocking - Map-based command mocking vs complex setup
  • ✅ Environment testing - Built-in environment variable verification
  • ✅ Cross-platform testing - Same tests work across all environments
  • ✅ Comprehensive coverage - All command execution paths testable

Acceptance Criteria

  • Snap migrated from direct exec.Command to CommandRunner
  • Constructor naming follows APT/YUM patterns
  • Dependency injection implemented for testing
  • Comprehensive test coverage added (target: >80%)
  • LC_ALL=C automatically handled
  • Interactive mode works consistently
  • All existing functionality preserved
  • Documentation updated

Priority

Medium Priority - Important architectural consistency improvement, but lower priority than core APT/YUM functionality.

Dependencies

  • Requires Issue #27 (APT CommandRunner Migration) completion
  • Should follow same patterns established in APT migration

Related Issues

  • Part of Issue #20 - CommandRunner Pattern Implementation
  • Preceded by Issue #27 (APT CommandRunner Migration)
  • Followed by Issue #20-3 (Flatpak CommandRunner Migration)

Implementation Notes

  • Follow proven patterns from APT migration (Issue #27)
  • Leverage existing CommandRunner interface and MockCommandRunner
  • Maintain backward compatibility during migration
  • Use established testing frameworks and patterns
Ngôn ngữ chính
Go
Star
17
Fork
8
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của bluet/syspkg

Tất cả issue của bluet/syspkg

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.