Skip to content

Commit

Permalink
Review Fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
asalan316 committed Dec 3, 2024
1 parent 3fc59be commit f1324b2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 9 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ mta-release: mta-build
@echo " - building mtar release '${VERSION}' to dir: '${DEST}' "

.PHONY: acceptance-release
acceptance-release: generate-fakes generate-openapi-generated-clients-and-servers clean-acceptance go-mod-tidy go-mod-vendor build-test-app
acceptance-release: clean-acceptance go-mod-tidy go-mod-vendor build-test-app
@echo " - building acceptance test release '${VERSION}' to dir: '${DEST}' "
@mkdir -p ${DEST}
${AUTOSCALER_DIR}/scripts/compile-acceptance-tests.sh
Expand Down
14 changes: 6 additions & 8 deletions src/autoscaler/models/binding_configs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ import (

var _ = Describe("BindingConfigs", func() {

Describe("GetBindingConfigAndPolicy", func() {
var bindingConfig *BindingConfig

Context("GetBindingConfigAndPolicy", func() {
var (
scalingPolicy *ScalingPolicy
customMetricStrategy string
Expand All @@ -20,7 +22,7 @@ var _ = Describe("BindingConfigs", func() {
result, err = GetBindingConfigAndPolicy(scalingPolicy, customMetricStrategy)
})

Context("when both scaling policy and custom metric strategy are present", func() {
When("both scaling policy and custom metric strategy are present", func() {
BeforeEach(func() {
scalingPolicy = &ScalingPolicy{
InstanceMax: 5,
Expand Down Expand Up @@ -48,7 +50,7 @@ var _ = Describe("BindingConfigs", func() {
})
})

Context("when only scaling policy is present", func() {
When("only scaling policy is present", func() {
BeforeEach(func() {
scalingPolicy = &ScalingPolicy{
InstanceMax: 5,
Expand All @@ -73,7 +75,7 @@ var _ = Describe("BindingConfigs", func() {
})
})

Context("when policy is not found", func() {
When("policy is not found", func() {
BeforeEach(func() {
scalingPolicy = nil
customMetricStrategy = CustomMetricsBoundApp
Expand All @@ -86,10 +88,6 @@ var _ = Describe("BindingConfigs", func() {
})
})

var (
bindingConfig *BindingConfig
)

Context("GetCustomMetricsStrategy", func() {
It("should return the correct custom metrics strategy", func() {
bindingConfig = &BindingConfig{
Expand Down

0 comments on commit f1324b2

Please sign in to comment.