Fix configuration schema forbidding custom bytecode provider #1764
Conversation
| } | ||
|
|
||
| [Test] | ||
| public void ByteCodeProvider() |
There was a problem hiding this comment.
The first test of the fixture still succeeds because the configuration is invalid by lack of session-factory node. That is not a schema validation error, but a hard-coded check in the parsing.
There was a problem hiding this comment.
I understood that later, but did not report it here. Can you rename that test to something more meaningful than InvalidConfig?
To be squashed.
| { | ||
| [Test] | ||
| public void InvalidConfig() | ||
| public void ThrowIfSessionFactoryIsMissing() |
There was a problem hiding this comment.
Strictly speaking <session-factory /> element is not required. It is required only when config is loaded not from app.config I guess.
There was a problem hiding this comment.
Yes, but the name is already long enough I find. ThrowIfSessionFactoryIsMissingWhenNotFromAppConfig?
There was a problem hiding this comment.
Yes, something like that. Or, better: SessionFactoryIsRequiredWhenConfigurationIsLoadedNotFromAppConfig
To be squashed.
2 participants
No description provided.