mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-05-31 21:08:27 +01:00
Fix xUnit1030: Do not call ConfigureAwait in test method
This commit is contained in:
@@ -41,10 +41,10 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
{
|
||||
var client = _factory.CreateClient();
|
||||
|
||||
using var response = await client.GetAsync("Users/Public").ConfigureAwait(false);
|
||||
using var response = await client.GetAsync("Users/Public");
|
||||
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||
var users = await JsonSerializer.DeserializeAsync<UserDto[]>(
|
||||
await response.Content.ReadAsStreamAsync().ConfigureAwait(false), _jsonOpions).ConfigureAwait(false);
|
||||
await response.Content.ReadAsStreamAsync(), _jsonOpions);
|
||||
// User are hidden by default
|
||||
Assert.NotNull(users);
|
||||
Assert.Empty(users);
|
||||
@@ -55,12 +55,12 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
public async Task GetUsers_Valid_Success()
|
||||
{
|
||||
var client = _factory.CreateClient();
|
||||
client.DefaultRequestHeaders.AddAuthHeader(_accessToken ??= await AuthHelper.CompleteStartupAsync(client).ConfigureAwait(false));
|
||||
client.DefaultRequestHeaders.AddAuthHeader(_accessToken ??= await AuthHelper.CompleteStartupAsync(client));
|
||||
|
||||
using var response = await client.GetAsync("Users").ConfigureAwait(false);
|
||||
using var response = await client.GetAsync("Users");
|
||||
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||
var users = await JsonSerializer.DeserializeAsync<UserDto[]>(
|
||||
await response.Content.ReadAsStreamAsync().ConfigureAwait(false), _jsonOpions).ConfigureAwait(false);
|
||||
await response.Content.ReadAsStreamAsync(), _jsonOpions);
|
||||
Assert.NotNull(users);
|
||||
Assert.Single(users);
|
||||
Assert.False(users![0].HasConfiguredPassword);
|
||||
@@ -71,9 +71,9 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
public async Task Me_Valid_Success()
|
||||
{
|
||||
var client = _factory.CreateClient();
|
||||
client.DefaultRequestHeaders.AddAuthHeader(_accessToken ??= await AuthHelper.CompleteStartupAsync(client).ConfigureAwait(false));
|
||||
client.DefaultRequestHeaders.AddAuthHeader(_accessToken ??= await AuthHelper.CompleteStartupAsync(client));
|
||||
|
||||
_ = await AuthHelper.GetUserDtoAsync(client).ConfigureAwait(false);
|
||||
_ = await AuthHelper.GetUserDtoAsync(client);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
@@ -90,10 +90,10 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
Name = TestUsername
|
||||
};
|
||||
|
||||
using var response = await CreateUserByName(client, createRequest).ConfigureAwait(false);
|
||||
using var response = await CreateUserByName(client, createRequest);
|
||||
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||
var user = await JsonSerializer.DeserializeAsync<UserDto>(
|
||||
await response.Content.ReadAsStreamAsync().ConfigureAwait(false), _jsonOpions).ConfigureAwait(false);
|
||||
await response.Content.ReadAsStreamAsync(), _jsonOpions);
|
||||
Assert.Equal(TestUsername, user!.Name);
|
||||
Assert.False(user.HasPassword);
|
||||
Assert.False(user.HasConfiguredPassword);
|
||||
@@ -121,7 +121,7 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
Name = username!
|
||||
};
|
||||
|
||||
using var response = await CreateUserByName(client, createRequest).ConfigureAwait(false);
|
||||
using var response = await CreateUserByName(client, createRequest);
|
||||
Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
|
||||
}
|
||||
|
||||
@@ -134,7 +134,7 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
// access token can't be null here as the previous test populated it
|
||||
client.DefaultRequestHeaders.AddAuthHeader(_accessToken!);
|
||||
|
||||
using var response = await client.DeleteAsync($"User/{Guid.NewGuid()}").ConfigureAwait(false);
|
||||
using var response = await client.DeleteAsync($"User/{Guid.NewGuid()}");
|
||||
Assert.Equal(HttpStatusCode.NotFound, response.StatusCode);
|
||||
}
|
||||
|
||||
@@ -150,11 +150,11 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
NewPw = "4randomPa$$word"
|
||||
};
|
||||
|
||||
using var response = await UpdateUserPassword(client, _testUserId, createRequest).ConfigureAwait(false);
|
||||
using var response = await UpdateUserPassword(client, _testUserId, createRequest);
|
||||
Assert.Equal(HttpStatusCode.NoContent, response.StatusCode);
|
||||
|
||||
var users = await JsonSerializer.DeserializeAsync<UserDto[]>(
|
||||
await client.GetStreamAsync("Users").ConfigureAwait(false), _jsonOpions).ConfigureAwait(false);
|
||||
await client.GetStreamAsync("Users"), _jsonOpions);
|
||||
var user = users!.First(x => x.Id.Equals(_testUserId));
|
||||
Assert.True(user.HasPassword);
|
||||
Assert.True(user.HasConfiguredPassword);
|
||||
@@ -173,11 +173,11 @@ namespace Jellyfin.Server.Integration.Tests.Controllers
|
||||
CurrentPw = "4randomPa$$word",
|
||||
};
|
||||
|
||||
using var response = await UpdateUserPassword(client, _testUserId, createRequest).ConfigureAwait(false);
|
||||
using var response = await UpdateUserPassword(client, _testUserId, createRequest);
|
||||
Assert.Equal(HttpStatusCode.NoContent, response.StatusCode);
|
||||
|
||||
var users = await JsonSerializer.DeserializeAsync<UserDto[]>(
|
||||
await client.GetStreamAsync("Users").ConfigureAwait(false), _jsonOpions).ConfigureAwait(false);
|
||||
await client.GetStreamAsync("Users"), _jsonOpions);
|
||||
var user = users!.First(x => x.Id.Equals(_testUserId));
|
||||
Assert.False(user.HasPassword);
|
||||
Assert.False(user.HasConfiguredPassword);
|
||||
|
||||
Reference in New Issue
Block a user