improve unit test and code coverage

Signed-off-by: Navid Yaghoobi <navidys@fedoraproject.org>

.github/workflows/codecov.yml

@@ -15,32 +15,34 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - uses: actions-rs/toolchain@v1
       with:
         toolchain: nightly
         override: true
-    - name: Build
-      run: cargo build --verbose
+    - name: Extract test data
+      run: make extract_test_data
     - name: Run tests
       run: cargo test --verbose
       env:
         CARGO_INCREMENTAL: '0'
         RUSTFLAGS: '-Zprofile -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'
         RUSTDOCFLAGS: '-Zprofile -Ccodegen-units=1 -Cinline-threshold=0 -Clink-dead-code -Coverflow-checks=off -Cpanic=abort -Zpanic_abort_tests'
     - name: rust-grcov
+      id: coverage
       # You may pin to the exact commit or the version.
       # uses: actions-rs/grcov@bb47b1ed7883a1502fa6875d562727ace2511248
       uses: actions-rs/grcov@v0.1
     - name: Codecov
       # You may pin to the exact commit or the version.
       # uses: codecov/codecov-action@81cd2dc8148241f03f5839d295e000b8f761e378
-      uses: codecov/codecov-action@v4
+      uses: codecov/codecov-action@v4.0.1
       env:
         CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
       with:
         # Repository upload token - get it from codecov.io. Required only for private repositories
         # token: # optional
         # Specify whether the Codecov output should be verbose
         verbose: true
-        fail_ci_if_error: true
+        fail_ci_if_error: true
+        file: ${{ steps.coverage.outputs.report }}

Makefile

@@ -18,7 +18,7 @@ validate: ## Validate code
 
 .PHONY: test
 test: extract_test_data ## Run unit tests
-	sudo $(CARGO) test
+	$(CARGO) test
 
 .PHONY: create_test_data
 create_test_data: ## create test data fixtures archive
@@ -59,7 +59,7 @@ crate-publish: ## Publish crate
 
 .PHONY: clean
 clean: ## Cleanup
-	sudo rm -rf target
+	rm -rf target
 	rm -rf ./test_data/fixtures
 
 #=================================================

README.md

@@ -1,4 +1,4 @@
-# procsys [![][img_crates]][crates] [![][img_doc]][doc]
+# procsys [![][img_crates]][crates] [![][img_doc]][doc] [![][img_codecov]][codecov]
 
 Rust library to retrieve system, kernel, and process metrics from the pseudo-filesystems /proc and /sys.
 
@@ -38,6 +38,8 @@ Licensed under the [MIT License](https://github.com/navidys/procsys/blob/main/LI
 
 [img_crates]: https://img.shields.io/crates/v/procsys.svg
 [img_doc]: https://img.shields.io/badge/rust-documentation-blue.svg
+[img_codecov]: https://codecov.io/gh/navidys/procsys/branch/main/graph/badge.svg
 
 [crates]: https://crates.io/crates/procsys
 [doc]: https://docs.rs/procsys/
+[codecov]: https://codecov.io/gh/navidys/procsys

src/sysfs/class_cooling.rs

@@ -113,15 +113,21 @@ mod tests {
     fn cooling_devices() {
         let cooling_class_path = Path::new("test_data/fixtures/sys/class/thermal/");
         let cdev = collect_from(cooling_class_path).expect("collecting cooling information");
-        assert!(cdev.len().eq(&2));
-        assert!(cdev[0].name.eq("cooling_device0"));
-        assert!(cdev[0].cur_state.eq(&0));
-        assert!(cdev[0].max_state.eq(&50));
-        assert!(cdev[0].cooling_type.eq("Processor"));
-
-        assert!(cdev[1].name.eq("cooling_device1"));
-        assert!(cdev[1].cur_state.eq(&-1));
-        assert!(cdev[1].max_state.eq(&27));
-        assert!(cdev[1].cooling_type.eq("intel_powerclamp"));
+
+        for cooling in cdev {
+            match cooling.name.as_str() {
+                "cooling_device0" => {
+                    assert!(cooling.cur_state.eq(&0));
+                    assert!(cooling.max_state.eq(&50));
+                    assert!(cooling.cooling_type.eq("Processor"));
+                }
+                "cooling_device1" => {
+                    assert!(cooling.cur_state.eq(&-1));
+                    assert!(cooling.max_state.eq(&27));
+                    assert!(cooling.cooling_type.eq("intel_powerclamp"));
+                }
+                _ => panic!("invalid cooling device: {}", cooling.name),
+            }
+        }
     }
 }

src/sysfs/class_dmi.rs

@@ -91,7 +91,7 @@ impl DMI {
 
 /// attempts to collect dmi information
 /// # Example
-/// ```
+/// ```no_run
 /// use procsys::sysfs::class_dmi;
 ///
 /// let dmi_info = class_dmi::collect().expect("dmi information");

src/sysfs/class_thermal.rs

@@ -146,18 +146,24 @@ mod tests {
         let tdev =
             collect_from(thermal_zone_class_path).expect("collecting system thermal information");
         assert!(tdev.len().eq(&2));
-        assert!(tdev[0].name.eq("thermal_zone0"));
-        assert!(tdev[0].zone_type.eq("bcm2835_thermal"));
-        assert!(tdev[0].policy.eq("step_wise"));
-        assert!(tdev[0].mode.is_none());
-        assert!(tdev[0].temp.eq(&49925));
-        assert!(tdev[0].passive.is_none());
-
-        assert!(tdev[1].name.eq("thermal_zone1"));
-        assert!(tdev[1].zone_type.eq("acpitz"));
-        assert!(tdev[1].policy.eq("step_wise"));
-        assert!(tdev[1].mode.unwrap());
-        assert!(tdev[1].temp.eq(&-44000));
-        assert!(tdev[1].passive.unwrap().eq(&0));
+        for thermal in tdev {
+            match thermal.name.as_str() {
+                "thermal_zone0" => {
+                    assert_eq!(thermal.zone_type, "bcm2835_thermal");
+                    assert_eq!(thermal.policy, "step_wise");
+                    assert!(thermal.mode.is_none());
+                    assert!(thermal.temp.eq(&49925));
+                    assert!(thermal.passive.is_none());
+                }
+                "thermal_zone1" => {
+                    assert_eq!(thermal.zone_type, "acpitz");
+                    assert_eq!(thermal.policy, "step_wise");
+                    assert!(thermal.mode.unwrap());
+                    assert!(thermal.temp.eq(&-44000));
+                    assert!(thermal.passive.unwrap().eq(&0));
+                }
+                _ => panic!("invalid thermal zone: {}", thermal.name),
+            }
+        }
     }
 }