Merge pull request #41 from BoolPurist/feat_better_output_ldap_list

Feat: better output  for listing LDAP users

CHANGELOG.md

@@ -6,8 +6,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+- Added possibility to output listing of LDAP user via pretty table format 
+  and the old machine-readable format via CLI flag in subcommand list
+
 ### Changed
 
+- Output of for LDAP user via the sub command list is now presented in a nicer ASCII table format
 - CLI argument for slurm, ldap and directory management can be toggled via cli and conf.toml individually
   CLI Option for a system is only available if a sub command supports it.
   slurm-only, ldap-only and dirs-only were replaced by new options

CONTRIBUTING.md

@@ -16,3 +16,26 @@ grep -nr "TODO:" src/*
 
 you can see all the places.
 
+# How to Test
+
+Before submitting run
+
+```sh
+cargo test
+```
+
+## Snapshot testing
+
+Some tests use a crate called [Insta](https://insta.rs/docs/cli/) to perform
+snapshot testing. This makes auditing and writing test with complex expected output 
+a lot easier.
+
+If you write an unit test which uses this tool or you change the code in a way which changes the snapshot
+then you should use the cargo insta plugin to review changed/created snapshots.
+
+You can install this cargo plugin via
+
+```sh
+cargo install cargo-insta
+```
+