Merge pull request #415 from tbjgolden/master

fix: typo plural minutes
jkbrzt · Aug 21, 2020 · 07580aa · 07580aa
2 parents a96ba52 + ab9c564
commit 07580aa
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/nlp/totext.ts b/src/nlp/totext.ts
@@ -192,7 +192,7 @@ export default class ToText {
     this.add(
       this.plural(this.options.interval!)
         ? gettext('minutes')
-        : gettext('minutes')
+        : gettext('minute')
     )
   }
 

diff --git a/test/nlp.test.ts b/test/nlp.test.ts
@@ -79,6 +79,18 @@ describe('NLP', () => {
     expect(rule.toText()).to.equal('every day')
   })
 
+  it('shows correct text for every minute', () => {
+    const options = { 'freq': RRule.MINUTELY };
+    const rule = new RRule(options);
+    expect(rule.toText()).to.equal('every minute');
+  });
+
+  it('shows correct text for every (plural) minutes', () => {
+    const options = { 'freq': RRule.MINUTELY, 'interval': 2 };
+    const rule = new RRule(options);
+    expect(rule.toText()).to.equal('every 2 minutes');
+  });
+
   it('by default formats \'until\' correctly', () => {
     const rrule = new RRule({
       freq: RRule.WEEKLY,