@media-quest/builder 0.0.25 → 0.0.26

package/package.json

@@ -1,15 +1,15 @@
-{
-  "name": "@media-quest/builder",
-  "version": "0.0.25",
-  "description": "Builder library for Media-quest schemas",
-  "main": "src/public-api.ts",
-  "license": "MIT",
-  "scripts": {
-    "tsc:check:builder": "tsc --watch --noEmit",
-    "clean": "rimraf dist",
-    "buildXXX": "npm run clean && tsup src/public-api.ts --sourcemap inline --format cjs,esm --dts"
-  },
-  "dependencies": {
-    "@media-quest/engine": "0.0.25"
-  }
-}
+{
+  "name": "@media-quest/builder",
+  "version": "0.0.26",
+  "description": "Builder library for Media-quest schemas",
+  "main": "src/public-api.ts",
+  "license": "MIT",
+  "scripts": {
+    "tsc:check:builder": "tsc --watch --noEmit",
+    "clean": "rimraf dist",
+    "buildXXX": "npm run clean && tsup src/public-api.ts --sourcemap inline --format cjs,esm --dts"
+  },
+  "dependencies": {
+    "@media-quest/engine": "0.0.26"
+  }
+}

package/src/Builder-option.ts

@@ -1,64 +1,64 @@
-import { BuilderObject } from "./BuilderObject";
-import type { BuilderOptionTheme } from "./theme/IDefaultTheme";
-import { DefaultTheme } from "./theme/IDefaultTheme";
-import { AudioFile } from "./media-files";
-import { OptionID } from "./primitives/ID";
-
-export interface BuilderOptionDto {
-  readonly id: OptionID;
-  readonly value: number;
-  readonly label: string;
-  readonly labelAudio?: AudioFile;
-}
-
-export class BuilderOption extends BuilderObject<"builder-question-option", BuilderOptionDto> {
-  readonly objectType = "builder-question-option";
-  id: OptionID;
-  value: number;
-  label = "";
-  private _labelAudioFile: AudioFile | false = false;
-  get labelAudioFile() {
-    return this._labelAudioFile;
-  }
-  set labelAudioFile(audioFile: AudioFile | false) {
-    this._labelAudioFile = audioFile;
-  }
-
-  private constructor(dto: BuilderOptionDto) {
-    super(dto);
-    this.id = dto.id;
-    this.value = dto.value;
-    this.label = dto.label;
-    // this.theme = dto.theme;
-  }
-  public static create(value: number, label: string) {
-    const id = OptionID.create();
-    const dto: BuilderOptionDto = {
-      id,
-      value,
-      label,
-    };
-    const instance = new BuilderOption(dto);
-    return instance;
-  }
-
-  public static fromJson(dto: BuilderOptionDto) {
-    const instance = new BuilderOption(dto);
-    return instance;
-  }
-
-  toJson(): BuilderOptionDto {
-    const dto: BuilderOptionDto = {
-      id: this.id,
-      value: this.value,
-      label: this.label,
-    };
-    return dto;
-  }
-
-  clone(): BuilderOptionDto {
-    const dto = this.toJson();
-    const cloneDto: BuilderOptionDto = { ...dto, id: OptionID.create() };
-    return cloneDto;
-  }
-}
+import { BuilderObject } from "./BuilderObject";
+import type { BuilderOptionTheme } from "./theme/IDefaultTheme";
+import { DefaultTheme } from "./theme/IDefaultTheme";
+import { AudioFile } from "./media-files";
+import { OptionID } from "./primitives/ID";
+
+export interface BuilderOptionDto {
+  readonly id: OptionID;
+  readonly value: number;
+  readonly label: string;
+  readonly labelAudio?: AudioFile;
+}
+
+export class BuilderOption extends BuilderObject<"builder-question-option", BuilderOptionDto> {
+  readonly objectType = "builder-question-option";
+  id: OptionID;
+  value: number;
+  label = "";
+  private _labelAudioFile: AudioFile | false = false;
+  get labelAudioFile() {
+    return this._labelAudioFile;
+  }
+  set labelAudioFile(audioFile: AudioFile | false) {
+    this._labelAudioFile = audioFile;
+  }
+
+  private constructor(dto: BuilderOptionDto) {
+    super(dto);
+    this.id = dto.id;
+    this.value = dto.value;
+    this.label = dto.label;
+    // this.theme = dto.theme;
+  }
+  public static create(value: number, label: string) {
+    const id = OptionID.create();
+    const dto: BuilderOptionDto = {
+      id,
+      value,
+      label,
+    };
+    const instance = new BuilderOption(dto);
+    return instance;
+  }
+
+  public static fromJson(dto: BuilderOptionDto) {
+    const instance = new BuilderOption(dto);
+    return instance;
+  }
+
+  toJson(): BuilderOptionDto {
+    const dto: BuilderOptionDto = {
+      id: this.id,
+      value: this.value,
+      label: this.label,
+    };
+    return dto;
+  }
+
+  clone(): BuilderOptionDto {
+    const dto = this.toJson();
+    const cloneDto: BuilderOptionDto = { ...dto, id: OptionID.create() };
+    return cloneDto;
+  }
+}

package/src/Builder-page.spec.ts

@@ -16,7 +16,6 @@ const deleteIdsFromPage = (page: BuilderPageDto) => {
     });
   };
   U.deleteProp(page, "id");
-  page.questions.forEach(deleteIdsFromQuestion);
   deleteIdsFromQuestion(page.defaultQuestion);
   return page;
 };
@@ -64,7 +63,6 @@ const questionPageDto: BuilderPageDto = {
   // id: "p1" as BuilderObjectId.PageID,
   id: PageID.create(),
   prefix: PagePrefix.fromStringOrThrow("pxx"),
-  questions: [],
 };
 
 const multiQuestionPageDto: BuilderPageDto = {
@@ -92,56 +90,6 @@ const multiQuestionPageDto: BuilderPageDto = {
 
   id: PageID.create(),
   prefix: PagePrefix.fromStringOrThrow("pxx"),
-  questions: [
-    {
-      id: QuestionID.dummy.a,
-      prefix: "a",
-      text: "har du..",
-      _type: "select-one",
-      // options: []
-      options: [
-        {
-          id: "q3-opt1" as OptionID,
-          value: 0,
-          label: "Nei",
-        },
-        {
-          id: "q3-opt2" as OptionID,
-          value: 1,
-          label: "Ja",
-        },
-        {
-          id: "q3-opt3" as OptionID,
-          value: 9,
-          label: "Vet ikke",
-        },
-      ],
-    },
-    {
-      id: QuestionID.create(),
-      prefix: "a",
-      text: "har du..",
-      _type: "select-one",
-      // options: []
-      options: [
-        {
-          id: OptionID.dummy.a,
-          value: 0,
-          label: "Nei",
-        },
-        {
-          id: OptionID.dummy.b,
-          value: 1,
-          label: "Ja",
-        },
-        {
-          id: OptionID.dummy.c,
-          value: 9,
-          label: "Vet ikke",
-        },
-      ],
-    },
-  ],
 };
 
 let page = BuilderPage.create("info-page", PagePrefix.fromStringOrThrow("a"));
@@ -160,39 +108,6 @@ describe("Builder Page", () => {
     expect(multiQuestionPageDto).toStrictEqual(asJson2);
     // expect(page1.prefix).toBe(pageDto.prefix);
   });
-  test("Can change question type", () => {
-    expect(page.questions.length).toBe(0);
-    page.addQuestion("select-one");
-    page.addQuestion("select-one");
-    expect(page.questions.length).toBe(2);
-    page.pageType = "question";
-    expect(page.questions.length).toBe(0);
-  });
-  test("Can add and delete questions", () => {
-    page.pageType = "multi-select";
-    expect(page.questions.length).toBe(1);
-    const q1 = page.addQuestion("select-one");
-    const q2 = page.addQuestion("select-one");
-    expect(page.questions.length).toBe(3);
-    page.deleteQuestion(q1);
-    expect(page.questions.length).toBe(2);
-    page.deleteQuestion(q2);
-    expect(page.questions.length).toBe(1);
-    // page.pageType = 'question';
-    // expect(page.questions.length).toBe(0);
-  });
-  test("Can add question at concrete index. ", () => {
-    page.pageType = "multi-select";
-    expect(page.questions.length).toBe(1);
-    const q1 = page.addQuestion("select-one", 0);
-    expect(page.questions[0].id).toBe(q1.id);
-    const q2 = page.addQuestion("select-one", 1);
-    expect(page.questions[1].id).toBe(q2.id);
-    expect(page.questions.length).toBe(3);
-    const q3 = page.addQuestion("email", 2);
-    expect(page.questions[2]).toBe(q3);
-    expect(q3.type).toBe("email");
-  });
   test("Can clone a page and everything except id is equal ", () => {
     const page1 = BuilderPage.create("info-page", PagePrefix.fromStringOrThrow("as"));
     const clone1 = BuilderPage.fromJson(page1.clone());
@@ -220,85 +135,12 @@ describe("Builder Page", () => {
     const testDtoIds = (dto: BuilderPageDto) => {
       const instance = BuilderPage.fromJson(dto);
       const clone = BuilderPage.fromJson(instance.clone());
-      const p2QuestionIds = instance.questions.map((q) => q.id);
-      const cloneQuestionIds = clone.questions.map((q) => q.id);
 
-      // expect(instance.id).to.not.eq(clone.id);
-      // expect(instance.defaultQuestion.id).to.not.eq(clone.defaultQuestion.id);
-      expect(p2QuestionIds.length).toBe(cloneQuestionIds.length);
-      p2QuestionIds.forEach((id, index) => {
-        expect(id).not.toBe(cloneQuestionIds[index]);
-      });
-
-      instance.questions.forEach((q, qIndex) => {
-        const cloneQ = clone.questions[qIndex];
-        expect(cloneQ).toBeDefined();
-        expect(q.id).not.toBe(cloneQ.id);
-        q.options.forEach((op, opIndex) => {
-          const cloneOp = cloneQ.options[opIndex];
-          expect(cloneOp).toBeDefined();
-          expect(op.id).not.toBe(cloneOp.id);
-        });
-      });
-
-      // expect(p)
+      testDtoIds(questionPageDto);
+      testDtoIds(multiQuestionPageDto);
     };
-    testDtoIds(questionPageDto);
-    testDtoIds(multiQuestionPageDto);
-  });
-  test("Can insert a new Question at index", () => {
-    const page1 = BuilderPage.create("form", pxx);
-    const q1 = page1.addQuestion("select-one");
-    const q2 = page1.addQuestion("select-many");
-    expect(page1.questions.length).toBe(3);
-    expect(page1.questions[1]).toBe(q1);
-    const q1Clone = BuilderQuestion.fromJson(q1.clone());
-    const result10 = page1.insertQuestion(q1Clone, 10);
-    expect(result10).toBeFalsy();
-    const result1 = page1.insertQuestion(q1Clone, 1);
-    expect(result1).toBeTruthy();
-    expect(page1.questions[1]).toBe(q1Clone);
-  });
-  test("Can not  insert a duplicate question", () => {
-    const page1 = BuilderPage.create("form", pxx);
-    const q1 = page1.addQuestion("select-one");
-    expect(page1.questions.length).toBe(2);
-    expect(page1.questions[1]).toBe(q1);
-    const result1 = page1.insertQuestion(q1, 0);
-    expect(page1.questions.length).toBe(2);
-    expect(result1).toBeFalsy();
-  });
-
-  test("Can move a question", () => {
-    const page1 = BuilderPage.create("form", pxx);
-    const q1 = page1.addQuestion("select-many");
-    const q2 = page1.addQuestion("select-one");
-    const q3 = page1.addQuestion("select-one");
-    const q4 = page1.addQuestion("select-one");
-    expect(page1.questions.length).toBe(5);
-    const m1 = page1.moveQuestion(q1, 0);
-    expect(page1.questions.length).toBe(5);
-    expect(page1.questions[0]).toBe(q1);
-    expect(m1).toBeTruthy();
-    const illegalMove = page1.moveQuestion(q2, -1);
-    expect(illegalMove).toBeFalsy();
-    expect(page1.questions.length).toBe(5);
-    const m3 = page1.moveQuestion(q4, 1);
-    expect(m3).toBe(true);
-    expect(page1.questions[1]).toBe(q4);
   });
 
-  test("Can not move a question that is not in questions-array", () => {
-    const prefix = PagePrefix.fromStringOrThrow("as1");
-    const page1 = BuilderPage.create("form", prefix);
-    const q1 = page1.addQuestion("select-many");
-    const q2 = page1.addQuestion("select-one");
-    expect(page1.questions.length).toBe(3);
-    const newQuestion = BuilderQuestion.create("select-one");
-    const m1 = page1.moveQuestion(newQuestion, 0);
-    expect(page1.questions.length).toBe(3);
-    expect(m1).toBe(false);
-  });
   test("Can get all rule-variables.", () => {
     const prefix = PagePrefix.fromStringOrThrow("as1");
     const page = BuilderPage.create("question", prefix);

package/src/Builder-page.ts

@@ -7,7 +7,7 @@ import type { BuilderMainVideoDto } from "./BuilderMainVideoDto";
 import type { BuilderMainImageDto } from "./BuilderMainImageDto";
 import type { BuilderMainTextDto } from "./BuilderMainText";
 import { BuilderMainText } from "./BuilderMainText";
-import { BuilderVariableOption, QuestionVariable } from "./rulebuilder/RuleVariable";
+import { RuleVariableOption, RuleQuestionVariable } from "./rulebuilder";
 import { DUtil } from "@media-quest/engine";
 import { PagePrefix, PagePrefixValue } from "./primitives/page-prefix";
 import { VarID } from "./primitives/varID";
@@ -24,11 +24,9 @@ export interface BuilderPageDto {
   mainText: BuilderMainTextDto;
   nextButton: BuilderOptionDto;
   defaultQuestion: BuilderQuestionDto;
-  questions: Array<BuilderQuestionDto>;
   mainMedia?: BuilderMainImageDto | BuilderMainVideoDto;
   autoplaySequence: Array<string>;
   tags: ReadonlyArray<string>;
-  // sumScoreVariables: Array<BuilderObjectId.SumScoreVariableId>
 }
 
 export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
@@ -36,7 +34,6 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
   readonly id: PageID;
   private _pageType: BuilderPageType;
   private _prefix: PagePrefix;
-  private _questions: Array<BuilderQuestion> = [];
   private readonly _tags: Set<string>;
   private _backgroundColor = "#FFFFFF";
   mainMedia: BuilderMainVideoDto | BuilderMainImageDto | false = false;
@@ -63,7 +60,7 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
       nextButton: nextButtonDto,
       mainText: mainTextDto,
       prefix: _prefix,
-      questions: [],
+      // questions: [],
       tags: [],
     };
 
@@ -84,74 +81,11 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
     this.mainText = BuilderMainText.fromJson(dto.mainText);
     this.nextButton = BuilderOption.fromJson(dto.nextButton);
     this.defaultQuestion = BuilderQuestion.fromJson(dto.defaultQuestion);
-    this._questions = dto.questions.map((q) => BuilderQuestion.fromJson(q));
     const tagList: string[] = Array.isArray(dto.tags) ? dto.tags : [];
     this._tags = new Set(tagList);
     if (dto.mainMedia) {
       this.mainMedia = dto.mainMedia;
     }
-    this.updateRows();
-  }
-
-  insertQuestion(question: BuilderQuestion, atIndex: number): boolean {
-    const validIndexFn = U.isInRange(0, this._questions.length);
-    if (!validIndexFn(atIndex)) {
-      return false;
-    }
-    const hasQuestion = !!this._questions.find((q) => q.id === question.id);
-    if (hasQuestion) {
-      return false;
-    }
-    this._questions.splice(atIndex, 0, question);
-    return true;
-  }
-
-  addQuestion(type: BuilderQuestionType, atIndex = -1): BuilderQuestion {
-    const question = BuilderQuestion.create(type);
-    if (atIndex < this._questions.length && atIndex >= 0) {
-      this._questions.splice(atIndex, 0, question);
-    } else {
-      this._questions.push(question);
-    }
-    return question;
-  }
-
-  /**
-   * Move a question in questions-array
-   * @param question (reference)
-   * @param toIndex
-   */
-  moveQuestion(question: BuilderQuestion, toIndex: number): boolean {
-    const validToIndexFn = U.isInRange(0, this._questions.length);
-    if (!validToIndexFn(toIndex)) {
-      return false;
-    }
-    const currentIndex = this._questions.indexOf(question);
-    if (currentIndex < 0) {
-      return false;
-    }
-    this._questions.splice(currentIndex, 1);
-    this._questions.splice(toIndex, 0, question);
-    return true;
-  }
-
-  deleteQuestion(question: BuilderQuestion) {
-    this._questions = this._questions.filter((q) => q !== question);
-    // TODO EMIT DELETED QUESTION.
-    this.updateRows();
-  }
-  private updateRows() {
-    if (this._pageType === "question" || this._pageType === "info-page") {
-      this._questions = [];
-    }
-    if (this._pageType === "form" && this._questions.length === 0) {
-      this._questions = [];
-      this.addQuestion("text");
-    }
-    if (this._pageType === "multi-select" && this._questions.length === 0) {
-      this._questions = [];
-      this.addQuestion("text");
-    }
   }
 
   public addTag(tag: string) {
@@ -163,15 +97,13 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
 
   set pageType(value: BuilderPageType) {
     this._pageType = value;
-    // TODO Emit All questions that are deleted? Listen for removed variables??
-    this.updateRows();
   }
 
   getQuestionVariables(
     modulePrefix: SchemaPrefix,
     pageNumber: number,
-  ): ReadonlyArray<QuestionVariable> {
-    const variables: QuestionVariable[] = [];
+  ): ReadonlyArray<RuleQuestionVariable> {
+    const variables: RuleQuestionVariable[] = [];
 
     if (this._pageType === "question") {
       const pagePrefix = this.prefix;
@@ -180,9 +112,9 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
       const op = this.defaultQuestion.options.map((o) => {
         const label = o.label;
         const value = o.value;
-        return new BuilderVariableOption(label, value);
+        return new RuleVariableOption(label, value);
       });
-      const singleVar = new QuestionVariable(varId, label, op, pageNumber);
+      const singleVar = new RuleQuestionVariable(varId, label, op, pageNumber);
       variables.push(singleVar);
     }
     return variables;
@@ -204,7 +136,6 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
   }
 
   toJson(): BuilderPageDto {
-    const questions = this._questions.map((q) => q.toJson());
     const mainText = this.mainText.toJson();
     const nextButton = this.nextButton.toJson();
     const mainMedia = this.mainMedia;
@@ -217,7 +148,6 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
       tags: [...this.tags],
       prefix: this._prefix.value,
       defaultQuestion: this.defaultQuestion.toJson(),
-      questions,
     };
     if (mainMedia) {
       dto.mainMedia = mainMedia;
@@ -229,15 +159,12 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
     const dto = this.toJson();
     const defaultQuestionClone = this.defaultQuestion.clone();
     const mainTextClone = JSON.parse(JSON.stringify(this.mainText));
-    // const pagesClone = this
-    const questionsClone = this.questions.map((q) => q.clone());
     const newId = PageID.create();
     const clone: BuilderPageDto = {
       ...dto,
       id: newId,
       defaultQuestion: defaultQuestionClone,
       mainText: mainTextClone,
-      questions: questionsClone,
     };
     // const cloneDto
     return clone;
@@ -252,8 +179,4 @@ export class BuilderPage extends BuilderObject<"builder-page", BuilderPageDto> {
       this._backgroundColor = color;
     }
   }
-
-  get questions(): ReadonlyArray<BuilderQuestion> {
-    return this._questions;
-  }
 }

package/src/Builder-question.spec.ts

@@ -1,68 +1,68 @@
-import type { BuilderQuestionDto } from "./Builder-question";
-import { BuilderQuestion } from "./Builder-question";
-import { OptionID, QuestionID } from "./primitives/ID";
-
-const question1: BuilderQuestionDto = {
-  id: QuestionID.validateOrThrow("id-for-question-1"),
-  prefix: "q1",
-  text: "sadf",
-  options: [],
-  _type: "select-one",
-};
-
-const question2: BuilderQuestionDto = {
-  id: QuestionID.validateOrThrow("id-for-question-2"),
-  prefix: "a",
-  text: "har du..",
-  _type: "select-one",
-  // options: []
-  options: [
-    {
-      id: "q3-opt1" as OptionID,
-      value: 0,
-      label: "Nei",
-    },
-    {
-      id: "q3-opt2" as OptionID,
-      value: 1,
-      label: "Ja",
-    },
-    {
-      id: "q3-opt3" as OptionID,
-      value: 9,
-      label: "Vet ikke",
-    },
-  ],
-};
-
-describe("Builder Question", () => {
-  test("Can create question from json", () => {
-    const q1 = BuilderQuestion.fromJson(question1);
-    const q1AsJson = q1.toJson();
-    expect(question1).toStrictEqual(q1AsJson);
-    const q2Instance = BuilderQuestion.fromJson(question2);
-    const q2AsJson = q2Instance.toJson();
-    expect(question2).toStrictEqual(q2AsJson);
-  });
-  test("Can add options", () => {
-    const q1 = BuilderQuestion.fromJson(question1);
-    expect(q1.options.length).toBe(0);
-    q1.addOption("Ja", 1);
-    expect(q1.options.length).toBe(1);
-    const nei = q1.addOption("Nei", 0, 0);
-    expect(q1.options.length).toBe(2);
-    expect(q1.options[0]).toBe(nei);
-    const vetIkke = q1.addOption("Vet-ikke", 9);
-    expect(q1.options.length).toBe(3);
-    expect(q1.options[2]).toBe(vetIkke);
-  });
-  test("Can delete options", () => {
-    const q = BuilderQuestion.fromJson(question2);
-    expect(q.options.length).toBe(3);
-    const o1 = q.options[0];
-    const o2 = q.options[1];
-    const o3 = q.options[2];
-    const didDelete = q.deleteOption(o1);
-    expect(q.options.length).toBe(2);
-  });
-});
+import type { BuilderQuestionDto } from "./Builder-question";
+import { BuilderQuestion } from "./Builder-question";
+import { OptionID, QuestionID } from "./primitives/ID";
+
+const question1: BuilderQuestionDto = {
+  id: QuestionID.validateOrThrow("id-for-question-1"),
+  prefix: "q1",
+  text: "sadf",
+  options: [],
+  _type: "select-one",
+};
+
+const question2: BuilderQuestionDto = {
+  id: QuestionID.validateOrThrow("id-for-question-2"),
+  prefix: "a",
+  text: "har du..",
+  _type: "select-one",
+  // options: []
+  options: [
+    {
+      id: "q3-opt1" as OptionID,
+      value: 0,
+      label: "Nei",
+    },
+    {
+      id: "q3-opt2" as OptionID,
+      value: 1,
+      label: "Ja",
+    },
+    {
+      id: "q3-opt3" as OptionID,
+      value: 9,
+      label: "Vet ikke",
+    },
+  ],
+};
+
+describe("Builder Question", () => {
+  test("Can create question from json", () => {
+    const q1 = BuilderQuestion.fromJson(question1);
+    const q1AsJson = q1.toJson();
+    expect(question1).toStrictEqual(q1AsJson);
+    const q2Instance = BuilderQuestion.fromJson(question2);
+    const q2AsJson = q2Instance.toJson();
+    expect(question2).toStrictEqual(q2AsJson);
+  });
+  test("Can add options", () => {
+    const q1 = BuilderQuestion.fromJson(question1);
+    expect(q1.options.length).toBe(0);
+    q1.addOption("Ja", 1);
+    expect(q1.options.length).toBe(1);
+    const nei = q1.addOption("Nei", 0, 0);
+    expect(q1.options.length).toBe(2);
+    expect(q1.options[0]).toBe(nei);
+    const vetIkke = q1.addOption("Vet-ikke", 9);
+    expect(q1.options.length).toBe(3);
+    expect(q1.options[2]).toBe(vetIkke);
+  });
+  test("Can delete options", () => {
+    const q = BuilderQuestion.fromJson(question2);
+    expect(q.options.length).toBe(3);
+    const o1 = q.options[0];
+    const o2 = q.options[1];
+    const o3 = q.options[2];
+    const didDelete = q.deleteOption(o1);
+    expect(q.options.length).toBe(2);
+  });
+});