Skip to content

Commit 5d7b51f

Browse files
authored
#136 Add public properties for Name and Locator to Form (#142) +semver: minor
1 parent b686152 commit 5d7b51f

File tree

1 file changed

+20
-20
lines changed
  • Aquality.Selenium/src/Aquality.Selenium/Forms

1 file changed

+20
-20
lines changed

Aquality.Selenium/src/Aquality.Selenium/Forms/Form.cs

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -12,28 +12,37 @@ namespace Aquality.Selenium.Forms
1212
public abstract class Form
1313
{
1414
/// <summary>
15-
/// Instance of logger <see cref="Logging.Logger">
15+
/// Constructor with parameters.
1616
/// </summary>
17-
/// <value>Logger instance.</value>
18-
protected Logger Logger => Logger.Instance;
17+
/// <param name="locator">Unique locator of the form.</param>
18+
/// <param name="name">Name of the form.</param>
19+
protected Form(By locator, string name)
20+
{
21+
Locator = locator;
22+
Name = name;
23+
}
1924

2025
/// <summary>
21-
/// Locator for specified form
26+
/// Locator of specified form.
2227
/// </summary>
23-
protected readonly By Locator;
28+
public By Locator { get; }
2429

2530
/// <summary>
26-
/// Name of specified form
31+
/// Name of specified form.
32+
/// </summary>
33+
public string Name { get; }
34+
35+
/// <summary>
36+
/// Instance of logger <see cref="Logging.Logger">
2737
/// </summary>
28-
protected readonly string Name;
38+
/// <value>Logger instance.</value>
39+
protected Logger Logger => Logger.Instance;
2940

3041
/// <summary>
3142
/// Element factory <see cref="IElementFactory">
3243
/// </summary>
3344
/// <value>Element factory.</value>
34-
protected IElementFactory ElementFactory => new ElementFactory();
35-
36-
private ILabel FormLabel => ElementFactory.GetLabel(Locator, Name);
45+
protected IElementFactory ElementFactory => new ElementFactory();
3746

3847
/// <summary>
3948
/// Return form state for form locator
@@ -47,16 +56,7 @@ public abstract class Form
4756
/// </summary>
4857
public Size Size => FormLabel.GetElement().Size;
4958

50-
/// <summary>
51-
/// Constructor with parameters
52-
/// </summary>
53-
/// <param name="locator">Unique locator of the form.</param>
54-
/// <param name="name">Name of the form.</param>
55-
protected Form(By locator, string name)
56-
{
57-
Locator = locator;
58-
Name = name;
59-
}
59+
private ILabel FormLabel => ElementFactory.GetLabel(Locator, Name);
6060

6161
/// <summary>
6262
/// Scroll form without scrolling entire page

0 commit comments

Comments
 (0)