Skip to content

Commit

Permalink
Increase test coverage (#131)
Browse files Browse the repository at this point in the history
* Start of passing tests

* Update TODO.md

* Add StyleCop; fix for various rules

* Update TODO

* Add P/Invoke tests

* test: EnumForceUInt64

* Test: EnumForceUInt16

* Test: EnumForceUInt8

* test: EnumForceSInt8

* Finish enum tests

* Fix workflow for installing Clang

* Fix workflow for C#

* Ignore CA1711 + CA1720

* Ignore CA1050 + CA1716

* Ignore CA1051 + CA1062

* Ignore CA1822 and silence CA5392

* Ignore CA5392

* Update ClangInstaller failure message

* Call Setup method for bindgen C# test

* Update ClangInstaller failure message

* Bump ClangInstaller logs to info

* Use `/lib` path native library for tests

* Install clang for Ubuntu

* Update

* Update Clang installation

* Fix CMake library builder

* Add additional logging for CMake library builder

* Use full path for native library

* Fix full path for native library Windows

---------

Co-authored-by: Lucas Girouard-Stranks <[email protected]>
  • Loading branch information
lithiumtoast and lithiumtoast authored Feb 7, 2023
1 parent df14898 commit 31b84fd
Show file tree
Hide file tree
Showing 530 changed files with 2,857 additions and 7,429 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"

DOWNLOAD_DIR="/tmp/c2cs"
INSTALL_DIR="$DIR/../lib"
INSTALL_DIR="$DIR/../../lib"

mkdir -p "$DOWNLOAD_DIR"
mkdir -p "$INSTALL_DIR"
Expand Down
17 changes: 5 additions & 12 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,7 @@ jobs:
matrix:
platform:
- { name: Windows x64, os: windows-latest, shell: bash }
# Using Ubuntu 18.04 because of GitHub Actions Runner bug: https://github.com/actions/runner/issues/1819#issuecomment-1098641309
- { name: Linux x64, os: ubuntu-18.04, shell: bash }
- { name: Linux x64, os: ubuntu-latest, shell: bash }
- { name: macOS x64, os: macos-latest, shell: bash }
defaults:
run:
Expand All @@ -39,25 +38,19 @@ jobs:
- name: "Install Clang: Windows"
if: matrix.platform.os == 'windows-latest'
run: |
source "${{ github.workspace }}\scripts\install-clang-x64-windows.sh"
source "${{ github.workspace }}\.github\scripts\install-clang-x64-windows.sh"
- name: "Install Clang: Linux"
if: matrix.platform.os == 'ubuntu-18.04'
if: matrix.platform.os == 'ubuntu-latest'
run: |
sudo apt-get update
sudo apt-get install gcc-aarch64-linux-gnu
source "${{ github.workspace }}/scripts/install-clang-x64-ubuntu_18_04.sh"
- name: "Install Clang: macOS"
if: matrix.platform.os == 'macos-latest'
run: |
source "${{ github.workspace }}/scripts/install-clang-macos.sh"
sudo apt-get install gcc-aarch64-linux-gnu llvm-14 clang-14
- name: ".NET"
uses: bottlenoselabs/github-actions-dotnet@v3
with:
dotnet-sdk-version: "7.0.x"
solution-or-project: "${{ github.workspace }}/C2CS.sln"
solution-or-project: "${{ github.workspace }}/src/cs/C2CS.sln"
is-enabled-upload-myget: "${{ matrix.platform.os == 'ubuntu-18.04' && ( github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') ) }}"
is-enabled-upload-nuget: "${{ matrix.platform.os == 'ubuntu-18.04' && startsWith(github.ref, 'refs/tags/v') }}"
myget-access-token: "${{ secrets.MYGET_ACCESS_TOKEN }}"
Expand Down
118 changes: 0 additions & 118 deletions C2CS.sln

This file was deleted.

32 changes: 0 additions & 32 deletions C2CS.sln.DotSettings

This file was deleted.

2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
MIT License

Copyright (c) 2020 Lucas Girouard-Stranks (https://github.com/lithiumtoast)
Copyright (c) 2023 Bottlenose Labs Inc. (https://github.com/bottlenoselabs)

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
Expand Down
3 changes: 0 additions & 3 deletions TODO.md

This file was deleted.

Loading

0 comments on commit 31b84fd

Please sign in to comment.