remove descriptionHtml method from lambda steps

allure-framework · Aug 2, 2023 · abd88f5 · abd88f5
1 parent a3f2984
commit abd88f5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 37 deletions.
diff --git a/packages/allure-js-commons/src/AllureCommandStep.ts b/packages/allure-js-commons/src/AllureCommandStep.ts
@@ -59,8 +59,6 @@ export interface AllureCommandStep<T = MetadataMessage> {
   ): void | Promise<void>;
 
   description(content: string): void | Promise<void>;
-
-  descriptionHtml(content: string): void | Promise<void>;
 }
 
 export class AllureCommandStepExecutable implements AllureCommandStep {
@@ -206,10 +204,6 @@ export class AllureCommandStepExecutable implements AllureCommandStep {
     this.metadata.description = content;
   }
 
-  descriptionHtml(content: string): void {
-    this.metadata.descriptionHtml = content;
-  }
-
   async step(name: string, body: StepBodyFunction): Promise<void> {
     if (!this.metadata.steps) {
       this.metadata.steps = [];
@@ -257,7 +251,6 @@ export class AllureCommandStepExecutable implements AllureCommandStep {
             parameters: [],
             steps,
             description,
-            descriptionHtml,
           },
         ],
       });
@@ -281,7 +274,6 @@ export class AllureCommandStepExecutable implements AllureCommandStep {
             parameters: [],
             steps,
             description,
-            descriptionHtml,
           },
         ],
       });

diff --git a/packages/allure-js-commons/test/specs/AllureCommandStep.spec.ts b/packages/allure-js-commons/test/specs/AllureCommandStep.spec.ts
@@ -480,40 +480,20 @@ describe("AllureCommandStep.start()", () => {
   });
 
   describe("description", () => {
-    describe("plain", () => {
-      let metadata: MetadataMessage;
+    let metadata: MetadataMessage;
 
-      beforeEach(async () => {
-        metadata = await currentStep.start((step) => {
-          step.description("foo");
-        });
-      });
-
-      it("doesn't provide description to metadata", () => {
-        expect(metadata.description).eq(undefined);
-      });
-
-      it("adds description to the step", () => {
-        expect(metadata.steps![0].description).eq("foo");
+    beforeEach(async () => {
+      metadata = await currentStep.start((step) => {
+        step.description("foo");
       });
     });
 
-    describe("html", () => {
-      let metadata: MetadataMessage;
-
-      beforeEach(async () => {
-        metadata = await currentStep.start((step) => {
-          step.descriptionHtml("foo");
-        });
-      });
-
-      it("doesn't provide html description to metadata", () => {
-        expect(metadata.descriptionHtml).eq(undefined);
-      });
+    it("doesn't provide description to metadata", () => {
+      expect(metadata.description).eq(undefined);
+    });
 
-      it("adds html description to the step", () => {
-        expect(metadata.steps![0].descriptionHtml).eq("foo");
-      });
+    it("adds description to the step", () => {
+      expect(metadata.steps![0].description).eq("foo");
     });
   });