• Jason Rhinelander's avatar
    Update all remaining tests to new test styles · 391c7544
    Jason Rhinelander authored
    This udpates all the remaining tests to the new test suite code and
    comment styles started in #898.  For the most part, the test coverage
    here is unchanged, with a few minor exceptions as noted below.
    
    - test_constants_and_functions: this adds more overload tests with
      overloads with different number of arguments for more comprehensive
      overload_cast testing.  The test style conversion broke the overload
      tests under MSVC 2015, prompting the additional tests while looking
      for a workaround.
    
    - test_eigen: this dropped the unused functions `get_cm_corners` and
      `get_cm_corners_const`--these same tests were duplicates of the same
      things provided (and used) via ReturnTester methods.
    
    - test_opaque_types: this test had a hidden dependence on ExampleMandA
      which is now fixed by using the global UserType which suffices for the
      relevant test.
    
    - test_methods_and_attributes: this required some additions to UserType
      to make it usable as a replacement for the test's previous SimpleType:
      UserType gained a value mutator, and the `value` property is not
      mutable (it was previously readonly).  Some overload tests were also
      added to better test overload_cast (as described above).
    
    - test_numpy_array: removed the untemplated mutate_data/mutate_data_t:
      the templated versions with an empty parameter pack expand to the same
      thing.
    
    - test_stl: this was already mostly in the new style; this just tweaks
      things a bit, localizing a class, and adding some missing
      `// test_whatever` comments.
    
    - test_virtual_functions: like `test_stl`, this was mostly in the new
      test style already, but needed some `// test_whatever` comments.
      This commit also moves the inherited virtual example code to the end
      of the file, after the main set of tests (since it is less important
      than the other tests, and rather length); it also got renamed to
      `test_inherited_virtuals` (from `test_inheriting_repeat`) because it
      tests both inherited virtual approaches, not just the repeat approach.
    391c7544
pybind11_tests.cpp 3.62 KB