Files
tasks/common/headers/version_checker_test.go
Michael SnowdenandGitHub d5c62e926f Make all Go source file names snake case (#4558)
<!-- Describe what has changed in this PR -->
**What changed?**
I renamed all of our Go source code files so that they're snake case
instead of camel case.

<!-- Tell your future self why have you made these changes -->
**Why?**
This was an annoying inconsistency we carried over from much older code
in the repo.

<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->
**How did you test it?**
The renaming process was part of an automated script
(https://gist.github.com/MichaelSnowden/47576fffe27aa1cae6b26eaccec7ab55),
so it should be consistent. Also, I don't see us doing anything in our
codebase that involves parsing our own source files by path:

<img width="687" alt="image"
src="https://github.com/temporalio/temporal/assets/5942963/c6b873b0-d841-40d1-8b0f-2fe9d8079a53">

*PS: here's a link to the GPT convo I had to generate this renaming
script:
https://chat.openai.com/share/7f2d011a-a8da-4fa4-9c6d-e09102fdd49f*

<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->
**Potential risks**
Mainly merge conflicts, but I used `git mv` so that these are renames
instead of deletes/adds, so that shouldn't be a big issue.

<!-- Is this PR a hotfix candidate or require that a notification be
sent to the broader community? (Yes/No) -->
**Is hotfix candidate?**
No.
2023-07-03 14:23:56 -04:00

142 lines
4.2 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 headers
import (
"context"
"strings"
"testing"
"github.com/stretchr/testify/require"
"github.com/stretchr/testify/suite"
"go.temporal.io/api/serviceerror"
)
type (
VersionCheckerSuite struct {
*require.Assertions
suite.Suite
}
)
func TestVersionCheckerSuite(t *testing.T) {
suite.Run(t, new(VersionCheckerSuite))
}
func (s *VersionCheckerSuite) SetupTest() {
s.Assertions = require.New(s.T())
}
func (s *VersionCheckerSuite) TestClientSupported() {
serverVersion := "22.8.78"
myFeature := "my-new-feature-flag"
testCases := []struct {
callContext context.Context
expectErr bool
supportsMyFeature bool
}{
{
callContext: context.Background(),
expectErr: false,
},
{
callContext: s.constructCallContext("", "unknown-client", "", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("0.0.0", "", "", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("0.0.0", "unknown-client", "", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("malformed-version", ClientNameGoSDK, "", ""),
expectErr: true,
},
{
callContext: s.constructCallContext("3.0.1", ClientNameGoSDK, "", ""),
expectErr: true,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, "", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, "<23.1.0", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, ">23.1.0", ""),
expectErr: true,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, "<1.0.0 >=3.5.6 || >22.0.0", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("", ClientNameGoSDK, "<1.0.0 >=3.5.6 || >22.0.0", ""),
expectErr: false,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, "", myFeature),
supportsMyFeature: true,
},
{
callContext: s.constructCallContext("2.4.5", ClientNameGoSDK, "",
strings.Join([]string{"another-feature", myFeature, "third-feature"}, SupportedFeaturesHeaderDelim)),
supportsMyFeature: true,
},
}
versionChecker := NewVersionChecker(map[string]string{
ClientNameGoSDK: "<3.0.0",
}, serverVersion)
for caseIndex, tc := range testCases {
err := versionChecker.ClientSupported(tc.callContext)
if tc.expectErr {
s.Errorf(err, "Case #%d", caseIndex)
switch err.(type) {
case *serviceerror.InvalidArgument, *serviceerror.ClientVersionNotSupported, *serviceerror.ServerVersionNotSupported:
default:
s.Fail("error has wrong type: %T", err)
}
} else {
s.NoErrorf(err, "Case #%d", caseIndex)
}
if tc.callContext != nil {
s.Equal(tc.supportsMyFeature, versionChecker.ClientSupportsFeature(tc.callContext, myFeature))
}
}
}
func (s *VersionCheckerSuite) constructCallContext(clientVersion, clientName, supportedServerVersions, supportedFeatures string) context.Context {
return SetVersionsForTests(context.Background(), clientVersion, clientName, supportedServerVersions, supportedFeatures)
}