Add resource path to the exception message when parsing XML mapper failed (#1172)

* print error detail when xmlConfigBuilder parse error

* modify get location from context to variables

Author: iMouseWu

src/main/java/org/apache/ibatis/builder/xml/XMLMapperBuilder.java

@@ -117,7 +117,7 @@ private void configurationElement(XNode context) {
       sqlElement(context.evalNodes("/mapper/sql"));
       buildStatementFromContext(context.evalNodes("select|insert|update|delete"));
     } catch (Exception e) {
-      throw new BuilderException("Error parsing Mapper XML. Cause: " + e, e);
+      throw new BuilderException("Error parsing Mapper XML.The XML location is " + resource + " Cause: " + e, e);
     }
   }
 

src/test/java/org/apache/ibatis/builder/ProblemMapper.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+       Copyright 2010-2016 the original author or authors.
+
+       Licensed under the Apache License, Version 2.0 (the "License");
+       you may not use this file except in compliance with the License.
+       You may obtain a copy of the License at
+
+          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.
+
+-->
+<!DOCTYPE mapper
+    PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN"
+    "http://mybatis.org/dtd/mybatis-3-mapper.dtd">
+<mapper namespace="org.mybatis.spring.TestProblemMapper">
+
+    <select id="findProblemTest">
+        SELECT *
+        FROM table_test WHERE id = #{}
+    </select>
+</mapper>

src/test/java/org/apache/ibatis/builder/XmlMapperBuilderTest.java

@@ -24,8 +24,10 @@
 import org.apache.ibatis.mapping.ResultSetType;
 import org.apache.ibatis.mapping.StatementType;
 import org.apache.ibatis.session.Configuration;
+import org.junit.Rule;
 import org.apache.ibatis.type.TypeHandler;
 import org.junit.Test;
+import org.junit.rules.ExpectedException;
 
 import static com.googlecode.catchexception.apis.BDDCatchException.*;
 import static org.assertj.core.api.BDDAssertions.then;
@@ -34,6 +36,9 @@
 
 public class XmlMapperBuilderTest {
 
+  @Rule
+  public ExpectedException expectedEx = ExpectedException.none();
+
   @Test
   public void shouldSuccessfullyLoadXMLMapperFile() throws Exception {
     Configuration configuration = new Configuration();
@@ -167,6 +172,17 @@ public void useCacheRefNamespaceIsUndefined() {
       .hasMessage("No cache for namespace 'eee' could be found.");
   }
 
+  @Test
+  public void shouldFailedLoadXMLMapperFile() throws Exception {
+    expectedEx.expect(BuilderException.class);
+    expectedEx.expectMessage("Error parsing Mapper XML.The XML location is org/apache/ibatis/builder/ProblemMapper.xml");
+    Configuration configuration = new Configuration();
+    String resource = "org/apache/ibatis/builder/ProblemMapper.xml";
+    InputStream inputStream = Resources.getResourceAsStream(resource);
+    XMLMapperBuilder builder = new XMLMapperBuilder(inputStream, configuration, resource, configuration.getSqlFragments());
+    builder.parse();
+  }
+
 //  @Test
 //  public void shouldNotLoadTheSameNamespaceFromTwoResourcesWithDifferentNames() throws Exception {
 //    Configuration configuration = new Configuration();