From 12579b9afa20612734355c6d0d10481857dbd184 Mon Sep 17 00:00:00 2001
From: Elias Dorneles <eliasdorneles@gmail.com>
Date: Sun, 9 Aug 2015 01:21:39 -0300
Subject: [PATCH] warning when ambiguous root arguments and minor cleanups

---
 scrapy/linkextractors/sgml.py |  3 +--
 scrapy/selector/unified.py    |  7 +++++--
 tests/test_selector.py        | 24 +++++++++++++++---------
 3 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/scrapy/linkextractors/sgml.py b/scrapy/linkextractors/sgml.py
index d045baa24..4a6a24254 100644
--- a/scrapy/linkextractors/sgml.py
+++ b/scrapy/linkextractors/sgml.py
@@ -127,11 +127,10 @@ class SgmlLinkExtractor(FilteringLinkExtractor):
     def extract_links(self, response):
         base_url = None
         if self.restrict_xpaths:
-            sel = response.selector
             base_url = get_base_url(response)
             body = u''.join(f
                             for x in self.restrict_xpaths
-                            for f in sel.xpath(x).extract()
+                            for f in response.xpath(x).extract()
                             ).encode(response.encoding, errors='xmlcharrefreplace')
         else:
             body = response.body
diff --git a/scrapy/selector/unified.py b/scrapy/selector/unified.py
index e229b10b8..a8f80c84d 100644
--- a/scrapy/selector/unified.py
+++ b/scrapy/selector/unified.py
@@ -33,10 +33,13 @@ class Selector(ParselSelector, object_ref):
     def __init__(self, response=None, text=None, type=None, root=None, _root=None, **kwargs):
         st = _st(response, type or self._default_type)
 
-        if root is None and _root is not None:
+        if _root is not None:
             warnings.warn("Argument `_root` is deprecated, use `root` instead",
                           ScrapyDeprecationWarning, stacklevel=2)
-            root = _root
+            if root is None:
+                root = _root
+            else:
+                warnings.warn("Ignoring deprecated `_root` argument, using provided `root`")
 
         if text is not None:
             response = _response_from_text(text, st)
diff --git a/tests/test_selector.py b/tests/test_selector.py
index d9660c674..dc37da86b 100644
--- a/tests/test_selector.py
+++ b/tests/test_selector.py
@@ -8,7 +8,6 @@ from scrapy.http import TextResponse, HtmlResponse, XmlResponse
 from scrapy.selector import Selector
 from scrapy.selector.lxmlsel import XmlXPathSelector, HtmlXPathSelector, XPathSelector
 from lxml import etree
-from tests import mock
 
 
 class SelectorTestCase(unittest.TestCase):
@@ -39,14 +38,21 @@ class SelectorTestCase(unittest.TestCase):
         self.assertEqual([x.extract() for x in sel.xpath("concat(//input[@name='a']/@value, //input[@name='b']/@value)")],
                          [u'12'])
 
-    @mock.patch('scrapy.selector.unified.warnings')
-    def test_deprecated_root_argument(self, warnings):
-        root = etree.fromstring(u'<html/>')
-        sel = self.sscls(_root=root)
-        self.assertIs(root, sel.root)
-        warnings.warn.assert_called_once_with(
-            'Argument `_root` is deprecated, use `root` instead',
-            mock.ANY, stacklevel=2)
+    def test_deprecated_root_argument(self):
+        with warnings.catch_warnings(record=True) as w:
+            root = etree.fromstring(u'<html/>')
+            sel = self.sscls(_root=root)
+            self.assertIs(root, sel.root)
+            self.assertEqual(str(w[-1].message),
+                             'Argument `_root` is deprecated, use `root` instead')
+
+    def test_deprecated_root_argument_ambiguous(self):
+        with warnings.catch_warnings(record=True) as w:
+            _root = etree.fromstring(u'<xml/>')
+            root = etree.fromstring(u'<html/>')
+            sel = self.sscls(_root=_root, root=root)
+            self.assertIs(root, sel.root)
+            self.assertIn('Ignoring deprecated `_root` argument', str(w[-1].message))
 
     def test_representation_slice(self):
         body = u"<p><input name='{}' value='\xa9'/></p>".format(50 * 'b')