Skip to content

Commit

Permalink
Merge branch 'main' of https://github.com/microsoft/promptflow into d…
Browse files Browse the repository at this point in the history
…evs/peiwen/add_error_handling
  • Loading branch information
PeiwenGaoMS committed Dec 7, 2023
2 parents ed7ac82 + eb48933 commit d50020d
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -304,8 +304,8 @@ def log_event(*args, **kwargs):
# device name removed
assert "ai.cloud.roleInstance" not in envelope.tags
assert "ai.device.id" not in envelope.tags
# role not scrubbed for test scenario
assert envelope.tags["ai.cloud.role"] == os.path.basename(sys.argv[0])
# role name should be scrubbed or kept in whitelist
assert envelope.tags["ai.cloud.role"] in [os.path.basename(sys.argv[0]), "***"]

with patch.object(PromptFlowSDKLogHandler, "emit") as mock_logger:
mock_logger.side_effect = log_event
Expand Down

0 comments on commit d50020d

Please sign in to comment.