chenlin/Mujoco_WASM
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't use designated initializer in engine_resource_test.

Browse Source 
MuJoCo codebase is currently C++17, not C++20.

PiperOrigin-RevId: 525507811
Change-Id: Iaa1b5a271e811bee34418be79583fc5bdda3bee9
This commit is contained in:
Saran Tunyasuvunakool
2023-04-19 11:41:41 -07:00
committed by Copybara-Service
parent c6e35d255c
commit 34e62aa3c1
1 changed files with 21 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/engine/engine_resource_test.cc
+21 -22
View File
@@ -67,8 +67,8 @@ void close_str(mjResource* resource) {
} }
TEST_F(ResourceTest, RegisterProviderSuccess) { TEST_F(ResourceTest, RegisterProviderSuccess) {
mjpResourceProvider provider = {.prefix = "myprefix", mjpResourceProvider provider = {"myprefix", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
int count1 = mjp_resourceProviderCount(); int count1 = mjp_resourceProviderCount();
int i = mjp_registerResourceProvider(&provider); int i = mjp_registerResourceProvider(&provider);
@@ -79,7 +79,8 @@ TEST_F(ResourceTest, RegisterProviderSuccess) {
} }
TEST_F(ResourceTest, RegisterProviderMissingCallbacks) { TEST_F(ResourceTest, RegisterProviderMissingCallbacks) {
mjpResourceProvider provider = {.prefix = "myprefix"}; mjpResourceProvider provider = {"myprefix", nullptr, nullptr, nullptr,
nullptr};
// install warning handler // install warning handler
static char warning[1024]; static char warning[1024];
@@ -96,8 +97,7 @@ TEST_F(ResourceTest, RegisterProviderMissingCallbacks) {
} }
TEST_F(ResourceTest, RegisterProviderMissingPrefix) { TEST_F(ResourceTest, RegisterProviderMissingPrefix) {
mjpResourceProvider provider = {.prefix = "", mjpResourceProvider provider = {"", open_nop, read_nop, close_nop, nullptr};
.open = open_nop, .read = read_nop, .close = close_nop};
// install warning handler // install warning handler
static char warning[1024]; static char warning[1024];
@@ -114,11 +114,11 @@ TEST_F(ResourceTest, RegisterProviderMissingPrefix) {
} }
TEST_F(ResourceTest, RegisterProviderSubPrefix) { TEST_F(ResourceTest, RegisterProviderSubPrefix) {
mjpResourceProvider provider = {.prefix = "prefix", mjpResourceProvider provider = {"prefix", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
mjpResourceProvider provider2 = {.prefix = "pre", mjpResourceProvider provider2 = {"pre", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
// install warning handler // install warning handler
static char warning[1024]; static char warning[1024];
@@ -138,11 +138,11 @@ TEST_F(ResourceTest, RegisterProviderSubPrefix) {
} }
TEST_F(ResourceTest, RegisterProviderSuperPrefix) { TEST_F(ResourceTest, RegisterProviderSuperPrefix) {
mjpResourceProvider provider = {.prefix = "prefix", mjpResourceProvider provider = {"prefix", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
mjpResourceProvider provider2 = {.prefix = "prefix2", mjpResourceProvider provider2 = {"prefix2", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
// install warning handler // install warning handler
static char warning[1024]; static char warning[1024];
@@ -162,11 +162,11 @@ TEST_F(ResourceTest, RegisterProviderSuperPrefix) {
} }
TEST_F(ResourceTest, RegisterProviderSame) { TEST_F(ResourceTest, RegisterProviderSame) {
mjpResourceProvider provider = {.prefix = "prefix", mjpResourceProvider provider = {"prefix", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
mjpResourceProvider provider2 = {.prefix = "prefix", mjpResourceProvider provider2 = {"prefix", open_nop, read_nop, close_nop,
.open = open_nop, .read = read_nop, .close = close_nop}; nullptr};
int i1 = mjp_registerResourceProvider(&provider); int i1 = mjp_registerResourceProvider(&provider);
int count1 = mjp_resourceProviderCount(); int count1 = mjp_resourceProviderCount();
@@ -180,8 +180,8 @@ TEST_F(ResourceTest, RegisterProviderSame) {
} }
TEST_F(ResourceTest, GeneralTest) { TEST_F(ResourceTest, GeneralTest) {
mjpResourceProvider provider = {.prefix = "str://", mjpResourceProvider provider = {"str://", open_str, read_str, close_str,
.open = open_str, .read = read_str, .close = close_str}; nullptr};
// register resource provider // register resource provider
int i = mjp_registerResourceProvider(&provider); int i = mjp_registerResourceProvider(&provider);
@@ -197,12 +197,11 @@ TEST_F(ResourceTest, GeneralTest) {
EXPECT_THAT(buffer, StrEq("Hello World")); EXPECT_THAT(buffer, StrEq("Hello World"));
mju_closeResource(resource); mju_closeResource(resource);
} }
TEST_F(ResourceTest, GeneralTestFailure) { TEST_F(ResourceTest, GeneralTestFailure) {
mjpResourceProvider provider = {.prefix = "str://", mjpResourceProvider provider = {"str://", open_str, read_str, close_str,
.open = open_str, .read = read_str, .close = close_str}; nullptr};
// register resource provider // register resource provider
int i = mjp_registerResourceProvider(&provider); int i = mjp_registerResourceProvider(&provider);