Skip to content
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

Organizations support #1727

Merged
merged 17 commits into from
Oct 15, 2018
Merged

Conversation

ashleygould
Copy link
Contributor

This patch implements about 30% of the AWS Organizations endpoints. It covers basics for organizational units and accounts. No support for service control policy yet.

@coveralls
Copy link

coveralls commented Jul 20, 2018

Coverage Status

Coverage decreased (-0.05%) to 92.507% when pulling 7a88e63 on ashleygould:organizations_support into dfa7935 on spulec:master.

@corrjo
Copy link
Contributor

corrjo commented Jul 29, 2018

Exciting!

@spulec
Copy link
Collaborator

spulec commented Sep 22, 2018

This is looking great. If we can get rebased against master and Travis passing, will be happy to merge.

endpoints covers so far:
- create_organization
- describe_organization
- create_account
- describe_account
- list_accounts

all tests passing.
could use some advise from maintaners.
create_organizational_unit
describe_organizational_unit
list_organizational_units_for_parents
list_parents
list_accounts_for_parent
move_account
account_id -> id
account_name -> name
refactered classes: FakeRoot inherits from FakeOrganizationsUnit
add root_id attribute to class FakeOrganization
dropped 'roots' attribute from class OrganizationaBackend
ready for pull request
did a little cleanup refactoring
local tests pass
added exception handling in class OrganizationsBackend
@spulec spulec merged commit 6d14911 into getmoto:master Oct 15, 2018
@spulec
Copy link
Collaborator

spulec commented Oct 15, 2018

This is awesome. Thanks for all the work and welcome to Moto!

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.

4 participants