refactor: fix unit and documentation tests

[SanchithHegde]

Type of Change

• Bugfix
• New feature
• Enhancement
• Refactoring
• Dependency updates
• Documentation
• CI/CD

Description

This PR enables the usage of sccache for caching compilation artifacts in the CI-pr workflow, and fixes the existing unit and documentation tests. I'll be taking up running the unit and documentation on pull requests in a separate PR, with some more CI workflow improvments.

Motivation and Context

Fixing unit tests so that they can be run as part of pull request checks.

How did you test it?

I've run unit tests locally with the following command (with locally running Postgres and Redis servers), and they pass:

FEATURES=$(cargo metadata --all-features --format-version 1 | \
  jq -r '
    [ ( .workspace_members | sort ) as $package_ids # Store workspace crate package IDs in `package_ids` array
    | .packages[] | select( IN(.id; $package_ids[]) ) | .features | keys[] ] | unique # Select all unique features from all workspace crates
    | del( .[] | select( any( . ; . == ("aws_kms", "aws_s3", "release", "vergen") ) ) ) # Exclude some features from features list
    | join(",") # Construct a comma-separated string of features for passing to `cargo`
  ')
cargo nextest run --config-file .github/nextest.toml --profile ci -E 'kind(lib)' --features "${FEATURES}" --status-level none

Further, running documentation tests using the following command succeeds:

cargo test --doc --features "${FEATURES}"

I'm unable to include a screenshot of the documentation test run, since the output is too long.

Checklist

• I formatted the code cargo +nightly fmt --all
• I addressed lints thrown by cargo clippy
• I reviewed the submitted code
• I added unit tests for my changes where possible

[pixincreate]

CI changes, LGTM.

[prajjwalkumar17]

Approved on behalf of routing.

[SanchithHegde]

I've rebased the PR on a recent main branch, updated the PR to only fix unit and documentation tests, and enabled the use of sccache in the CI-pr workflow. I'll be adding the unit and documentation tests in a separate PR, possibly as a separate job, once I figure out the arguments that we would pass to cargo nextest. I've updated the PR description to reflect the same.

[pixincreate]

CI stuff, LGTM!