Fix clang-format issue, don't allow failure on test
Issue/feature addressed
This MR fixes a clang-format issue, and removes allow_failure
from the jobs so that CI won't pass unless formatting is correct.
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).