Skip to content
Snippets Groups Projects
Commit 3069355a authored by Nat Goodspeed's avatar Nat Goodspeed
Browse files

DEV-34522: [COMMENTS ONLY] clean up comments about (formerly) disabled tests

parent cbad737f
No related branches found
No related tags found
No related merge requests found
...@@ -236,13 +236,13 @@ target_link_libraries( ...@@ -236,13 +236,13 @@ target_link_libraries(
include(LLAddBuildTest) include(LLAddBuildTest)
SET(llcommon_TEST_SOURCE_FILES SET(llcommon_TEST_SOURCE_FILES
# WARNING: Please don't write tests against LLCommon or LLMath until this issue is resolved: https://jira.lindenlab.com/jira/browse/DEV-29456
# lllazy.cpp
) )
LL_ADD_PROJECT_UNIT_TESTS(llcommon "${llcommon_TEST_SOURCE_FILES}") LL_ADD_PROJECT_UNIT_TESTS(llcommon "${llcommon_TEST_SOURCE_FILES}")
#set(TEST_DEBUG on) #set(TEST_DEBUG on)
set(test_libs llcommon ${LLCOMMON_LIBRARIES} ${WINDOWS_LIBRARIES}) set(test_libs llcommon ${LLCOMMON_LIBRARIES} ${WINDOWS_LIBRARIES})
# Have to treat lllazy test as an integration test until this issue is resolved:
# https://jira.lindenlab.com/jira/browse/DEV-29456
LL_ADD_INTEGRATION_TEST(lllazy lllazy.cpp "${test_libs}") LL_ADD_INTEGRATION_TEST(lllazy lllazy.cpp "${test_libs}")
# *TODO - reenable these once tcmalloc libs no longer break the build. # *TODO - reenable these once tcmalloc libs no longer break the build.
......
...@@ -85,7 +85,8 @@ add_library (llmath ${llmath_SOURCE_FILES}) ...@@ -85,7 +85,8 @@ add_library (llmath ${llmath_SOURCE_FILES})
include(LLAddBuildTest) include(LLAddBuildTest)
SET(llmath_TEST_SOURCE_FILES SET(llmath_TEST_SOURCE_FILES
# WARNING: Please don't write tests against LLCommon or LLMath until this issue is resolved: https://jira.lindenlab.com/jira/browse/DEV-29456 # nat 2009-08-28: found this commented out and considered implementing it
# using LL_ADD_INTEGRATION_TEST, but there's no llvolume_test.cpp source?
# llvolume.cpp # llvolume.cpp
) )
LL_ADD_PROJECT_UNIT_TESTS(llmath "${llmath_TEST_SOURCE_FILES}") LL_ADD_PROJECT_UNIT_TESTS(llmath "${llmath_TEST_SOURCE_FILES}")
......
...@@ -240,8 +240,4 @@ IF (NOT LINUX AND VIEWER) ...@@ -240,8 +240,4 @@ IF (NOT LINUX AND VIEWER)
) )
LL_ADD_PROJECT_UNIT_TESTS(llmessage "${llmessage_TEST_SOURCE_FILES}") LL_ADD_PROJECT_UNIT_TESTS(llmessage "${llmessage_TEST_SOURCE_FILES}")
# Commented out - see rationale at bottom of newview's build file + poppy 2009-06-05
# Don't make llmessage depend on llsdmessage_test because ADD_COMM_BUILD_TEST depends on llmessage!
# ADD_COMM_BUILD_TEST(llsdmessage "" "${CMAKE_CURRENT_SOURCE_DIR}/tests/test_llsdmessage_peer.py")
ENDIF (NOT LINUX AND VIEWER) ENDIF (NOT LINUX AND VIEWER)
...@@ -1511,11 +1511,6 @@ endif (INSTALL) ...@@ -1511,11 +1511,6 @@ endif (INSTALL)
include(LLAddBuildTest) include(LLAddBuildTest)
SET(viewer_TEST_SOURCE_FILES SET(viewer_TEST_SOURCE_FILES
llagentaccess.cpp llagentaccess.cpp
# Not *actually* a unit test, it's an integration test.
# Because it won't work in the new unit test iface, i've commented out
# and notified Nat. Delete this when it's replaced!
# + poppy & brad 2009-06-05
# llcapabilitylistener.cpp
) )
set_source_files_properties( set_source_files_properties(
${viewer_TEST_SOURCE_FILES} ${viewer_TEST_SOURCE_FILES}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment