bramp / bramp/ffmpeg-cli-wrapper
Integration tests across multiple FFmpeg versions
- 主要言語
- Java
- スター
- 1.9k
- フォーク
- 424
- 平均マージ
- 5時間 35分
- マージ済み PR(30日)
- 2
説明
## Goal
Test the library against multiple FFmpeg versions to ensure compatibility and catch version-specific regressions.
## Current State
- CI runs on **JDK 11, 17, 21** but only one FFmpeg version (`release` via `setup-ffmpeg`).
- Unit tests mock FFmpeg — they don't exercise real FFmpeg behavior.
- No integration tests that verify actual transcoding, probing, or option handling against a real FFmpeg binary.
## Plan
### 1. Add FFmpeg version matrix to GitHub Actions
- Extend the CI matrix to include multiple FFmpeg versions (e.g. `5.1`, `6.1`, `7.1`, `release`).
- Use `FedericoCarboni/setup-ffmpeg` which supports version pinning, or download specific builds from ffmpeg.org.
- Matrix: `{java: latest} x {ffmpeg: [5.1, 6.1, 7.1, release]}`.
### 2. Create integration test suite
- New test class(es) under `src/test/java/.../integration/`, gated behind a Maven profile (e.g. `-Pintegration`) so they don't run in normal unit test builds.
- Tests should cover:
- **Probing**: `FFprobe.probe()` against sample files — verify parsed codecs, streams, format, duration.
- **Transcoding**: Basic transcode (e.g. mp4 to mkv, audio extraction) — verify output file is valid.
- **Builder options**: Exercise every builder option (`setVideoCodec`, `setAudioCodec`, `setFormat`, `setVideoFilter`, `setComplexFilter`, `setFrames`, HLS output, etc.) and verify the resulting FFmpeg command succeeds.
- **Progress listener**: Run a real transcode with `ProgressListener` and verify progress events are received.
- **Error handling**: Feed invalid input and verify exceptions are thrown with useful messages.
- **Edge cases**: Unicode filenames, paths with spaces, large filter strings.
### 3. Sample media files
- Include small (< 1MB) sample media files in `src/test/resources/` for integration tests.
- Cover common formats: mp4 (h264+aac), mkv, webm, mp3, wav, etc.
### 4. CI workflow update
- Add a separate job or workflow (`integration-test.yml`) that runs the integration profile.
- Only run on `push` to main and PRs (not on every commit to avoid long CI times).
This is related to #31 (documenting supported FFmpeg versions) — once we have the test matrix, we can confidently document which versions are supported.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
まず、既存の GitHub Actions CI 設定と Maven のテスト設定を確認します。src/test/java/.../integration/ に統合プロファイルとテストクラスを追加し、src/test/resources/ にサンプルメディアを配置したうえで、計画で説明されている分離された integration-test.yml ワークフローを追加します。完了条件は、バージョンマトリクスで実際の統合テストスイートが実行され、指定された FFmpeg バージョン全体で一覧にある動作が検証されることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- ci-cd, testing
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100