On second thought, not replacing it fro dry-run might not be necessary since the original TestExecuter it delegates to should already handle dry-run mode correctly. If that's the case, I think we should only keep the func test.
Makes sense, I've removed the dry-run logic, and only kept the test
jean-andre-gauthier
commented
1 year ago
Makes sense, I've removed the dry-run logic, and only kept the test