{"body":"If we're going to build utilities off of this, we should have good test\ncoverage.","files":[{"path":"src\/utils\/__tests__\/traverseAllChildren-test.js","status":"A"}],"parents":["946029c9217362cd6cd638c967e9b31659983c22"],"commit":{"revision":"5d4f9034824ca0af60db9661628f0b5b0f165c50","authorName":"Jordan Walke","authoredAt":"2013-07-16T11:29:09-07:00","subject":"[React Children] Step Three: New test case for `traverseChildren`.","authorEmail":"jordanjcw@fb.com","shortRevision":"5d4f903482"},"patch":"diff --git a\/src\/utils\/__tests__\/traverseAllChildren-test.js b\/src\/utils\/__tests__\/traverseAllChildren-test.js\nnew file mode 100644\nindex 0000000000..071aa9c6fa\n--- \/dev\/null\n+++ b\/src\/utils\/__tests__\/traverseAllChildren-test.js\n@@ -0,0 +1,233 @@\n+\/**\n+ * Copyright 2013 Facebook, Inc.\n+ *\n+ * Licensed under the Apache License, Version 2.0 (the \"License\");\n+ * you may not use this file except in compliance with the License.\n+ * You may obtain a copy of the License at\n+ *\n+ * http:\/\/www.apache.org\/licenses\/LICENSE-2.0\n+ *\n+ * Unless required by applicable law or agreed to in writing, software\n+ * distributed under the License is distributed on an \"AS IS\" BASIS,\n+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n+ * See the License for the specific language governing permissions and\n+ * limitations under the License.\n+ *\n+ * @emails react-core\n+ * @jsx React.DOM\n+ *\/\n+\n+\"use strict\";\n+\n+describe('traverseAllChildren', function() {\n+ var traverseAllChildren;\n+ var React;\n+ beforeEach(function() {\n+ traverseAllChildren = require('traverseAllChildren');\n+ React = require('React');\n+ });\n+\n+\n+ it('should support identity for simple', function() {\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var simpleKid = ;\n+\n+ \/\/ Jasmine doesn't provide a way to test that the fn was invoked with scope.\n+ var instance =

{simpleKid}<\/div>;\n+ traverseAllChildren(instance.props.children, traverseFn, traverseContext);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ simpleKid,\n+ '[simple]',\n+ 0\n+ );\n+ expect(traverseContext.length).toEqual(1);\n+ });\n+\n+ it('should treat single arrayless child as being in array', function() {\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var simpleKid = ;\n+ var instance =

{simpleKid}<\/div>;\n+ traverseAllChildren(instance.props.children, traverseFn, traverseContext);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ simpleKid,\n+ '[0]',\n+ 0\n+ );\n+ expect(traverseContext.length).toEqual(1);\n+ });\n+\n+ it('should treat single child in array as expected', function() {\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var simpleKid = ;\n+ var instance =

{[simpleKid]}<\/div>;\n+ traverseAllChildren(instance.props.children, traverseFn, traverseContext);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ simpleKid,\n+ '[0]',\n+ 0\n+ );\n+ expect(traverseContext.length).toEqual(1);\n+ });\n+\n+ it('should be called for each child', function() {\n+ var zero =

;\n+ var one = null;\n+ var two =

;\n+ var three = null;\n+ var four =

;\n+\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var instance = (\n+

\n+ {zero}\n+ {one}\n+ {two}\n+ {three}\n+ {four}\n+ <\/div>\n+ );\n+\n+ traverseAllChildren(instance.props.children, traverseFn, traverseContext);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ zero,\n+ '[keyZero]',\n+ 0\n+ );\n+ expect(traverseFn).toHaveBeenCalledWith(traverseContext, one, '[1]', 1);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ two,\n+ '[keyTwo]',\n+ 2\n+ );\n+ expect(traverseFn).toHaveBeenCalledWith(traverseContext, three, '[3]', 3);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ four,\n+ '[keyFour]',\n+ 4\n+ );\n+ });\n+\n+ \/\/ Todo: test that nums\/strings are converted to ReactComponents.\n+\n+ it('should be called for each child in nested structure', function() {\n+ var zero =

;\n+ var one = null;\n+ var two =

;\n+ var three = null;\n+ var four =

;\n+ var five =

;\n+ \/\/ Name precedence is as follows:\n+ \/\/ 1. JavaScript Object key if in a JavaScript object:\n+ \/\/ 2. If grouped in an Array, the `key` attribute.\n+ \/\/ 3. The array index if in a JavaScript Array.\n+\n+\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var instance = (\n+

{\n+ [{\n+ firstHalfKey: [zero, one, two],\n+ secondHalfKey: [three, four],\n+ keyFive: five\n+ }]\n+ }<\/div>\n+ );\n+\n+ traverseAllChildren(instance.props.children, traverseFn, traverseContext);\n+ expect(traverseFn.calls.length).toBe(6);\n+ expect(traverseContext.length).toEqual(6);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ zero,\n+ '[0]{firstHalfKey}[keyZero]',\n+ 0\n+ );\n+\n+ expect(traverseFn)\n+ .toHaveBeenCalledWith(traverseContext, one, '[0]{firstHalfKey}[1]', 1);\n+\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ two,\n+ '[0]{firstHalfKey}[keyTwo]',\n+ 2\n+ );\n+\n+ expect(traverseFn)\n+ .toHaveBeenCalledWith(traverseContext, three, '[0]{secondHalfKey}[0]', 3);\n+\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ four,\n+ '[0]{secondHalfKey}[keyFour]',\n+ 4\n+ );\n+\n+ expect(traverseFn)\n+ .toHaveBeenCalledWith(traverseContext, five, '[0]{keyFive}', 5);\n+ });\n+\n+ it('should retain key across two mappings', function() {\n+ var zeroForceKey =

;\n+ var oneForceKey =

;\n+ var traverseContext = [];\n+ var traverseFn =\n+ jasmine.createSpy().andCallFake(function (context, kid, key, index) {\n+ context.push(true);\n+ });\n+\n+ var forcedKeys = (\n+

\n+ {zeroForceKey}\n+ {oneForceKey}\n+ <\/div>\n+ );\n+\n+ traverseAllChildren(forcedKeys.props.children, traverseFn, traverseContext);\n+ expect(traverseContext.length).toEqual(2);\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ zeroForceKey,\n+ '[keyZero]',\n+ 0\n+ );\n+ expect(traverseFn).toHaveBeenCalledWith(\n+ traverseContext,\n+ oneForceKey,\n+ '[keyOne]',\n+ 1\n+ );\n+ });\n+\n+});\n"}

dillon1000/react

Files changed