Prepare 2.0.0-beta.12 release (Part 3) (#221)

Improvements to test reliability
openai · Sep 20, 2024 · bbac88f · bbac88f
1 parent d9bc167
commit bbac88f
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/tests/Assistants/Assistants.VectorStoresTests.cs b/tests/Assistants/Assistants.VectorStoresTests.cs
@@ -18,7 +18,6 @@ namespace OpenAI.Tests.VectorStores;
 
 [TestFixture(true)]
 [TestFixture(false)]
-[Parallelizable(ParallelScope.Fixtures)]
 [Category("Assistants")]
 public class VectorStoresTests : SyncAsyncTestBase
 {

diff --git a/tests/Assistants/AssistantsSmokeTests.cs b/tests/Assistants/AssistantsSmokeTests.cs
@@ -7,7 +7,6 @@ namespace OpenAI.Tests.Assistants;
 
 #pragma warning disable OPENAI001
 
-[Parallelizable(ParallelScope.Fixtures)]
 [Category("Assistants")]
 [Category("Smoke")]
 public class AssistantsSmokeTests

diff --git a/tests/Assistants/AssistantsTests.cs b/tests/Assistants/AssistantsTests.cs
@@ -20,7 +20,6 @@ namespace OpenAI.Tests.Assistants;
 
 [TestFixture(true)]
 [TestFixture(false)]
-[Parallelizable(ParallelScope.Fixtures)]
 [Category("Assistants")]
 public class AssistantsTests : SyncAsyncTestBase
 {

diff --git a/tests/Models/ModelsTests.cs b/tests/Models/ModelsTests.cs
@@ -92,7 +92,9 @@ public void DeleteModelCanParseServiceError()
             ex = Assert.Throws<ClientResultException>(() => client.DeleteModel("fake_id"));
         }
 
-        Assert.That(ex.Status, Is.EqualTo(403));
+        // If the model exists but the user doesn't own it, the service returns 403.
+        // If the model doesn't exist at all, the service returns 404.
+        Assert.That(ex.Status, Is.EqualTo(404));
     }
 
     [Test]