From d5e970b93fc0e5ab9d284c265b170eecee2ad0ad Mon Sep 17 00:00:00 2001
From: yungsters <yungsters@fb.com>
Date: Sun, 28 Jul 2013 01:05:13 -0700
Subject: [PATCH] Fix `Danger` test failures.

The original tests were flawed because the `Danger` module exploits the fact that all React-generated markup has at least one attribute. This allows the module to extract node names from markup strings faster.

However, the tests were passing in strings of markup with no attributes.

Also, this fixes a test failure due to the test trying to set text content into a `<tr>` which is typically disallowed by browsers (and PhantomJS). This changes it to use `<td>` instead.
---
 src/dom/__tests__/Danger-test.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/dom/__tests__/Danger-test.js b/src/dom/__tests__/Danger-test.js
index 62bccb41d1..de1ca88b44 100644
--- a/src/dom/__tests__/Danger-test.js
+++ b/src/dom/__tests__/Danger-test.js
@@ -59,7 +59,7 @@ describe('Danger', function() {
 
     it('should render lists of markup with similar `nodeName`', function() {
       var renderedMarkup = Danger.dangerouslyRenderMarkup(
-        ['<p>1</p>', '<p>2</p>', '<p>3</p>']
+        ['<p id="A">1</p>', '<p id="B">2</p>', '<p id="C">3</p>']
       );
 
       expect(renderedMarkup.length).toBe(3);
@@ -75,13 +75,13 @@ describe('Danger', function() {
 
     it('should render lists of markup with different `nodeName`', function() {
       var renderedMarkup = Danger.dangerouslyRenderMarkup(
-        ['<p>1</p>', '<tr>2</tr>', '<p>3</p>']
+        ['<p id="A">1</p>', '<td id="B">2</td>', '<p id="C">3</p>']
       );
 
       expect(renderedMarkup.length).toBe(3);
 
       expect(renderedMarkup[0].nodeName).toBe('P');
-      expect(renderedMarkup[1].nodeName).toBe('TR');
+      expect(renderedMarkup[1].nodeName).toBe('TD');
       expect(renderedMarkup[2].nodeName).toBe('P');
 
       expect(renderedMarkup[0].innerHTML).toBe('1');