CSS локаторы и совместимость с браузерами

breakmt · 28.Июль.2015 14:59:05

Возник вопрос - при использовании локаторов в виде CSS - нужно ли думать о их совместимости с браузерами? Вопрос возник потому что тут указано что nth-child не поддерживается в IE8, но я все же попробовал запустить тест на IE8 с локатором

.tableProc tr:nth-child(2) td:nth-child(2)

и элемент был найдет. Как это так?

st_eremin · 28.Июль.2015 20:51:07

уверены что поддерживает? проверил у себя - не работает

breakmt · 29.Июль.2015 06:03:45

Да, проверил еще раз - работает. Версия IE 8.0.7601.17514.

А не может быть дело в IEDriverServer? У меня стоит 2.46

dzhariy · 29.Июль.2015 06:32:43

nth-child не поддерживается IE8, за счет того, что в то время, эта функция не была реализована.
Вы можете воспроизвести это даже в IE11 поставив эмуляцию IE8.

Внутри WebDriver для поиска элемента вызывается функция JavaScript (а точнее DOM):
document.querySelector

Тем не менее, многие разработчики использую polyfill – хаки позволяющие добавить старым браузером функционал из новых версий. JavaScript это позволяет сделать.

Можете, например, в консоле браузера выполнить команду:

document.querySelector = alert;

И после каждого вызова вместо поиска селектора вы увидите сообщение с текстом селектора.

document.querySelector("a:nth-child(2)");

Я подозреваю, то на том сайте который вы тестируете как раз используется такой polyfill.

vmaximv · 29.Июль.2015 06:45:55

Это “косяк” IEDriverServer’a

D 2015-07-29 09:26:10:984 ElementFinder.cpp(40) Element location strategy is CSS selectors, but document does not support CSS selectors. Falling back to using the Sizzle JavaScript CSS selector engine.
T 2015-07-29 09:26:10:984 ElementFinder.cpp(170) Entering ElementFinder::FindElementUsingSizzle

github.com

SeleniumHQ/selenium/blob/trunk/cpp/iedriver/ElementFinder.cpp#L305


      
          if (result == WD_SUCCESS) {
            *found_elements = Json::Value(Json::arrayValue);
            if (!get_element_count_script_wrapper.ResultIsInteger()) {
              LOG(WARN) << "Found elements count is not integer";
              result = EUNEXPECTEDJSERROR;
            } else {
              long length = get_element_count_script_wrapper.result().lVal;
              std::wstring get_next_element_script = L"(function(){return function() {return arguments[0][arguments[1]];}})();";
              for (long i = 0; i < length; ++i) {
                Script get_element_script_wrapper(doc, get_next_element_script, 2);
                get_element_script_wrapper.AddArgument(snapshot);
                get_element_script_wrapper.AddArgument(i);
                result = get_element_script_wrapper.Execute();
                if (result == WD_SUCCESS) {
                  Json::Value json_element;
                  get_element_script_wrapper.ConvertResultToJsonValue(executor,
                                                                  &json_element);
                  found_elements->append(json_element);
                } else {
                  LOG(WARN) << "Unable to get " << i << " found element";
                }

Т.е. используется sizzle, вместо нативной реализации. Рискну предположить, что в других версиях ИЕ будет тоже самое.
Как только всплывет (или кто-то отпишется баг-репортом, или само-собой - когда “выпилят” sizzle) - это закроют - и все вернется на круги своя.
Так что верьте лучше “табличкам”

dzhariy · 29.Июль.2015 07:31:46

Точно А я посмотрел на

TODO: Add support for using sizzle to locate elements

github.com

SeleniumHQ/selenium/blob/01399fffecd5a20af6f31aed6cb0043c9c5cde65/javascript/atoms/locators/css.js#L18


      
          //
          //   http://www.apache.org/licenses/LICENSE-2.0
          //
          // Unless required by applicable law or agreed to in writing,
          // software distributed under the License is distributed on an
          // "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
          // KIND, either express or implied.  See the License for the
          // specific language governing permissions and limitations
          // under the License.
          
          
// TODO: Add support for using sizzle to locate elements
          
          
goog.provide('bot.locators.css');
          
          
goog.require('bot.Error');
          goog.require('bot.ErrorCode');
          goog.require('bot.userAgent');
          goog.require('goog.dom.NodeType');
          goog.require('goog.string');
          goog.require('goog.userAgent');

И на этом успокоился, приняв решение что его там нет