fix(ModuleImports): This PR fixes the module import resolver for module of type BaseClassDataType

[rshah-evertz]

Description

Currently, there is a bug in datamodel-codegenerator's import resolver. While parsing and resolving the imports for a module, if the data model is of data type BaseClassDataType and the length of split reference_path is greater than the length of split current_module, the final output_path resolved is incorrect.

Example

current_module: "api.schemas"
reference: "api.this.that.ClassToImport"
relative(current_module, reference) returns: ('.this', 'that')
resulting from_ value for model of instance BaseClassDataType: .thisthat

This PR fixes this error by ensuring that when joining the tuple returned by the relative method it is checked that the first item in the tuple ends with a .

Is this acceptable, @koxudaxi?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@76fc354). Click here to learn what that means.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1862   +/-   ##
=======================================
  Coverage        ?   98.89%           
=======================================
  Files           ?       37           
  Lines           ?     4163           
  Branches        ?      969           
=======================================
  Hits            ?     4117           
  Misses          ?       29           
  Partials        ?       17           

Flag	Coverage Δ
unittests	98.55% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[codspeed-hq]

CodSpeed Performance Report

Merging #1862 will not alter performance

_{Comparing rshah-evertz:fix-imports-for-base-class-data-type (5968f9b) with main (76fc354)}

Summary

✅ 29 untouched benchmarks

[koxudaxi]

@rshah-evertz
I'm sorry for late reply.
Looks good.
Could you please add a unit test for your changes?

Thank you!!

[nwicht]

@rshah-evertz I have the same issue, and your fix would be great.

Do you need help to write the unit test?

[rshah-evertz]

I actually do need help with the unit test. How do I regenerate the expected files with the default timestamp of 2019-07-26T00:00:00+00:00? @koxudaxi

[rshah-evertz]

I've changed the existing nested directory structure to test the change I've made. Also, manually fixed the timestamps of newly generated expected files because I couldn't figure out how to run the codegen with the default timestamp.

[rshah-evertz]

Unsure about why codecov is failing. Any ideas? @koxudaxi