Sage-Bionetworks / data_curator

Data and metadata ingress app
Apache License 2.0
10 stars 21 forks source link

Test schematic authentication in shinyapps deploy #412

Open afwillia opened 1 year ago

afwillia commented 1 year ago

Is your feature request related to a problem? Please describe. When deploying DCA to shinyapps.io, issues with schematic authentication may not be discovered until users interact with specific elements of the app. Testing the schematic calls that the app makes before deployment will ensure authentication works properly before release. This was brought up after launching DCA for VEOIBD. The app launched, but google sheets would not generate because .synapseConfig had bad credentials.

Describe the solution you'd like In the shinyapps deployment workflow, add test calls to schematic that require authentication. If they fail, abort the deployment and return the error. In particular this will help ensure our secrets are configured correctly and have not expired.

How important is this feature? Select from the options below: • 🏝 Low - it's an enhancement but not crucial for work

milen-sage commented 1 year ago

Note this is using local REST API for testing.