It is. There is literally zero other way to make that function unit testable. What are you gonna compare that data with in a test if it's thrown into IO?
By definition all unit testable functions have to return data that can be asserted. You throw that data to IO it's not unit testable.
IO is testable via an integration tests. But not unit tests. Which is what my query exactly specified. I specified unit tests.
That doesn't change the fact that it's not a valid refactoring. If you can't make it unit testable without changing behavior, then it should tell you that.
Replacing a function that does `print("hello world")` with a function that does `return "hello world"` isn't a valid way to make it unit testable.
Alright fine, I can concede to this. ChatGPT should not have given me the best alternative but it should have given me the exact technically correct answer. You're right.
By definition all unit testable functions have to return data that can be asserted. You throw that data to IO it's not unit testable.
IO is testable via an integration tests. But not unit tests. Which is what my query exactly specified. I specified unit tests.