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

suggestion #1 #1

Open
rwu2359 opened this issue Aug 15, 2017 · 5 comments
Open

suggestion #1 #1

rwu2359 opened this issue Aug 15, 2017 · 5 comments

Comments

@rwu2359
Copy link

rwu2359 commented Aug 15, 2017

In a project I am currently working on, there is a need for some validation data to be included in the header, so that the first part of documentation or the actual script can be quickly scanned.

so we are using a ValidateDate, and also because the types of validation required is dependant on a risk analysis matrix, we are also adding Complexity - which if course FMPerception gives us very nicely..

the in-house rule then is that if the validation date is completed the date is removed, and the script change can not be put into production until they have been tested and validated...

@toddgeist
Copy link
Contributor

how would we implement that?

@rwu2359
Copy link
Author

rwu2359 commented Aug 15, 2017

/# @validationDate
/# @validationComplexity
/# @validationTests

am keen to push further on test-driven development than I have done till now. It should be possible to create a file whose sole purpose was to test many scripts by sending both good and bad data parameters - an extension of what you have done with the CustomFunctions file... just musing out loud.

@rwu2359
Copy link
Author

rwu2359 commented Aug 16, 2017

other thing that would be good in here is
/# @dependantCustom
/# @dependantExternal

for both CustomFunctions and any external (plugin dependencies) ??
again consistency when search with FMPerception could highlight all the scripts you need to test if you have change a CF - just becasue the function works in the tests file doesnt mean you ahve dealt with the Law of Unintended Consequences

@toddgeist
Copy link
Contributor

jsdoc already has @requires. Isn't that good enough?

@rwu2359
Copy link
Author

rwu2359 commented Aug 16, 2017

Yes, for some reason I didnt see that when reading the docs...... :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants