Commit 6026538a by Corentin Wallez

Revert "Use named value-parameterized tests for dEQP gtests"

This reverts commit 0db7b155. BUG= Change-Id: Ia2de8268bd51d865ad87875c55cb5bfcc51e29f6 Reviewed-on: https://chromium-review.googlesource.com/297760Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org> Tested-by: 's avatarCorentin Wallez <cwallez@chromium.org>
parent b92c133e
...@@ -148,7 +148,7 @@ void dEQPCaseList::initialize() ...@@ -148,7 +148,7 @@ void dEQPCaseList::initialize()
if (inString.substr(0, 4) == "TEST") if (inString.substr(0, 4) == "TEST")
{ {
std::string dEQPName = inString.substr(6); std::string dEQPName = inString.substr(6);
std::string gTestName = dEQPName.substr(dEQPName.find('.') + 1); std::string gTestName = dEQPName.substr(11);
std::replace(gTestName.begin(), gTestName.end(), '.', '_'); std::replace(gTestName.begin(), gTestName.end(), '.', '_');
// Occurs in some luminance tests // Occurs in some luminance tests
...@@ -172,12 +172,6 @@ class dEQPTest : public testing::TestWithParam<size_t> ...@@ -172,12 +172,6 @@ class dEQPTest : public testing::TestWithParam<size_t>
return testing::Range<size_t>(0, GetCaseList().numCases()); return testing::Range<size_t>(0, GetCaseList().numCases());
} }
static std::string GetCaseGTestName(size_t caseIndex)
{
const auto &caseInfo = GetCaseList().getCaseInfo(caseIndex);
return caseInfo.mGTestName;
}
static const dEQPCaseList &GetCaseList() static const dEQPCaseList &GetCaseList()
{ {
static dEQPCaseList sCaseList(TestModuleIndex); static dEQPCaseList sCaseList(TestModuleIndex);
...@@ -216,26 +210,32 @@ class dEQP_EGL : public dEQPTest<2> ...@@ -216,26 +210,32 @@ class dEQP_EGL : public dEQPTest<2>
{ {
}; };
#ifdef ANGLE_DEQP_GLES2_TESTS
// TODO(jmadill): add different platform configs, or ability to choose platform // TODO(jmadill): add different platform configs, or ability to choose platform
#define ANGLE_INSTANTIATE_DEQP_TEST_CASE(DEQP_TEST) \ TEST_P(dEQP_GLES2, Default)
TEST_P(DEQP_TEST, Default) { runTest(); } \ {
\ runTest();
INSTANTIATE_TEST_CASE_P(, DEQP_TEST, DEQP_TEST::GetTestingRange(), \ }
[](const testing::TestParamInfo<size_t> &info) \
{ \
return DEQP_TEST::GetCaseGTestName(info.param); \
})
#ifdef ANGLE_DEQP_GLES2_TESTS INSTANTIATE_TEST_CASE_P(, dEQP_GLES2, dEQP_GLES2::GetTestingRange());
ANGLE_INSTANTIATE_DEQP_TEST_CASE(dEQP_GLES2);
#endif #endif
#ifdef ANGLE_DEQP_GLES3_TESTS #ifdef ANGLE_DEQP_GLES3_TESTS
ANGLE_INSTANTIATE_DEQP_TEST_CASE(dEQP_GLES3); TEST_P(dEQP_GLES3, Default)
{
runTest();
}
INSTANTIATE_TEST_CASE_P(, dEQP_GLES3, dEQP_GLES3::GetTestingRange());
#endif #endif
#ifdef ANGLE_DEQP_EGL_TESTS #ifdef ANGLE_DEQP_EGL_TESTS
ANGLE_INSTANTIATE_DEQP_TEST_CASE(dEQP_EGL); TEST_P(dEQP_EGL, Default)
{
runTest();
}
INSTANTIATE_TEST_CASE_P(, dEQP_EGL, dEQP_EGL::GetTestingRange());
#endif #endif
} // anonymous namespace } // anonymous namespace
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment