Skip to content

Commit a6d8d13

Browse files
author
Christian
committed
adding missing test cases
1 parent 8750593 commit a6d8d13

File tree

7 files changed

+172
-1
lines changed

7 files changed

+172
-1
lines changed

‎src/ServiceControl.Config.Tests/Validation/AddAuditInstanceValidationTests.cs‎

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -339,6 +339,33 @@ public void Audit_hostname_can_be_an_ip_address_when_adding_audit_instance(strin
339339
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.AuditHostName)), Is.Empty);
340340
}
341341

342+
[TestCase("hostname with spaces")]
343+
[TestCase("bad@hostname")]
344+
[TestCase("bad#hostname")]
345+
[TestCase("badhostname...")]
346+
[TestCase("badhostname[/")]
347+
public void Audit_hostname_cannot_contain_invalid_characters_when_adding_audit_instance(string invalidHostname)
348+
{
349+
var viewModel = new ServiceControlAddViewModel
350+
{
351+
InstallAuditInstance = true,
352+
SubmitAttempted = true,
353+
AuditHostName = invalidHostname
354+
};
355+
356+
viewModel.NotifyOfPropertyChange(nameof(viewModel.AuditHostName));
357+
358+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
359+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.AuditHostName));
360+
361+
Assert.Multiple(() =>
362+
{
363+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
364+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
365+
"Hostname validation should display the exact error message 'Hostname is not valid'");
366+
});
367+
}
368+
342369
#endregion
343370

344371
#region Portnumber

‎src/ServiceControl.Config.Tests/Validation/AddErrorInstanceValidationTests.cs‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -320,6 +320,8 @@ public void Error_hostname_can_not_be_null_when_adding_error_instance()
320320

321321
[TestCase("192.168.1.1")]
322322
[TestCase("256.0.0.0")]
323+
[TestCase("::1")]
324+
[TestCase("2001:0db8:85a3:0000:0000:8a2e:0370:7334")]
323325
public void Error_hostname_can_be_an_ip_address_when_adding_error_instance(string ipAddress)
324326
{
325327
var viewModel = new ServiceControlAddViewModel
@@ -336,6 +338,33 @@ public void Error_hostname_can_be_an_ip_address_when_adding_error_instance(strin
336338
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.ErrorHostName)), Is.Empty);
337339
}
338340

341+
[TestCase("hostname with spaces")]
342+
[TestCase("bad@hostname")]
343+
[TestCase("bad#hostname")]
344+
[TestCase("badhostname...")]
345+
[TestCase("badhostname[/")]
346+
public void Error_hostname_cannot_contain_invalid_characters_when_adding_error_instance(string invalidHostname)
347+
{
348+
var viewModel = new ServiceControlAddViewModel
349+
{
350+
InstallErrorInstance = true,
351+
SubmitAttempted = true,
352+
ErrorHostName = invalidHostname
353+
};
354+
355+
viewModel.NotifyOfPropertyChange(nameof(viewModel.ErrorHostName));
356+
357+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
358+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.ErrorHostName));
359+
360+
Assert.Multiple(() =>
361+
{
362+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
363+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
364+
"Hostname validation should display the exact error message 'Hostname is not valid'");
365+
});
366+
}
367+
339368
#endregion
340369

341370
#region Portnumber

‎src/ServiceControl.Config.Tests/Validation/AddMonitoringInstanceValidationTests.cs‎

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,8 @@ public void Monitoring_hostname_cannot_be_null_when_adding_monitoring_instance()
127127

128128
[TestCase("192.168.1.1")]
129129
[TestCase("256.0.0.0")]
130+
[TestCase("::1")]
131+
[TestCase("2001:0db8:85a3:0000:0000:8a2e:0370:7334")]
130132
public void Monitoring_hostname_can_be_an_ip_address_when_adding_monitoring_instance(string ipAddress)
131133
{
132134
var viewModel = new MonitoringAddViewModel
@@ -142,6 +144,32 @@ public void Monitoring_hostname_can_be_an_ip_address_when_adding_monitoring_inst
142144
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.HostName)), Is.Empty);
143145
}
144146

147+
[TestCase("hostname with spaces")]
148+
[TestCase("bad@hostname")]
149+
[TestCase("bad#hostname")]
150+
[TestCase("badhostname...")]
151+
[TestCase("badhostname[/")]
152+
public void Monitoring_hostname_cannot_contain_invalid_characters_when_adding_monitoring_instance(string invalidHostname)
153+
{
154+
var viewModel = new MonitoringAddViewModel
155+
{
156+
SubmitAttempted = true,
157+
HostName = invalidHostname
158+
};
159+
160+
viewModel.NotifyOfPropertyChange(nameof(viewModel.HostName));
161+
162+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
163+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.HostName));
164+
165+
Assert.Multiple(() =>
166+
{
167+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
168+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
169+
"Hostname validation should display the exact error message 'Hostname is not valid'");
170+
});
171+
}
172+
145173
#endregion
146174

147175
#region Portnumber

‎src/ServiceControl.Config.Tests/Validation/EditAuditInstanceValidationTests.cs‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
namespace ServiceControl.Config.Tests.Validation
22
{
33
using System.ComponentModel;
4+
using System.Linq;
45
using NUnit.Framework;
56
using ServiceControlInstaller.Engine.Instances;
67
using UI.InstanceAdd;
@@ -127,6 +128,8 @@ public void Audit_hostname_can_not_be_null_when_editing_audit_instance()
127128

128129
[TestCase("192.168.1.1")]
129130
[TestCase("256.0.0.0")]
131+
[TestCase("::1")]
132+
[TestCase("2001:0db8:85a3:0000:0000:8a2e:0370:7334")]
130133
public void Audi_hostname_can_be_an_ip_address_when_editing_an_audit_instance(string ipAddress)
131134
{
132135
var viewModel = new ServiceControlAuditEditViewModel
@@ -142,6 +145,32 @@ public void Audi_hostname_can_be_an_ip_address_when_editing_an_audit_instance(st
142145
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.HostName)), Is.Empty);
143146
}
144147

148+
[TestCase("hostname with spaces")]
149+
[TestCase("bad@hostname")]
150+
[TestCase("bad#hostname")]
151+
[TestCase("badhostname...")]
152+
[TestCase("badhostname[/")]
153+
public void Audit_hostname_cannot_contain_invalid_characters_when_editing_audit_instance(string invalidHostname)
154+
{
155+
var viewModel = new ServiceControlAuditEditViewModel
156+
{
157+
SubmitAttempted = true,
158+
HostName = invalidHostname
159+
};
160+
161+
viewModel.NotifyOfPropertyChange(nameof(viewModel.HostName));
162+
163+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
164+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.HostName));
165+
166+
Assert.Multiple(() =>
167+
{
168+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
169+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
170+
"Hostname validation should display the exact error message 'Hostname is not valid'");
171+
});
172+
}
173+
145174
#endregion
146175

147176
#region Portnumber

‎src/ServiceControl.Config.Tests/Validation/EditErrorInstanceValidationTests.cs‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
namespace ServiceControl.Config.Tests.Validation
22
{
33
using System.ComponentModel;
4+
using System.Linq;
45
using NUnit.Framework;
56
using ServiceControlInstaller.Engine.Instances;
67
using UI.InstanceAdd;
@@ -120,6 +121,8 @@ public void Error_hostname_can_not_be_null_when_editing_error_instance()
120121

121122
[TestCase("192.168.1.1")]
122123
[TestCase("256.0.0.0")]
124+
[TestCase("::1")]
125+
[TestCase("2001:0db8:85a3:0000:0000:8a2e:0370:7334")]
123126
public void Error_hostname_can_be_an_ip_address_when_editing_an_error_instance(string ipAddress)
124127
{
125128
var viewModel = new ServiceControlEditViewModel
@@ -135,6 +138,32 @@ public void Error_hostname_can_be_an_ip_address_when_editing_an_error_instance(s
135138
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.HostName)), Is.Empty);
136139
}
137140

141+
[TestCase("hostname with spaces")]
142+
[TestCase("bad@hostname")]
143+
[TestCase("bad#hostname")]
144+
[TestCase("badhostname...")]
145+
[TestCase("badhostname[/")]
146+
public void Error_hostname_cannot_contain_invalid_characters_when_editing_error_instance(string invalidHostname)
147+
{
148+
var viewModel = new ServiceControlEditViewModel
149+
{
150+
SubmitAttempted = true,
151+
HostName = invalidHostname
152+
};
153+
154+
viewModel.NotifyOfPropertyChange(nameof(viewModel.HostName));
155+
156+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
157+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.HostName));
158+
159+
Assert.Multiple(() =>
160+
{
161+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
162+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
163+
"Hostname validation should display the exact error message 'Hostname is not valid'");
164+
});
165+
}
166+
138167
#endregion
139168

140169
#region Portnumber

‎src/ServiceControl.Config.Tests/Validation/EditMonitoringInstanceValidationTests.cs‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
namespace ServiceControl.Config.Tests.Validation
22
{
33
using System.ComponentModel;
4+
using System.Linq;
45
using NUnit.Framework;
56
using UI.InstanceEdit;
67

@@ -42,6 +43,8 @@ public void Monitoring_hostname_cannot_be_null_when_editing_monitoring_instance(
4243

4344
[TestCase("192.168.1.1")]
4445
[TestCase("256.0.0.0")]
46+
[TestCase("::1")]
47+
[TestCase("2001:0db8:85a3:0000:0000:8a2e:0370:7334")]
4548
public void Monitoring_hostname_can_be_an_ip_address_when_editing_a_monitoring_instance(string ipAddress)
4649
{
4750
var viewModel = new MonitoringEditViewModel
@@ -56,6 +59,32 @@ public void Monitoring_hostname_can_be_an_ip_address_when_editing_a_monitoring_i
5659

5760
Assert.That(notifyErrorInfo.GetErrors(nameof(viewModel.HostName)), Is.Empty);
5861
}
62+
63+
[TestCase("hostname with spaces")]
64+
[TestCase("bad@hostname")]
65+
[TestCase("bad#hostname")]
66+
[TestCase("badhostname...")]
67+
[TestCase("badhostname[/")]
68+
public void Monitoring_hostname_cannot_contain_invalid_characters_when_editing_monitoring_instance(string invalidHostname)
69+
{
70+
var viewModel = new MonitoringEditViewModel
71+
{
72+
SubmitAttempted = true,
73+
HostName = invalidHostname
74+
};
75+
76+
viewModel.NotifyOfPropertyChange(nameof(viewModel.HostName));
77+
78+
var notifyErrorInfo = GetNotifyErrorInfo(viewModel);
79+
var errors = notifyErrorInfo.GetErrors(nameof(viewModel.HostName));
80+
81+
Assert.Multiple(() =>
82+
{
83+
Assert.That(errors, Is.Not.Empty, "Hostname validation should exist and trigger for invalid hostnames");
84+
Assert.That(errors.Cast<string>().Any(error => error.Contains("Hostname is not valid")), Is.True,
85+
"Hostname validation should display the exact error message 'Hostname is not valid'");
86+
});
87+
}
5988
#endregion
6089

6190
#region Portnumber

‎src/ServiceControl.Config/UI/SharedInstanceEditor/SharedMonitoringEditorViewModelValidator.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ protected SharedMonitoringEditorViewModelValidator()
99
{
1010
RuleFor(x => x.HostName)
1111
.NotEmpty()
12-
.ValidHostname(); // Removed the .When(x => x.SubmitAttempted) condition
12+
.ValidHostname();
1313

1414
RuleFor(x => x.LogPath)
1515
.NotEmpty()

0 commit comments

Comments
 (0)