Dedent fixes for Python 3.12

[Harsha-Nori]

Aiming to fix #942

This should handle general use cases across Python versions, but further fixes are probably still needed for:

• closure functions
• nested f-strings (with arbitary nesting now possible in python 3.12 thanks to the new python parser improvements 😵‍💫)

I think these are both esoteric enough that we can kick the can down the road, but I'll keep thinking about it.

@mmoskal

[Harsha-Nori]

Also tag @riedgar-ms -- I may have put the tests in the wrong place, I saw "test_call_embeddings" inside of "test_model" which feels spiritually similar to the dedent semantics, but I'm really not sure where the right place is in the new framework. Happy to move these anywhere really

[riedgar-ms]

Looks like a reasonable location to me.

[hudson-ai]

Nice @Harsha-Nori! The solution is also a lot more readable than what was there before.

Regarding location, I think it's fine where it is, but I'd consider making a new file test_decorator.py or something of that sort in tests/unit/. If we later want to add some tests about inferring stateful/statelessness, etc. I think that we'd want to put those in there too.

[riedgar-ms]

Can these commented blocks (in the tests as well) be deleted before merge, please?

[Harsha-Nori]

Is it OK if we leave them in? The logic here is annoyingly tricky and while it's not feature complete yet, will be important to replicate when we extend the decorator to nested function definitions

[hudson-ai]

@riedgar-ms @Harsha-Nori any objections to merging this?

[Harsha-Nori]

Thanks for approving @hudson-ai and @riedgar-ms!