Merge pull request #250 from ashawley/fix-fail-empty-attrib

Fix failure for empty attribute match

Author: ashawley

shared/src/main/scala/scala/xml/NodeSeq.scala

@@ -116,6 +116,7 @@ abstract class NodeSeq extends AbstractSeq[Node] with immutable.Seq[Node] with S
     that match {
       case ""                                        => fail
       case "_"                                       => makeSeq(!_.isAtom)
+      case "@"                                       => fail
       case _ if (that(0) == '@' && this.length == 1) => atResult
       case _                                         => makeSeq(_.label == that)
     }

shared/src/test/scala/scala/xml/AttributeTest.scala

@@ -154,4 +154,23 @@ class AttributeTest {
     assertEquals(List(Group(Seq(Text("1"))), Group(Seq(Text("2")))), barList)
   }
 
+  @Test(expected=classOf[IllegalArgumentException])
+  def invalidAttributeFailForOne: Unit = {
+    <x/> \ "@"
+  }
+
+  @Test(expected=classOf[IllegalArgumentException])
+  def invalidAttributeFailForMany: Unit = {
+    <x><y/><z/></x>.child \ "@"
+  }
+
+  @Test(expected=classOf[IllegalArgumentException])
+  def invalidEmptyAttributeFailForOne: Unit = {
+    <x/> \@ ""
+  }
+
+  @Test(expected=classOf[IllegalArgumentException])
+  def invalidEmptyAttributeFailForMany: Unit = {
+    <x><y/><z/></x>.child \@ ""
+  }
 }