-
Notifications
You must be signed in to change notification settings - Fork 417
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
chore: Replace parsing function for saving granted object names #2813
Conversation
71a09fd
to
b5a1a69
Compare
Integration tests failure for 71a09fd6163e97f561cc69ae6172329e76568a8d |
Integration tests failure for b5a1a69a533b5c62773b49fcb60773d9bb5a27ab |
fc1dc0b
to
2d5e8d9
Compare
Integration tests failure for 4a8b235ca07bf0f38556dff2d363b96602fb35ca |
Integration tests failure for 8c947f1d294cc30f8d1e4d73062ffde50c4be53e |
Integration tests failure for fc1dc0bd48a257c21cd73379215a26b2cfc8accf |
Integration tests failure for 2d5e8d999d4b610a197ba0f8764a897528513a79 |
@@ -511,7 +515,7 @@ func TestAcc_GrantOwnership_OnFuture_InSchema_ToAccountRole(t *testing.T) { | |||
In: &sdk.ShowGrantsIn{ | |||
Schema: sdk.Pointer(sdk.NewDatabaseObjectIdentifier(databaseName, schemaName)), | |||
}, | |||
}, sdk.ObjectTypeTable, accountRoleName, fmt.Sprintf("%s.%s.<TABLE>", databaseName, schemaName)), | |||
}, sdk.ObjectTypeTable, accountRoleName, fmt.Sprintf("\"%s\".\"%s\".\"<TABLE>\"", databaseName, schemaName)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: use ` for readability
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
Integration tests cancelled for ed4b2a8d7830a122fc3b65381d3d5016d36ebee7 |
Integration tests cancelled for 31c635fa2ce64e6bdddc32edc6c5b213666e323f |
🤖 I have created a release *beep* *boop* --- ## [0.91.0](v0.90.0...v0.91.0) (2024-05-16) ### 🎉 **What's new:** * add snowflake_grant_application_role resource ([#2690](#2690)) ([838d241](838d241)) * datasource database role ([#2731](#2731)) ([319ddc3](319ddc3)) ### 🔧 **Misc** * Add few documentation adjustments ([#2789](#2789)) ([6db8bf3](6db8bf3)) * Bump dependencies ([#2802](#2802)) ([54ea6bc](54ea6bc)) * Replace parsing function for saving granted object names ([#2813](#2813)) ([175cfc7](175cfc7)) ### 🐛 **Bug fixes:** * Fix sweepers ([#2800](#2800)) ([a01115e](a01115e)) * Invoke SetId immediately after alert creation ([#2786](#2786)) ([181b4d0](181b4d0)) * Update the tests after snowflake bugfix ([#2806](#2806)) ([6843c1a](6843c1a)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
Add better parsing of identifiers returned by SHOW GRANTS. It should resolve some of our issues with failing tests (they sometimes fail because they depend on the generated name of a resource and whether they will be quoted in the output provided by Snowflake).
New
ParseObjectIdentifier
function is based onDecodeSnowflakeParameterID
function used in resources. As a fallback (becauseParseObjectIdentifier
can fail)NewObjectIdentifierFromFullyQualifiedName
will be used.