Commit Graph

738 Commits

Author SHA1 Message Date
Dan Funk e043a5ff63
Merge pull request #38 from sartography/feature/investigators_reference_file
Adding a new reference file that provides greater details about the i…
2020-05-12 10:01:48 -04:00
Dan Funk e3573bb7df Really just flailing now as I try to get tests to pass in Jenkins when they all work just fine on my box. 2020-05-12 08:48:26 -04:00
Dan Funk 99503fb832 missed a file in the last commit. 2020-05-11 17:42:24 -04:00
Dan Funk b7c11fd893 Merge branch 'master' into feature/investigators_reference_file 2020-05-11 17:36:37 -04:00
Dan Funk 420d4af5b5 Merge branch 'master' into feature/investigators_reference_file 2020-05-11 17:10:57 -04:00
Dan Funk 182162540a
Merge pull request #42 from sartography/feature/new_script_engine
Updated to use the latest script engine / evaluation engine that crea…
2020-05-11 17:09:54 -04:00
Dan Funk 02f8764056 Updated to use the latest script engine / evaluation engine that creates a single location where all values used in BPMN/DMN are processed. Right now this is a python based interpreter, but we will eventually base this on FEEL expressions.
The validation process needs to take the api model into account so we catch errors with bad file names.
2020-05-11 17:04:05 -04:00
Aaron Louie 02118800ca Prints out LDAP URL 2020-05-11 16:22:42 -04:00
Aaron Louie d8f7b6cc9f Adds dev, demo, and training branches 2020-05-11 12:05:16 -04:00
Aaron Louie 27d8e0b51b
Merge pull request #41 from sartography/staging
Staging
2020-05-08 20:37:46 -04:00
Aaron Louie b27cad8233
Merge pull request #40 from sartography/testing
Testing
2020-05-08 18:09:36 -04:00
Aaron Louie 9cb1a23a28
Merge pull request #39 from sartography/chore/github_branches_to_docker_tags
Chore/GitHub branches to docker tags
2020-05-07 16:47:45 -04:00
Dan Funk c34000d369
Merge branch 'master' into feature/investigators_reference_file 2020-05-07 16:47:24 -04:00
Aaron Louie a1ad11c07d Cleans up debugging stuff 2020-05-07 14:27:32 -04:00
Aaron Louie 06f14daa86 Fixes commit substring syntax 2020-05-07 14:24:15 -04:00
Aaron Louie fceda2c796 Tries defining commit in deploy script 2020-05-07 14:14:25 -04:00
Dan Funk da7cae51b8 Adding a new reference file that provides greater details about the investigators related to a study.
Improving the study_info script documentation to provide detailed examples of values returned based on arguments.
Making the tests a little more targetted and less subject to breaking through better mocks.
Allow all tests to pass even when ther protocol builder mock isn't running locally.
Removing the duplication of reference files in tests and static, as this seems silly to me at the moment.
2020-05-07 13:57:24 -04:00
Aaron Louie 9af91bed8b Printing out some stuff for debugging 2020-05-07 13:49:17 -04:00
Aaron Louie 7463664ae0
Merge pull request #37 from sartography/chore/github_branches_to_docker_tags
Chore/GitHub branches to docker tags
2020-05-07 11:43:04 -04:00
Aaron Louie 5c510c5a9a Moves PB_BASE_URL into matrix env var 2020-05-07 10:56:09 -04:00
Aaron Louie 559494aee2 Fails build if any docker or aws command errors out 2020-05-07 10:47:24 -04:00
Aaron Louie 9ba00c1320 Moves commit var back up to global 2020-05-07 10:42:19 -04:00
Aaron Louie f4e293e8fa
Merge pull request #36 from sartography/chore/github_branches_to_docker_tags
Builds and deploys Docker image if build passes on testing, staging, …
2020-05-07 10:35:37 -04:00
Aaron Louie 26251a199f Moves commit variable into deploy script 2020-05-07 10:25:06 -04:00
Aaron Louie 827e840103 Builds and deploys Docker image if build passes on testing, staging, or master branches 2020-05-07 10:15:14 -04:00
Aaron Louie 9afcd753c4 No longer requires task_id 2020-05-06 14:27:28 -04:00
Dan Funk 1eb52ed268 Removing duplicate migration. 2020-05-06 13:28:41 -04:00
Dan Funk 1571986c0e I had to give up and live with the idea that we can only render documentation on the current task, not on the previous or next tasks. I think this is ok. If you want to view a task, you need to make it the active task to assure all the parts and pieces are in place. 2020-05-06 13:01:38 -04:00
Dan Funk 8ded625c7d Merge remote-tracking branch 'origin/chore/update_specs' into feature/previous_task
# Conflicts:
#	Pipfile.lock

Assuring that all documents from the xls spreadsheet are loaded when doing validations.
Fixing some failed tests.
2020-05-06 11:46:19 -04:00
Dan Funk 07e58e923d Merge remote-tracking branch 'origin/chore/update_specs' into feature/previous_task
# Conflicts:
#	Pipfile.lock

Assuring that all documents from the xls spreadsheet are loaded when doing validations.
2020-05-06 11:25:50 -04:00
Dan Funk 9629b36e92 Setting JSON_SORT_KEYS to false, assuring that Flask does not resort all data returned to the front end.
Updating Spiff Workflow which has some critical behavioral changes around MultiInstance.
2020-05-06 10:59:49 -04:00
Aaron Louie 6c36a32411 Adds DSP 2020-05-06 10:37:43 -04:00
Aaron Louie a8827d78c4 Fixes typo 2020-05-05 20:49:07 -04:00
Aaron Louie 4ecb0cb3a3 Updates BPMN files 2020-05-05 16:15:38 -04:00
Dan Funk ce76b0d3e8 Merge branch 'feature/protocol_status' into feature/previous_task 2020-05-04 11:12:12 -04:00
Dan Funk 714b5f3be0 Merge branch 'feature/protocol_status' into feature/previous_task
# Conflicts:
#	crc/services/study_service.py
2020-05-04 11:08:36 -04:00
Dan Funk 2699f5c65c Refactor the stats models, and assure they are very correct across all tests with the workflow api.
I noticed we were saving the workflow every time we loaded it up, rather than only when we were making changes to it.  Refactored this to be a little more careful.
Centralized the saving of the workflow into one location in the processor, so we can make sure we update all the details about that workflow every time we save.
The workflow service has a method that will log any task action taken in a consistent way.
The stats models were removed from the API completely.  Will wait for a use case for dealing with this later.
2020-05-04 10:57:09 -04:00
Aaron Louie 28d5727851 Merge branch 'master' into feature/protocol_status 2020-05-01 22:55:59 -04:00
Aaron Louie 098cf10926 Fixes Jinja code to match new structure of StudyInfo documents 2020-05-01 22:49:52 -04:00
Aaron Louie 6a96ccffa2 Updates pipenv hashes. Fixes typo in SpiffWorkflow file name 2020-05-01 21:54:01 -04:00
Dan Funk 1f5002680a Initial work on a "Previous" task. 2020-05-01 12:11:39 -04:00
Dan Funk 7c743d65ed Adding tests around parallel which pass great, but ran into a bug down in Spiff that I'm passing back to Kelly with a test in that repo. More coming on this. 2020-04-30 15:39:11 -04:00
Dan Funk bec59a71d7 Deleteing stuff is a damn mess, but this is a little cleaner. 2020-04-29 16:07:39 -04:00
Dan Funk f1f8b91c9c Refactor the document details scripts. Now there is one script, it returns data in a consistent format, and has all the details required. The script is located in StudyInfo, with the argument documents. Make note that it returns a dictionary of ALL the documents, with a field to mark which ones are required according to the protocol builder. Others may become required if a workflow determines such, in which case the workflow will enforce this, and the document will have a count > 0, and additional details in a list of files within the document. I modified the XLS file to use lower case variable names, because it disturbed me, and we have to reference them frequently. Removed devious "as_object" variable on get_required_docs, so it behaves like the other methods all the time, and returns a dictionary. All the core business logic for finding the documents list now resides in the StudyService.
Because this changes the endpoint for all existing document details, I've modified all the test and static bpmn files to use the new format.
Shorting up the SponsorsList.xls file makes for slightly faster tests. seems senseless to load 5000 everytime we reset the data.
Tried to test all of this carefully in the test_study_details_documents.py test.
2020-04-29 15:08:11 -04:00
Aaron Louie beb86f0453 Adds protocol script to study service 2020-04-29 10:21:24 -04:00
Dan Funk 13333fb306 Adding an API endpoint that allows setting the token on the workflow to a specific task.
Added error checking such that attempting to submit data for a task that is not in the "READY" state throws an error message.
For some reason I'm getting lots of errors in the tests as they try to hit API endpoints they were not hitting before, so adding a number of mocks to some of the study service tests.
2020-04-28 17:25:53 -04:00
Aaron Louie 5e30e20b67 Updates pipenv lock file 2020-04-28 15:13:39 -04:00
Dan Funk 0088364b1d Merge branch 'master' of github.com:sartography/cr-connect-workflow 2020-04-28 13:48:52 -04:00
Dan Funk 3e3a249e3c Verifying Sub-Process works, and adding a field to expose a hint as to the sub-process in which the task occurs.
Because the name field is now used to expose workflow/sub-process information on tasks, we can't use it to store the workflow_version, so that is now just stored on the database model.  Which is much cleaner and removes a duplication.
2020-04-28 13:48:44 -04:00
Aaron Louie 663a1a00a8 Fixes file name so it matches workflow spec ID 2020-04-28 11:50:10 -04:00