feat: news.natgeo parser (#88)

* feat: natgeo parser

For some reason, the local copy of the article didn’t grab the author
name in it, so I couldn’t figure out how to parse it. The generic
parser took a name of an author of a paper mentioned in the article,
and thought that was the author name, which was funny.

I cleaned a large block quote that didn’t make sense as it was shown in
the preview, although I noticed that the Mercury chrome extension
didn’t even display it.

* fix: add date_published transform

* fix: date_published assertion

* disable: author assertion, generlize author selector

* rm: author assertion

* fix: image lead

* fix: guard agaist missing img url

* fix: generalize dek and title selectors

Author: janetleekim

Diff for: fixtures/news.nationalgeographic.com/1481919545107.html

@@ -37,6 +37,7 @@ export * from './www.reuters.com';
 export * from './mashable.com';
 export * from './www.chicagotribune.com';
 export * from './www.vox.com';
+export * from './news.nationalgeographic.com';
 export * from './www.nationalgeographic.com';
 export * from './www.latimes.com';
 export * from './pagesix.com';

Diff for: src/extractors/custom/index.js

@@ -0,0 +1,64 @@
+export const NewsNationalgeographicComExtractor = {
+  domain: 'news.nationalgeographic.com',
+
+  title: {
+    selectors: [
+      'h1',
+      'h1.main-title',
+    ],
+  },
+
+  author: {
+    selectors: [
+      '.byline-component__contributors b span',
+    ],
+  },
+
+  date_published: {
+    selectors: [
+       ['meta[name="article:published_time"]', 'value'],
+    ],
+    format: 'ddd MMM DD HH:mm:ss zz YYYY',
+    timezone: 'EST',
+  },
+
+  dek: {
+    selectors: [
+      '.article__deck',
+    ],
+  },
+
+  lead_image_url: {
+    selectors: [
+      ['meta[name="og:image"]', 'value'],
+    ],
+  },
+
+  content: {
+    selectors: [
+      ['.parsys.content', '.__image-lead__'],
+      '.content',
+    ],
+
+    // Is there anything in the content you selected that needs transformed
+    // before it's consumable content? E.g., unusual lazy loaded images
+    transforms: {
+      '.parsys.content': ($node, $) => {
+        const $imgSrc = $node.find('.image.parbase.section')
+            .find('.picturefill')
+            .first()
+            .data('platform-src');
+        if ($imgSrc) {
+          $node.prepend($(`<img class="__image-lead__" src="${$imgSrc}"/>`));
+        }
+      },
+    },
+
+    // Is there anything that is in the result that shouldn't be?
+    // The clean selectors will remove anything that matches from
+    // the result
+    clean: [
+      '.pull-quote.pull-quote--large',
+    ],
+  },
+};

Diff for: src/extractors/custom/news.nationalgeographic.com/index.js

@@ -0,0 +1,87 @@
+import assert from 'assert';
+import fs from 'fs';
+import URL from 'url';
+import cheerio from 'cheerio';
+
+import Mercury from 'mercury';
+import getExtractor from 'extractors/get-extractor';
+import { excerptContent } from 'utils/text';
+
+describe('NewsNationalgeographicComExtractor', () => {
+  describe('initial test case', () => {
+    let result;
+    let url;
+    beforeAll(() => {
+      url =
+        'http://news.nationalgeographic.com/energy/2015/08/150803-gender-bias-affects-office-heating-cooling-temperatures/';
+      const html =
+        fs.readFileSync('./fixtures/news.nationalgeographic.com/1481919545107.html');
+      result =
+        Mercury.parse(url, html, { fallback: false });
+    });
+
+    it('is selected properly', () => {
+      // This test should be passing by default.
+      // It sanity checks that the correct parser
+      // is being selected for URLs from this domain
+      const extractor = getExtractor(url);
+      assert.equal(extractor.domain, URL.parse(url).hostname);
+    });
+
+    it('returns the title', async () => {
+    // To pass this test, fill out the title selector
+    // in ./src/extractors/custom/news.nationalgeographic.com/index.js.
+      const { title } = await result;
+
+    // Update these values with the expected values from
+    // the article.
+      assert.equal(title, 'Here’s Why Your Office May Be Too Hot or Cold: Gender Bias');
+    });
+
+    it('returns the date_published', async () => {
+    // To pass this test, fill out the date_published selector
+    // in ./src/extractors/custom/news.nationalgeographic.com/index.js.
+      const { date_published } = await result;
+
+    // Update these values with the expected values from
+    // the article.
+      assert.equal(date_published, '2015-08-03T17:45:00.000Z');
+    });
+
+    it('returns the dek', async () => {
+    // To pass this test, fill out the dek selector
+    // in ./src/extractors/custom/news.nationalgeographic.com/index.js.
+      const { dek } = await result;
+
+    // Update these values with the expected values from
+    // the article.
+      assert.equal(dek, 'Do you argue about the temperature in your office or home? Find out what often decides it, and tell us your preference.');
+    });
+
+    it('returns the lead_image_url', async () => {
+    // To pass this test, fill out the lead_image_url selector
+    // in ./src/extractors/custom/news.nationalgeographic.com/index.js.
+      const { lead_image_url } = await result;
+
+    // Update these values with the expected values from
+    // the article.
+      assert.equal(lead_image_url, 'http://news.nationalgeographic.com/content/dam/news/2015/08/03/temperaturegenderbias/02tempgenderbias.ngsversion.1438795800319.jpg');
+    });
+
+    it('returns the content', async () => {
+      // To pass this test, fill out the content selector
+      // in ./src/extractors/custom/news.nationalgeographic.com/index.js.
+      // You may also want to make use of the clean and transform
+      // options.
+      const { content } = await result;
+
+      const $ = cheerio.load(content || '');
+
+      const first13 = excerptContent($('*').first().text(), 13);
+
+      // Update these values with the expected values from
+      // the article.
+      assert.equal(first13, 'Many couples fight about it at home. No, it’s not money, sex, or');
+    });
+  });
+});