Skip to content

Commit

Permalink
Updating Insights class to remove fields which are not available thro…
Browse files Browse the repository at this point in the history
…ugh the API

Summary: Some of the fields are not exposed through the Insights edge any longer. This diff removes those fields from the insights class.

Test Plan:
Integration tests and Unit tests pass
ritu-mbp:python-ads-sdk ritu$ python -m facebookads.test.unit
............................
----------------------------------------------------------------------
Ran 28 tests in 0.013s

OK
ritu-mbp:python-ads-sdk ritu$
ritu-mbp:python-ads-sdk ritu$ python3 -m facebookads.test.unit
............................
----------------------------------------------------------------------
Ran 28 tests in 0.016s

OK
ritu-mbp:python-ads-sdk ritu$ python3 -m facebookads.test.integration
....................
----------------------------------------------------------------------
Ran 20 tests in 52.802s

OK
ritu-mbp:python-ads-sdk ritu$ python -m facebookads.test.integration
....................
----------------------------------------------------------------------
Ran 20 tests in 43.585s

OK
  • Loading branch information
Rituparna Mukherjee committed Oct 8, 2015
1 parent 79daf6c commit ec6c9d2
Show file tree
Hide file tree
Showing 2 changed files with 54 additions and 56 deletions.
Loading

0 comments on commit ec6c9d2

Please sign in to comment.