-
Notifications
You must be signed in to change notification settings - Fork 94
feat: add unit tests in ci #201
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
base: master
Are you sure you want to change the base?
Conversation
@maxday why add a new unittest directory? |
It seems to me that existing tests are integration tests (which requires a more complex infra to be able to run them with CodeBuild and the SDK). |
Can you think of a way to combine both tests? It's not ideal to we have two different sets of tests that use different versions of the testing library. |
Description of changes:
gtest
framework viaCMakeLists.txt
(1.12.0
)noop.dummy_test
, as a first step for having unit tests in CI.clang-tidy
not to complain on external testing dependenciesTesting:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.