exasol / error-code-crawler-maven-plugin

Validator and crawler for exasol-error-codes in Java code
MIT License
1 stars 1 forks source link

Assigning the message to a variable should be allowed #61

Closed morazow closed 2 years ago

morazow commented 2 years ago

Assigning an error message (with .toString() call) to a variable should be allowed. Only the builder must not be assigned.

Context: https://github.com/exasol/error-reporting-java/pull/32

pj-spoelders commented 2 years ago

It works when you use the helper method, you don't get these errors then: Normally you wouldn't be able to use the c'tor like in these tests because it's not public.

Also, using the error crawler on the actual library that defines the parsed errors doesn't make a whole lot of sense to me, neither does crawling the actual tests for error messages since the errors in the error catalog are supposed to be customer facing ones, right?

I'll close this for now.