-
Notifications
You must be signed in to change notification settings - Fork 2
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
Person properties plugin unit tests #6
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rties-plugin-unit-tests
fixed spelling error in method name
Added missing unit tests for person properties plugin |
bischoffz
added a commit
that referenced
this pull request
Mar 8, 2023
shawnhatch
pushed a commit
that referenced
this pull request
Mar 10, 2023
* initial impl of proj dir * initial working input for global properties * update to address maven warnings * remove prototype proto * make interface for common methods * rename proto * stochastics proto * clean up * update poms to work more cohesively * stochastics plugin parsing complete * slight refactor of getting a value * clean up prototype classes * fix project warning * add interface for translator builders * implement translatorBuilder interface clean up adding default plugin descriptors * parser should ignore unknown fields by default * allow to change behavior towards unknown fields and default values * fix runtime error * significant refactor * clean up imports * refactor to use context * refactor to add parsing into lifecycle * add a parser context * update input json to match output json * add support for writing output refactor * small change to exceptions * implement focalBundle * focal bundle * initial impl of new design for bundle * fix compile errors * convert global propertier to new bundle * convert stochastics bundle to new design * remove depricated code * remove depricated code * add dependency support add plugin bundle id remove depricated code * reorginization pass #1 * reorganization pass #2 * reorganization pass #3 * reorganization pass #4 * reorganization pass #5 * reorganization pass #6 * fix ide error... again add suppress to generated files * rename proto package * add people plugin * change dir name * reorganization pass #7 * add groups-plugin * fix bugs * remove un-needed addition of resource * update paths to be relative * code clean up * add support for enum within an any add support for marker interfaces * reorganize translators to be more in line with how a modler would use them * add maven plugin to supproess warning in generated proto files * update stochastics to use new design * fix bug slight refactor * add translator for timetrackignpolicy and update propdef translator to use it update path in propertiesPlugin * refactor/reorg of master translator methods * add back additions to map that caused propertiesPlugin to fail rename write method * refactor global properties to use new design add globalPropId translator * refactor to aid in actual reuse of PropDefMapInput and PropValMapInput * remove translator that is no longer needed * add globalpropIdinput * change to better reflect actual use cases * convert global properties to use standard plugindata for test * convert people plugin to add check in app
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PersonPropertyInteractionReport
PersonPropertyReport
PersonPropertyReport$Builder
PersonPropertyDefinitionEvent
PersonPropertyDefinitionInitialization
PersonPropertyDefinitionInitialization$Builder
PersonPropertyInitialization
PersonPropertyLabeler
PropertyFilter
PersonPropertiesActionSupport
TestAuxiliaryPersonPropertyId
TestPersonPropertyId