From b99fd9368443cd66bffa397d47f1f4150adf383a Mon Sep 17 00:00:00 2001 From: Ben Alpert Date: Sun, 24 Nov 2013 01:21:21 -0500 Subject: [PATCH] 'getElementsByClassName' doesn't exist in IE8 Fixes #532. --- src/core/__tests__/ReactMountDestruction-test.js | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/core/__tests__/ReactMountDestruction-test.js b/src/core/__tests__/ReactMountDestruction-test.js index e476dd0864b..6dc330862dc 100644 --- a/src/core/__tests__/ReactMountDestruction-test.js +++ b/src/core/__tests__/ReactMountDestruction-test.js @@ -43,17 +43,13 @@ describe('ReactMount', function() { React.renderComponent(instanceTwo, secondRootDiv); // Test that two react roots are rendered in isolation - expect(firstRootDiv.getElementsByClassName('firstReactDiv').length) - .toBe(1); - expect(secondRootDiv.getElementsByClassName('secondReactDiv').length) - .toBe(1); + expect(firstRootDiv.firstChild.className).toBe('firstReactDiv'); + expect(secondRootDiv.firstChild.className).toBe('secondReactDiv'); // Test that after unmounting each, they are no longer in the document. React.unmountComponentAtNode(firstRootDiv); - expect(firstRootDiv.getElementsByClassName('firstReactDiv').length) - .toBe(0); + expect(firstRootDiv.firstChild).toBeNull(); React.unmountComponentAtNode(secondRootDiv); - expect(secondRootDiv.getElementsByClassName('secondReactDiv').length) - .toBe(0); + expect(secondRootDiv.firstChild).toBeNull(); }); });