Skip to content

Added more tests to cover variable p and SEM bases definitions

Spencer Sherwin requested to merge redesign/MoreTests into feature/redesign

Issue/feature addressed

Currently the test suite did not check for variable p basis definition and SEM expansions

Proposed solution

Add in a few more default files with variable p and SEM bases definitions and add them to the test suites

Implementation

As above

Tests

Added additional tests using the current test suite implementations.

Suggested reviewers

Jacques, myself and possibly Josh for the sycl side of things

Notes

Please add any other information that could be useful for reviewers.

Checklist

  • [ ] Functions and classes, or changes to them, are documented.

    [ ] User guide/documentation is updated.

    [ ] Changelog is updated.

  • Suitable tests added for new functionality.

  • Contributed code is correctly formatted. (See the contributing guidelines).

  • [ ] License added to any new files.

  • No extraneous files have been added (e.g. compiler output or test data files).

Merge request reports

Loading