744
68%

Ran 17 Feb 2017 04:36PM UTC

Jobs 1

Files 162

Run time 9s

Badge

Embed ▾

pending completion

Build # 744

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Delete deprecated Account fields (#547) ✂️️

Delete deprecated Account fields

- Replaces references to Account.number with Account.identifier.
- Looks like the d08d0b37788a_.py migration copied account.identifier over to account.number.
- With the deprecated fields gone, the `aws_account.py` (`AWSAccountManager` class)  no longer needs to override `_populate_account()` or `lookup_account_by_identifier()`
- Replace any references to `Account.s3_name` or `Account.role_nam`e with code that checks the custom tags. (If there are queries filtering on these fields, may need to create a sqlalchemy hybrid property/method.)
- Replace `security_monkey.common.untils::add_account()` with `account_manager.create()`. (See `manage.py::add_account()`) https://github.com/Netflix/security_monkey/blob/develop/security_monkey/common/utils.py#L104
- Create a new DB migration:
-- Migration should copy over `Account.s3_name` to new custom tag field
-- Migration should copy over `Account.role_name` to new custom tag field
-- Migration should remove `Account.s3_name` and `Account.role_name` and `Account.number` fields
- Tests pass and travis builds.

Run Details

4733 of 8949 relevant lines covered (52.89%)

0.53 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	744.1	17 Feb 2017 04:36PM UTC	162	52.89	Travis Job 744.1

Source Files on build 744

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

Netflix / security_monkey / 744
68%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 744

Netflix / security_monkey / 744 68%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 744

Netflix / security_monkey / 744
68%

README BADGES
x