Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatic tests on all platforms #128

Merged
merged 109 commits into from
Aug 6, 2020
Merged

Automatic tests on all platforms #128

merged 109 commits into from
Aug 6, 2020

Conversation

bcebere
Copy link
Member

@bcebere bcebere commented Aug 4, 2020

Description

Activate automatic tests for Windows and MacOS.

Fixes #88

Checklist

@bcebere bcebere changed the title [WIP] Automatic tests on all platforms Automatic tests on all platforms Aug 5, 2020
@bcebere bcebere requested a review from youben11 August 5, 2020 21:22
@bcebere bcebere requested a review from philomath213 August 5, 2020 21:42
Copy link
Member

@youben11 youben11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing work Bogdan! I guess we only missing Bazel on Windows and we can say that we have a full cross platform tests, no more failure on release!

@youben11 youben11 added Build Related to the build of the package Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase labels Aug 6, 2020
@bcebere bcebere requested review from philomath213 and removed request for philomath213 August 6, 2020 09:15
@youben11 youben11 merged commit 3f7cf2a into master Aug 6, 2020
@bcebere bcebere deleted the crossplatform_tests branch August 12, 2020 08:24
pierreeliseeflory pushed a commit to pierreeliseeflory/TenSEAL that referenced this pull request Apr 27, 2022
* workflow: move ubuntu depends to script

* move linter checks to dedicated job

* add tests folder to clang-format

* test failure

* lint tests

* add macos script

* yaml error

* debug

* disable other jobs for now

* osx debug

* debug osx

* osx BUILD issue

* debug osx

* debug

* osx debug

* debug

* debug

* debug

* debug osx proto

* debug

* debug

* Revert "debug"

This reverts commit 6d0fa9e.

* debug

* debug osx

* debug osx

* cleanup

* revert proto makefile

* add windows

* Revert "revert proto makefile"

This reverts commit 4dfa891.

* debug windows

* add build scripts

* msbuild bbreak

* typo

* debug windows

* debug windows

* debug windows

* protoc test

* debug windows

* drop protoc action

* debug windows

* debug windows

* debug windows

* windows hacks

* trial&error

* trial&error

* debug

* debug

* debug

* debug

* build protobuf from scratch for windows

* case insensitive windows

* remove caches

* debug windows

* debug

* debug

* windows bbreak

* typo

* bbreak

* debug windows

* windows debug

* update bat script

* update windows

* link errors

* windows issues

* debug

* fix env path

* build proto release

* windows issues

* windows issues

* debug windows

* update linked libraries

* windows: dynamic runtime

* debug windows

* i hate windows

* debug

* bbreak

* debug windows MT vs MD

* debug

* update gtest

* windows bbreak

* add sealapi

* windows debug

* debug

* pytest: windows issues

* windows fixes

* update seal tests

* enable bazel

* enable pytest for tenseal

* use MTd for tests

* gtest bbreak

* yaml error

* yaml errors

* yaml fixes

* windows debug

* gtest issue on windows

* enable pytest

* treat warnings as errors

* full tests

* cleanup

* Update CMakeLists.txt

* small explanation on MT vs MD

* fix some warnings

* remove protobuf from the main cmake

* separate bazel workflow

* update release yamls

* windows publish scripts

Co-authored-by: Ayoub Benaissa <ayouben9@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Related to the build of the package Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automate tests on MacOS and Windows
2 participants