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

[CI] Run unit tests #476

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[CI] Run unit tests #476

wants to merge 6 commits into from

Commits on Apr 17, 2022

  1. Fix gtest install

    GoogleTest is packaged in subdirectory.
    
      ls -lah  downloads/googletest-release-1.11.0/*
        drwxrwxr-x 2 4.0K Jun 11  2021 ci
        drwxrwxr-x 8 4.0K Jun 11  2021 googlemock
        drwxrwxr-x 9 4.0K Jun 11  2021 googletest
        # ....
    
    First download the zip file, then unzip to a tmp dir and finaly move googletest
    to tools directory
    FabioBatSilva committed Apr 17, 2022
    Configuration menu
    Copy the full SHA
    64632ad View commit details
    Browse the repository at this point in the history

Commits on Apr 18, 2022

  1. Configuration menu
    Copy the full SHA
    5aab324 View commit details
    Browse the repository at this point in the history
  2. Use -isystem to include gtest

    This will ignore warning in gtest
    since it is not compatible with some of the flags beeing enforced
    
      gtest.cc:1266:43:
      error: implicit conversion from ‘float’ to ‘double’ to match other
      operand of binary expression [-Werror=double-promotion]
       1266 |         costs[l_i + 1][r_i + 1] = replace + 1.00001;
            |                                   ~~~~~~~~^~~~~~~~~
    FabioBatSilva committed Apr 18, 2022
    Configuration menu
    Copy the full SHA
    9c599db View commit details
    Browse the repository at this point in the history
  3. Compile utils_math with -O1

    Compile with -O1 to make inline functions compatible with gcc
    
      utils_math.c:582: undefined reference to `utils_truncate_number'
        error: ld returned 1 exit status
    FabioBatSilva committed Apr 18, 2022
    Configuration menu
    Copy the full SHA
    64b649b View commit details
    Browse the repository at this point in the history
  4. Uncomment assertions for vector_2d on edge of range

     Unused variable ret causes error :
    
     unittest.cpp: In member function ‘virtual void Saturate2dVector_ValsOnEdgeOfRange_Test::TestBody()’:
      unittest.cpp:291:9: error: variable ‘ret’ set but not used [-Werror=unused-but-set-variable]
        291 |    bool ret;
            |         ^~~
    FabioBatSilva committed Apr 18, 2022
    Configuration menu
    Copy the full SHA
    1a89222 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    4f11e2b View commit details
    Browse the repository at this point in the history