xmlx/xmlx_test.go

// This work is subject to the CC0 1.0 Universal (CC0 1.0) Public Domain Dedication
// license. Its contents can be found at:
// http://creativecommons.org/publicdomain/zero/1.0/

package xmlx

import "testing"

func TestLoadLocal(t *testing.T) {
	doc := New()

	if err := doc.LoadFile("test.xml", nil); err != nil {
		t.Error(err.Error())
		return
	}

	if len(doc.Root.Children) == 0 {
		t.Errorf("Root node has no children.")
		return
	}
}

func TestWildcard(t *testing.T) {
	doc := New()

	if err := doc.LoadFile("test2.xml", nil); err != nil {
		t.Error(err.Error())
		return
	}

	list := doc.SelectNodes("ns", "*")

	if len(list) != 1 {
		t.Errorf("Wrong number of child elements. Expected 1, got %d.", len(list))
		return
	}
}

func TestWildcardRecursive(t *testing.T) {
	doc := New()

	if err := doc.LoadFile("test2.xml", nil); err != nil {
		t.Error(err.Error())
		return
	}

	list := doc.SelectNodesRecursive("ns", "*")

	if len(list) != 7 {
		t.Errorf("Wrong number of child elements. Expected 7, got %d.", len(list))
		return
	}
}

func _TestLoadRemote(t *testing.T) {
	doc := New()

	if err := doc.LoadUri("http://blog.golang.org/feeds/posts/default", nil); err != nil {
		t.Error(err.Error())
		return
	}

	if len(doc.Root.Children) == 0 {
		t.Errorf("Root node has no children.")
		return
	}
}

func TestSave(t *testing.T) {
	doc := New()

	if err := doc.LoadFile("test.xml", nil); err != nil {
		t.Errorf("LoadFile(): %s", err)
		return
	}

	IndentPrefix = "\t"
	if err := doc.SaveFile("test1.xml"); err != nil {
		t.Errorf("SaveFile(): %s", err)
		return
	}
}

func TestNodeSearch(t *testing.T) {
	doc := New()

	if err := doc.LoadFile("test1.xml", nil); err != nil {
		t.Errorf("LoadFile(): %s", err)
		return
	}

	if node := doc.SelectNode("", "item"); node == nil {
		t.Errorf("SelectNode(): No node found.")
		return
	}

	nodes := doc.SelectNodes("", "item")
	if len(nodes) == 0 {
		t.Errorf("SelectNodes(): no nodes found.")
		return
	}

	ch := doc.SelectNode("", "channel")
	// Test that SelectNodes properly selects multiple nodes
	links := ch.SelectNodes("", "link")
	if len(links) != 8 {
		t.Errorf("SelectNodes(): Expected 8, Got %d", len(links))
		return
	}
}

type Image struct {
	Title       string `xml:"title"`
	Url         string `xml:"url"`
	Link        string `xml:"link"`
	Description string `xml:"description"`
	Width       int    `xml:"width"`
	Height      int    `xml:"height"`
}

func TestUnmarshal(t *testing.T) {
	doc := New()
	err := doc.LoadFile("test1.xml", nil)

	if err != nil {
		t.Errorf("LoadFile(): %s", err)
		return
	}

	node := doc.SelectNode("", "image")
	if node == nil {
		t.Errorf("SelectNode(): No node found.")
		return
	}

	var img Image
	if err = node.Unmarshal(&img); err != nil {
		t.Errorf("Unmarshal(): %s", err)
		return
	}

	if img.Title != "WriteTheWeb" {
		t.Errorf("Image.Title has incorrect value. Got '%s', expected 'WriteTheWeb'.", img.Title)
		return
	}
}

func TestStringNamespaces(t *testing.T) {
	doc := New()
	err := doc.LoadFile("test3.xml", nil)

	if err != nil {
		t.Errorf("LoadFile(): %s", err)
		return
	}

	expected := `<root xmlns:foo="http:/example.org/foo">
  <child foo:bar="1">
    <grandchild xmlns:foo="">
      <great-grandchild bar="2">&#xA;      </great-grandchild>
    </grandchild>
  </child>
</root>
`

	if got := doc.Root.String(); got != expected {
		t.Fatalf("expected: %s\ngot: %s\n", expected, got)
	}
}

func TestStringEscaping(t *testing.T) {
	doc := New()
	err := doc.LoadFile("test4.xml", nil)

	if err != nil {
		t.Errorf("LoadFile(): %s", err)
		return
	}

	expected := `<body>  &lt;https://example.com/file/fm/SU0vRk0xLzIwMTMwOTEwLzA1MDA0MS5ybXdhdGVzdEByZXV0ZXJzLmNvbTEzNzg4NDU1OTk4OTA/Screen%20Shot%202013-09-10%20at%2021.33.54.png&gt; File Attachment:-Screen Shot 2013-09-10 at 21.33.54.png  </body>
`

	if got := doc.Root.String(); got != expected {
		t.Fatalf("expected: %s\ngot: %s\n", expected, got)
	}
}

func TestElementNodeValueFetch(t *testing.T) {
	data := `<car><color>
	r<cool />
	ed</color><brand>BMW</brand><price>50
	<cheap />.25</price><count>6
	<small />2
	</count><available>
	Tr
	<found />
	ue</available></car>`
	doc := New()

	if err := doc.LoadString(data, nil); nil != err {
		t.Fatalf("LoadString(): %s", err)
	}

	carN := doc.SelectNode("", "car")
	if v := carN.S("", "brand"); v != "BMW" {
		t.Errorf("Failed to get brand as string, got: '%s', wanted: 'BMW'", v)
	}
	if v := carN.S("", "color"); v != "red" {
		t.Errorf("Failed to get color as string, got: '%s', wanted: 'red'", v)
	}

	if v := carN.I("", "count"); v != 62 {
		t.Errorf("Failed to get count using I, got: %v, wanted: 62", v)
	}
	if v := carN.I8("", "count"); v != 62 {
		t.Errorf("Failed to get count using I8, got: %v, wanted: 62", v)
	}
	if v := carN.I16("", "count"); v != 62 {
		t.Errorf("Failed to get count using I16, got: %v, wanted: 62", v)
	}
	if v := carN.I32("", "count"); v != 62 {
		t.Errorf("Failed to get count using I32, got: %v, wanted: 62", v)
	}
	if v := carN.I64("", "count"); v != 62 {
		t.Errorf("Failed to get count using I64, got: %v, wanted: 62", v)
	}
	if v := carN.U("", "count"); v != 62 {
		t.Errorf("Failed to get count using U, got: %v, wanted: 62", v)
	}
	if v := carN.U8("", "count"); v != 62 {
		t.Errorf("Failed to get count using U8, got: %v, wanted: 62", v)
	}
	if v := carN.U16("", "count"); v != 62 {
		t.Errorf("Failed to get count using U16, got: %v, wanted: 62", v)
	}
	if v := carN.U32("", "count"); v != 62 {
		t.Errorf("Failed to get count using U32, got: %v, wanted: 62", v)
	}
	if v := carN.U64("", "count"); v != 62 {
		t.Errorf("Failed to get count using U64, got: %v, wanted: 62", v)
	}

	if v := carN.F32("", "price"); v != 50.25 {
		t.Errorf("Failed to get price using F32, got: %v, wanted: 50.25", v)
	}
	if v := carN.F64("", "price"); v != 50.25 {
		t.Errorf("Failed to get price using F64, got: %v, wanted: 50.25", v)
	}

	if v := carN.B("", "available"); v != true {
		t.Errorf("Failed to get availability using B, got: %v, wanted: true", v)
	}
}
-												Switch to more liberay CC0 public domain license

											
										
										
											2011-03-19 13:50:46 +00:00
+								// This work is subject to the CC0 1.0 Universal (CC0 1.0) Public Domain Dedication
 								// license. Its contents can be found at:
 								// http://creativecommons.org/publicdomain/zero/1.0/
-												Reworked README and xmlx/entitymap.go. Now includes conversion of named xml entities instead of just numeric ones. Eg: &pi; and &#960; to a unicode character and vice-versa. It also came to my attention that lack of a proper opensource license might put people off of using this package. I therefor added a 1-clause BSD license under which this code now falls.

											
										
										
											2010-09-26 20:59:14 +00:00
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+								package xmlx
 								import "testing"
-												Added *document.LoadUri() method to load documents from the web (over http)

											
										
										
											2009-11-25 01:50:06 +00:00
+								func TestLoadLocal(t *testing.T) {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									doc := New()
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									if err := doc.LoadFile("test.xml", nil); err != nil {
-												gofix error

											
										
										
											2011-11-02 15:50:45 +00:00
+										t.Error(err.Error())
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
 									if len(doc.Root.Children) == 0 {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("Root node has no children.")
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
 								}
-												Add wildcard support for node names. Allows selecting of all child nodes with a specific namespace (See issue #4).

											
										
										
											2011-07-13 03:17:00 +00:00
+								func TestWildcard(t *testing.T) {
 									doc := New()
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									if err := doc.LoadFile("test2.xml", nil); err != nil {
-												gofix error

											
										
										
											2011-11-02 15:50:45 +00:00
+										t.Error(err.Error())
-												Add wildcard support for node names. Allows selecting of all child nodes with a specific namespace (See issue #4).

											
										
										
											2011-07-13 03:17:00 +00:00
+										return
 									}
-												Rolled back mistaken bugfix

											
										
										
											2014-01-29 21:27:00 +00:00
+									list := doc.SelectNodes("ns", "*")
-												Reworked the tests just a bit

											
										
										
											2013-10-25 21:31:10 +00:00
+									if len(list) != 1 {
 										t.Errorf("Wrong number of child elements. Expected 1, got %d.", len(list))
-												Making the recursive SelectNodes a separate function and adding tests

											
										
										
											2012-07-29 19:28:03 +00:00
+										return
 									}
 								}
 								func TestWildcardRecursive(t *testing.T) {
 									doc := New()
 									if err := doc.LoadFile("test2.xml", nil); err != nil {
 										t.Error(err.Error())
 										return
 									}
 									list := doc.SelectNodesRecursive("ns", "*")
 									if len(list) != 7 {
 										t.Errorf("Wrong number of child elements. Expected 7, got %d.", len(list))
-												Add wildcard support for node names. Allows selecting of all child nodes with a specific namespace (See issue #4).

											
										
										
											2011-07-13 03:17:00 +00:00
+										return
 									}
 								}
 								func _TestLoadRemote(t *testing.T) {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									doc := New()
-												Added *document.LoadUri() method to load documents from the web (over http)

											
										
										
											2009-11-25 01:50:06 +00:00
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									if err := doc.LoadUri("http://blog.golang.org/feeds/posts/default", nil); err != nil {
-												gofix error

											
										
										
											2011-11-02 15:50:45 +00:00
+										t.Error(err.Error())
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										return
-												Added *document.LoadUri() method to load documents from the web (over http)

											
										
										
											2009-11-25 01:50:06 +00:00
+									}
 									if len(doc.Root.Children) == 0 {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("Root node has no children.")
 										return
-												Added *document.LoadUri() method to load documents from the web (over http)

											
										
										
											2009-11-25 01:50:06 +00:00
+									}
 								}
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+								func TestSave(t *testing.T) {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									doc := New()
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									if err := doc.LoadFile("test.xml", nil); err != nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("LoadFile(): %s", err)
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
-												Adds optional indented output for the Node.String() and Node.Bytes()
methods. Set the global `IndentPrefix` var to a tab or spaces to
enable indented output. The existing API has not been changed.

This addresses issue #7.

											
										
										
											2012-11-25 22:46:56 +00:00
+									IndentPrefix = "\t"
-												Updated code to use more recent go idioms. Yielding more compact xml processing. Hopefully better performant.

											
										
										
											2010-05-26 00:24:44 +00:00
+									if err := doc.SaveFile("test1.xml"); err != nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("SaveFile(): %s", err)
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
 								}
 								func TestNodeSearch(t *testing.T) {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									doc := New()
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									if err := doc.LoadFile("test1.xml", nil); err != nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("LoadFile(): %s", err)
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
-												Updated code to use more recent go idioms. Yielding more compact xml processing. Hopefully better performant.

											
										
										
											2010-05-26 00:24:44 +00:00
+									if node := doc.SelectNode("", "item"); node == nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("SelectNode(): No node found.")
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
-												Rolled back mistaken bugfix

											
										
										
											2014-01-29 21:27:00 +00:00
+									nodes := doc.SelectNodes("", "item")
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									if len(nodes) == 0 {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("SelectNodes(): no nodes found.")
 										return
-													new file:   README
	new file:   src/Makefile
	new file:   src/document.go
	new file:   src/io.go
	new file:   src/node.go
	new file:   src/test.xml
	new file:   src/xmlx_test.go

											
										
										
											2009-11-23 04:16:27 +00:00
+									}
-												Fixed inadvertent recursion in SelectNodes. Updated/add tests for this behavior

											
										
										
											2013-10-25 20:44:32 +00:00
 									ch := doc.SelectNode("", "channel")
-												Rolled back mistaken bugfix

											
										
										
											2014-01-29 21:27:00 +00:00
+									// Test that SelectNodes properly selects multiple nodes
-												Reworked the tests just a bit

											
										
										
											2013-10-25 21:31:10 +00:00
+									links := ch.SelectNodes("", "link")
 									if len(links) != 8 {
-												Rolled back mistaken bugfix

											
										
										
											2014-01-29 21:27:00 +00:00
+										t.Errorf("SelectNodes(): Expected 8, Got %d", len(links))
-												Fixed inadvertent recursion in SelectNodes. Updated/add tests for this behavior

											
										
										
											2013-10-25 20:44:32 +00:00
+										return
 									}
 								}
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+								type Image struct {
-												Run gofmt and fix the Unmarshal test.

											
										
										
											2012-02-29 10:42:39 +00:00
+									Title       string `xml:"title"`
 									Url         string `xml:"url"`
 									Link        string `xml:"link"`
 									Description string `xml:"description"`
 									Width       int    `xml:"width"`
 									Height      int    `xml:"height"`
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+								}
 								func TestUnmarshal(t *testing.T) {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									doc := New()
-												Remove CharsetFunc as field for Document type and instead supply it as a parameter for all Document.LoadXXX methods. There should be no need to store the function pointer in the Document struct.

											
										
										
											2012-02-29 10:21:35 +00:00
+									err := doc.LoadFile("test1.xml", nil)
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
 									if err != nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("LoadFile(): %s", err)
 										return
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+									}
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+									node := doc.SelectNode("", "image")
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+									if node == nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("SelectNode(): No node found.")
 										return
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+									}
-												Run gofmt and fix the Unmarshal test.

											
										
										
											2012-02-29 10:42:39 +00:00
+									var img Image
-												Refactored some code to be faster and a little more elegant.

											
										
										
											2010-08-22 03:07:38 +00:00
+									if err = node.Unmarshal(&img); err != nil {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("Unmarshal(): %s", err)
 										return
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+									}
 									if img.Title != "WriteTheWeb" {
-												fixed some slice copy routines

											
										
										
											2010-05-06 03:36:48 +00:00
+										t.Errorf("Image.Title has incorrect value. Got '%s', expected 'WriteTheWeb'.", img.Title)
 										return
-												Added *node.Unmarshal() method

											
										
										
											2009-12-02 18:38:35 +00:00
+									}
 								}
-												fix: String() handled ns prefixes incorrectly

Previously parsing a document like this:

  <root xmlns:foo='http:/example.org/foo' foo:bar='1'/>

Would produce an incorrect document when turning it back into a string:

  <root xmlns:foo='http:/example.org/foo' http:/example.org/foo:bar='1'/>

This patch fixes this by implementing a proper lookup for namespace
prefixes.

											
										
										
											2013-09-03 22:35:22 +00:00
-												Implement text nodes / content escaping

This removes the support for re-indenting XML, but allows to faithfully
parse / stringify a document. Beautification could be added again later
on if desired.

											
										
										
											2013-09-10 21:18:13 +00:00
+								func TestStringNamespaces(t *testing.T) {
-												fix: String() handled ns prefixes incorrectly

Previously parsing a document like this:

  <root xmlns:foo='http:/example.org/foo' foo:bar='1'/>

Would produce an incorrect document when turning it back into a string:

  <root xmlns:foo='http:/example.org/foo' http:/example.org/foo:bar='1'/>

This patch fixes this by implementing a proper lookup for namespace
prefixes.

											
										
										
											2013-09-03 22:35:22 +00:00
+									doc := New()
 									err := doc.LoadFile("test3.xml", nil)
 									if err != nil {
 										t.Errorf("LoadFile(): %s", err)
 										return
 									}
 									expected := `<root xmlns:foo="http:/example.org/foo">
-												Implement text nodes / content escaping

This removes the support for re-indenting XML, but allows to faithfully
parse / stringify a document. Beautification could be added again later
on if desired.

											
										
										
											2013-09-10 21:18:13 +00:00
+								  <child foo:bar="1">
 								    <grandchild xmlns:foo="">
 								      <great-grandchild bar="2">&#xA;      </great-grandchild>
 								    </grandchild>
 								  </child>
-												fix: String() handled ns prefixes incorrectly

Previously parsing a document like this:

  <root xmlns:foo='http:/example.org/foo' foo:bar='1'/>

Would produce an incorrect document when turning it back into a string:

  <root xmlns:foo='http:/example.org/foo' http:/example.org/foo:bar='1'/>

This patch fixes this by implementing a proper lookup for namespace
prefixes.

											
										
										
											2013-09-03 22:35:22 +00:00
+								</root>
 								`
 									if got := doc.Root.String(); got != expected {
 										t.Fatalf("expected: %s\ngot: %s\n", expected, got)
 									}
 								}
-												Implement text nodes / content escaping

This removes the support for re-indenting XML, but allows to faithfully
parse / stringify a document. Beautification could be added again later
on if desired.

											
										
										
											2013-09-10 21:18:13 +00:00
 								func TestStringEscaping(t *testing.T) {
 									doc := New()
 									err := doc.LoadFile("test4.xml", nil)
 									if err != nil {
 										t.Errorf("LoadFile(): %s", err)
 										return
 									}
 									expected := `<body>  &lt;https://example.com/file/fm/SU0vRk0xLzIwMTMwOTEwLzA1MDA0MS5ybXdhdGVzdEByZXV0ZXJzLmNvbTEzNzg4NDU1OTk4OTA/Screen%20Shot%202013-09-10%20at%2021.33.54.png&gt; File Attachment:-Screen Shot 2013-09-10 at 21.33.54.png  </body>
 								`
 									if got := doc.Root.String(); got != expected {
 										t.Fatalf("expected: %s\ngot: %s\n", expected, got)
 									}
 								}
-												Fix the value fetcher to deal with NT_TEXT nodes.

Due to the introduction of the NT_TEXT nodes, all of the simple value fetchers
broke.  To fix, make a GetValue function fetch all NT_TEXT nodes for
NT_ELEMENT nodes, and put them together, using string.TrimSpace to remove
extra, maintaining compatibility.  Then change the .S function to use GetValue
to fetch a node's content. Also, make all the other value fetchers use .S to
get a string representation to convert, centralizing the implementation.  Use
.S to centralize the handling of a not found node.

Also, implement a series of tests to verify all the fetcher's functionality.
The test is based on the test produced by Tim Jurcka.

											
										
										
											2013-09-30 05:09:47 +00:00
 								func TestElementNodeValueFetch(t *testing.T) {
 									data := `<car><color>
 									r<cool />
 									ed</color><brand>BMW</brand><price>50
 									<cheap />.25</price><count>6
 									<small />2
 									</count><available>
 									Tr
 									<found />
 									ue</available></car>`
 									doc := New()
 									if err := doc.LoadString(data, nil); nil != err {
 										t.Fatalf("LoadString(): %s", err)
 									}
 									carN := doc.SelectNode("", "car")
 									if v := carN.S("", "brand"); v != "BMW" {
 										t.Errorf("Failed to get brand as string, got: '%s', wanted: 'BMW'", v)
 									}
 									if v := carN.S("", "color"); v != "red" {
 										t.Errorf("Failed to get color as string, got: '%s', wanted: 'red'", v)
 									}
 									if v := carN.I("", "count"); v != 62 {
 										t.Errorf("Failed to get count using I, got: %v, wanted: 62", v)
 									}
 									if v := carN.I8("", "count"); v != 62 {
 										t.Errorf("Failed to get count using I8, got: %v, wanted: 62", v)
 									}
 									if v := carN.I16("", "count"); v != 62 {
 										t.Errorf("Failed to get count using I16, got: %v, wanted: 62", v)
 									}
 									if v := carN.I32("", "count"); v != 62 {
 										t.Errorf("Failed to get count using I32, got: %v, wanted: 62", v)
 									}
 									if v := carN.I64("", "count"); v != 62 {
 										t.Errorf("Failed to get count using I64, got: %v, wanted: 62", v)
 									}
 									if v := carN.U("", "count"); v != 62 {
 										t.Errorf("Failed to get count using U, got: %v, wanted: 62", v)
 									}
 									if v := carN.U8("", "count"); v != 62 {
 										t.Errorf("Failed to get count using U8, got: %v, wanted: 62", v)
 									}
 									if v := carN.U16("", "count"); v != 62 {
 										t.Errorf("Failed to get count using U16, got: %v, wanted: 62", v)
 									}
 									if v := carN.U32("", "count"); v != 62 {
 										t.Errorf("Failed to get count using U32, got: %v, wanted: 62", v)
 									}
 									if v := carN.U64("", "count"); v != 62 {
 										t.Errorf("Failed to get count using U64, got: %v, wanted: 62", v)
 									}
 									if v := carN.F32("", "price"); v != 50.25 {
 										t.Errorf("Failed to get price using F32, got: %v, wanted: 50.25", v)
 									}
 									if v := carN.F64("", "price"); v != 50.25 {
 										t.Errorf("Failed to get price using F64, got: %v, wanted: 50.25", v)
 									}
 									if v := carN.B("", "available"); v != true {
 										t.Errorf("Failed to get availability using B, got: %v, wanted: true", v)
 									}
 								}