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
fixed a bug in service now #21955
fixed a bug in service now #21955
Conversation
Merge remote-tracking branch 'origin/master' into omer_servicenow_mirroring_bug
Merge remote-tracking branch 'origin/master' into omer_servicenow_mirroring_bug
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool
|
||
@pytest.mark.parametrize('ticket_state, expected_res', [('1', 'Other'), | ||
('7', 'Resolved')]) | ||
def test_converts_state_close_reason(ticket_state, expected_res): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add docstrings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@users.noreply.github.com>
Merge remote-tracking branch 'origin/master' into omer_servicenow_mirroring_bug
Merge branch 'omer_servicenow_mirroring_bug' of github.com:demisto/content into omer_servicenow_mirroring_bug
@@ -6,7 +6,8 @@ | |||
|
|||
|
|||
# disable insecure warnings | |||
requests.packages.urllib3.disable_warnings() | |||
import urllib3 | |||
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you add the specific exception?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's just change it to:
urllib3.disable_warnings()
Co-authored-by: Shelly Tzohar <45915502+Shellyber@users.noreply.github.com>
Link to the unit tests coverage report: |
Status
Related Issues
fixes: link to the issue
Description
There was a bug that closedNotes was in the place of closedReason and the there was no mirror in for the closedReson fixed the 2 issues.
Screenshots
Paste here any images that will help the reviewer
Minimum version of Cortex XSOAR
Does it break backward compatibility?
Must have