From 66586bf622a603a88bc9b98677ff9e38531b1464 Mon Sep 17 00:00:00 2001 From: Rachel Fenichel Date: Fri, 3 Nov 2017 15:15:32 -0700 Subject: [PATCH] Add a test for case-sensitivity in the variable names, ids, etc. in the flyout --- tests/jsunit/xml_test.js | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/tests/jsunit/xml_test.js b/tests/jsunit/xml_test.js index 700f632f6..1a6dbce3d 100644 --- a/tests/jsunit/xml_test.js +++ b/tests/jsunit/xml_test.js @@ -364,3 +364,28 @@ function test_variablesToDom_noVariables() { assertEquals(1, resultDom.children.length); xmlTest_tearDown(); } + +function test_variableFieldXml_caseSensitive() { + var id = 'testId'; + var type = 'testType'; + var name = 'testName'; + + var mockVariableModel = { + type: type, + name: name, + getId: function() { + return id; + } + }; + + var generatedXml = Blockly.Variables.generateVariableFieldXml_(mockVariableModel); + // The field contains this XML tag as a result of how we're generating this + // XML. This is not desirable, but the goal of this test is to make sure + // we're preserving case-sensitivity. + var xmlns = 'xmlns="http://www.w3.org/1999/xhtml"'; + var goldenXml = + '' + name + ''; + assertEquals(goldenXml, generatedXml); +}