Test for vector-valued ZhangLi u

[lang-m]

This is related to ubermag/micromagneticmodel#75 and ubermag/oommfc#133

Merging this test will also require some changes in mumax3c: either we can support the same functionality or disable the test. I will try that when I am on a suitable machine.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3fe64cb) to head (7c2c39e).

❗ Current head 7c2c39e differs from pull request most recent head 72b892d

Please upload reports for the commit 72b892d to get more accurate results.

Additional details and impacted files

@@            Coverage Diff            @@
##            master       #48   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           14        14           
=========================================
  Hits            14        14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lang-m]

@samjrholt Can you please review the new test test_vector_scalar.

[lang-m]

I re-wrote all tests for Zhang Li, the diff is not really meaningful. The new tests all check some sort of DW motion.

Testing the new methods with mumax3c remains to be done.

[lang-m]

Support in mumax3c is implemented in ubermag/mumax3c#88

[samjrholt]

Looks good in general. Should we also have a test with a vector field? I think that is the only case which isn't covered at the moment.

[samjrholt]

Looks good in general. Should we also have a test with a vector field? I think that is the only case which isn't covered at the moment.

@lang-m did you implement this or decide it wasn't needed?

[lang-m]

There was a test already: test_field_vector_u. Current is just in x direction but that should not matter because we mainly need to tests our conversion code.