mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-04 02:04:03 +00:00
## What changed? <!-- Describe what has changed in this PR --> - Switch to uber gomock - Generate workflowservice and operatorservice mock inside server repo, instead of using imports from api-go to avoid making breaking changes there. ## Why? <!-- Tell your future self why have you made these changes --> - google gomock is no longer maintained and doesn't support generics ## How did you test it? <!-- How have you verified this change? Tested locally? Added a unit test? Checked in staging env? --> ## Potential risks <!-- Assuming the worst case, what can be broken when deploying this change to production? --> ## Documentation <!-- Have you made sure this change doesn't falsify anything currently stated in `docs/`? If significant new behavior is added, have you described that in `docs/`? --> ## Is hotfix candidate? <!-- Is this PR a hotfix candidate or does it require a notification to be sent to the broader community? (Yes/No) -->
435 lines
13 KiB
Go
435 lines
13 KiB
Go
// The MIT License
|
|
//
|
|
// Copyright (c) 2020 Temporal Technologies Inc. All rights reserved.
|
|
//
|
|
// Copyright (c) 2020 Uber Technologies, Inc.
|
|
//
|
|
// Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
// of this software and associated documentation files (the "Software"), to deal
|
|
// in the Software without restriction, including without limitation the rights
|
|
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
// copies of the Software, and to permit persons to whom the Software is
|
|
// furnished to do so, subject to the following conditions:
|
|
//
|
|
// The above copyright notice and this permission notice shall be included in
|
|
// all copies or substantial portions of the Software.
|
|
//
|
|
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
// THE SOFTWARE.
|
|
|
|
package quotas
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
"github.com/stretchr/testify/suite"
|
|
"go.uber.org/mock/gomock"
|
|
)
|
|
|
|
type (
|
|
priorityStageRateLimiterSuite struct {
|
|
suite.Suite
|
|
*require.Assertions
|
|
|
|
controller *gomock.Controller
|
|
highPriorityRateLimiter *MockRateLimiter
|
|
lowPriorityRateLimiter *MockRateLimiter
|
|
highPriorityReservation *MockReservation
|
|
lowPriorityReservation *MockReservation
|
|
highPriorityAPIName string
|
|
lowPriorityAPIName string
|
|
|
|
rateLimiter *PriorityRateLimiterImpl
|
|
}
|
|
)
|
|
|
|
func TestPriorityStageRateLimiterSuite(t *testing.T) {
|
|
s := new(priorityStageRateLimiterSuite)
|
|
suite.Run(t, s)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) SetupSuite() {
|
|
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TearDownSuite() {
|
|
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) SetupTest() {
|
|
s.Assertions = require.New(s.T())
|
|
|
|
s.controller = gomock.NewController(s.T())
|
|
s.highPriorityRateLimiter = NewMockRateLimiter(s.controller)
|
|
s.lowPriorityRateLimiter = NewMockRateLimiter(s.controller)
|
|
s.highPriorityReservation = NewMockReservation(s.controller)
|
|
s.lowPriorityReservation = NewMockReservation(s.controller)
|
|
|
|
s.highPriorityAPIName = "high-priority"
|
|
s.lowPriorityAPIName = "low-priority"
|
|
apiToPriority := map[string]int{
|
|
s.highPriorityAPIName: 0,
|
|
s.lowPriorityAPIName: 2,
|
|
}
|
|
priorityToRateLimiters := map[int]RequestRateLimiter{
|
|
0: NewRequestRateLimiterAdapter(s.highPriorityRateLimiter),
|
|
2: NewRequestRateLimiterAdapter(s.lowPriorityRateLimiter),
|
|
}
|
|
s.rateLimiter = NewPriorityRateLimiter(func(req Request) int {
|
|
return apiToPriority[req.API]
|
|
}, priorityToRateLimiters)
|
|
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TearDownTest() {
|
|
s.controller.Finish()
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestAllow_HighPriority_Allow() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.highPriorityRateLimiter.EXPECT().AllowN(now, token).Return(true)
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.lowPriorityReservation)
|
|
|
|
allow := s.rateLimiter.Allow(now, req)
|
|
s.True(allow)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestAllow_HighPriority_Disallow() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.highPriorityRateLimiter.EXPECT().AllowN(now, token).Return(false)
|
|
|
|
allow := s.rateLimiter.Allow(now, req)
|
|
s.False(allow)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestAllow_LowPriority_Allow() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.lowPriorityRateLimiter.EXPECT().AllowN(now, token).Return(true)
|
|
|
|
allow := s.rateLimiter.Allow(now, req)
|
|
s.True(allow)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestAllow_LowPriority_Disallow() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.lowPriorityRateLimiter.EXPECT().AllowN(now, token).Return(false)
|
|
|
|
allow := s.rateLimiter.Allow(now, req)
|
|
s.False(allow)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestReserve_HighPriority_OK() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(true)
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.highPriorityReservation)
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.lowPriorityReservation)
|
|
|
|
reservation := s.rateLimiter.Reserve(now, req)
|
|
s.Equal(NewPriorityReservation(
|
|
s.highPriorityReservation,
|
|
[]Reservation{s.lowPriorityReservation},
|
|
), reservation)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestReserve_HighPriority_NotOK() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(false)
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.highPriorityReservation)
|
|
|
|
reservation := s.rateLimiter.Reserve(now, req)
|
|
s.Equal(s.highPriorityReservation, reservation)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestReserve_LowPriority_OK() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true)
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.lowPriorityReservation)
|
|
|
|
reservation := s.rateLimiter.Reserve(now, req)
|
|
s.Equal(NewPriorityReservation(
|
|
s.lowPriorityReservation,
|
|
[]Reservation{},
|
|
), reservation)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestReserve_LowPriority_NotOK() {
|
|
now := time.Now()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(false)
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(now, token).Return(s.lowPriorityReservation)
|
|
|
|
reservation := s.rateLimiter.Reserve(now, req)
|
|
s.Equal(s.lowPriorityReservation, reservation)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_HighPriority_AlreadyExpired() {
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_LowPriority_AlreadyExpired() {
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_HighPriority_NotExpired_WithExpiration_Error() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), time.Second)
|
|
defer cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
highPriorityReservationDelay := 2 * time.Second
|
|
s.highPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(highPriorityReservationDelay)
|
|
s.highPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
s.lowPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.highPriorityReservation)
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_LowPriority_NotExpired_WithExpiration_Error() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), time.Second)
|
|
defer cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
lowPriorityReservationDelay := 3 * time.Second
|
|
s.lowPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(lowPriorityReservationDelay)
|
|
s.lowPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_HighPriority_NotExpired_WithExpiration_Cancelled() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), 100*time.Second)
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
go func() {
|
|
time.Sleep(4 * time.Second)
|
|
cancel()
|
|
}()
|
|
|
|
highPriorityReservationDelay := 20 * time.Second
|
|
s.highPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(highPriorityReservationDelay)
|
|
s.highPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
s.lowPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.highPriorityReservation)
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_LowPriority_NotExpired_WithExpiration_Cancelled() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), 100*time.Second)
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
go func() {
|
|
time.Sleep(4 * time.Second)
|
|
cancel()
|
|
}()
|
|
|
|
lowPriorityReservationDelay := 30 * time.Second
|
|
s.lowPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(lowPriorityReservationDelay)
|
|
s.lowPriorityReservation.EXPECT().CancelAt(gomock.Any())
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.Error(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_HighPriority_NotExpired_WithExpiration_NoError() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), 4*time.Second)
|
|
defer cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
highPriorityReservationDelay := 2 * time.Second
|
|
s.highPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(highPriorityReservationDelay)
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.highPriorityReservation)
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.NoError(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_LowPriority_NotExpired_WithExpiration_NoError() {
|
|
ctx, cancel := context.WithTimeout(context.Background(), 4*time.Second)
|
|
defer cancel()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
lowPriorityReservationDelay := 2 * time.Second
|
|
s.lowPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(lowPriorityReservationDelay)
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.NoError(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_HighPriority_NotExpired_WithoutExpiration() {
|
|
ctx := context.Background()
|
|
token := 1
|
|
req := Request{
|
|
API: s.highPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
highPriorityReservationDelay := 2 * time.Second
|
|
s.highPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(highPriorityReservationDelay)
|
|
|
|
s.highPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.highPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.highPriorityReservation)
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.NoError(err)
|
|
}
|
|
|
|
func (s *priorityStageRateLimiterSuite) TestWait_LowPriority_NotExpired_WithoutExpiration() {
|
|
ctx := context.Background()
|
|
token := 1
|
|
req := Request{
|
|
API: s.lowPriorityAPIName,
|
|
Token: token,
|
|
Caller: "",
|
|
}
|
|
|
|
lowPriorityReservationDelay := 3 * time.Second
|
|
s.lowPriorityReservation.EXPECT().DelayFrom(gomock.Any()).Return(lowPriorityReservationDelay)
|
|
|
|
s.lowPriorityReservation.EXPECT().OK().Return(true).AnyTimes()
|
|
s.lowPriorityRateLimiter.EXPECT().ReserveN(gomock.Any(), token).Return(s.lowPriorityReservation)
|
|
|
|
err := s.rateLimiter.Wait(ctx, req)
|
|
s.NoError(err)
|
|
}
|