Merge pull request #116 from bradymholt/fix/105

Fix verbose DOW specified with * DOM
bradymholt · Jun 25, 2019 · 68bad2f · 68bad2f
2 parents 2f630dc + b73df0c
commit 68bad2f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/expressionDescriptor.ts b/src/expressionDescriptor.ts
@@ -394,6 +394,9 @@ export class ExpressionDescriptor {
             let offSetDays = lastDayOffSetMatches[1];
             description = StringUtilities.format(this.i18n.commaDaysBeforeTheLastDayOfTheMonth(), offSetDays);
             break;
+          } else if (expression == "*" && this.expressionParts[5] != "*"){
+            // * dayOfMonth and dayOfWeek specified so use dayOfWeek verbiage instead
+            return "";
           } else {
             description = this.getSegmentDescription(
               expression,

diff --git a/test/cronstrue.ts b/test/cronstrue.ts
@@ -482,6 +482,10 @@ describe("Cronstrue", function() {
     it("30 4 1 * *", function() {
       assert.equal(construe.toString(this.test.title, { verbose: true }), "At 04:30 AM, on day 1 of the month");
     });
+
+    it("0 13 * * 1", function() {
+      assert.equal(construe.toString(this.test.title, { verbose: true}), "At 01:00 PM, only on Monday");
+    });
   });
 
   describe("errors", function() {