Use absolute_path instead of fs::canonicalize

[scampi]

This is a continuation of #3324:

• add a test for relative path to modules
• tested the fix proposed by @topecongiro works on a windows environment.

I think the comments regarding canonical paths by @rivy and @Xanewok can be revisited later once some fixes on rust itself are done (see list of open issues #1873 (comment)). I'll open an issue to track this.

Close #1873

[rivy]

Any progress on this?
Using Windows cargo fmt is broken for lots of repos because of this...

refs: #1873 , #3324, #3603, #3688

[Diggsey]

Bump...

[rivy]

Ping @topecongiro @scampi , this has been languishing for almost a year.
What can we do to push it forward?
As I said before, this is the base cause of issues in lots of repos.

[calebcartwright]

At this point it might be easiest if someone wanted to grab the commits or changes and try from a new PR, especially considering all the subsequent changes

[topecongiro]

Closing in favor of #4131.