Merge pull request #50 from Financial-Times/add-video-modifier

Add 'type' of content as a modifier

src/presenters/teaser-presenter.js

@@ -1,3 +1,5 @@
+const hyphenatePascalCase = require('../utils/hyphenate-pascal-case');
+
 const ONE_HOUR = 1000 * 60 * 60;
 const MAX_RELATED_CONTENT = 3;
 const HEADSHOT_BASE_URL = 'https://www.ft.com/__origami/service/image/v2/images/raw/';
@@ -51,6 +53,9 @@ const TeaserPresenter = class TeaserPresenter {
 		if (this.data.isEditorsChoice) {
 			mods.push('highlight');
 		}
+		if (this.data.type) {
+			mods.push(hyphenatePascalCase(this.data.type));
+		}
 		return mods;
 	}
 

src/utils/hyphenate-pascal-case.js

@@ -0,0 +1,4 @@
+module.exports = value => {
+	return value.replace(/([a-z])([A-Z])/g, '$1-$2')
+		.toLowerCase();
+};

tests/presenters/teaser-presenter.spec.js

@@ -33,13 +33,13 @@ describe('Teaser Presenter', () => {
 			it('returns has-headshot when correct template and has-headshot', () => {
 				const content = Object.assign({}, articleOpinionAuthorFixture, {template: 'standard'});
 				subject = new Presenter(content);
-				expect(subject.classModifiers).to.deep.equal(['has-headshot']);
+				expect(subject.classModifiers).to.include('has-headshot');
 			});
 
 			it('does not add a modifier if wrong template but has headshot', () => {
 				const content = Object.assign({}, articleOpinionAuthorFixture, {template: 'no-headshot'});
 				subject = new Presenter(content);
-				expect(subject.classModifiers.length).to.equal(0);
+				expect(subject.classModifiers).to.not.include('has-headshot');
 			});
 
 			it('does not add a modifier if correct template but does not have a headshot', () => {
@@ -98,6 +98,26 @@ describe('Teaser Presenter', () => {
 			});
 		});
 
+		context('type', () => {
+			it('returns `article` when content has `type` set to `Article`', () => {
+				const content = {type: 'Article'};
+				subject = new Presenter(content);
+				expect(subject.classModifiers).to.deep.equal(['article']);
+			});
+
+			it('returns `video` when content has `type` set to `Video`', () => {
+				const content = {type: 'Video'};
+				subject = new Presenter(content);
+				expect(subject.classModifiers).to.deep.equal(['video']);
+			});
+
+			it('returns `live-blog` when content has `type` set to `LiveBlog`', () => {
+				const content = {type: 'LiveBlog'};
+				subject = new Presenter(content);
+				expect(subject.classModifiers).to.deep.equal(['live-blog']);
+			});
+		});
+
 	});
 
 	context('displayTag', () => {

tests/utils/hyphenate-pascal-case.spec.js

@@ -0,0 +1,23 @@
+require('chai').should();
+
+const hyphenatePascalCase = require('../../src/utils/hyphenate-pascal-case');
+
+describe('Utils', () => {
+
+	describe('Hyphenate Pascal Case', () => {
+
+		it('should exist', () => {
+			hyphenatePascalCase.should.exist;
+		});
+
+		it('should hyphenate pascal case, e.g. `FooBar` to `foo-bar`', () => {
+			hyphenatePascalCase('FooBar').should.equal('foo-bar');
+		});
+
+		it('should handle a single ‘word’', () => {
+			hyphenatePascalCase('Foo').should.equal('foo');
+		});
+
+	});
+
+});