Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MODE-1155 Added test to verify I18n messages exist for related annotations #82

Merged
merged 1 commit into from Apr 20, 2011

Conversation

rhauch
Copy link
Contributor

@rhauch rhauch commented Apr 20, 2011

Added several unit tests that check the existence and validity of I18n constants and messages for all annotations referencing them. These annotations, such as @Label, @description, and @category, reference the I18n messages by string value rather than I18n reference.

Note that a helper method was added to the AbstractI18nTest that is used in each module that uses I18n constants. This allows test methods to be easily added for any object that has such annotations.

Adding these tests pointed out quite a few mismatches between the string values and the I18n message keys. These were corrected, and all unit and integration tests now pass.

…tions

Added several unit tests that check the existence and validity of I18n constants and messages for all annotations referencing them. These annotations, such as @Label, @description, and @category, reference the I18n messages by string value rather than I18n reference.

Note that a helper method was added to the AbstractI18nTest that is used in each module that uses I18n constants. This allows test methods to be easily added for any object that has such annotations.

Adding these tests pointed out quite a few mismatches between the string values and the I18n message keys. These were corrected, and all unit and integration tests now pass.
@rhauch rhauch merged commit da4ce23 into ModeShape:master Apr 20, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant