Skip to content

Comments

Error handling#4

Open
chp2001 wants to merge 10 commits intoJoshCu:mainfrom
chp2001:error_handling
Open

Error handling#4
chp2001 wants to merge 10 commits intoJoshCu:mainfrom
chp2001:error_handling

Conversation

@chp2001
Copy link
Contributor

@chp2001 chp2001 commented Feb 11, 2026

Adds:

  • Additional error handling/context for cli.rs, routing.rs, and main.rs.
  • Added basic in-file unit tests for cli.rs and config.rs.
    • More thorough testing of arguments to the program will likely require extensive refactoring.
  • Added testing suite for main.rs that verifies the test case works as expected.

Modifies:

  • Added static OUTPUT_TYPE variable to main.rs along with a match statement to suppress rustc's warnings about unused enum elements. This should serve as a placeholder until the relevant features are added.
  • Added unused _config_dir variable to the main function in `main.rs' to suppress the warnings about the CLI Config struct config_dir member going unused. Given that using the configuration data is on the roadmap, this was preferable to outright removal.
  • Slight refactoring in routing.rs to suppress unused variable warnings.
  • Applied rust-fmt to full project to improve code consistency and format.

PR likely is not complete, as there are probably additional areas that need error handling added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant