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

Use pFUnit 4 for testing #70

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Use pFUnit 4 for testing #70

wants to merge 4 commits into from

Commits on Aug 21, 2023

  1. Switch to use pFUnit 4.7 

    Includes necessary modifications to CMakeLists.txt and the tests
themselves.

At the moment produces large number of warnings from the tests and
CMake.
    @Mikolaj-A-Kowalski
    Mikolaj-A-Kowalski committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    f4e5abd View commit details
    Browse the repository at this point in the history

  2. Remove easy to fix warnings. 

    Mostly unused variables and unintended conversions.
    @Mikolaj-A-Kowalski
    Mikolaj-A-Kowalski committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    8612baa View commit details
    Browse the repository at this point in the history

  3. Supress unused variable warning due to pFUnit bug

    @Mikolaj-A-Kowalski
    Mikolaj-A-Kowalski committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    3170b67 View commit details
    Browse the repository at this point in the history

Commits on Nov 22, 2023

  1. Change the Docker image used for tests 

    Now uses a version with PFUnit 4.7.3
    @Mikolaj-A-Kowalski
    Mikolaj-A-Kowalski committed Nov 22, 2023
    Configuration menu
    Copy the full SHA
    3cf6571 View commit details
    Browse the repository at this point in the history