Merge pull request #707 from Zerotask/list-command-progress-info

feat(list): added progress info

src/exercise.rs

@@ -1,6 +1,6 @@
-use std::env;
 use regex::Regex;
 use serde::Deserialize;
+use std::env;
 use std::fmt::{self, Display, Formatter};
 use std::fs::{self, remove_file, File};
 use std::io::Read;
@@ -132,8 +132,7 @@ path = "{}.rs""#,
                 } else {
                     "Failed to write 📎 Clippy 📎 Cargo.toml file."
                 };
-                fs::write(CLIPPY_CARGO_TOML_PATH, cargo_toml)
+                fs::write(CLIPPY_CARGO_TOML_PATH, cargo_toml).expect(cargo_toml_error_msg);
-                    .expect(cargo_toml_error_msg);
                 // To support the ability to run the clipy exercises, build
                 // an executable, in addition to running clippy. With a
                 // compilation failure, this would silently fail. But we expect

src/main.rs

@@ -138,13 +138,19 @@ fn main() {
             println!("{:<17}\t{:<46}\t{:<7}", "Name", "Path", "Status");
         }
         let filters = list_m.value_of("filter").unwrap_or_default().to_lowercase();
+        let mut exercises_done: u16 = 0;
         exercises.iter().for_each(|e| {
             let fname = format!("{}", e.path.display());
             let filter_cond = filters
                 .split(',')
                 .filter(|f| !f.trim().is_empty())
                 .any(|f| e.name.contains(&f) || fname.contains(&f));
-            let status = if e.looks_done() { "Done" } else { "Pending" };
+            let status = if e.looks_done() {
+                exercises_done = exercises_done + 1;
+                "Done"
+            } else {
+                "Pending"
+            };
             let solve_cond = {
                 (e.looks_done() && list_m.is_present("solved"))
                     || (!e.looks_done() && list_m.is_present("unsolved"))
@@ -173,6 +179,13 @@ fn main() {
                 }
             }
         });
+        let percentage_progress = exercises_done as f32 / exercises.len() as f32 * 100.0;
+        println!(
+            "Progress: You completed {} / {} exercises ({:.2} %).",
+            exercises_done,
+            exercises.len(),
+            percentage_progress
+        );
         std::process::exit(0);
     }
 
@@ -314,7 +327,7 @@ fn watch(exercises: &[Exercise], verbose: bool) -> notify::Result<()> {
                             .chain(
                                 exercises
                                     .iter()
-                                    .filter(|e| !e.looks_done() && !filepath.ends_with(&e.path))
+                                    .filter(|e| !e.looks_done() && !filepath.ends_with(&e.path)),
                             );
                         clear_screen();
                         match verify(pending_exercises, verbose) {

src/ui.rs

@@ -1,14 +1,10 @@
 macro_rules! warn {
     ($fmt:literal, $ex:expr) => {{
-        use std::env;
         use console::{style, Emoji};
+        use std::env;
         let formatstr = format!($fmt, $ex);
         if env::var("NO_EMOJI").is_ok() {
-            println!(
+            println!("{} {}", style("!").red(), style(formatstr).red());
-                "{} {}",
-                style("!").red(),
-                style(formatstr).red()
-            );
         } else {
             println!(
                 "{} {}",
@@ -21,15 +17,11 @@ macro_rules! warn {
 
 macro_rules! success {
     ($fmt:literal, $ex:expr) => {{
-        use std::env;
         use console::{style, Emoji};
+        use std::env;
         let formatstr = format!($fmt, $ex);
         if env::var("NO_EMOJI").is_ok() {
-            println!(
+            println!("{} {}", style("✓").green(), style(formatstr).green());
-                "{} {}",
-                style("✓").green(),
-                style(formatstr).green()
-            );
         } else {
             println!(
                 "{} {}",

src/verify.rs

@@ -1,7 +1,7 @@
-use std::env;
 use crate::exercise::{CompiledExercise, Exercise, Mode, State};
 use console::style;
 use indicatif::ProgressBar;
+use std::env;
 
 // Verify that the provided container of Exercise objects
 // can be compiled and run without any failures.