Skip to content

Trust basic acc test#1818

Merged
jtopjian merged 4 commits intogophercloud:masterfrom
jtopjian:trust-basic-acc-test
Jan 15, 2020
Merged

Trust basic acc test#1818
jtopjian merged 4 commits intogophercloud:masterfrom
jtopjian:trust-basic-acc-test

Conversation

@jtopjian
Copy link
Copy Markdown
Contributor

@jtopjian jtopjian commented Jan 13, 2020

For #1642
Supersedes #1645

/cc @ozerovandrei

ozerovandrei and others added 4 commits January 13, 2020 02:06
Implement Create and Extract functions for v3/extenstions/trusts package
with unit tests and docs.

This commit also adds Role struct to this package.
Add TestTrustCRUD acceptance test with some helpers to create and delete
trusts.
Remove required tag from the "impersonation" argument since it's boolean
so we can just don't set "omitempty" tag to always send it in the Create
request.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 77.077% when pulling d074f75 on jtopjian:trust-basic-acc-test into 4f96abf on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jan 13, 2020

Build succeeded.

@jtopjian jtopjian requested a review from ozerovandrei January 13, 2020 03:57
Copy link
Copy Markdown
Contributor

@ozerovandrei ozerovandrei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jtopjian jtopjian merged commit 7daa96f into gophercloud:master Jan 15, 2020
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.

3 participants