Remove changed integration test and add unit test for new method

[Copilot]

Description and Context: What and Why?

This PR removes the DefectivePark91aWrapper from the integration test test_logpdf_gaussian_process.py and creates a dedicated unit test for the _filter_failed_evaluations method in the AdaptiveSampling class.

Changes Made

• Reverted integration test: Removed DefectivePark91aWrapper class and itertools import from tests/integration_tests/models/test_logpdf_gaussian_process.py, restoring the driver to use park91a_hifi_on_grid directly
• Added unit test: Created tests/unit_tests/iterators/test_adaptive_sampling.py with comprehensive tests for _filter_failed_evaluations:
  • test_filter_failed_evaluations_no_failures - Tests when there are no NaN values
  • test_filter_failed_evaluations_some_failures - Tests when some evaluations contain NaN
  • test_filter_failed_evaluations_all_failures - Tests when all evaluations fail
  • test_filter_failed_evaluations_multiple_calls - Tests that failed evaluations accumulate across multiple calls

Related Issues and Pull Requests

• Related to the commit that introduced _filter_failed_evaluations method in AdaptiveSampling

Interested Parties

Note: More information on the merge request procedure in QUEENS can be found in the Submit a pull request section in the CONTRIBUTING.md file.

Original prompt

remove the changed integration test and do a unit test instead, testing the newly added method

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[rjoussen]

@copilot just write the python files, I will test myself

[Copilot]

@copilot just write the python files, I will test myself

The changes are complete in commit a7c1a58. I've removed the DefectivePark91aWrapper from the integration test and created a new unit test file at tests/unit_tests/iterators/test_adaptive_sampling.py with tests for the _filter_failed_evaluations method.