/*
Copyright 2019 The Knative Authors

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

package autoscaling

import (
	"context"
	"reflect"
	"testing"

	"github.com/google/go-cmp/cmp"

	"knative.dev/serving/pkg/autoscaler/config/autoscalerconfig"
)

func TestValidateAnnotations(t *testing.T) {
	cases := []struct {
		name          string
		annotations   map[string]string
		expectErr     string
		configMutator func(*autoscalerconfig.Config)
	}{{
		name:        "nil annotations",
		annotations: nil,
	}, {
		name:        "empty annotations",
		annotations: map[string]string{},
	}, {
		name:        "invalid class",
		annotations: map[string]string{ClassAnnotationKey: "unsupported.knative.dev"},
		expectErr:   "invalid value: unsupported.knative.dev: " + ClassAnnotationKey,
	}, {
		name:        "non-knative.dev domain class",
		annotations: map[string]string{ClassAnnotationKey: "some.other.domain"},
	}, {
		name:        "minScale is 0",
		annotations: map[string]string{MinScaleAnnotationKey: "0"},
	}, {
		name:        "maxScale is 0",
		annotations: map[string]string{MaxScaleAnnotationKey: "0"},
	}, {
		name:        "minScale is -1",
		annotations: map[string]string{MinScaleAnnotationKey: "-1"},
		expectErr:   "expected 0 <= -1 <= 2147483647: " + MinScaleAnnotationKey,
	}, {
		name:        "maxScale is huuuuuuuge",
		annotations: map[string]string{MaxScaleAnnotationKey: "2147483648"},
		expectErr:   "expected 0 <= 2147483648 <= 2147483647: " + MaxScaleAnnotationKey,
	}, {
		name:        "maxScale is -1",
		annotations: map[string]string{MaxScaleAnnotationKey: "-1"},
		expectErr:   "expected 0 <= -1 <= 2147483647: " + MaxScaleAnnotationKey,
	}, {
		name:        "minScale is foo",
		annotations: map[string]string{MinScaleAnnotationKey: "foo"},
		expectErr:   "invalid value: foo: " + MinScaleAnnotationKey,
	}, {
		name:        "maxScale is bar",
		annotations: map[string]string{MaxScaleAnnotationKey: "bar"},
		expectErr:   "invalid value: bar: " + MaxScaleAnnotationKey,
	}, {
		name:        "max/minScale is bar",
		annotations: map[string]string{MaxScaleAnnotationKey: "bar", MinScaleAnnotationKey: "bar"},
		expectErr:   "invalid value: bar: " + MaxScaleAnnotationKey + ", " + MinScaleAnnotationKey,
	}, {
		name:        "minScale is 5",
		annotations: map[string]string{MinScaleAnnotationKey: "5"},
	}, {
		name:        "maxScale is 2",
		annotations: map[string]string{MaxScaleAnnotationKey: "2"},
	}, {
		name:        "minScale is 2, maxScale is 5",
		annotations: map[string]string{MinScaleAnnotationKey: "2", MaxScaleAnnotationKey: "5"},
	}, {
		name:        "minScale is 5, maxScale is 2",
		annotations: map[string]string{MinScaleAnnotationKey: "5", MaxScaleAnnotationKey: "2"},
		expectErr:   "max-scale=2 is less than min-scale=5: " + MaxScaleAnnotationKey + ", " + MinScaleAnnotationKey,
	}, {
		name: "minScale is 0, maxScale is 0",
		annotations: map[string]string{
			MinScaleAnnotationKey: "0",
			MaxScaleAnnotationKey: "0",
		},
	}, {
		name: "maxScale is greater than MaxScaleLimit when in create",
		configMutator: func(config *autoscalerconfig.Config) {
			config.MaxScaleLimit = 10
		},
		annotations: map[string]string{MaxScaleAnnotationKey: "11"},
		expectErr:   "expected 1 <= 11 <= 10: " + MaxScaleAnnotationKey,
	}, {
		name: "maxScale is greater than MaxScaleLimit when in create and default MaxScale is set",
		configMutator: func(config *autoscalerconfig.Config) {
			config.MaxScaleLimit = 10
			config.MaxScale = 11
		},
		annotations: map[string]string{MaxScaleAnnotationKey: "20"},
		expectErr:   "expected 1 <= 20 <= 10: " + MaxScaleAnnotationKey,
	}, {
		name: "maxScale is explicitly set to 0 when MaxScaleLimit and default MaxScale are set",
		configMutator: func(config *autoscalerconfig.Config) {
			config.MaxScaleLimit = 10
			config.MaxScale = 1
		},
		annotations: map[string]string{MaxScaleAnnotationKey: "0"},
		expectErr:   "max-scale=0 (unlimited), must be less than 10: " + MaxScaleAnnotationKey,
	}, {
		name: "maxScale is not set when both MaxScaleLimit and default MaxScale are set",
		configMutator: func(config *autoscalerconfig.Config) {
			config.MaxScaleLimit = 10
			config.MaxScale = 11
		},
	}, {
		name: "maxScale is less than MaxScaleLimit",
		configMutator: func(config *autoscalerconfig.Config) {
			config.MaxScaleLimit = 10
		},
		annotations: map[string]string{MaxScaleAnnotationKey: "9"},
	}, {
		name:        "min-scale, activation-scale, max-scale all set appropriately",
		annotations: map[string]string{MinScaleAnnotationKey: "1", ActivationScaleKey: "2", MaxScaleAnnotationKey: "3"},
	}, {
		name:        "min-scale is greater than activation-scale",
		annotations: map[string]string{MinScaleAnnotationKey: "3", ActivationScaleKey: "2"},
		expectErr:   "min-scale=3 is greater than activation-scale=2: " + ActivationScaleKey,
	}, {
		name:        "max-scale is less than activation-scale",
		annotations: map[string]string{MaxScaleAnnotationKey: "1", ActivationScaleKey: "2"},
		expectErr:   "max-scale=1 is less than activation-scale=2: " + ActivationScaleKey,
	}, {
		name: "valid algorithm on KPA",
		annotations: map[string]string{
			MetricAggregationAlgorithmKey: MetricAggregationAlgorithmLinear,
			ClassAnnotationKey:            KPA,
		},
	}, {
		name: "empty algorithm",
		annotations: map[string]string{
			MetricAggregationAlgorithmKey: "",
			ClassAnnotationKey:            KPA,
		},
	}, {
		name: "random algorithm on KPA",
		annotations: map[string]string{
			MetricAggregationAlgorithmKey: "random-selection",
			ClassAnnotationKey:            KPA,
		},
		expectErr: "invalid value: random-selection: " + MetricAggregationAlgorithmKey,
	}, {
		name: "random algorithm on non KPA",
		annotations: map[string]string{
			MetricAggregationAlgorithmKey: "random-selection",
			ClassAnnotationKey:            "of-keys",
		},
	}, {
		name:        "panic window percentage bad",
		annotations: map[string]string{PanicWindowPercentageAnnotationKey: "-1"},
		expectErr:   "expected 1 <= -1 <= 100: " + PanicWindowPercentageAnnotationKey,
	}, {
		name:        "panic window percentage bad2",
		annotations: map[string]string{PanicWindowPercentageAnnotationKey: "202"},
		expectErr:   "expected 1 <= 202 <= 100: " + PanicWindowPercentageAnnotationKey,
	}, {
		name:        "panic window percentage bad3",
		annotations: map[string]string{PanicWindowPercentageAnnotationKey: "fifty"},
		expectErr:   "invalid value: fifty: " + PanicWindowPercentageAnnotationKey,
	}, {
		name:        "panic window percentage good",
		annotations: map[string]string{PanicThresholdPercentageAnnotationKey: "210"},
	}, {
		name:        "panic threshold percentage bad2",
		annotations: map[string]string{PanicThresholdPercentageAnnotationKey: "109"},
		expectErr:   "expected 110 <= 109 <= 1000: " + PanicThresholdPercentageAnnotationKey,
	}, {
		name:        "panic threshold percentage bad2.5",
		annotations: map[string]string{PanicThresholdPercentageAnnotationKey: "10009"},
		expectErr:   "expected 110 <= 10009 <= 1000: " + PanicThresholdPercentageAnnotationKey,
	}, {
		name:        "panic threshold percentage bad3",
		annotations: map[string]string{PanicThresholdPercentageAnnotationKey: "fifty"},
		expectErr:   "invalid value: fifty: " + PanicThresholdPercentageAnnotationKey,
	}, {
		name:        "target negative",
		annotations: map[string]string{TargetAnnotationKey: "-11"},
		expectErr:   "target -11 should be at least 0.01: " + TargetAnnotationKey,
	}, {
		name:        "target 0",
		annotations: map[string]string{TargetAnnotationKey: "0"},
		expectErr:   "target 0 should be at least 0.01: " + TargetAnnotationKey,
	}, {}, {
		name:        "invalid target",
		annotations: map[string]string{TargetAnnotationKey: "100}"},
		expectErr:   "invalid value: 100}: " + TargetAnnotationKey,
	}, {
		name:        "target okay",
		annotations: map[string]string{TargetAnnotationKey: "11"},
	}, {
		name:        "TBC negative",
		annotations: map[string]string{TargetBurstCapacityKey: "-11"},
		expectErr:   "invalid value: -11: " + TargetBurstCapacityKey,
	}, {
		name:        "TBC 0",
		annotations: map[string]string{TargetBurstCapacityKey: "0"},
	}, {
		name:        "TBC 19880709",
		annotations: map[string]string{TargetBurstCapacityKey: "19870709"},
	}, {
		name:        "TBC -1",
		annotations: map[string]string{TargetBurstCapacityKey: "-1"},
	}, {
		name:        "TBC invalid",
		annotations: map[string]string{TargetBurstCapacityKey: "qarashen"},
		expectErr:   "invalid value: qarashen: " + TargetBurstCapacityKey,
	}, {
		name:        "TU too small",
		annotations: map[string]string{TargetUtilizationPercentageKey: "0"},
		expectErr:   "expected 1 <= 0 <= 100: " + TargetUtilizationPercentageKey,
	}, {
		name:        "TU too big",
		annotations: map[string]string{TargetUtilizationPercentageKey: "101"},
		expectErr:   "expected 1 <= 101 <= 100: " + TargetUtilizationPercentageKey,
	}, {
		name:        "TU invalid",
		annotations: map[string]string{TargetUtilizationPercentageKey: "dghyak"},
		expectErr:   "invalid value: dghyak: " + TargetUtilizationPercentageKey,
	}, {
		name:        "window invalid",
		annotations: map[string]string{WindowAnnotationKey: "jerry-was-a-racecar-driver"},
		expectErr:   "invalid value: jerry-was-a-racecar-driver: " + WindowAnnotationKey,
	}, {
		name:        "window too short",
		annotations: map[string]string{WindowAnnotationKey: "1s"},
		expectErr:   "expected 6s <= 1s <= 1h0m0s: " + WindowAnnotationKey,
	}, {
		name:        "window too long",
		annotations: map[string]string{WindowAnnotationKey: "365h"},
		expectErr:   "expected 6s <= 365h <= 1h0m0s: " + WindowAnnotationKey,
	}, {
		name:        "window too precise",
		annotations: map[string]string{WindowAnnotationKey: "1m9s82ms"},
		expectErr:   "must be specified with at most second precision: " + WindowAnnotationKey,
	}, {
		name:        "annotation /window is valid for class KPA",
		annotations: map[string]string{WindowAnnotationKey: "7s", ClassAnnotationKey: KPA},
		expectErr:   "",
	}, {
		name:        "annotation /window is valid for other than HPA and KPA class",
		annotations: map[string]string{WindowAnnotationKey: "7s", ClassAnnotationKey: "test"},
		expectErr:   "",
	}, {
		name:        "value too long and valid class for /window annotation",
		annotations: map[string]string{WindowAnnotationKey: "365h", ClassAnnotationKey: KPA},
		expectErr:   "expected 6s <= 365h <= 1h0m0s: " + WindowAnnotationKey,
	}, {
		name:        "invalid format and valid class for /window annotation",
		annotations: map[string]string{WindowAnnotationKey: "jerry-was-a-racecar-driver", ClassAnnotationKey: KPA},
		expectErr:   "invalid value: jerry-was-a-racecar-driver: " + WindowAnnotationKey,
	}, {
		name:        "valid 0 last pod scaledown timeout",
		annotations: map[string]string{ScaleToZeroPodRetentionPeriodKey: "0"},
	}, {
		name:        "valid positive last pod scaledown timeout",
		annotations: map[string]string{ScaleToZeroPodRetentionPeriodKey: "21m31s"},
	}, {
		name:        "invalid positive last pod scaledown timeout",
		annotations: map[string]string{ScaleToZeroPodRetentionPeriodKey: "311m"},
		expectErr:   "expected 0s <= 311m <= 1h0m0s: " + ScaleToZeroPodRetentionPeriodKey,
	}, {
		name:        "invalid negative last pod scaledown timeout",
		annotations: map[string]string{ScaleToZeroPodRetentionPeriodKey: "-42s"},
		expectErr:   "expected 0s <= -42s <= 1h0m0s: " + ScaleToZeroPodRetentionPeriodKey,
	}, {
		name:        "invalid last pod scaledown timeout",
		annotations: map[string]string{ScaleToZeroPodRetentionPeriodKey: "twenty-two-minutes-and-five-seconds"},
		expectErr:   "invalid value: twenty-two-minutes-and-five-seconds: " + ScaleToZeroPodRetentionPeriodKey,
	}, {
		name:        "valid 0 scale down delay",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "0"},
	}, {
		name:        "valid positive scale down delay",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "21m31s"},
	}, {
		name:        "invalid positive scale down delay",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "311m"},
		expectErr:   "expected 0s <= 311m <= 1h0m0s: " + ScaleDownDelayAnnotationKey,
	}, {
		name:        "invalid positive scale down delay - too precise",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "42.5s"},
		expectErr:   "must be specified with at most second precision: " + ScaleDownDelayAnnotationKey,
	}, {
		name:        "invalid negative scale down delay",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "-42s"},
		expectErr:   "expected 0s <= -42s <= 1h0m0s: " + ScaleDownDelayAnnotationKey,
	}, {
		name:        "invalid scale down delay",
		annotations: map[string]string{ScaleDownDelayAnnotationKey: "twenty-two-minutes-and-five-seconds"},
		expectErr:   "invalid value: twenty-two-minutes-and-five-seconds: " + ScaleDownDelayAnnotationKey,
	}, {
		name: "all together now fail",
		annotations: map[string]string{
			PanicThresholdPercentageAnnotationKey: "fifty",
			PanicWindowPercentageAnnotationKey:    "-11",
			MinScaleAnnotationKey:                 "-4",
			MaxScaleAnnotationKey:                 "never",
		},
		expectErr: "expected 0 <= -4 <= 2147483647: " + MinScaleAnnotationKey + "\nexpected 1 <= -11 <= 100: " + PanicWindowPercentageAnnotationKey + "\ninvalid value: fifty: " + PanicThresholdPercentageAnnotationKey + "\ninvalid value: never: " + MaxScaleAnnotationKey,
	}, {
		name: "all together now, succeed",
		annotations: map[string]string{
			PanicThresholdPercentageAnnotationKey: "125",
			PanicWindowPercentageAnnotationKey:    "75",
			MinScaleAnnotationKey:                 "5",
			MaxScaleAnnotationKey:                 "8",
			WindowAnnotationKey:                   "1984s",
		},
	}, {
		name:        "invalid metric for default class(KPA)",
		annotations: map[string]string{MetricAnnotationKey: CPU},
		expectErr:   "invalid value: cpu: " + MetricAnnotationKey,
	}, {
		name:        "invalid metric for HPA class",
		annotations: map[string]string{MetricAnnotationKey: "", ClassAnnotationKey: HPA},
		expectErr:   "invalid value: : " + MetricAnnotationKey,
	}, {
		name:        "valid class KPA with metric RPS",
		annotations: map[string]string{MetricAnnotationKey: RPS},
	}, {
		name:        "valid class KPA with metric Concurrency",
		annotations: map[string]string{MetricAnnotationKey: Concurrency},
	}, {
		name:        "valid class HPA with metric CPU",
		annotations: map[string]string{ClassAnnotationKey: HPA, MetricAnnotationKey: CPU},
	}, {
		name:        "other than HPA and KPA class",
		annotations: map[string]string{ClassAnnotationKey: "other", MetricAnnotationKey: RPS},
	}, {
		name:        "initial scale is zero but cluster doesn't allow",
		annotations: map[string]string{InitialScaleAnnotationKey: "0"},
		expectErr:   "invalid value: 0: " + InitialScaleAnnotationKey + "=0 not allowed by cluster",
	}, {
		name: "initial scale is zero and cluster allows",
		configMutator: func(config *autoscalerconfig.Config) {
			config.AllowZeroInitialScale = true
		},
		annotations: map[string]string{InitialScaleAnnotationKey: "0"},
	}, {
		name:        "initial scale is greater than 0",
		annotations: map[string]string{InitialScaleAnnotationKey: "2"},
	}, {
		name:        "initial scale is less than 0",
		annotations: map[string]string{InitialScaleAnnotationKey: "-1"},
		expectErr:   "invalid value: -1: " + InitialScaleAnnotationKey + " must be greater than 0",
	}, {
		name:        "initial scale non-parseable",
		annotations: map[string]string{InitialScaleAnnotationKey: "invalid"},
		expectErr:   "invalid value: invalid: autoscaling.knative.dev/initial-scale",
	}}
	for _, c := range cases {
		t.Run(c.name, func(t *testing.T) {
			cfg := defaultConfig()
			if c.configMutator != nil {
				c.configMutator(cfg)
			}
			ctx := context.Background()
			if got, want := ValidateAnnotations(ctx, cfg, c.annotations).Error(), c.expectErr; !reflect.DeepEqual(got, want) {
				t.Errorf("\nErr = %q,\nwant: %q, diff(-want,+got):\n%s", got, want, cmp.Diff(want, got))
			}
		})
	}
}

func defaultConfig() *autoscalerconfig.Config {
	return &autoscalerconfig.Config{
		AllowZeroInitialScale: false,
		MaxScaleLimit:         0,
	}
}
