From f26cd40ad65b6e1b8ec1c0ae3b56b4e7ccdd9318 Mon Sep 17 00:00:00 2001
From: Ashley Grant <ashley@ashleygrant.com>
Date: Wed, 9 Mar 2016 14:49:54 -0500
Subject: [PATCH] feat(syntax-interpreter): single quoted strings in options
 attributes

---
 src/syntax-interpreter.js       | 14 ++++++++++-
 test/syntax-interpreter.spec.js | 43 +++++++++++++++++++++++++++++++++
 2 files changed, 56 insertions(+), 1 deletion(-)

diff --git a/src/syntax-interpreter.js b/src/syntax-interpreter.js
index 651191b..f6bf23c 100644
--- a/src/syntax-interpreter.js
+++ b/src/syntax-interpreter.js
@@ -111,11 +111,13 @@ export class SyntaxInterpreter {
     let current;
     let i;
     let ii;
+    let inString = false;
+    let inEscape = false
 
     for (i = 0, ii = attrValue.length; i < ii; ++i) {
       current = attrValue[i];
 
-      if (current === ';') {
+      if (current === ';' && !inString) {
         info = language.inspectAttribute(resources, name, target.trim());
         language.createAttributeInstruction(resources, element, info, instruction, context);
 
@@ -128,9 +130,19 @@ export class SyntaxInterpreter {
       } else if (current === ':' && name === null) {
         name = target.trim();
         target = '';
+      } else if (current === '\\') {
+        target += current;
+        inEscape = true;
+          continue;
       } else {
         target += current;
+
+        if(name !== null && inEscape === false && current === '\'') {
+          inString = !inString;
+        }
       }
+
+      inEscape = false;
     }
 
     if (name !== null) {
diff --git a/test/syntax-interpreter.spec.js b/test/syntax-interpreter.spec.js
index 5d36904..5e04496 100644
--- a/test/syntax-interpreter.spec.js
+++ b/test/syntax-interpreter.spec.js
@@ -152,6 +152,49 @@ describe('SyntaxInterpreter', () => {
       expect(instruction.attributes.key).toBe('foo');
       expect(instruction.attributes.value).toBe('bar');
     });
+  });
+
+  describe('options attributes', () => {
+    var interpreter, info;
 
+    beforeAll(() => {
+      interpreter = new SyntaxInterpreter(new Parser(), new ObserverLocator(), new EventManager());
+
+      interpreter.language = {
+        inspectAttribute(resources, attrName, attrValue) {
+          return {
+            attrName: attrName,
+            attrValue: attrValue
+          };
+        },
+        createAttributeInstruction() {
+          return;
+        }
+      }
+
+      info = {
+        attrName: 'custom',
+        command: null,
+        defaultBindingMode: 1
+      };
+    });
+
+    it('handles a semicolon inside a string of an options attribute', () => {
+      info.attrValue = "foo: 'bar;';";
+      var instruction = interpreter.options({}, null, info, null);
+      expect(instruction.attributes['foo']).toBe("'bar;'");
+    });
+
+    it('handles an escaped single quote inside a string of an options attribute', () => {
+      info.attrValue = "foo: 'bar\\'';";
+      var instruction = interpreter.options({}, null, info, null);
+      expect(instruction.attributes['foo']).toBe("'bar\\''");
+    });
+
+    it('handles an escaped single quote and a semicolon inside a string of an options attribute', () => {
+      info.attrValue = "foo: 'bar\\';';";
+      var instruction = interpreter.options({}, null, info, null);
+      expect(instruction.attributes['foo']).toBe("'bar\\';'");
+    });
   });
 });