Skip to content

Commit 83504c0

Browse files
committed
Use extension method to assert no errors on api response
1 parent 7a26f58 commit 83504c0

File tree

3 files changed

+29
-28
lines changed

3 files changed

+29
-28
lines changed

Tests/ApiResponseExtensions.cs

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
using MatthiWare.FinancialModelingPrepApi.Model;
2+
using Xunit;
3+
4+
namespace Tests
5+
{
6+
public static class ApiResponseExtensions
7+
{
8+
public static void AssertNoErrors<T>(this ApiResponse<T> response)
9+
where T : class
10+
{
11+
Assert.NotNull(response);
12+
Assert.False(response.HasError, response.Error);
13+
}
14+
}
15+
}

Tests/CompanyValuation/CompanyValuationTests.cs

Lines changed: 11 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,7 @@ public async Task GetCompanyProfileTests()
2121

2222
var result = await api.CompanyValuation.GetCompanyProfileAsync("AAPL");
2323

24-
Assert.NotNull(result);
25-
Assert.False(result.HasError);
24+
result.AssertNoErrors();
2625
Assert.Equal("AAPL", result.Data.symbol);
2726
}
2827

@@ -44,8 +43,7 @@ public async Task GetSymbolsList()
4443

4544
var result = await api.CompanyValuation.GetSymbolsListAsync();
4645

47-
Assert.NotNull(result);
48-
Assert.False(result.HasError);
46+
result.AssertNoErrors();
4947
Assert.NotEmpty(result.Data);
5048
}
5149

@@ -56,8 +54,7 @@ public async Task GetETFList()
5654

5755
var result = await api.CompanyValuation.GetETFListAsync();
5856

59-
Assert.NotNull(result);
60-
Assert.False(result.HasError);
57+
result.AssertNoErrors();
6158
Assert.NotEmpty(result.Data);
6259
}
6360

@@ -68,8 +65,7 @@ public async Task GetTradableSymbolsList()
6865

6966
var result = await api.CompanyValuation.GetTradableSymbolsListAsync();
7067

71-
Assert.NotNull(result);
72-
Assert.False(result.HasError);
68+
result.AssertNoErrors();
7369
Assert.NotEmpty(result.Data);
7470
}
7571

@@ -80,8 +76,7 @@ public async Task GetEnterpriseValue()
8076

8177
var result = await api.CompanyValuation.GetEnterpriseValueAsync("AAPL", Period.Annual, 5);
8278

83-
Assert.NotNull(result);
84-
Assert.False(result.HasError);
79+
result.AssertNoErrors();
8580
Assert.NotEmpty(result.Data);
8681
Assert.Equal(5, result.Data.Count);
8782
Assert.All(result.Data, data => Assert.Equal("AAPL", data.symbol));
@@ -94,8 +89,7 @@ public async Task GetIncomeStatement()
9489

9590
var result = await api.CompanyValuation.GetIncomeStatementAsync("AAPL", Period.Annual, 5);
9691

97-
Assert.NotNull(result);
98-
Assert.False(result.HasError);
92+
result.AssertNoErrors();
9993
Assert.NotEmpty(result.Data);
10094
Assert.Equal(5, result.Data.Count);
10195
Assert.All(result.Data, data => Assert.Equal("AAPL", data.Symbol));
@@ -108,8 +102,7 @@ public async Task GetCashFlowStatement()
108102

109103
var result = await api.CompanyValuation.GetCashFlowStatementAsync("AAPL", Period.Annual, 5);
110104

111-
Assert.NotNull(result);
112-
Assert.False(result.HasError);
105+
result.AssertNoErrors();
113106
Assert.NotEmpty(result.Data);
114107
Assert.Equal(5, result.Data.Count);
115108
Assert.All(result.Data, data => Assert.Equal("AAPL", data.Symbol));
@@ -122,8 +115,7 @@ public async Task GetBalanceSheetStatement()
122115

123116
var result = await api.CompanyValuation.GetBalanceSheetStatementAsync("AAPL", Period.Annual, 5);
124117

125-
Assert.NotNull(result);
126-
Assert.False(result.HasError);
118+
result.AssertNoErrors();
127119
Assert.NotEmpty(result.Data);
128120
Assert.Equal(5, result.Data.Count);
129121
Assert.All(result.Data, data => Assert.Equal("AAPL", data.Symbol));
@@ -136,8 +128,7 @@ public async Task GetStockNewsAsync()
136128

137129
var result = await api.CompanyValuation.GetStockNewsAsync("AAPL", 5);
138130

139-
Assert.NotNull(result);
140-
Assert.False(result.HasError);
131+
result.AssertNoErrors();
141132
Assert.NotEmpty(result.Data);
142133
Assert.Equal(5, result.Data.Count);
143134
Assert.All(result.Data, data => Assert.Equal("AAPL", data.Symbol));
@@ -150,8 +141,7 @@ public async Task GetCompanyRatingAsync()
150141

151142
var result = await api.CompanyValuation.GetCompanyRatingAsync("AAPL");
152143

153-
Assert.NotNull(result);
154-
Assert.False(result.HasError);
144+
result.AssertNoErrors();
155145
Assert.Equal("AAPL", result.Data.Symbol);
156146
}
157147

@@ -162,8 +152,7 @@ public async Task GetHistoricalCompanyRatingAsync()
162152

163153
var result = await api.CompanyValuation.GetHistoricalCompanyRatingAsync("AAPL", 5);
164154

165-
Assert.NotNull(result);
166-
Assert.False(result.HasError);
155+
result.AssertNoErrors();
167156
Assert.NotEmpty(result.Data);
168157
Assert.Equal(5, result.Data.Count);
169158
Assert.All(result.Data, data => Assert.Equal("AAPL", data.Symbol));

Tests/MarketIndexes/MarketIndexesTests.cs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ public async Task GetDowJonesCompaniesTests()
2020

2121
var result = await api.MarketIndexes.GetDowJonesCompaniesAsync();
2222

23-
Assert.NotNull(result);
24-
Assert.False(result.HasError);
23+
result.AssertNoErrors();
2524
Assert.NotEmpty(result.Data);
2625
}
2726

@@ -32,8 +31,7 @@ public async Task GetNasdaqCompaniesTests()
3231

3332
var result = await api.MarketIndexes.GetNasdaqCompaniesAsync();
3433

35-
Assert.NotNull(result);
36-
Assert.False(result.HasError);
34+
result.AssertNoErrors();
3735
Assert.NotEmpty(result.Data);
3836
}
3937

@@ -44,8 +42,7 @@ public async Task GetSP500CompaniesTests()
4442

4543
var result = await api.MarketIndexes.GetSP500CompaniesAsync();
4644

47-
Assert.NotNull(result);
48-
Assert.False(result.HasError);
45+
result.AssertNoErrors();
4946
Assert.NotEmpty(result.Data);
5047
}
5148
}

0 commit comments

Comments
 (0)