formatLabel: dont print -Yes- as label

diff --git a/src/lib/helpers.js b/src/lib/helpers.js
index a2e5d82c78d2bebc1427e167065e07a90edcc6d8..de1cfd41b2fd3dad879a784645ab2f8effeee352 100644 (file)
--- a/src/lib/helpers.js
+++ b/src/lib/helpers.js
@@ -41,8 +41,8 @@ function formatLabel(aPlace) {
     return s && s[0].toUpperCase() + s.slice(1);
   }
 
-  if (aPlace.type && aPlace.type === 'yes' && aPlace.class) {
-    return capitalize(aPlace.class.replace(/_/g, ' '));
+  if (aPlace.type && aPlace.type === 'yes' && aPlace.category) {
+    return capitalize(aPlace.category.replace(/_/g, ' '));
   }
   if (aPlace.type) {
     return capitalize(aPlace.type.replace(/_/g, ' '));

diff --git a/test/unit/helpers.js b/test/unit/helpers.js
new file mode 100644 (file)
index 0000000..9570059
--- /dev/null
+++ b/test/unit/helpers.js
@@ -0,0 +1,37 @@
+const assert = require('assert');
+const helpers = require('../../src/lib/helpers');
+
+describe('Helpers', function () {
+
+  it('.formatLabel', function () {
+    // not enough data
+    assert.equal(helpers.formatLabel({}), '');
+
+    // if label set, that becomes the label
+    assert.equal(helpers.formatLabel({ label: 'A Label' }), 'A Label');
+
+    // type, but nicely formatted
+    assert.equal(helpers.formatLabel({ category: 'highway', type: 'bus_stop' }), 'Bus stop');
+
+    // type=yes, so we use the category
+    assert.equal(helpers.formatLabel({ category: 'building', type: 'yes' }), 'Building');
+  });
+
+  it('.wikipediaLink', function () {
+    assert.equal(
+      helpers.wikipediaLink({}),
+      ''
+    );
+
+    assert.equal(
+      helpers.wikipediaLink({ calculated_wikipedia: 'de:Brandenburg Gate' }),
+      '<a href="https://de.wikipedia.org/wiki/Brandenburg Gate" target="_blank">de:Brandenburg Gate</a>'
+    );
+
+    // title includes HTML
+    assert.equal(
+      helpers.wikipediaLink({ calculated_wikipedia: 'en:Slug & Lattuce' }),
+      '<a href="https://en.wikipedia.org/wiki/Slug &amp; Lattuce" target="_blank">en:Slug &amp; Lattuce</a>'
+    );
+  });
+});