put back *another* bit of required behavior - <3 tests!

Author: alexcjohnson

src/lib/dates.js

@@ -423,9 +423,12 @@ function formatTime(x, tr) {
          */
         var sec = Math.min(mod(x / ONESEC, 60), MAXSECONDS[tr]);
 
-        timeStr += ':' + (
-            (100 + sec).toFixed(tr).substr(1).replace(/[\.]?0*$/, '')
-        );
+        var secStr = (100 + sec).toFixed(tr).substr(1);
+        if(tr > 0) {
+            secStr = secStr.replace(/0+$/, '').replace(/[\.]$/, '');
+        }
+
+        timeStr += ':' + secStr;
     }
     return timeStr;
 }

test/jasmine/tests/lib_date_test.js

@@ -590,6 +590,8 @@ describe('dates', function() {
         it('should remove extra fractional second zeros', function() {
             expect(Lib.formatDate(0.1, '', 4)).toBe('00:00:00.0001\nJan 1, 1970');
             expect(Lib.formatDate(0.1, '', 3)).toBe('00:00:00\nJan 1, 1970');
+            expect(Lib.formatDate(0.1, '', 0)).toBe('00:00:00\nJan 1, 1970');
+            expect(Lib.formatDate(0.1, '', 'S')).toBe('00:00:00\nJan 1, 1970');
             expect(Lib.formatDate(0.1, '', 3, 'coptic'))
                 .toBe('00:00:00\nKoi 23, 1686');