Skip to content

Commit

Permalink
[SymForce] Fix test include paths
Browse files Browse the repository at this point in the history
The tests shouldn't have to include these headers directly, the
generated functions include them.

Reviewers: danny
Topic: sym-test-include-paths
GitOrigin-RevId: 478f4e601402ab1a762f43b92360706703f6c374
  • Loading branch information
aaron-skydio authored and chao-qu-skydio committed May 11, 2023
1 parent 5bb0bdd commit c2283db
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
6 changes: 4 additions & 2 deletions test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,10 @@ file(MAKE_DIRECTORY ${CMAKE_BINARY_DIR}/bin/tests)
# symforce_test library

file(GLOB_RECURSE SYMFORCE_TEST_SOURCES CONFIGURE_DEPENDS symforce_function_codegen_test_data/**/*.cc)
file(GLOB_RECURSE SYMFORCE_TEST_HEADERS CONFIGURE_DEPENDS symforce_function_codegen_test_data/**/*.hpp)
file(GLOB_RECURSE SYMFORCE_TEST_HEADERS CONFIGURE_DEPENDS include/*.h)
file(GLOB_RECURSE SYMFORCE_TEST_HEADERS CONFIGURE_DEPENDS
symforce_function_codegen_test_data/**/*.hpp
include/*.h
)
add_library(
symforce_test
${SYMFORCE_LIBRARY_TYPE}
Expand Down
1 change: 0 additions & 1 deletion test/custom_function_replacement_symengine_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

#include <catch2/catch_test_macros.hpp>

#include "include/custom_function_replacement_header.h"
#include "symforce_function_codegen_test_data/symengine/custom_function_replacement.h"

TEST_CASE("Generated function with function replacement is correct", "[custom_func_replacement]") {
Expand Down
1 change: 0 additions & 1 deletion test/custom_function_replacement_sympy_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

#include <catch2/catch_test_macros.hpp>

#include "include/custom_function_replacement_header.h"
#include "symforce_function_codegen_test_data/sympy/custom_function_replacement.h"

TEST_CASE("Generated function with function replacement is correct", "[custom_func_replacement]") {
Expand Down

0 comments on commit c2283db

Please sign in to comment.