Fixed collection selectorMethod references

Author: quigamdev

src/Core/Riganti.Selenium.Core/ElementWrapperCollection.cs

@@ -100,10 +100,12 @@ private void SetReferences(List<TElement> elms, string selector, Func<string, By
             foreach (var ew in elms)
             {
                 ew.Selector = selector;
-                ew.SelectMethod = selectMethod;
                 ew.ParentWrapper = this;
             }
         }
+        /// <summary>
+        /// Throws exception when collection of elements does not contain any elements.
+        /// </summary>
         public IElementWrapperCollection<TElement, TBrowser> ThrowIfSequenceEmpty()
         {
             if (!GetCollection().Any())
@@ -113,6 +115,9 @@ public IElementWrapperCollection<TElement, TBrowser> ThrowIfSequenceEmpty()
             return this;
         }
 
+        /// <summary>
+        /// Throws exception when collection of elements does contain more then one element. 
+        /// </summary>
         public IElementWrapperCollection<TElement, TBrowser> ThrowIfSequenceContainsMoreThanOneElement()
         {
             if (Count > 1)
@@ -122,7 +127,7 @@ public IElementWrapperCollection<TElement, TBrowser> ThrowIfSequenceContainsMore
             return this;
         }
         /// <summary>
-        /// Throws exception when lenght of a sequence is empty or the sequence contains more than one element.
+        /// Throws exception when length of a sequence is empty or the sequence contains more than one element.
         /// </summary>
         /// <returns></returns>
         public IElementWrapperCollection<TElement, TBrowser> ThrowIfEmptyOrMoreThanOne()
@@ -287,7 +292,7 @@ public TElement FirstOrDefault(string selector, Func<string, By> tmpSelectMethod
         IElementWrapperCollection<TElement1, TBrowser1> ISeleniumWrapperCollection.Convert<TElement1, TBrowser1>()
         {
             return new ElementWrapperCollection<TElement1, TBrowser1>(() => GetCollection().Cast<TElement1>(),
-                Selector, SelectMethod ,BrowserWrapper, ParentWrapper);
+                Selector, SelectMethod, BrowserWrapper, ParentWrapper);
         }
     }
 }

src/Samples/Riganti.Selenium.Core.Samples.New/Views/Test/Index.cshtml

@@ -1,31 +1,35 @@
 <div id="displayed">
-    visible element
-    <div>
-        <p>
-            a
-        </p>
-        <p>
-            b
-            <span></span>
-        </p>
-        <p>
-            c
-            <span>
-                This is what I want.
-            </span>
-        </p>
-    </div>
+	visible element
+	<div>
+		<p>
+			a
+		</p>
+		<p>
+			b
+			<span></span>
+		</p>
+		<p>
+			c
+			<span>
+				This is what I want.
+			</span>
+		</p>
+	</div>
 </div>
 <div id="non-displayed" style="display: none;">
-    Invisible element
+	Invisible element
 </div>
 <div id="displayed-zero-draw-rec" style="width: 0px;height: 0px">
-    Invisible element
+	Invisible element
 </div>
 
 <div id="content" class="test">
-    Lorem Ipsum je demonstrativní výplňový text používaný v tiskařském a knihařském průmyslu. Lorem Ipsum je považováno za standard v této oblasti už od začátku 16. století, kdy dnes neznámý tiskař vzal kusy textu a na jejich základě vytvořil speciální vzorovou knihu. Jeho odkaz nevydržel pouze pět století, on přežil i nástup elektronické sazby v podstatě beze změny. Nejvíce popularizováno bylo Lorem Ipsum v šedesátých letech 20. století, kdy byly vydávány speciální vzorníky s jeho pasážemi a později pak díky počítačovým DTP programům jako Aldus PageMaker.
+	Lorem Ipsum je demonstrativní výplňový text používaný v tiskařském a knihařském průmyslu. Lorem Ipsum je považováno za standard v této oblasti už od začátku 16. století, kdy dnes neznámý tiskař vzal kusy textu a na jejich základě vytvořil speciální vzorovou knihu. Jeho odkaz nevydržel pouze pět století, on přežil i nástup elektronické sazby v podstatě beze změny. Nejvíce popularizováno bylo Lorem Ipsum v šedesátých letech 20. století, kdy byly vydávány speciální vzorníky s jeho pasážemi a později pak díky počítačovým DTP programům jako Aldus PageMaker.
 </div>
 
 <button id="dis-button" disabled>disabled button</button>
-<input id="submit-button" type="submit" value="Submit Button Test" />
+<input id="submit-button" type="submit" value="Submit Button Test" />
+
+<div data-ui="selector-test">
+	<span data-ui="selector-test-inner"></span>
+</div>

src/Tests/Riganti.Selenium.Core.Samples.Assert.Tests/ScrollToTests.cs

@@ -26,4 +26,5 @@ public void ScrollToChild()
         }
     }
 
+
 }

src/Tests/Riganti.Selenium.Core.Samples.Assert.Tests/SelectorsTests.cs

@@ -0,0 +1,35 @@
+using OpenQA.Selenium;
+using Riganti.Selenium.AssertApi;
+using System.Security.Principal;
+using Xunit;
+using Xunit.Abstractions;
+
+namespace Riganti.Selenium.Core.Samples.AssertApi.Tests
+{
+
+    public class SelectorsTests : AppSeleniumTest
+    {
+        public SelectorsTests(ITestOutputHelper output) : base(output)
+        {
+
+        }
+        public By SelectByDataUi(string selector)
+            => SelectBy.CssSelector($"[data-ui='{selector}']");
+
+        [Fact]
+        public void SelectByCssSelectorTest()
+        {
+            RunInAllBrowsers(browser =>
+            {
+                browser.NavigateToUrl("/test/Index");
+
+                var div = browser.Single("selector-test", SelectByDataUi);
+                var span = div.Single("span");
+            });
+        }
+
+
+    }
+
+
+}