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

box · Feb 24, 2025 · 5dfd1c5 · 5dfd1c5
1 parent 51d137a
commit 5dfd1c5
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 23 deletions.
diff --git a/.codegen.json b/.codegen.json
@@ -1 +1 @@
-{ "engineHash": "89ce9ce", "specHash": "06fc5f7", "version": "1.8.0" }
+{ "engineHash": "ef00939", "specHash": "06fc5f7", "version": "1.8.0" }
diff --git a/Box.Sdk.Gen.Tests.Integration/Test/LegalHoldPolicies/LegalHoldPoliciesManagerTests.cs b/Box.Sdk.Gen.Tests.Integration/Test/LegalHoldPolicies/LegalHoldPoliciesManagerTests.cs
@@ -12,6 +12,20 @@ public class LegalHoldPoliciesManagerTests {
         public LegalHoldPoliciesManagerTests() {
             client = new CommonsManager().GetDefaultClient();
         }
+        [TestMethod]
+        public async System.Threading.Tasks.Task TestCreateNotOngoingLegalHoldPolicy() {
+            string legalHoldPolicyName = Utils.GetUUID();
+            const string legalHoldDescription = "test description";
+            System.DateTimeOffset filterStartedAt = Utils.DateTimeFromString(dateTime: "2021-01-01T00:00:00-08:00");
+            System.DateTimeOffset filterEndedAt = Utils.DateTimeFromString(dateTime: "2022-01-01T00:00:00-08:00");
+            LegalHoldPolicy legalHoldPolicy = await client.LegalHoldPolicies.CreateLegalHoldPolicyAsync(requestBody: new CreateLegalHoldPolicyRequestBody(policyName: legalHoldPolicyName) { Description = legalHoldDescription, IsOngoing = false, FilterStartedAt = filterStartedAt, FilterEndedAt = filterEndedAt });
+            Assert.IsTrue(legalHoldPolicy.PolicyName == legalHoldPolicyName);
+            Assert.IsTrue(legalHoldPolicy.Description == legalHoldDescription);
+            Assert.IsTrue(Utils.DateTimeToString(dateTime: NullableUtils.Unwrap(legalHoldPolicy.FilterStartedAt)) == Utils.DateTimeToString(dateTime: filterStartedAt));
+            Assert.IsTrue(Utils.DateTimeToString(dateTime: NullableUtils.Unwrap(legalHoldPolicy.FilterEndedAt)) == Utils.DateTimeToString(dateTime: filterEndedAt));
+            await client.LegalHoldPolicies.DeleteLegalHoldPolicyByIdAsync(legalHoldPolicyId: legalHoldPolicy.Id);
+        }
+
         [TestMethod]
         public async System.Threading.Tasks.Task TestCreateUpdateGetDeleteLegalHoldPolicy() {
             string legalHoldPolicyName = Utils.GetUUID();
@@ -30,19 +44,5 @@ public async System.Threading.Tasks.Task TestCreateUpdateGetDeleteLegalHoldPolic
             await client.LegalHoldPolicies.DeleteLegalHoldPolicyByIdAsync(legalHoldPolicyId: legalHoldPolicyId);
         }
 
-        [TestMethod]
-        public async System.Threading.Tasks.Task TestCreateNotOngoingLegalHoldPolicy() {
-            string legalHoldPolicyName = Utils.GetUUID();
-            const string legalHoldDescription = "test description";
-            System.DateTimeOffset filterStartedAt = Utils.DateTimeFromString(dateTime: "2021-01-01T00:00:00-08:00");
-            System.DateTimeOffset filterEndedAt = Utils.DateTimeFromString(dateTime: "2022-01-01T00:00:00-08:00");
-            LegalHoldPolicy legalHoldPolicy = await client.LegalHoldPolicies.CreateLegalHoldPolicyAsync(requestBody: new CreateLegalHoldPolicyRequestBody(policyName: legalHoldPolicyName) { Description = legalHoldDescription, IsOngoing = false, FilterStartedAt = filterStartedAt, FilterEndedAt = filterEndedAt });
-            Assert.IsTrue(legalHoldPolicy.PolicyName == legalHoldPolicyName);
-            Assert.IsTrue(legalHoldPolicy.Description == legalHoldDescription);
-            Assert.IsTrue(Utils.DateTimeToString(dateTime: NullableUtils.Unwrap(legalHoldPolicy.FilterStartedAt)) == Utils.DateTimeToString(dateTime: filterStartedAt));
-            Assert.IsTrue(Utils.DateTimeToString(dateTime: NullableUtils.Unwrap(legalHoldPolicy.FilterEndedAt)) == Utils.DateTimeToString(dateTime: filterEndedAt));
-            await client.LegalHoldPolicies.DeleteLegalHoldPolicyByIdAsync(legalHoldPolicyId: legalHoldPolicy.Id);
-        }
-
     }
 }
diff --git a/Box.Sdk.Gen.Tests.Integration/Test/MetadataTemplates/MetadataTemplatesManagerTests.cs b/Box.Sdk.Gen.Tests.Integration/Test/MetadataTemplates/MetadataTemplatesManagerTests.cs
@@ -18,16 +18,29 @@ public MetadataTemplatesManagerTests() {
         [TestMethod]
         public async System.Threading.Tasks.Task TestMetadataTemplates() {
             string templateKey = string.Concat("key", Utils.GetUUID());
-            MetadataTemplate template = await client.MetadataTemplates.CreateMetadataTemplateAsync(requestBody: new CreateMetadataTemplateRequestBody(scope: "enterprise", displayName: templateKey) { TemplateKey = templateKey, Fields = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.String, key: "testName", displayName: "testName")}) });
+            MetadataTemplate template = await client.MetadataTemplates.CreateMetadataTemplateAsync(requestBody: new CreateMetadataTemplateRequestBody(scope: "enterprise", displayName: templateKey) { TemplateKey = templateKey, Fields = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.String, key: "testName", displayName: "testName"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Float, key: "age", displayName: "age"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Date, key: "birthDate", displayName: "birthDate"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Enum, key: "countryCode", displayName: "countryCode") { Options = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "US"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "CA")}) },new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.MultiSelect, key: "sports", displayName: "sports") { Options = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "basketball"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "football"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "tennis")}) }}) });
             Assert.IsTrue(template.TemplateKey == templateKey);
             Assert.IsTrue(template.DisplayName == templateKey);
-            Assert.IsTrue(NullableUtils.Unwrap(template.Fields).Count == 1);
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields).Count == 5);
             Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[0].Key == "testName");
             Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[0].DisplayName == "testName");
+            Assert.IsTrue(StringUtils.ToStringRepresentation(NullableUtils.Unwrap(template.Fields)[0].Type) == "string");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[1].Key == "age");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[1].DisplayName == "age");
+            Assert.IsTrue(StringUtils.ToStringRepresentation(NullableUtils.Unwrap(template.Fields)[1].Type) == "float");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[2].Key == "birthDate");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[2].DisplayName == "birthDate");
+            Assert.IsTrue(StringUtils.ToStringRepresentation(NullableUtils.Unwrap(template.Fields)[2].Type) == "date");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[3].Key == "countryCode");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[3].DisplayName == "countryCode");
+            Assert.IsTrue(StringUtils.ToStringRepresentation(NullableUtils.Unwrap(template.Fields)[3].Type) == "enum");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[4].Key == "sports");
+            Assert.IsTrue(NullableUtils.Unwrap(template.Fields)[4].DisplayName == "sports");
+            Assert.IsTrue(StringUtils.ToStringRepresentation(NullableUtils.Unwrap(template.Fields)[4].Type) == "multiSelect");
             MetadataTemplate updatedTemplate = await client.MetadataTemplates.UpdateMetadataTemplateAsync(scope: UpdateMetadataTemplateScope.Enterprise, templateKey: templateKey, requestBody: Array.AsReadOnly(new [] {new UpdateMetadataTemplateRequestBody(op: UpdateMetadataTemplateRequestBodyOpField.AddField) { FieldKey = "newfieldname", Data = new Dictionary<string, object>() { { "type", "string" }, { "displayName", "newFieldName" } } }}));
-            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields).Count == 2);
-            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields)[1].Key == "newfieldname");
-            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields)[1].DisplayName == "newFieldName");
+            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields).Count == 6);
+            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields)[5].Key == "newfieldname");
+            Assert.IsTrue(NullableUtils.Unwrap(updatedTemplate.Fields)[5].DisplayName == "newFieldName");
             MetadataTemplate getMetadataTemplate = await client.MetadataTemplates.GetMetadataTemplateByIdAsync(templateId: template.Id);
             Assert.IsTrue(getMetadataTemplate.Id == template.Id);
             MetadataTemplate getMetadataTemplateSchema = await client.MetadataTemplates.GetMetadataTemplateAsync(scope: GetMetadataTemplateScope.Enterprise, templateKey: NullableUtils.Unwrap(template.TemplateKey));

diff --git a/docs/LegalHoldPolicies.md b/docs/LegalHoldPolicies.md
@@ -50,7 +50,7 @@ See the endpoint docs at
 
 <!-- sample post_legal_hold_policies -->
 ```
-await client.LegalHoldPolicies.CreateLegalHoldPolicyAsync(requestBody: new CreateLegalHoldPolicyRequestBody(policyName: legalHoldPolicyName) { Description = legalHoldDescription, IsOngoing = true });
+await client.LegalHoldPolicies.CreateLegalHoldPolicyAsync(requestBody: new CreateLegalHoldPolicyRequestBody(policyName: legalHoldPolicyName) { Description = legalHoldDescription, IsOngoing = false, FilterStartedAt = filterStartedAt, FilterEndedAt = filterEndedAt });
 ```
 
 ### Arguments
@@ -148,7 +148,7 @@ See the endpoint docs at
 
 <!-- sample delete_legal_hold_policies_id -->
 ```
-await client.LegalHoldPolicies.DeleteLegalHoldPolicyByIdAsync(legalHoldPolicyId: legalHoldPolicyId);
+await client.LegalHoldPolicies.DeleteLegalHoldPolicyByIdAsync(legalHoldPolicyId: legalHoldPolicy.Id);
 ```
 
 ### Arguments

diff --git a/docs/MetadataTemplates.md b/docs/MetadataTemplates.md
@@ -266,7 +266,7 @@ See the endpoint docs at
 
 <!-- sample post_metadata_templates_schema -->
 ```
-await client.MetadataTemplates.CreateMetadataTemplateAsync(requestBody: new CreateMetadataTemplateRequestBody(scope: "enterprise", displayName: templateKey) { TemplateKey = templateKey, Fields = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.String, key: "testName", displayName: "testName")}) });
+await client.MetadataTemplates.CreateMetadataTemplateAsync(requestBody: new CreateMetadataTemplateRequestBody(scope: "enterprise", displayName: templateKey) { TemplateKey = templateKey, Fields = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.String, key: "testName", displayName: "testName"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Float, key: "age", displayName: "age"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Date, key: "birthDate", displayName: "birthDate"),new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.Enum, key: "countryCode", displayName: "countryCode") { Options = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "US"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "CA")}) },new CreateMetadataTemplateRequestBodyFieldsField(type: CreateMetadataTemplateRequestBodyFieldsTypeField.MultiSelect, key: "sports", displayName: "sports") { Options = Array.AsReadOnly(new [] {new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "basketball"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "football"),new CreateMetadataTemplateRequestBodyFieldsOptionsField(key: "tennis")}) }}) });
 ```
 
 ### Arguments
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		{ "engineHash": "89ce9ce", "specHash": "06fc5f7", "version": "1.8.0" }
		{ "engineHash": "ef00939", "specHash": "06fc5f7", "version": "1.8.0" }