Description of changes:
A couple unit tests are incorrectly using "src.fmeval" instead of just "fmeval" when patching functions. As a result, tests like test_get_user_agent_extra are calling the actual function instead. In hindsight, I should've used a different return_value when patching fmeval.model_runners.util.get_fmeval_package_version so that this would've been easily caught.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Description of changes: A couple unit tests are incorrectly using
"src.fmeval"instead of just"fmeval"when patching functions. As a result, tests liketest_get_user_agent_extraare calling the actual function instead. In hindsight, I should've used a differentreturn_valuewhen patchingfmeval.model_runners.util.get_fmeval_package_versionso that this would've been easily caught.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.