Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mauriciowittenberg
Copy link
Contributor

@mauriciowittenberg mauriciowittenberg commented Sep 22, 2023

unpack security_definitions inside goblet.handlers.routes.OpenApiSpec

make securityDefinitions in openapispec tests GConfig instances test write spec to yaml

add test to write securityDefinitions to yaml

closes #437

make securityDefinitions in openapispec tests GConfig instances
test write spec to yaml
@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02% 🎉

Comparison is base (2c29389) 89.47% compared to head (3de66c8) 89.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #438      +/-   ##
==========================================
+ Coverage   89.47%   89.50%   +0.02%     
==========================================
  Files          62       62              
  Lines        5589     5601      +12     
==========================================
+ Hits         5001     5013      +12     
  Misses        588      588              
Files Changed Coverage Δ
goblet/handlers/routes.py 93.44% <100.00%> (+0.02%) ⬆️
goblet/tests/test_openapispec.py 95.27% <100.00%> (+0.21%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@anovis anovis merged commit e3cc83f into goblet:main Sep 22, 2023
quajones pushed a commit to quajones/goblet that referenced this pull request Oct 19, 2023
unpack security_definitions inside goblet.handlers.routes.OpenApiSpec

make securityDefinitions in openapispec tests GConfig instances test
write spec to yaml

add test to write securityDefinitions to yaml
@mauriciowittenberg mauriciowittenberg deleted the fix/openapi_spec-securityDefinitions branch October 23, 2023 23:52
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

Successfully merging this pull request may close these issues.

[Openapispec] - error creating spec

2 participants