Fix minor wallet issues

[zees-dev]

Changelog

• added tempfile to create test dirs for testing
  • updated test functions and util functions accordingly (removed redundant helper functions and replaced references)
• added rust-toolchain.toml with stable rust 1.80.0
  • minor refactors to CI workflows to match the rust-toolchain.toml rust version
• git ignoring IDE directory (.vscode/)
• fixed new_at_index function to print checksum wallet address (currently printing bech32 address to stdout)
• updated ensure_no_wallet_exists function to handle the following cases:
  • where wallet path is a directory
  • where wallet path is an empty file
    • this also addresses: Wallet already exists warning is not correct #186
• added unit tests for ensure_no_wallet_exists

[kayagokalp]

Left a comment about rust-toolchain because I don't think it is doing what we are expecting to do, let's discuss first before merging

[kayagokalp]

Nice! Thanks for cleaning those little but frustrating issues 🚀 I left couple of final nits. A final note would be to splitting this type of PRs into more focused ones (as we squash-merge this commit and the message of it won't be very descriptive).

For this PR this is fine to merge as it is as the diff is very small but in general it might be better to separate things based on what they do as we are not rebase-merging or merging via merge commit.

[JoshuaBatty]

I guess I meant, what is content? I can see that it's optional, but I don't understand what content is made up of.

[zees-dev]

Content could be anything, just populates file with data. Doesn't have to be specific format.
P.s. Feel free to suggest a description (or improvement)

[kayagokalp]

How about something like:

/// Represents the possible serialized states of a wallet.
/// Used primarily for simulating wallet creation and serialization processes.
enum WalletSerializedState {
    Empty,
    WithData(String),
}

/// Simulates the serialization of a wallet to a file, optionally including dummy data.
/// Primarily used to test if checks for wallet file existence are functioning correctly.
fn serialize_wallet_to_file(wallet_path: &Path, state: WalletSerializedState) {
    // Create the wallet file if it does not exist.
    if !wallet_path.exists() {
        fs::File::create(wallet_path).unwrap();
    }

    // Write content to the wallet file based on the specified state.
    if let WalletSerializedState::WithData(data) = state {
        fs::write(wallet_path, data).unwrap();
    }
}

Leaving this as a suggestion only maybe Josh has something else in mind

[zees-dev]

thanks; updated: #220

[JoshuaBatty]

Thanks Kaya, that's perfect.