Fix bat cache --clear not clearing the --target dir if specified

src/bin/bat/assets.rs

@@ -1,6 +1,8 @@
 use std::borrow::Cow;
 use std::fs;
 use std::io;
+use std::path::Path;
+use std::path::PathBuf;
 
 use clap::crate_version;
 
@@ -18,10 +20,15 @@ pub fn cache_dir() -> Cow<'static, str> {
     PROJECT_DIRS.cache_dir().to_string_lossy()
 }
 
-pub fn clear_assets() {
-    clear_asset("themes.bin", "theme set cache");
-    clear_asset("syntaxes.bin", "syntax set cache");
-    clear_asset("metadata.yaml", "metadata file");
+pub fn clear_assets(matches: &clap::ArgMatches) {
+    let target_dir = matches
+        .get_one::<String>("target")
+        .map(Path::new)
+        .unwrap_or_else(|| PROJECT_DIRS.cache_dir());
+
+    clear_asset(target_dir.join("themes.bin"), "theme set cache");
+    clear_asset(target_dir.join("syntaxes.bin"), "syntax set cache");
+    clear_asset(target_dir.join("metadata.yaml"), "metadata file");
 }
 
 pub fn assets_from_cache_or_binary(use_custom_assets: bool) -> Result<HighlightingAssets> {
@@ -50,9 +57,8 @@ pub fn assets_from_cache_or_binary(use_custom_assets: bool) -> Result<Highlighti
     Ok(custom_assets.unwrap_or_else(HighlightingAssets::from_binary))
 }
 
-fn clear_asset(filename: &str, description: &str) {
+fn clear_asset(path: PathBuf, description: &str) {
     print!("Clearing {} ... ", description);
-    let path = PROJECT_DIRS.cache_dir().join(filename);
     match fs::remove_file(&path) {
         Err(err) if err.kind() == io::ErrorKind::NotFound => {
             println!("skipped (not present)");

src/bin/bat/main.rs

@@ -67,7 +67,7 @@ fn run_cache_subcommand(matches: &clap::ArgMatches) -> Result<()> {
         #[cfg(not(feature = "build-assets"))]
         println!("bat has been built without the 'build-assets' feature. The 'cache --build' option is not available.");
     } else if matches.get_flag("clear") {
-        clear_assets();
+        clear_assets(matches);
     }
 
     Ok(())

tests/integration_tests.rs

@@ -892,11 +892,8 @@ fn config_read_arguments_from_file() {
         .stdout(predicate::eq("dummy-pager-from-config\n").normalize());
 }
 
-// Ignore this test for now as `bat cache --clear` only targets the default cache dir.
-// `bat cache --clear` must clear the `--target` dir for this test to pass.
 #[cfg(unix)]
 #[test]
-#[ignore]
 fn cache_clear() {
     let src_dir = "cache_source";
     let tmp_dir = tempdir().expect("can create temporary directory");