Add support for single element lists (again)

src/index.js

@@ -73,9 +73,10 @@ export function parseValue(value) {
 }
 
 export function parseList(list) {
-  return `(${list
+  const listContent = list
     .map(value => parseValue(value))
-    .join(',')})`;
+    .join(',');
+  return listContent ? `(${listContent},)` : `()`;
 }
 
 export function parseMap(map) {

test/fixtures-json5/empty-lists/style.scss

@@ -0,0 +1,5 @@
+@import 'variables.json5';
+
+body {
+  color: type-of($colors);
+}

test/fixtures-json5/empty-lists/variables.json5

@@ -0,0 +1,3 @@
+{
+  'colors': [], // Empty array
+}

test/fixtures-json5/one-element-lists/style.scss

@@ -0,0 +1,5 @@
+@import 'variables.json5';
+
+body {
+  color: type-of($colors);
+}

test/fixtures-json5/one-element-lists/variables.json5

@@ -0,0 +1,3 @@
+{
+  'colors': ['#c33'], // Array with one item
+}

test/fixtures/empty-lists/style.scss

@@ -0,0 +1,5 @@
+@import 'variables.json';
+
+body {
+  color: type-of($colors);
+}

test/fixtures/empty-lists/variables.json

@@ -0,0 +1,3 @@
+{
+  "colors": []
+}

test/fixtures/one-element-lists/style.scss

@@ -0,0 +1,5 @@
+@import 'variables.json';
+
+body {
+  color: type-of($colors);
+}

test/fixtures/one-element-lists/variables.json

@@ -0,0 +1,3 @@
+{
+  "colors": ["#c33"]
+}

test/index.js

@@ -51,6 +51,24 @@ describe('Import type test (JSON)', function() {
     expect(result.css.toString()).to.eql(EXPECTATION);
   });
 
+  it('imports one element lists correctly', function() {
+    let result = sass.renderSync({
+      file: './test/fixtures/one-element-lists/style.scss',
+      importer: jsonImporter(),
+    });
+
+    expect(result.css.toString()).to.eql('body {\n  color: list; }\n');
+  });
+
+  it('imports empty lists correctly', function() {
+    let result = sass.renderSync({
+      file: './test/fixtures/empty-lists/style.scss',
+      importer: jsonImporter(),
+    });
+
+    expect(result.css.toString()).to.eql('body {\n  color: list; }\n');
+  });
+
   it('imports maps', function() {
     let result = sass.renderSync({
       file: './test/fixtures/maps/style.scss',
@@ -174,6 +192,24 @@ describe('Import type test (JSON5)', function() {
     expect(result.css.toString()).to.eql(EXPECTATION);
   });
 
+  it('imports one element lists correctly', function() {
+    let result = sass.renderSync({
+      file: './test/fixtures-json5/one-element-lists/style.scss',
+      importer: jsonImporter(),
+    });
+
+    expect(result.css.toString()).to.eql('body {\n  color: list; }\n');
+  });
+
+  it('imports empty lists correctly', function() {
+    let result = sass.renderSync({
+      file: './test/fixtures-json5/empty-lists/style.scss',
+      importer: jsonImporter(),
+    });
+
+    expect(result.css.toString()).to.eql('body {\n  color: list; }\n');
+  });
+
   it('imports maps', function() {
     let result = sass.renderSync({
       file: './test/fixtures-json5/maps/style.scss',
@@ -279,11 +315,11 @@ describe('Import type test (JSON5)', function() {
 });
 
 describe('parseValue', function() {
-  it('returns comma-separated items wrapped in parentheses for an array', function() {
-    expect(parseValue(['some', 'entries'])).to.eql('(some,entries)');
+  it('returns comma-separated items wrapped in parentheses with trailing comma for an array', function() {
+    expect(parseValue(['some', 'entries'])).to.eql('(some,entries,)');
   });
 
-  it('calls comma-separated key value pairs wrapped in parentheses for an object', function() {
+  it('returns comma-separated key value pairs wrapped in parentheses for an object', function() {
     expect(parseValue({'key1': 'value1', 'key2': 'value2'})).to.eql('(key1: value1,key2: value2)');
   });