demisto / demisto-sdk / 9ebfa881-ef08-4120-b757-454934dc755b

81%

push

circleci

Use Content Graph in Format (#3389)

* id_set to graph

* added the flag --graph/--no-graph and removed the id_set flag and everything related to it

* fixed mypy errors

* added unit tests and changed the place of the graph updating to be in only one place

* fixed a mypy error

* fixed a mypy error

* added a function that checks whether the graph is necessary

* added a test and updated the old format tests to have -ngr. updated the BaseContent class to have a name field

* updated the Changelog

* fixed a test

* updated the unit tests I added and removed unnecessary unit tests

* updated test test_remove_non_existent_fields

* removed unnecessary test

* fixed cr notes

* fixed failing tests and changed graph.search to use path

* fixed cr notes and e2e unit tests

* fixed the last e2e unit test

* added import os in format_integration_test.py

* moved the unit tests with the graph to demisto_sdk/commands/content_graph/tests

* created OLDEST_INCIDENT_FIELD_SUPPORTED_VERSION and reverted the change in OLDEST_SUPPORTED_VERSION, and updated the tests accordingly

* pre-commit changes

* removed unnecessary test

* moved format_with_graph_test.py to the correct directory

* working E2E graph tests

* added -nv flag to the E2E unit tests

* pre-commit changes

* updated in the code after the merge from master

* pre-commit changes

* added mocker.patch to update_content_graph()

* added checks on the object that returns from graph.search()

* fixed cr comments

* pre-commit changes

* added back the deletion of the kwargs[graph] when needed

87 of 87 new or added lines in 8 files covered. (100.0%)

31222 of 38646 relevant lines covered (80.79%)

2.42 hits per line