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

Add check for test data key exists #141

Closed
rkl-xc opened this issue Sep 18, 2020 · 2 comments
Closed

Add check for test data key exists #141

rkl-xc opened this issue Sep 18, 2020 · 2 comments
Assignees
@occupant23
Labels
codeDone The issue is rechecked for development branch codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop feature Low Priority
Milestone
v4.1.0

Comments

@rkl-xc
Copy link
Contributor

rkl-xc commented Sep 18, 2020

For data driven automated tests it would be useful to have a DataUtils.exist("key") like method, that checks whether a key is defined in the dataset or not. As a workaround this works with Neodymium-4.0.0: while (DataUtils.asString("key", null) != null) { do stuff }
@occupant23 occupant23 added this to the v4.1.0 milestone Sep 24, 2020
@occupant23 occupant23 self-assigned this Sep 24, 2020
@occupant23 occupant23 added docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop feature Low Priority labels Sep 24, 2020
occupant23 pushed a commit that referenced this issue Sep 24, 2020
#141 - Add check for test data key exists
2e24878
@occupant23 occupant23 added the codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there label Sep 24, 2020
@occupant23
Copy link
Contributor

Code and documentation are ready for test.

@oomelianchuk
Copy link
Contributor

IMO the changes are good, I have nothing to complain about

@occupant23 occupant23 added the codeDone The issue is rechecked for development branch label Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@occupant23 occupant23

Labels
codeDone The issue is rechecked for development branch codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop feature Low Priority
Projects
None yet
Milestone
v4.1.0
Development

No branches or pull requests
3 participants
@occupant23 @oomelianchuk @rkl-xc