[libcxx] [test] Fix Container::insert(value_type const&) tests
Several unit tests meaning to test the behavior of lvalue insertion incorrectly pass rvalues. Fixes bug PR # 27394
Paths
| Differential D44411
[libcxx] [test] Fix Container::insert(value_type const&) tests ClosedPublic Authored by jloser on Mar 12 2018, 7:49 PM.
Details
Diff Detail Event TimelineComment Actions Have you verified that we're not losing test coverage here? That is, are you sure we still have tests for the rvalue overloads in other test files?
jloser retitled this revision from [libc++] Fix Container::insert(value_type const&) tests to [libcxx] [test] Fix Container::insert(value_type const&) tests.Apr 8 2018, 1:46 PM Comment Actions Use v3 rather than an rvalue of 3 in libcxx/test/std/containers/associative/multiset/insert_cv.pass.cpp This revision is now accepted and ready to land.Apr 8 2018, 3:00 PM Comment Actions
Yep. These containers already have tests for inserts with rvalues. No loss in test coverage here.
Revision Contents
Diff 141573 libcxx/test/std/containers/associative/multiset/insert_cv.pass.cpp
libcxx/test/std/containers/associative/set/insert_cv.pass.cpp
libcxx/test/std/containers/unord/unord.map/unord.map.modifiers/insert_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.map/unord.map.modifiers/insert_hint_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.multimap/unord.multimap.modifiers/insert_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.multimap/unord.multimap.modifiers/insert_hint_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.multiset/insert_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.multiset/insert_hint_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.set/insert_const_lvalue.pass.cpp
libcxx/test/std/containers/unord/unord.set/insert_hint_const_lvalue.pass.cpp
|
Should be v3?