Skip to content

[FSSDK-11458] Python - Add SDK Multi-Region Support for Data Hosting #459

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.

Sign up for GitHub

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

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

[FSSDK-11458] Python - Add SDK Multi-Region Support for Data Hosting #459

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
d9e8f83
[FSSDK-11458] Python - Add SDK Multi-Region Support for Data Hosting
esrakartalOpt Jul 2, 2025
0866962
Fix lint issues
esrakartalOpt Jul 2, 2025
5a0e19d
Fix lint issue
esrakartalOpt Jul 2, 2025
730fede
Fix errors
esrakartalOpt Jul 2, 2025
47ccb94
Add region on base test
esrakartalOpt Jul 2, 2025
a7d74ac
Update Region as enum
esrakartalOpt Jul 2, 2025
8a65e0f
Delete blank lines
esrakartalOpt Jul 2, 2025
24d2fcd
Fix lint issue
esrakartalOpt Jul 2, 2025
ba5beee
Fix lint issue
esrakartalOpt Jul 2, 2025
17c9280
Fix test errors
esrakartalOpt Jul 2, 2025
155ab25
Add enum
esrakartalOpt Jul 2, 2025
ba049b6
Fix region value
esrakartalOpt Jul 2, 2025
a777fa0
Fix type issue
esrakartalOpt Jul 2, 2025
c1f1693
Correct the event url
esrakartalOpt Jul 2, 2025
6de0889
Add region in params
esrakartalOpt Jul 2, 2025
83afabe
Fix region param
esrakartalOpt Jul 2, 2025
158aab3
Add region to create
esrakartalOpt Jul 2, 2025
0a0cdef
Fix test cases
esrakartalOpt Jul 2, 2025
f68d678
Fix test cases
esrakartalOpt Jul 2, 2025
81cd042
Fix lint
esrakartalOpt Jul 2, 2025
9d2c892
Add region
esrakartalOpt Jul 2, 2025
3c1332f
Fix test cases
esrakartalOpt Jul 4, 2025
11523cf
Fix tests
esrakartalOpt Jul 4, 2025
1af68db
Cast to string
esrakartalOpt Jul 4, 2025
7a09f1d
Fix tests
esrakartalOpt Jul 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Correct the event url
  • Loading branch information
@esrakartalOpt
esrakartalOpt committed Jul 2, 2025
commit c1f16938914c669ece074276356d82ba0d2b07ef
35 changes: 21 additions & 14 deletions tests/test_event_builder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ def test_create_impression_event(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -155,7 +155,7 @@ def test_create_impression_event__with_attributes(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -208,7 +208,7 @@ def test_create_impression_event_when_attribute_is_not_in_datafile(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -280,7 +280,7 @@ def side_effect(*args, **kwargs):

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -345,7 +345,7 @@ def test_create_impression_event__with_user_agent_when_bot_filtering_is_enabled(

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -409,7 +409,7 @@ def test_create_impression_event__with_empty_attributes_when_bot_filtering_is_en

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -479,7 +479,7 @@ def test_create_impression_event__with_user_agent_when_bot_filtering_is_disabled

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -515,6 +515,7 @@ def test_create_conversion_event(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -525,7 +526,7 @@ def test_create_conversion_event(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -563,6 +564,7 @@ def test_create_conversion_event__with_attributes(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -573,7 +575,7 @@ def test_create_conversion_event__with_attributes(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -618,6 +620,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_enabled(
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -631,7 +634,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_enabled(

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -681,6 +684,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_disabled
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -694,7 +698,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_disabled

self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -735,6 +739,7 @@ def test_create_conversion_event__with_event_tags(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -749,7 +754,7 @@ def test_create_conversion_event__with_event_tags(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -788,6 +793,7 @@ def test_create_conversion_event__with_invalid_event_tags(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -802,7 +808,7 @@ def test_create_conversion_event__with_invalid_event_tags(self):
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down Expand Up @@ -843,6 +849,7 @@ def test_create_conversion_event__when_event_is_used_in_multiple_experiments(sel
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand All @@ -857,7 +864,7 @@ def test_create_conversion_event__when_event_is_used_in_multiple_experiments(sel
)
self._validate_event_object(
event_obj,
event_builder.EventBuilder.EVENTS_URL,
event_builder.EventBuilder.EVENTS_URLS.get('US'),
expected_params,
event_builder.EventBuilder.HTTP_VERB,
event_builder.EventBuilder.HTTP_HEADERS,
Expand Down
7 changes: 7 additions & 0 deletions tests/test_event_factory.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -582,6 +582,7 @@ def test_create_conversion_event(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -629,6 +630,7 @@ def test_create_conversion_event__with_attributes(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -683,6 +685,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_enabled(
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -744,6 +747,7 @@ def test_create_conversion_event__with_user_agent_when_bot_filtering_is_disabled
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -796,6 +800,7 @@ def test_create_conversion_event__with_event_tags(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -848,6 +853,7 @@ def test_create_conversion_event__with_invalid_event_tags(self):
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down Expand Up @@ -902,6 +908,7 @@ def test_create_conversion_event__when_event_is_used_in_multiple_experiments(sel
'enrich_decisions': True,
'anonymize_ip': False,
'revision': '42',
'region': 'US',
}

with mock.patch('time.time', return_value=42.123), mock.patch(
Expand Down
Loading