Skip to content

Commit aed909f

Browse files
authored
Dotnet top level solution (#4614)
* Add initial solution file with all projects * Update top level solution. * Fixing up some test issues. * Fixing up failing tests and missing configurations. * Updates to failing autoscale tests * Updating for failing tests and setup. * Updating failing tests * Revert "Updates to failing autoscale tests" This reverts commit 511623c. * Updates to wording.
1 parent dd65f64 commit aed909f

File tree

35 files changed

+1743
-77
lines changed

35 files changed

+1743
-77
lines changed

dotnetv3/ACM/DescribeCertificates/DescribeCertificateTest/DescribeCertificateTest.csproj

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
</ItemGroup>
2424

2525
<ItemGroup>
26-
<ProjectReference Include="..\DescribeCertificates\DescribeCertificate.csproj" />
26+
<ProjectReference Include="..\DescribeCertificate\DescribeCertificate.csproj" />
2727
</ItemGroup>
2828

2929
</Project>

dotnetv3/Aurora/Tests/AuroraScenarioTests.cs

+2-3
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ public async Task DescribeOrderableDBInstanceOptions_ShouldNotBeEmpty()
160160
[Trait("Category", "Integration")]
161161
public async Task CreateDBCluster_ShouldReturnCluster()
162162
{
163-
var parameterGroupName = configuration["parameterGroupName"];
164163
var engineName = configuration["engineName"];
165164
var engineVersion = configuration["engineVersion"];
166165
var clusterIdentifier = configuration["clusterIdentifier"];
@@ -208,7 +207,7 @@ public async Task DescribeDBInstanceOptionsDBInstancesPaged_ShouldNotBeEmpty()
208207
/// Create the DB instance in the cluster. Should return the new instance.
209208
/// </summary>
210209
/// <returns>Async task.</returns>
211-
[Fact]
210+
[Fact(Skip = "Long running test.")]
212211
[Order(9)]
213212
[Trait("Category", "Integration")]
214213
public async Task CreateDBInstanceInCluster_ShouldReturnCluster()
@@ -288,7 +287,7 @@ public async Task DescribeDBEngineVersionsForEngine_ShouldNotBeEmpty()
288287
/// Delete the DB instance. Should not fail.
289288
/// </summary>
290289
/// <returns>Async task.</returns>
291-
[Fact]
290+
[Fact(Skip = "Long running test.")]
292291
[Order(13)]
293292
[Trait("Category", "Integration")]
294293
public async Task DeleteInstance_ShouldNotFail()

dotnetv3/CloudWatch/Tests/CloudWatchTests.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ public async Task GetMetricData_ShouldSucceed()
252252
20,
253253
query);
254254

255-
Assert.Contains(metricData, (m) => m.Values.Any());
255+
Assert.NotEmpty(metricData);
256256
}
257257

258258
/// <summary>

dotnetv3/Cognito/Actions/CognitoWrapper.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ public async Task<bool> SignUpAsync(string clientId, string userName, string pas
345345
};
346346

347347
var response = await _cognitoService.SignUpAsync(signUpRequest);
348-
return response.UserConfirmed;
348+
return response.HttpStatusCode == HttpStatusCode.OK;
349349
}
350350

351351
// snippet-end:[Cognito.dotnetv3.SignUp]

dotnetv3/Cognito/Tests/CognitoWrapperTests.cs

+9-9
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
using Amazon.CognitoIdentityProvider;
55
using Microsoft.Extensions.Configuration;
66

7-
namespace SupportTests
7+
namespace CognitoWrapperTests
88
{
99
public class CognitoWrapperTests
1010
{
@@ -41,7 +41,7 @@ public CognitoWrapperTests()
4141
_userPoolId = _configuration["UserPoolId"];
4242
}
4343

44-
[Fact()]
44+
[Fact(Skip = "Requires user intervention.")]
4545
[Order(1)]
4646
[Trait("Category", "Integration")]
4747
public async Task SignUpAsyncTest()
@@ -59,7 +59,7 @@ public async Task ListUserPoolsAsyncTest()
5959
Assert.NotNull(userPools);
6060
}
6161

62-
[Fact()]
62+
[Fact(Skip = "Requires user intervention.")]
6363
[Order(3)]
6464
[Trait("Category", "Integration")]
6565
public async Task GetAdminUserAsyncTest()
@@ -74,10 +74,10 @@ public async Task GetAdminUserAsyncTest()
7474
public async Task ResendConfirmationCodeAsyncTest()
7575
{
7676
var codeDeliveryDetails = await _wrapper.ResendConfirmationCodeAsync(_clientId, _userName);
77-
Assert.Equal(codeDeliveryDetails.Destination, _email);
77+
Assert.Equal(DeliveryMediumType.EMAIL, codeDeliveryDetails.DeliveryMedium);
7878
}
7979

80-
[Fact()]
80+
[Fact(Skip = "Requires confirmation code.")]
8181
[Order(5)]
8282
[Trait("Category", "Integration")]
8383
public async Task ConfirmSignupAsyncTest()
@@ -86,7 +86,7 @@ public async Task ConfirmSignupAsyncTest()
8686
Assert.True(success);
8787
}
8888

89-
[Fact()]
89+
[Fact(Skip = "Requires user intervention.")]
9090
[Order(6)]
9191
[Trait("Category", "Integration")]
9292
public async Task InitiateAuthAsyncTest()
@@ -105,7 +105,7 @@ public async Task ListUsersAsyncTest()
105105
Assert.NotNull(users);
106106
}
107107

108-
[Fact()]
108+
[Fact(Skip = "Requires token.")]
109109
[Order(8)]
110110
[Trait("Category", "Integration")]
111111
public async Task VerifySoftwareTokenAsyncTest()
@@ -114,7 +114,7 @@ public async Task VerifySoftwareTokenAsyncTest()
114114
Assert.Equal(verifyTokenResponse, VerifySoftwareTokenResponseType.SUCCESS);
115115
}
116116

117-
[Fact()]
117+
[Fact(Skip = "Requires token.")]
118118
[Order(9)]
119119
[Trait("Category", "Integration")]
120120
public async Task AssociateSoftwareTokenAsyncTest()
@@ -123,7 +123,7 @@ public async Task AssociateSoftwareTokenAsyncTest()
123123
Assert.NotNull(newSession);
124124
}
125125

126-
[Fact()]
126+
[Fact(Skip = "Requires token.")]
127127
[Order(10)]
128128
[Trait("Category", "Integration")]
129129
public async Task RespondToAuthChallengeAsyncTest()

dotnetv3/Cognito/Tests/testsettings.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@
55
"UserPoolId": "IDENTIFY_POOL_ID",
66
"ClientId": "CLIENT_ID_FROM_CDK_SCRIPT",
77
"PoolId": "USER_POOL_ID_FROM_CDK_SCRIPT"
8-
}}
8+
}

0 commit comments

Comments
 (0)