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.

Sign up for GitHub

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

Jump to bottom

Additional unit-tests and unit-test adjustment #499

Merged
merged 69 commits into from
May 25, 2023
Merged

Additional unit-tests and unit-test adjustment #499

merged 69 commits into from
May 25, 2023

Commits on May 5, 2023

  1. Convert existing tests to use new environment variables, fix mqtt311 … 

    …operation statistics test
    @TwistedTwigleg
    TwistedTwigleg committed May 5, 2023
    Configuration menu
    Copy the full SHA
    cafbdfd View commit details
    Browse the repository at this point in the history

  2. Add some MQTT311 tests akin to the MQTT5 tests. Just direct MQTT for now

    @TwistedTwigleg
    TwistedTwigleg committed May 5, 2023
    Configuration menu
    Copy the full SHA
    33aa94c View commit details
    Browse the repository at this point in the history

Commits on May 8, 2023

  1. use a UUID for IoT Core connection tests

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    e3ac79c View commit details
    Browse the repository at this point in the history

  2. Add the MQTT311 WS low-level tests, add tests to the CMakeLists

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    1a35733 View commit details
    Browse the repository at this point in the history

  3. Minor compiling fixes

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    3de4d3b View commit details
    Browse the repository at this point in the history

  4. Relax the operation statistics test just a bit

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    60e8c70 View commit details
    Browse the repository at this point in the history

  5. Add a bunch of MQTT311 credentials tests

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    78415d5 View commit details
    Browse the repository at this point in the history

  6. Run new builder CI in Codebuild

    @TwistedTwigleg
    TwistedTwigleg committed May 8, 2023
    Configuration menu
    Copy the full SHA
    4bb8cdb View commit details
    Browse the repository at this point in the history

Commits on May 9, 2023

  1. Add MQTT5 credentials connection tests, add MQTT311 will test

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    90e31b8 View commit details
    Browse the repository at this point in the history

  2. Removed unused variables

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    ee7af8c View commit details
    Browse the repository at this point in the history

  3. Make Windows happy

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    bef0948 View commit details
    Browse the repository at this point in the history

  4. Make the PKCS11 init strict in CI

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    c63f939 View commit details
    Browse the repository at this point in the history

  5. Use DUSE_OPENSSL=ON to run PKCS11 in CI, install the SSL development … 

    …lib if needed
    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    2a52be1 View commit details
    Browse the repository at this point in the history

  6. Try installing openssl11 before openssl

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    8c441b1 View commit details
    Browse the repository at this point in the history

  7. Nevermind, do not use openssl11-devel

    @TwistedTwigleg
    TwistedTwigleg committed May 9, 2023
    Configuration menu
    Copy the full SHA
    435e400 View commit details
    Browse the repository at this point in the history

Commits on May 10, 2023

  1. Only selectively set to use OpenSSL

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    1cc0912 View commit details
    Browse the repository at this point in the history

  2. Make running PKCS11 tests dependent on USE_OPENSSL, adjust Codebuild … 

    …accordingly
    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    74b25c8 View commit details
    Browse the repository at this point in the history

  3. Adjust Codebuild install steps

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    bf686fa View commit details
    Browse the repository at this point in the history

  4. Skip installing AWS CLI on Codebuild, split Alpine out into own matri… 

    …x so we can avoid running PKCS11 on it
    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    2fe7820 View commit details
    Browse the repository at this point in the history

  5. Fix byo-crypto CI and API init in Codebuild

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    72f9068 View commit details
    Browse the repository at this point in the history

  6. More BYO-Crpto fixes, further Codebuild adjustments

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    92f728d View commit details
    Browse the repository at this point in the history

  7. Fix compile issue

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    3942fb3 View commit details
    Browse the repository at this point in the history

  8. Fix PKCS11 tests being skipped sometimes, adjust CMakeLists for tests

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    9c5f7f4 View commit details
    Browse the repository at this point in the history

  9. Move AL2 to not run with USE_OPENSSL for now

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    57c47aa View commit details
    Browse the repository at this point in the history

  10. Hopefully fix Will test in MQTT5 on Codebuild

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    b026742 View commit details
    Browse the repository at this point in the history

  11. Did not fix the will thing after all

    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    568f898 View commit details
    Browse the repository at this point in the history

  12. Quick trivial commit to trigger CI to see if PKCS12 is fixed on MacOS… 

    … now
    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    8f26a74 View commit details
    Browse the repository at this point in the history

  13. Super quick test - try NOT using OpenSSL in Codebuild to see if that … 

    …fixes Will test
    @TwistedTwigleg
    TwistedTwigleg committed May 10, 2023
    Configuration menu
    Copy the full SHA
    86c966f View commit details
    Browse the repository at this point in the history

Commits on May 12, 2023

  1. Go back to using OpenSSL in Codebuild since it did not make a difference

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    788d56a View commit details
    Browse the repository at this point in the history

  2. Adjust the PKCS11 port and pass the CA file

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    47d3999 View commit details
    Browse the repository at this point in the history

  3. For now, skip PKCS11 tests on ManyLinux

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    fc9e365 View commit details
    Browse the repository at this point in the history

  4. Better adjust CI headers. Compile iOS to ios-armv8, which appears to … 

    …be the supported platform
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    21b2958 View commit details
    Browse the repository at this point in the history

  5. Set the bundle identifier if cross compiling on iOS to fix CI

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    0f7ca00 View commit details
    Browse the repository at this point in the history

  6. Try to fix iOS compile again

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    9507fe0 View commit details
    Browse the repository at this point in the history

  7. Try disabling code signing on XCode

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    b7564fd View commit details
    Browse the repository at this point in the history

  8. Another attempt at iOS code signing fix

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    427bc42 View commit details
    Browse the repository at this point in the history

  9. Try the last commit but set to run on APPLE instead of IOS for testin… 

    …g purposes
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    21207ae View commit details
    Browse the repository at this point in the history

  10. Try setting the properties a different way

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    a9cdfe2 View commit details
    Browse the repository at this point in the history

  11. Try a different way to disable code signing

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    74df3c1 View commit details
    Browse the repository at this point in the history

  12. Codebuild time. See if the environment tests are being skipped due to… 

    … missing environment variables
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    5679c9d View commit details
    Browse the repository at this point in the history

  13. Is it the net_test_case causing the skip in Codebuild?

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    541f1c5 View commit details
    Browse the repository at this point in the history

  14. Does it have anything to do with the code itself?

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    aff862b View commit details
    Browse the repository at this point in the history

  15. Is it due to returning -1

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    b9e038c View commit details
    Browse the repository at this point in the history

  16. Instead of assert or returning -1, try returning a big number when so… 

    …mething goes wrong
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    96b693a View commit details
    Browse the repository at this point in the history

  17. Adjust MQTT311 environment test to not assume the AWS credentials are… 

    … set, to see if that fixes the test on Codebuild
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    4f9b193 View commit details
    Browse the repository at this point in the history

  18. Use a working test, but make the env credentials provider

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    8dace70 View commit details
    Browse the repository at this point in the history

  19. Incorporate the provider back to it

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    0035078 View commit details
    Browse the repository at this point in the history

  20. Okay, we are getting somewhere. Keep narrowing down on where the skip is

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    948ff56 View commit details
    Browse the repository at this point in the history

  21. Check the credentials provider for null

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    b4378e3 View commit details
    Browse the repository at this point in the history

  22. Bring the next bit of code

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    dccbc50 View commit details
    Browse the repository at this point in the history

  23. Is it printf or ASSERT_TRUE(false) causing the skip?

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    91d2af2 View commit details
    Browse the repository at this point in the history

  24. See if this makes a difference

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    0944d0e View commit details
    Browse the repository at this point in the history

  25. Is it this specific print that causes a skip? Why

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    a39e5da View commit details
    Browse the repository at this point in the history

  26. Return -1 instead?

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    f4836c7 View commit details
    Browse the repository at this point in the history

  27. Throw an exception? Not sure everything else is a skip...

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    b965244 View commit details
    Browse the repository at this point in the history

  28. Skip the environment tests if the environment variables are not there… 

    …. Make most MQTT tests throw exceptions instead of errors
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    109c7d7 View commit details
    Browse the repository at this point in the history

  29. For MQTT311 tests, set the socket timeout options

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    7a971e5 View commit details
    Browse the repository at this point in the history

  30. Derp, tests might have been failing due to Codebuild setup rather tha… 

    …n the test itself
    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    1095107 View commit details
    Browse the repository at this point in the history

  31. Clean up unreachable code

    @TwistedTwigleg
    TwistedTwigleg committed May 12, 2023
    Configuration menu
    Copy the full SHA
    258fa90 View commit details
    Browse the repository at this point in the history

Commits on May 15, 2023

  1. CMakeLists cleanup

    @TwistedTwigleg
    TwistedTwigleg committed May 15, 2023
    Configuration menu
    Copy the full SHA
    f6ee0f2 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into unit_test_refactor

    @TwistedTwigleg
    TwistedTwigleg authored May 15, 2023
    Configuration menu
    Copy the full SHA
    f3c0182 View commit details
    Browse the repository at this point in the history

Commits on May 24, 2023

  1. Merge branch 'main' into unit_test_refactor

    @TwistedTwigleg
    TwistedTwigleg authored May 24, 2023
    Configuration menu
    Copy the full SHA
    8c502f5 View commit details
    Browse the repository at this point in the history

  2. CR changes: Improve wording and reduce duplicate code

    @TwistedTwigleg
    TwistedTwigleg committed May 24, 2023
    Configuration menu
    Copy the full SHA
    f04a873 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'unit_test_refactor' of https://github.com/awslabs/aws-c… 

    …rt-cpp into unit_test_refactor
    @TwistedTwigleg
    TwistedTwigleg committed May 24, 2023
    Configuration menu
    Copy the full SHA
    d600d89 View commit details
    Browse the repository at this point in the history

  4. CR: Add some helper functions to reduce duplicate code, use ASSERT again

    @TwistedTwigleg
    TwistedTwigleg committed May 24, 2023
    Configuration menu
    Copy the full SHA
    d982622 View commit details
    Browse the repository at this point in the history

  5. This should fix all the C++ tests...

    @TwistedTwigleg
    TwistedTwigleg committed May 24, 2023
    Configuration menu
    Copy the full SHA
    bd709bc View commit details
    Browse the repository at this point in the history

  6. Use the latest builder release

    @TwistedTwigleg
    TwistedTwigleg committed May 24, 2023
    Configuration menu
    Copy the full SHA
    d5d0a1f View commit details
    Browse the repository at this point in the history

Commits on May 25, 2023

  1. Adjust CMAKE to not force disabling code signing and instead just dis… 

    …able as part of CI
    @TwistedTwigleg
    TwistedTwigleg committed May 25, 2023
    Configuration menu
    Copy the full SHA
    cf8bcc3 View commit details
    Browse the repository at this point in the history

  2. Forgot to set the release builder in Codebuild. This fixes that

    @TwistedTwigleg
    TwistedTwigleg committed May 25, 2023
    Configuration menu
    Copy the full SHA
    271f49f View commit details
    Browse the repository at this point in the history