direct_play_nice 0.2.0-beta.1

CLI program that converts video files to direct-play-compatible formats.
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125

#![cfg(feature = "ffmpeg-cli-tests")]

mod common;

use assert_cmd::prelude::*;
use common::{ensure_ffmpeg_present, gen_problem_input};
use predicates::str;
use std::error::Error;
use std::fs;
use std::path::{Path, PathBuf};
use std::process::Command;
use std::sync::{Mutex, OnceLock};
use tempfile::TempDir;

fn cli_lock() -> std::sync::MutexGuard<'static, ()> {
    static LOCK: OnceLock<Mutex<()>> = OnceLock::new();
    LOCK.get_or_init(|| Mutex::new(()))
        .lock()
        .unwrap_or_else(|poisoned| poisoned.into_inner())
}

fn make_input(tmp: &TempDir) -> Result<PathBuf, Box<dyn Error>> {
    let (source, _) = gen_problem_input(tmp);
    Ok(source)
}

fn skip_subtitles_and_use_reasonable_audio(cmd: &mut Command) {
    cmd.arg("--sub-mode")
        .arg("skip")
        .arg("--audio-quality")
        .arg("192k");
}

fn run_cli(input: &Path, output: &Path, extra_args: &[&str]) -> Result<(), Box<dyn Error>> {
    let _guard = cli_lock();
    let mut cmd = Command::new(assert_cmd::cargo::cargo_bin!("direct_play_nice"));
    cmd.arg("-s").arg("chromecast_1st_gen");
    skip_subtitles_and_use_reasonable_audio(&mut cmd);
    cmd.arg(input).arg(output);
    for arg in extra_args {
        cmd.arg(arg);
    }
    cmd.assert().success().stdout(str::is_empty());
    Ok(())
}

#[test]
fn delete_source_defaults_to_false() -> Result<(), Box<dyn Error>> {
    ensure_ffmpeg_present();
    let tmp = TempDir::new()?;
    let input = make_input(&tmp)?;
    let output = tmp.path().join("out.mp4");

    run_cli(&input, &output, &[])?;

    assert!(input.exists(), "input file should remain by default");
    Ok(())
}

#[test]
fn delete_source_config_true_overridden_by_cli_false() -> Result<(), Box<dyn Error>> {
    ensure_ffmpeg_present();
    let tmp = TempDir::new()?;
    let config_path = tmp.path().join("config.toml");
    fs::write(&config_path, "delete_source = true\n")?;

    let input = make_input(&tmp)?;
    let output = tmp.path().join("out.mp4");

    let _guard = cli_lock();
    let mut cmd = Command::new(assert_cmd::cargo::cargo_bin!("direct_play_nice"));
    cmd.arg("--config-file")
        .arg(&config_path)
        .arg("-s")
        .arg("chromecast_1st_gen");
    skip_subtitles_and_use_reasonable_audio(&mut cmd);
    cmd.arg(&input).arg(&output).arg("--delete-source=false");
    cmd.assert().success().stdout(str::is_empty());

    assert!(
        input.exists(),
        "CLI should override config delete_source=true"
    );
    Ok(())
}

#[test]
fn delete_source_flag_deletes_input() -> Result<(), Box<dyn Error>> {
    ensure_ffmpeg_present();
    let tmp = TempDir::new()?;
    let input = make_input(&tmp)?;
    let output = tmp.path().join("out.mp4");

    run_cli(&input, &output, &["--delete-source"])?;

    assert!(!input.exists(), "--delete-source should remove input file");
    Ok(())
}

#[test]
fn delete_source_config_true_respected_without_cli_override() -> Result<(), Box<dyn Error>> {
    ensure_ffmpeg_present();
    let tmp = TempDir::new()?;
    let config_path = tmp.path().join("config.toml");
    fs::write(&config_path, "delete_source = true\n")?;

    let input = make_input(&tmp)?;
    let output = tmp.path().join("out.mp4");

    let _guard = cli_lock();
    let mut cmd = Command::new(assert_cmd::cargo::cargo_bin!("direct_play_nice"));
    cmd.arg("--config-file")
        .arg(&config_path)
        .arg("-s")
        .arg("chromecast_1st_gen");
    skip_subtitles_and_use_reasonable_audio(&mut cmd);
    cmd.arg(&input).arg(&output);
    cmd.assert().success().stdout(str::is_empty());

    assert!(
        !input.exists(),
        "config delete_source=true should remove input when not overridden"
    );
    Ok(())
}