finos / legend-studio

Legend Studio
https://legend.finos.org
Apache License 2.0
87 stars 114 forks source link

Support permissionScheme and usageRole in SnowflakeApp #3345

Closed YannanGao-gs closed 1 month ago

YannanGao-gs commented 2 months ago

Summary

Support permission_scope and usage in SnowflakeApp

related to https://github.com/finos/legend-engine/pull/2965

How did you test this change?

https://github.com/user-attachments/assets/eafd2cbb-2dc3-497c-ace8-242036ab1a20

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 4df2bede7b02c96a40bd27f70fae839f840e65d1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 26 packages | Name | Type | | --------------------------------------------- | ----- | | @finos/legend-application-studio | Patch | | @finos/legend-graph | Patch | | @finos/legend-application-studio-bootstrap | Patch | | @finos/legend-extension-assortment | Patch | | @finos/legend-extension-dsl-data-quality | Patch | | @finos/legend-extension-dsl-data-space-studio | Patch | | @finos/legend-extension-dsl-diagram | Patch | | @finos/legend-extension-dsl-persistence | Patch | | @finos/legend-extension-dsl-service | Patch | | @finos/legend-extension-dsl-text | Patch | | @finos/legend-extension-store-flat-data | Patch | | @finos/legend-extension-store-relational | Patch | | @finos/legend-extension-store-service-store | Patch | | @finos/legend-vscode-extension-dependencies | Patch | | @finos/legend-application-pure-ide | Patch | | @finos/legend-application-query-bootstrap | Patch | | @finos/legend-application-query | Patch | | @finos/legend-application-repl | Patch | | @finos/legend-extension-dsl-data-space | Patch | | @finos/legend-lego | Patch | | @finos/legend-query-builder | Patch | | @finos/legend-application-studio-deployment | Patch | | @finos/legend-application-pure-ide-deployment | Patch | | @finos/legend-application-query-deployment | Patch | | @finos/legend-application-repl-deployment | Patch | | @finos/legend-server-showcase-deployment | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 58.49057% with 44 lines in your changes missing coverage. Please review.

Project coverage is 46.12%. Comparing base (8653469) to head (4df2bed). Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3345 +/- ## ========================================== - Coverage 46.21% 46.12% -0.09% ========================================== Files 2089 2089 Lines 357956 358140 +184 Branches 10900 10846 -54 ========================================== - Hits 165428 165204 -224 - Misses 191896 192304 +408 Partials 632 632 ``` | [Files](https://app.codecov.io/gh/finos/legend-studio/pull/3345?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos) | Coverage Δ | | |---|---|---| | [...rializationHelpers/V1\_DomainSerializationHelper.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureProtocol%2FserializationHelpers%2FV1_DomainSerializationHelper.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZVByb3RvY29sL3NlcmlhbGl6YXRpb25IZWxwZXJzL1YxX0RvbWFpblNlcmlhbGl6YXRpb25IZWxwZXIudHM=) | `100.00% <100.00%> (ø)` | | | [...ger/action/changeDetection/DomainObserverHelper.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Faction%2FchangeDetection%2FDomainObserverHelper.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL2FjdGlvbi9jaGFuZ2VEZXRlY3Rpb24vRG9tYWluT2JzZXJ2ZXJIZWxwZXIudHM=) | `36.33% <0.00%> (-0.12%)` | :arrow_down: | | [...el/packageableElements/function/V1\_SnowflakeApp.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Fmodel%2FpackageableElements%2Ffunction%2FV1_SnowflakeApp.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvbW9kZWwvcGFja2FnZWFibGVFbGVtZW50cy9mdW5jdGlvbi9WMV9Tbm93Zmxha2VBcHAudHM=) | `50.00% <20.00%> (-3.34%)` | :arrow_down: | | [.../pure/packageableElements/function/SnowflakeApp.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph%2Fmetamodel%2Fpure%2FpackageableElements%2Ffunction%2FSnowflakeApp.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC9tZXRhbW9kZWwvcHVyZS9wYWNrYWdlYWJsZUVsZW1lbnRzL2Z1bmN0aW9uL1Nub3dmbGFrZUFwcC50cw==) | `53.84% <55.55%> (+0.35%)` | :arrow_up: | | [packages/legend-graph/src/index.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Findex.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9pbmRleC50cw==) | `0.00% <0.00%> (ø)` | | | [...pureGraph/from/V1\_PackageableElementTransformer.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureGraph%2Ffrom%2FV1_PackageableElementTransformer.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZUdyYXBoL2Zyb20vVjFfUGFja2FnZWFibGVFbGVtZW50VHJhbnNmb3JtZXIudHM=) | `69.84% <0.00%> (-1.32%)` | :arrow_down: | | [...-activator/SnowflakeAppFunctionActivatorEditor.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-application-studio%2Fsrc%2Fcomponents%2Feditor%2Feditor-group%2Ffunction-activator%2FSnowflakeAppFunctionActivatorEditor.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXN0dWRpby9zcmMvY29tcG9uZW50cy9lZGl0b3IvZWRpdG9yLWdyb3VwL2Z1bmN0aW9uLWFjdGl2YXRvci9Tbm93Zmxha2VBcHBGdW5jdGlvbkFjdGl2YXRvckVkaXRvci50c3g=) | `89.75% <86.44%> (-0.71%)` | :arrow_down: | | [...ivator/SnowflakeAppFunctionActivatorEditorState.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-application-studio%2Fsrc%2Fstores%2Feditor%2Feditor-state%2Felement-editor-state%2Ffunction-activator%2FSnowflakeAppFunctionActivatorEditorState.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXN0dWRpby9zcmMvc3RvcmVzL2VkaXRvci9lZGl0b3Itc3RhdGUvZWxlbWVudC1lZGl0b3Itc3RhdGUvZnVuY3Rpb24tYWN0aXZhdG9yL1Nub3dmbGFrZUFwcEZ1bmN0aW9uQWN0aXZhdG9yRWRpdG9yU3RhdGUudHM=) | `49.30% <27.27%> (-1.83%)` | :arrow_down: | | [...rmation/pureGraph/to/V1\_ElementFirstPassBuilder.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3345?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureGraph%2Fto%2FV1_ElementFirstPassBuilder.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZUdyYXBoL3RvL1YxX0VsZW1lbnRGaXJzdFBhc3NCdWlsZGVyLnRz) | `77.19% <0.00%> (-0.89%)` | :arrow_down: | ... and [52 files with indirect coverage changes](https://app.codecov.io/gh/finos/legend-studio/pull/3345/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos)