improve test coverage

Signed-off-by: shmck <[email protected]>

Author: ShMcK

Diff for: src/utils/parse.ts

@@ -106,13 +106,12 @@ export function parse(params: ParseParams): any {
 
   // merge content and tutorial
   if (params.config.levels && params.config.levels.length) {
-    parsed.levels = params.config.levels.map(
-      (level: T.Level, levelIndex: number) => {
+    parsed.levels = params.config.levels
+      .map((level: T.Level, levelIndex: number) => {
         const levelContent = mdContent.levels[level.id];
 
         if (!levelContent) {
-          console.log(`Markdown content not found for ${level.id}`);
-          return;
+          return null;
         }
 
         level = { ...level, ...levelContent };
@@ -164,8 +163,8 @@ export function parse(params: ParseParams): any {
         );
 
         return level;
-      }
-    );
+      })
+      .filter((l: T.Level | null) => !!l);
   }
 
   return parsed;

Diff for: src/utils/validate.ts

@@ -15,6 +15,7 @@ export function validateSchema(json: any): boolean | PromiseLike<boolean> {
 
   if (!valid) {
     // log errors
+    /* istanbul ignore next */
     if (process.env.NODE_ENV !== "test") {
       jsonSchema.errors?.forEach((error: JsonSchema.ErrorObject) => {
         console.warn(

Diff for: tests/parse.test.ts

@@ -609,6 +609,64 @@ The third step
     expect(result.levels).toEqual(expected.levels);
   });
 
+  it("should handle steps with no solution", () => {
+    const md = `# Title
+    
+Description.
+
+## L1 Title 1
+
+First level content.
+
+### L1S1
+
+The first step
+
+`;
+    const config = {
+      levels: [
+        {
+          id: "L1",
+          steps: [
+            {
+              id: "L1S1",
+            },
+          ],
+        },
+      ],
+    };
+    const result = parse({
+      text: md,
+      config,
+      commits: {
+        L1S1Q: ["abcdefg1", "123456789"],
+      },
+    });
+    const expected = {
+      summary: {
+        description: "Description.",
+      },
+      levels: [
+        {
+          id: "L1",
+          title: "Title 1",
+          summary: "First level content.",
+          content: "First level content.",
+          steps: [
+            {
+              id: "L1S1",
+              content: "The first step",
+              setup: {
+                commits: ["abcdefg1", "123456789"],
+              },
+            },
+          ],
+        },
+      ],
+    };
+    expect(result.levels).toEqual(expected.levels);
+  });
+
   // config
   it("should parse the tutorial config", () => {
     const md = `# Title