SQLA DTO: Explicit error message for cases where a column has no type annotation

[provinzkraut]

Currently, when a SQLAlchemy DTO is constructed from a model that is missing a type annotation for an included column, a nondescript KeyError is raised.

This adds an ImproperlyConfiguredException with an explicit error message, pointing at the potential issue in these cases.

Pull Request Checklist

• New code has 100% test coverage
• (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
• (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

Close Issue(s)

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

[github-actions]

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/1852

[peterschutt]

Awesome, thanks for this! One thing I've had in the back of my mind is that we could infer type into from the column/relationship if type hints aren't available - but that's not to take anything away from this, and can be another PR. Thanks!

[provinzkraut]

One thing I've had in the back of my mind is that we could infer type into from the column/relationship if type hints aren't available

That's a good idea! I'll open an issue for this!