Fix the bug of asterisk

Author: noah

parser.go

@@ -127,11 +127,10 @@ func parseField(field string, b bound, t translater) (bitset, error) {
 // parseFieldExpr returns the bits indicated by the given expression:
 //   number | number "-" number [ "/" number ]
 func parseFieldExpr(fieldexpr string, b bound, t translater) (bitset, error) {
-	if fieldexpr == "*" {
-		return bitsetStar, nil
-	}
+	// Replace "*" into "min-max".
+	newexpr := strings.Replace(fieldexpr, "*", fmt.Sprintf("%d-%d", b.min, b.max), 1)
 
-	rangeAndStep := strings.Split(fieldexpr, "/")
+	rangeAndStep := strings.Split(newexpr, "/")
 	if !(len(rangeAndStep) == 1 || len(rangeAndStep) == 2) {
 		return 0, fmt.Errorf("Failed to parse the expr '%s', too many '/'", fieldexpr)
 	}
@@ -172,7 +171,7 @@ func parseFieldExpr(fieldexpr string, b bound, t translater) (bitset, error) {
 
 	// Parse the step, second.
 	step := 1
-	if len(rangeAndStep) == 2 {
+	if hasStep {
 		var err error
 		if step, err = strconv.Atoi(rangeAndStep[1]); err != nil {
 			return 0, fmt.Errorf("Failed to parse the expr '%s': %w", fieldexpr, err)

parser_test.go

@@ -23,7 +23,7 @@ func Test_parseField(t *testing.T) {
 			{
 				value: "*",
 				b:     boundMinute,
-				want:  bitsetStar,
+				want:  buildBitset(0, 59, 1),
 			},
 			{
 				value: "5",
@@ -51,6 +51,11 @@ func Test_parseField(t *testing.T) {
 				b:     boundMinute,
 				want:  (1 << 5) | (1 << 10) | (1 << 15) | (1 << 20),
 			},
+			{
+				value: "*/20",
+				b:     boundMinute,
+				want:  (1 << 0) | (1 << 20) | (1 << 40),
+			},
 		}
 
 		for _, c := range cases {