fix(project): Throws error when README does not contain a summary

lib/__tests__/__fixtures__/01-a-project-with-learning-objectives/README.md

@@ -1 +1,15 @@
 # A project
+
+## Índice
+
+Blah blah blah
+
+***
+
+## 1. Preámbulo
+
+Blah blah blah
+
+## 2. Resumen del proyecto
+
+Blah blah blah

lib/__tests__/__fixtures__/01-a-project-with-lo-needing-expansion/README.md

@@ -1 +1,15 @@
-# Foo
+# A project
+
+## Índice
+
+Blah blah blah
+
+***
+
+## 1. Preámbulo
+
+Blah blah blah
+
+## 2. Resumen del proyecto
+
+Blah blah blah

lib/__tests__/__fixtures__/01-a-project-with-unknown-learning-objective/README.md

@@ -1 +1,15 @@
 # A project
+
+## Índice
+
+Blah blah blah
+
+***
+
+## 1. Preámbulo
+
+Blah blah blah
+
+## 2. Resumen del proyecto
+
+Blah blah blah

lib/__tests__/__fixtures__/01-a-project-without-summary/README.md

@@ -0,0 +1,11 @@
+# A project
+
+## Índice
+
+Blah blah blah
+
+***
+
+## 1. Preámbulo
+
+Blah blah blah

lib/__tests__/__fixtures__/01-a-project-without-summary/project.yml

@@ -0,0 +1,14 @@
+learningObjectives:
+  - html/semantics
+  - css/selectors
+  - dom/selectors
+  - dom/events
+  - dom/manipulation
+  - js/data-types/primitive-vs-non-primitive
+  - js/data-types/strings
+  - js/variables
+  - js/conditionals
+  - js/functions
+  - js/semantics
+  - ux/user-understanding
+  - ux/prototyping

lib/__tests__/__fixtures__/01-a-project-without-track/README.md

@@ -1,3 +1,15 @@
 # A project
 
-BLah blah blah.
+## Índice
+
+Blah blah blah
+
+***
+
+## 1. Preámbulo
+
+Blah blah blah
+
+## 2. Resumen del proyecto
+
+Blah blah blah

lib/__tests__/__snapshots__/project.spec.js.snap

@@ -41,7 +41,7 @@ exports[`parseProject > parses a project with learning objectives validating aga
   "cover": null,
   "intl": {
     "es": {
-      "summary": null,
+      "summary": "<p>Blah blah blah</p>",
       "title": "A project",
     },
   },
@@ -75,7 +75,7 @@ exports[`parseProject > parses a project with learning objectives without valida
   "cover": null,
   "intl": {
     "es": {
-      "summary": null,
+      "summary": "<p>Blah blah blah</p>",
       "title": "A project",
     },
   },

lib/__tests__/project.spec.js

@@ -161,6 +161,7 @@ describe('parseProject', () => {
 
   it('extracts first paragraph of _resumen del proyecto_ as summary', () => {
     const p = resolveFixturePath('01-a-project-with-summary');
+    expect.assertions(2);
     return parseProject(p, {
       repo: 'Laboratoria/bootcamp',
       version: '1.0.0',
@@ -172,6 +173,19 @@ describe('parseProject', () => {
       });
   });
 
+  it('when no summary in project', () => {
+    const p = resolveFixturePath('01-a-project-without-summary');
+    expect.assertions(1);
+    return parseProject(p, {
+      repo: 'Laboratoria/bootcamp',
+      version: '1.0.0',
+      lo: path.join(__dirname, '__fixtures__', 'learning-objectives'),
+    }, pkg)
+      .catch((err) => {
+        expect(err.message).toBe('No project summary found');
+      });
+  });
+
   it('creates a thumbnail when file not present and has cover', () => {
     const p = resolveFixturePath('01-a-project-without-thumb');
     const thumbPath = path.join(p, 'thumb.png');

lib/project.js

@@ -18,7 +18,7 @@ const getSummary = (rootNode) => {
   ));
 
   if (headingIdx === -1 || rootNode.children[headingIdx + 1].type !== 'paragraph') {
-    return null;
+    throw new Error('No project summary found');
   }
 
   return parser.stringify(parser.runSync(rootNode.children[headingIdx + 1]));