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.

Sign up for GitHub

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

Jump to bottom

review Lookup Implementations to make it more flexible #19

Closed
caelcs opened this issue Mar 18, 2019 · 1 comment
Closed

review Lookup Implementations to make it more flexible #19

caelcs opened this issue Mar 18, 2019 · 1 comment
Milestone
VERSION-0.2.0

Comments

@caelcs
Copy link
Owner

caelcs commented Mar 18, 2019

No description provided.

@caelcs
Copy link
Owner Author

caelcs commented Mar 19, 2019

  • Removing RandomLookIp
  • DefaultLookUp should not use Optional as part of the map definition.
  • Simplify the logic to use Multiple sources for the value, Classes or Type, Supplier and Function.
  • Implement a extendible solution for new ways to generate values.

caelcs added a commit that referenced this issue Mar 19, 2019
@caelcs
[#19] DefaultLookUp support multiple types of source to resolve the v…
4ca28ff 
…alue and removed the randomLookup.

[#20] Refactor to remove dependency on Fyodor and replaced by Supplier.
[#21] Changes in the DSL to have separate methods for override based on the type.
@caelcs caelcs mentioned this issue Mar 19, 2019
Merged
caelcs pushed a commit that referenced this issue Mar 20, 2019
Refactor to give a nice shape to be extended for future requirements (#…
5abecd0 
…24)

[#19] DefaultLookUp support multiple types of source to resolve the value and removed the randomLookup.
[#20] Refactor to remove dependency on Fyodor and replaced by Supplier.
[#21] Changes in the DSL to have separate methods for override based on the type.

updating README
@caelcs caelcs added this to the VERSION-0.2.0 milestone Mar 20, 2019
@caelcs caelcs closed this as completed Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
VERSION-0.2.0
Development

No branches or pull requests
1 participant
@caelcs