Skip to content

chore: Delete deprecated employee endpoint tests #650

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

Merged
the-chris-mitchell merged 1 commit into from
Apr 30, 2026
+0 −42
Merged

chore: Delete deprecated employee endpoint tests #650

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
42 changes: 0 additions & 42 deletions Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@
};
string idempotencyKey = AutoFaker.Generate<string>();
bool? summarizeErrors = false;
var response = await instance.CreateBankTransactionsAsync(accessToken, xeroTenantId, bankTransactions, summarizeErrors, null, idempotencyKey ).ConfigureAwait(false);

Check warning on line 174 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<BankTransactions>(response);
}

Expand Down Expand Up @@ -226,7 +226,7 @@
Reference = "SUB 098801"
}
};
var response = await instance.CreateBankTransferAsync(accessToken, xeroTenantId, bankTransfers).ConfigureAwait(false);

Check warning on line 229 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<BankTransfers>(response);
}

Expand Down Expand Up @@ -324,7 +324,7 @@
}
};
bool? summarizeErrors = false;
var response = await instance.CreateBatchPaymentAsync(accessToken, xeroTenantId, batchPayments, summarizeErrors).ConfigureAwait(false);

Check warning on line 327 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<BatchPayments>(response);
}

Expand Down Expand Up @@ -498,20 +498,6 @@
Assert.IsType<Currencies>(response);
}

/// <summary>
/// Test CreateEmployees
/// </summary>
[Fact]
public async Task CreateEmployeesTest()
{
// TODO uncomment below to test the method and replace null with proper value
string xeroTenantId = AutoFaker.Generate<string>();
Employees employees = new Employees();
employees._Employees = new List<Employee> { new Employee() };
var response = await instance.CreateEmployeesAsync(accessToken, xeroTenantId, employees);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test CreateExpenseClaimHistory
/// </summary>
Expand Down Expand Up @@ -672,7 +658,7 @@
Date = DateTime.Now,
},
};
var response = await instance.CreateManualJournalsAsync(accessToken, xeroTenantId, manualJournals).ConfigureAwait(false);

Check warning on line 661 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<ManualJournals>(response);
}

Expand Down Expand Up @@ -1035,7 +1021,7 @@
{
Status = "DELETED",
};
var response = await instance.DeletePaymentAsync(accessToken, xeroTenantId, paymentID, paymentDelete).ConfigureAwait(false);

Check warning on line 1024 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<Payments>(response);
}

Expand Down Expand Up @@ -1455,7 +1441,7 @@
// TODO uncomment below to test the method and replace null with proper value
string xeroTenantId = AutoFaker.Generate<string>();
Guid contactID = Guid.NewGuid();
var response = await instance.GetContactCISSettingsAsync(accessToken, xeroTenantId, contactID).ConfigureAwait(false);

Check warning on line 1444 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<CISSettings>(response);
}

Expand Down Expand Up @@ -1630,34 +1616,6 @@
Assert.IsType<Currencies>(response);
}

/// <summary>
/// Test GetEmployee
/// </summary>
[Fact]
public async Task GetEmployeeTest()
{
// TODO uncomment below to test the method and replace null with proper value
string xeroTenantId = AutoFaker.Generate<string>();
Guid employeeID = AutoFaker.Generate<Guid>();
var response = await instance.GetEmployeeAsync(accessToken, xeroTenantId, employeeID);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test GetEmployees
/// </summary>
[Fact]
public async Task GetEmployeesTest()
{
// TODO uncomment below to test the method and replace null with proper value
string xeroTenantId = AutoFaker.Generate<string>();
DateTime? ifModifiedSince = AutoFaker.Generate<DateTime?>();
string where = AutoFaker.Generate<string>();
string order = AutoFaker.Generate<string>();
var response = await instance.GetEmployeesAsync(accessToken, xeroTenantId, ifModifiedSince, where, order);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test GetExpenseClaim
/// </summary>
Expand Down Expand Up @@ -1708,7 +1666,7 @@
// TODO uncomment below to test the method and replace null with proper value
string xeroTenantId = AutoFaker.Generate<string>();
Guid invoiceID = AutoFaker.Generate<Guid>();
var response = await instance.GetInvoiceAsync(accessToken, xeroTenantId, invoiceID).ConfigureAwait(false);

Check warning on line 1669 in Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs

View workflow job for this annotation

GitHub Actions / build-test-lint 

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.IsType<List<Invoice>>(response._Invoices);
}

Expand Down
Loading