Skip to content
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

Jump to bottom

PETOSS-575 Fix Linting Issues in Dotnet SDK: Fix Formatting #568

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

PETOSS-575 Fix Linting Issues in Dotnet SDK: Fix Formatting #568

Show file tree
Hide file tree
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
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
6 changes: 3 additions & 3 deletions .github/workflows/build-test-lint.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ jobs:
dotnet list package --outdated
working-directory: Xero-NetStandard

# - name: Validate Lint
# run: dotnet format --verify-no-changes
# working-directory: Xero-NetStandard
- name: Validate Lint
run: dotnet format --verify-no-changes
working-directory: Xero-NetStandard

# - name: Run Test
# run: dotnet test
Expand Down
5,458 changes: 2,729 additions & 2,729 deletions Xero.NetStandard.OAuth2.Test/Api/AccountingApiTests.cs
View file
Open in desktop

Large diffs are not rendered by default.

20 changes: 10 additions & 10 deletions Xero.NetStandard.OAuth2.Test/Api/BankFeedsApiTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public BankFeedsApiTests(ITestOutputHelper output)
}

// Methods are async but may not require it in specific cases
#pragma warning disable CS1998 // This async method lacks 'await' operators
#pragma warning disable CS1998 // This async method lacks 'await' operators
public async Task InitializeAsync()
{
var directory = Directory.GetCurrentDirectory();
Expand All @@ -59,7 +59,7 @@ public async Task DisposeAsync()
{
// Cleanup when everything is done.
}
#pragma warning restore CS1998 // This async method lacks 'await' operators
#pragma warning restore CS1998 // This async method lacks 'await' operators

/// <summary>
/// Test an instance of BankFeedsApi
Expand All @@ -71,7 +71,7 @@ public void InstanceTest()
//Assert.IsType(typeof(BankFeedsApi), instance, "instance is a BankFeedsApi");
}


/// <summary>
/// Test CreateFeedConnections
/// </summary>
Expand All @@ -82,7 +82,7 @@ public async Task CreateFeedConnectionsTest()
var response = await instance.CreateFeedConnectionsAsync(accessToken, xeroTenantId, feedConnections);
Assert.IsType<FeedConnections>(response);
}

/// <summary>
/// Test CreateStatements
/// </summary>
Expand All @@ -94,7 +94,7 @@ public async Task CreateStatementsTest()
var response = await instance.CreateStatementsAsync(accessToken, xeroTenantId, statements, idempotencyKey);
Assert.IsType<Statements>(response);
}

/// <summary>
/// Test DeleteFeedConnections
/// </summary>
Expand All @@ -105,7 +105,7 @@ public async Task DeleteFeedConnectionsTest()
var response = await instance.DeleteFeedConnectionsAsync(accessToken, xeroTenantId, feedConnections);
Assert.IsType<FeedConnections>(response);
}

/// <summary>
/// Test GetFeedConnection
/// </summary>
Expand All @@ -116,7 +116,7 @@ public async Task GetFeedConnectionTest()
var response = await instance.GetFeedConnectionAsync(accessToken, xeroTenantId, id);
Assert.IsType<FeedConnection>(response);
}

/// <summary>
/// Test GetFeedConnections
/// </summary>
Expand All @@ -128,7 +128,7 @@ public async Task GetFeedConnectionsTest()
var response = await instance.GetFeedConnectionsAsync(accessToken, xeroTenantId, page, pageSize);
Assert.IsType<FeedConnections>(response);
}

/// <summary>
/// Test GetStatement
/// </summary>
Expand All @@ -139,7 +139,7 @@ public async Task GetStatementTest()
var response = await instance.GetStatementAsync(accessToken, xeroTenantId, statementId);
Assert.IsType<Statement>(response);
}

/// <summary>
/// Test GetStatements
/// </summary>
Expand All @@ -153,7 +153,7 @@ public async Task GetStatementsTest()
var response = await instance.GetStatementsAsync(accessToken, xeroTenantId, page, pageSize, xeroApplicationId, xeroUserId);
Assert.IsType<Statements>(response);
}

}

}
64 changes: 32 additions & 32 deletions Xero.NetStandard.OAuth2.Test/Api/PayrollAuApiTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ public PayrollAuApiTests()
}

// Methods are async but may not require it in specific cases
#pragma warning disable CS1998 // This async method lacks 'await' operators
#pragma warning disable CS1998 // This async method lacks 'await' operators
public async Task InitializeAsync()
{
var directory = Directory.GetCurrentDirectory();
Expand All @@ -58,7 +58,7 @@ public async Task DisposeAsync()
{
// Cleanup when everything is done.
}
#pragma warning restore CS1998 // This async method lacks 'await' operators
#pragma warning restore CS1998 // This async method lacks 'await' operators

/// <summary>
/// Test an instance of PayrollAuApi
Expand All @@ -69,7 +69,7 @@ public void InstanceTest()
Assert.IsType<PayrollAuApi>(instance);
}


/// <summary>
/// Test CreateEmployee
/// </summary>
Expand All @@ -85,7 +85,7 @@ public async Task CreateEmployeeTest()
var response = await instance.CreateEmployeeAsync(accessToken, xeroTenantId, employees);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test CreateLeaveApplication
/// </summary>
Expand All @@ -97,7 +97,7 @@ public async Task CreateLeaveApplicationTest()
var response = await instance.CreateLeaveApplicationAsync(accessToken, xeroTenantId, leaveApplication);
Assert.IsType<LeaveApplications>(response);
}

/// <summary>
/// Test CreatePayItem
/// </summary>
Expand All @@ -108,7 +108,7 @@ public async Task CreatePayItemTest()
var response = await instance.CreatePayItemAsync(accessToken, xeroTenantId, payItem);
Assert.IsType<PayItems>(response);
}

/// <summary>
/// Test CreatePayRun
/// </summary>
Expand All @@ -122,7 +122,7 @@ public async Task CreatePayRunTest()
var response = await instance.CreatePayRunAsync(accessToken, xeroTenantId, payRun);
Assert.IsType<PayRuns>(response);
}

/// <summary>
/// Test CreatePayrollCalendar
/// </summary>
Expand All @@ -133,7 +133,7 @@ public async Task CreatePayrollCalendarTest()
var response = await instance.CreatePayrollCalendarAsync(accessToken, xeroTenantId, payrollCalendar);
Assert.IsType<PayrollCalendars>(response);
}

/// <summary>
/// Test CreateSuperfund
/// </summary>
Expand All @@ -147,7 +147,7 @@ public async Task CreateSuperfundTest()
var response = await instance.CreateSuperfundAsync(accessToken, xeroTenantId, superFund);
Assert.IsType<SuperFunds>(response);
}

/// <summary>
/// Test CreateTimesheet
/// </summary>
Expand All @@ -165,7 +165,7 @@ public async Task CreateTimesheetTest()
var response = await instance.CreateTimesheetAsync(accessToken, xeroTenantId, timesheet);
Assert.IsType<Timesheets>(response);
}

/// <summary>
/// Test GetEmployee
/// </summary>
Expand All @@ -176,7 +176,7 @@ public async Task GetEmployeeTest()
var response = await instance.GetEmployeeAsync(accessToken, xeroTenantId, employeeId);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test GetEmployees
/// </summary>
Expand All @@ -190,7 +190,7 @@ public async Task GetEmployeesTest()
var response = await instance.GetEmployeesAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test GetLeaveApplication
/// </summary>
Expand All @@ -201,7 +201,7 @@ public async Task GetLeaveApplicationTest()
var response = await instance.GetLeaveApplicationAsync(accessToken, xeroTenantId, leaveApplicationId);
Assert.IsType<LeaveApplications>(response);
}

/// <summary>
/// Test GetLeaveApplications
/// </summary>
Expand All @@ -215,7 +215,7 @@ public async Task GetLeaveApplicationsTest()
var response = await instance.GetLeaveApplicationsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<LeaveApplications>(response);
}

/// <summary>
/// Test GetPayItems
/// </summary>
Expand All @@ -229,7 +229,7 @@ public async Task GetPayItemsTest()
var response = await instance.GetPayItemsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<PayItems>(response);
}

/// <summary>
/// Test GetPayRun
/// </summary>
Expand All @@ -240,7 +240,7 @@ public async Task GetPayRunTest()
var response = await instance.GetPayRunAsync(accessToken, xeroTenantId, payRunID);
Assert.IsType<PayRuns>(response);
}

/// <summary>
/// Test GetPayRuns
/// </summary>
Expand All @@ -254,7 +254,7 @@ public async Task GetPayRunsTest()
var response = await instance.GetPayRunsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<PayRuns>(response);
}

/// <summary>
/// Test GetPayrollCalendar
/// </summary>
Expand All @@ -265,7 +265,7 @@ public async Task GetPayrollCalendarTest()
var response = await instance.GetPayrollCalendarAsync(accessToken, xeroTenantId, payrollCalendarID);
Assert.IsType<PayrollCalendars>(response);
}

/// <summary>
/// Test GetPayrollCalendars
/// </summary>
Expand All @@ -279,7 +279,7 @@ public async Task GetPayrollCalendarsTest()
var response = await instance.GetPayrollCalendarsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<PayrollCalendars>(response);
}

/// <summary>
/// Test GetPayslip
/// </summary>
Expand All @@ -290,7 +290,7 @@ public async Task GetPayslipTest()
var response = await instance.GetPayslipAsync(accessToken, xeroTenantId, payslipID);
Assert.IsType<PayslipObject>(response);
}

/// <summary>
/// Test GetSettings
/// </summary>
Expand All @@ -300,7 +300,7 @@ public async Task GetSettingsTest()
var response = await instance.GetSettingsAsync(accessToken, xeroTenantId);
Assert.IsType<SettingsObject>(response);
}

/// <summary>
/// Test GetSuperfund
/// </summary>
Expand All @@ -311,7 +311,7 @@ public async Task GetSuperfundTest()
var response = await instance.GetSuperfundAsync(accessToken, xeroTenantId, superFundID);
Assert.IsType<SuperFunds>(response);
}

/// <summary>
/// Test GetSuperfundProducts
/// </summary>
Expand All @@ -323,7 +323,7 @@ public async Task GetSuperfundProductsTest()
var response = await instance.GetSuperfundProductsAsync(accessToken, xeroTenantId, ABN, USI);
Assert.IsType<SuperFundProducts>(response);
}

/// <summary>
/// Test GetSuperfunds
/// </summary>
Expand All @@ -337,7 +337,7 @@ public async Task GetSuperfundsTest()
var response = await instance.GetSuperfundsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<SuperFunds>(response);
}

/// <summary>
/// Test GetTimesheet
/// </summary>
Expand All @@ -348,7 +348,7 @@ public async Task GetTimesheetTest()
var response = await instance.GetTimesheetAsync(accessToken, xeroTenantId, timesheetID);
Assert.IsType<TimesheetObject>(response);
}

/// <summary>
/// Test GetTimesheets
/// </summary>
Expand All @@ -362,7 +362,7 @@ public async Task GetTimesheetsTest()
var response = await instance.GetTimesheetsAsync(accessToken, xeroTenantId, ifModifiedSince, where, order, page);
Assert.IsType<Timesheets>(response);
}

/// <summary>
/// Test UpdateEmployee
/// </summary>
Expand All @@ -380,7 +380,7 @@ public async Task UpdateEmployeeTest()
var response = await instance.UpdateEmployeeAsync(accessToken, xeroTenantId, employeeId, employee, idempotencyKey);
Assert.IsType<Employees>(response);
}

/// <summary>
/// Test UpdateLeaveApplication
/// </summary>
Expand All @@ -392,7 +392,7 @@ public async Task UpdateLeaveApplicationTest()
var response = await instance.UpdateLeaveApplicationAsync(accessToken, xeroTenantId, leaveApplicationId, leaveApplication);
Assert.IsType<LeaveApplications>(response);
}

/// <summary>
/// Test UpdatePayRun
/// </summary>
Expand All @@ -408,7 +408,7 @@ public async Task UpdatePayRunTest()
var response = await instance.UpdatePayRunAsync(accessToken, xeroTenantId, payRunID, payRun, idempotencyKey);
Assert.IsType<PayRuns>(response);
}

/// <summary>
/// Test UpdatePayslip
/// </summary>
Expand All @@ -421,7 +421,7 @@ public async Task UpdatePayslipTest()
var response = await instance.UpdatePayslipAsync(accessToken, xeroTenantId, payslipID, payslipLines, idempotencyKey);
Assert.IsType<Payslips>(response);
}

/// <summary>
/// Test UpdateSuperfund
/// </summary>
Expand All @@ -437,7 +437,7 @@ public async Task UpdateSuperfundTest()
var response = await instance.UpdateSuperfundAsync(accessToken, xeroTenantId, superFundID, superFund, idempotencyKey);
Assert.IsType<SuperFunds>(response);
}

/// <summary>
/// Test UpdateTimesheet
/// </summary>
Expand All @@ -457,7 +457,7 @@ public async Task UpdateTimesheetTest()
var response = await instance.UpdateTimesheetAsync(accessToken, xeroTenantId, timesheetID, timesheet, idempotencyKey);
Assert.IsType<Timesheets>(response);
}

}

}
Loading