Avoid copying std::function objects needlessly in the test_throws infra

[randombit]

No description provided.

[Copilot]

Pull request overview

This PR optimizes the test_throws test infrastructure by changing how std::function objects are passed to avoid unnecessary copying. The change switches from passing std::function by const reference to pass-by-value with move semantics, which eliminates an extra copy when lambdas are passed as temporary objects.

Changes:

• Updated function signatures to take std::function<void()> by value instead of const reference
• Added std::move() calls when constructing ThrowExpectations objects to enable move semantics

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/tests/tests.h	Updated declarations and template implementations of test_throws and test_no_throw to use pass-by-value and std::move()
src/tests/tests.cpp	Updated non-template implementations of test_throws and test_no_throw to use pass-by-value and std::move()

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coveralls]

coverage: 90.065%. remained the same
when pulling e5c2e11 on jack/test-throw-no-copy
into 5f60b6b on master.