Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

Commit

Permalink
chore: Add windows-latest to test suite, fix some lints for Windows. (#…
Browse files Browse the repository at this point in the history
…250)

Fixes #237.
  • Loading branch information
jsantell authored Mar 1, 2023
1 parent f18db24 commit a96638d
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 3 deletions.
25 changes: 25 additions & 0 deletions .github/workflows/run_test_suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,31 @@ jobs:
swift build
swift test
run-test-suite-windows:
runs-on: windows-latest
steps:
- uses: actions/checkout@v2
- uses: Swatinem/rust-cache@v2
- uses: actions-rs/toolchain@v1
with:
override: true
profile: minimal
toolchain: stable
- name: 'Install environment packages'
run: |
choco install -y cmake protoc openssl
shell: sh
- name: 'Install IPFS Kubo'
uses: ibnesayeed/setup-ipfs@master
with:
ipfs_version: v0.17.0
run_daemon: true
- name: 'Run Rust native target tests'
uses: actions-rs/cargo@v1.0.3
with:
command: test
args: --features test_kubo

run-test-suite-linux:
runs-on: ubuntu-latest
steps:
Expand Down
6 changes: 3 additions & 3 deletions rust/noosphere/src/sphere/storage.rs
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,10 @@ impl StorageLayout {
fn get_scoped_path(path: &PathBuf, scope: &Did) -> PathBuf {
#[cfg(not(windows))]
let path_buf = path.join(scope.as_str());

#[cfg(windows)]
/// Windows does not allow `:` in file paths.
/// Replace `:` in scope/key with `_`.
// Windows does not allow `:` in file paths.
// Replace `:` in scope/key with `_`.
let path_buf = path.join(scope.as_str().replace(":", "_"));

path_buf
Expand Down

0 comments on commit a96638d

Please sign in to comment.