test: reorder function calls to fix test samples (box/box-codegen#666) (#492)

box-sdk-build · web-flow · commit 91ffc008ca64 · 2025-03-07T14:39:12.000+01:00
diff --git a/.codegen.json b/.codegen.json
@@ -1 +1 @@
-{ "engineHash": "5c674a3", "specHash": "137a375", "version": "1.12.0" }
+{ "engineHash": "ef00939", "specHash": "06fc5f7", "version": "1.12.0" }
diff --git a/docs/legal_hold_policies.md b/docs/legal_hold_policies.md
@@ -54,7 +54,11 @@ See the endpoint docs at
 
 ```python
 client.legal_hold_policies.create_legal_hold_policy(
-    legal_hold_policy_name, description=legal_hold_description, is_ongoing=True
+    legal_hold_policy_name,
+    description=legal_hold_description,
+    filter_started_at=filter_started_at,
+    filter_ended_at=filter_ended_at,
+    is_ongoing=False,
 )
 ```
 
@@ -158,7 +162,7 @@ See the endpoint docs at
 <!-- sample delete_legal_hold_policies_id -->
 
 ```python
-client.legal_hold_policies.delete_legal_hold_policy_by_id(legal_hold_policy_id)
+client.legal_hold_policies.delete_legal_hold_policy_by_id(legal_hold_policy.id)
 ```
 
 ### Arguments
diff --git a/docs/metadata_templates.md b/docs/metadata_templates.md
@@ -278,7 +278,36 @@ client.metadata_templates.create_metadata_template(
             type=CreateMetadataTemplateFieldsTypeField.STRING,
             key="testName",
             display_name="testName",
-        )
+        ),
+        CreateMetadataTemplateFields(
+            type=CreateMetadataTemplateFieldsTypeField.FLOAT,
+            key="age",
+            display_name="age",
+        ),
+        CreateMetadataTemplateFields(
+            type=CreateMetadataTemplateFieldsTypeField.DATE,
+            key="birthDate",
+            display_name="birthDate",
+        ),
+        CreateMetadataTemplateFields(
+            type=CreateMetadataTemplateFieldsTypeField.ENUM,
+            key="countryCode",
+            display_name="countryCode",
+            options=[
+                CreateMetadataTemplateFieldsOptionsField(key="US"),
+                CreateMetadataTemplateFieldsOptionsField(key="CA"),
+            ],
+        ),
+        CreateMetadataTemplateFields(
+            type=CreateMetadataTemplateFieldsTypeField.MULTISELECT,
+            key="sports",
+            display_name="sports",
+            options=[
+                CreateMetadataTemplateFieldsOptionsField(key="basketball"),
+                CreateMetadataTemplateFieldsOptionsField(key="football"),
+                CreateMetadataTemplateFieldsOptionsField(key="tennis"),
+            ],
+        ),
     ],
 )
 ```
diff --git a/docs/search.md b/docs/search.md
@@ -75,9 +75,23 @@ See the endpoint docs at
 
 ```python
 client.search.search_for_content(
-    query=keyword,
     ancestor_folder_ids=["0"],
-    trash_content=SearchForContentTrashContent.NON_TRASHED_ONLY,
+    mdfilters=[
+        MetadataFilter(
+            filters={
+                "stringField": "stringValue",
+                "dateField": MetadataFieldFilterDateRange(
+                    lt=date_time_from_string("2035-01-01T00:00:00Z"),
+                    gt=date_time_from_string("2035-01-03T00:00:00Z"),
+                ),
+                "floatField": MetadataFieldFilterFloatRange(lt=9.5, gt=10.5),
+                "enumField": "enumValue2",
+                "multiSelectField": ["multiSelectValue1", "multiSelectValue2"],
+            },
+            scope=MetadataFilterScopeField.ENTERPRISE,
+            template_key=template_key,
+        )
+    ],
 )
 ```
 
diff --git a/test/legal_hold_policies.py b/test/legal_hold_policies.py
@@ -1,9 +1,9 @@
+from box_sdk_gen.internal.utils import DateTime
+
 from box_sdk_gen.schemas.legal_hold_policy import LegalHoldPolicy
 
 from box_sdk_gen.schemas.legal_hold_policies import LegalHoldPolicies
 
-from box_sdk_gen.internal.utils import DateTime
-
 from box_sdk_gen.internal.utils import get_uuid
 
 from box_sdk_gen.internal.utils import date_time_from_string
@@ -17,6 +17,31 @@
 client: BoxClient = get_default_client()
 
 
+def testCreateNotOngoingLegalHoldPolicy():
+    legal_hold_policy_name: str = get_uuid()
+    legal_hold_description: str = 'test description'
+    filter_started_at: DateTime = date_time_from_string('2021-01-01T00:00:00-08:00')
+    filter_ended_at: DateTime = date_time_from_string('2022-01-01T00:00:00-08:00')
+    legal_hold_policy: LegalHoldPolicy = (
+        client.legal_hold_policies.create_legal_hold_policy(
+            legal_hold_policy_name,
+            description=legal_hold_description,
+            filter_started_at=filter_started_at,
+            filter_ended_at=filter_ended_at,
+            is_ongoing=False,
+        )
+    )
+    assert legal_hold_policy.policy_name == legal_hold_policy_name
+    assert legal_hold_policy.description == legal_hold_description
+    assert date_time_to_string(
+        legal_hold_policy.filter_started_at
+    ) == date_time_to_string(filter_started_at)
+    assert date_time_to_string(
+        legal_hold_policy.filter_ended_at
+    ) == date_time_to_string(filter_ended_at)
+    client.legal_hold_policies.delete_legal_hold_policy_by_id(legal_hold_policy.id)
+
+
 def testCreateUpdateGetDeleteLegalHoldPolicy():
     legal_hold_policy_name: str = get_uuid()
     legal_hold_description: str = 'test description'
@@ -44,28 +69,3 @@ def testCreateUpdateGetDeleteLegalHoldPolicy():
     )
     assert updated_legal_hold_policy.policy_name == updated_legal_hold_policy_name
     client.legal_hold_policies.delete_legal_hold_policy_by_id(legal_hold_policy_id)
-
-
-def testCreateNotOngoingLegalHoldPolicy():
-    legal_hold_policy_name: str = get_uuid()
-    legal_hold_description: str = 'test description'
-    filter_started_at: DateTime = date_time_from_string('2021-01-01T00:00:00-08:00')
-    filter_ended_at: DateTime = date_time_from_string('2022-01-01T00:00:00-08:00')
-    legal_hold_policy: LegalHoldPolicy = (
-        client.legal_hold_policies.create_legal_hold_policy(
-            legal_hold_policy_name,
-            description=legal_hold_description,
-            filter_started_at=filter_started_at,
-            filter_ended_at=filter_ended_at,
-            is_ongoing=False,
-        )
-    )
-    assert legal_hold_policy.policy_name == legal_hold_policy_name
-    assert legal_hold_policy.description == legal_hold_description
-    assert date_time_to_string(
-        legal_hold_policy.filter_started_at
-    ) == date_time_to_string(filter_started_at)
-    assert date_time_to_string(
-        legal_hold_policy.filter_ended_at
-    ) == date_time_to_string(filter_ended_at)
-    client.legal_hold_policies.delete_legal_hold_policy_by_id(legal_hold_policy.id)
diff --git a/test/metadata_templates.py b/test/metadata_templates.py
@@ -1,3 +1,5 @@
+from box_sdk_gen.internal.utils import to_string
+
 import pytest
 
 from box_sdk_gen.client import BoxClient
@@ -10,6 +12,10 @@
     CreateMetadataTemplateFieldsTypeField,
 )
 
+from box_sdk_gen.managers.metadata_templates import (
+    CreateMetadataTemplateFieldsOptionsField,
+)
+
 from box_sdk_gen.managers.metadata_templates import UpdateMetadataTemplateScope
 
 from box_sdk_gen.managers.metadata_templates import UpdateMetadataTemplateRequestBody
@@ -52,14 +58,56 @@ def testMetadataTemplates():
                 type=CreateMetadataTemplateFieldsTypeField.STRING,
                 key='testName',
                 display_name='testName',
-            )
+            ),
+            CreateMetadataTemplateFields(
+                type=CreateMetadataTemplateFieldsTypeField.FLOAT,
+                key='age',
+                display_name='age',
+            ),
+            CreateMetadataTemplateFields(
+                type=CreateMetadataTemplateFieldsTypeField.DATE,
+                key='birthDate',
+                display_name='birthDate',
+            ),
+            CreateMetadataTemplateFields(
+                type=CreateMetadataTemplateFieldsTypeField.ENUM,
+                key='countryCode',
+                display_name='countryCode',
+                options=[
+                    CreateMetadataTemplateFieldsOptionsField(key='US'),
+                    CreateMetadataTemplateFieldsOptionsField(key='CA'),
+                ],
+            ),
+            CreateMetadataTemplateFields(
+                type=CreateMetadataTemplateFieldsTypeField.MULTISELECT,
+                key='sports',
+                display_name='sports',
+                options=[
+                    CreateMetadataTemplateFieldsOptionsField(key='basketball'),
+                    CreateMetadataTemplateFieldsOptionsField(key='football'),
+                    CreateMetadataTemplateFieldsOptionsField(key='tennis'),
+                ],
+            ),
         ],
     )
     assert template.template_key == template_key
     assert template.display_name == template_key
-    assert len(template.fields) == 1
+    assert len(template.fields) == 5
     assert template.fields[0].key == 'testName'
     assert template.fields[0].display_name == 'testName'
+    assert to_string(template.fields[0].type) == 'string'
+    assert template.fields[1].key == 'age'
+    assert template.fields[1].display_name == 'age'
+    assert to_string(template.fields[1].type) == 'float'
+    assert template.fields[2].key == 'birthDate'
+    assert template.fields[2].display_name == 'birthDate'
+    assert to_string(template.fields[2].type) == 'date'
+    assert template.fields[3].key == 'countryCode'
+    assert template.fields[3].display_name == 'countryCode'
+    assert to_string(template.fields[3].type) == 'enum'
+    assert template.fields[4].key == 'sports'
+    assert template.fields[4].display_name == 'sports'
+    assert to_string(template.fields[4].type) == 'multiSelect'
     updated_template: MetadataTemplate = (
         client.metadata_templates.update_metadata_template(
             UpdateMetadataTemplateScope.ENTERPRISE,
@@ -73,9 +121,9 @@ def testMetadataTemplates():
             ],
         )
     )
-    assert len(updated_template.fields) == 2
-    assert updated_template.fields[1].key == 'newfieldname'
-    assert updated_template.fields[1].display_name == 'newFieldName'
+    assert len(updated_template.fields) == 6
+    assert updated_template.fields[5].key == 'newfieldname'
+    assert updated_template.fields[5].display_name == 'newFieldName'
     get_metadata_template: MetadataTemplate = (
         client.metadata_templates.get_metadata_template_by_id(template.id)
     )
diff --git a/test/search.py b/test/search.py
@@ -38,12 +38,12 @@
     SearchResultsWithSharedLinks,
 )
 
-from box_sdk_gen.managers.search import SearchForContentTrashContent
-
 from box_sdk_gen.schemas.metadata_filter import MetadataFilter
 
 from box_sdk_gen.schemas.metadata_filter import MetadataFilterScopeField
 
+from box_sdk_gen.managers.search import SearchForContentTrashContent
+
 from box_sdk_gen.internal.utils import get_uuid
 
 from box_sdk_gen.internal.utils import generate_byte_stream
@@ -151,29 +151,6 @@ def testCreateMetaDataQueryExecuteRead():
     client.files.delete_file_by_id(file.id)
 
 
-def testGetSearch():
-    keyword: str = 'test'
-    search: Union[SearchResults, SearchResultsWithSharedLinks] = (
-        client.search.search_for_content(
-            query=keyword,
-            ancestor_folder_ids=['0'],
-            trash_content=SearchForContentTrashContent.NON_TRASHED_ONLY,
-        )
-    )
-    assert len(search.entries) >= 0
-    assert to_string(search.type) == 'search_results_items'
-    search_with_shared_link: Union[SearchResults, SearchResultsWithSharedLinks] = (
-        client.search.search_for_content(
-            query=keyword,
-            ancestor_folder_ids=['0'],
-            trash_content=SearchForContentTrashContent.NON_TRASHED_ONLY,
-            include_recent_shared_links=True,
-        )
-    )
-    assert len(search_with_shared_link.entries) >= 0
-    assert to_string(search_with_shared_link.type) == 'search_results_with_shared_links'
-
-
 def testMetadataFilters():
     template_key: str = ''.join(['key', get_uuid()])
     template: MetadataTemplate = client.metadata_templates.create_metadata_template(
@@ -261,3 +238,26 @@ def testMetadataFilters():
         DeleteMetadataTemplateScope.ENTERPRISE, template.template_key
     )
     client.files.delete_file_by_id(file.id)
+
+
+def testGetSearch():
+    keyword: str = 'test'
+    search: Union[SearchResults, SearchResultsWithSharedLinks] = (
+        client.search.search_for_content(
+            query=keyword,
+            ancestor_folder_ids=['0'],
+            trash_content=SearchForContentTrashContent.NON_TRASHED_ONLY,
+        )
+    )
+    assert len(search.entries) >= 0
+    assert to_string(search.type) == 'search_results_items'
+    search_with_shared_link: Union[SearchResults, SearchResultsWithSharedLinks] = (
+        client.search.search_for_content(
+            query=keyword,
+            ancestor_folder_ids=['0'],
+            trash_content=SearchForContentTrashContent.NON_TRASHED_ONLY,
+            include_recent_shared_links=True,
+        )
+    )
+    assert len(search_with_shared_link.entries) >= 0
+    assert to_string(search_with_shared_link.type) == 'search_results_with_shared_links'

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-{ "engineHash": "5c674a3", "specHash": "137a375", "version": "1.12.0" }`
	`1`	`+{ "engineHash": "ef00939", "specHash": "06fc5f7", "version": "1.12.0" }`