Merge pull request #65 from zencoder/ParseDuration

Expose a useful function

Author: bcasinclair

mpd/duration.go

@@ -31,7 +31,7 @@ func (d Duration) MarshalXMLAttr(name xml.Name) (xml.Attr, error) {
 }
 
 func (d *Duration) UnmarshalXMLAttr(attr xml.Attr) error {
-	dur, err := parseDuration(attr.Value)
+	dur, err := ParseDuration(attr.Value)
 	if err != nil {
 		return err
 	}
@@ -153,7 +153,7 @@ func fmtInt(buf []byte, v uint64) int {
 	return w
 }
 
-func parseDuration(str string) (time.Duration, error) {
+func ParseDuration(str string) (time.Duration, error) {
 	if len(str) < 3 {
 		return 0, errors.New("At least one number and designator are required")
 	}

mpd/duration_test.go

@@ -36,7 +36,7 @@ func TestParseDuration(t *testing.T) {
 		"PT1004199059S": (1004199059 * time.Second).Seconds(),
 	}
 	for ins, ex := range in {
-		act, err := parseDuration(ins)
+		act, err := ParseDuration(ins)
 		require.NoError(t, err, ins)
 		require.EqualFloat64(t, ex, act.Seconds(), ins)
 	}
@@ -54,7 +54,7 @@ func TestParseBadDurations(t *testing.T) {
 		"-P20H":  `Duration cannot be negative`,                          // Negative duration doesn't make sense
 	}
 	for ins, msg := range in {
-		_, err := parseDuration(ins)
+		_, err := ParseDuration(ins)
 		require.EqualError(t, err, msg, fmt.Sprintf("Expected an error for: %s", ins))
 	}
 }