beego/client/cache/write_delete_test.go
Ming Deng 0bd2df91a1
Resolve conflicts among master branch and develop branch (#5286)
* feature extend readthrough for cache module (#5116)

* feature 增加readthrough

* feature: add write though for cache mode (#5117)

* feature: add writethough for cache mode

* feature add singleflight cache (#5119)

* build(deps): bump go.opentelemetry.io/otel/trace from 1.8.0 to 1.11.2

Bumps [go.opentelemetry.io/otel/trace](https://github.com/open-telemetry/opentelemetry-go) from 1.8.0 to 1.11.2.
- [Release notes](https://github.com/open-telemetry/opentelemetry-go/releases)
- [Changelog](https://github.com/open-telemetry/opentelemetry-go/blob/main/CHANGELOG.md)
- [Commits](https://github.com/open-telemetry/opentelemetry-go/compare/v1.8.0...v1.11.2)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/otel/trace
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix 5129: must set formatter after init the logger

* remove beego.vip

* build(deps): bump actions/stale from 5 to 7

Bumps [actions/stale](https://github.com/actions/stale) from 5 to 7.
- [Release notes](https://github.com/actions/stale/releases)
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/stale/compare/v5...v7)

---
updated-dependencies:
- dependency-name: actions/stale
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix 5079: only log msg when the channel is not closed (#5132)

* optimize test

* upgrade otel dependencies to v1.11.2

* format code

* Bloom filter cache (#5126)

* feature: add bloom filter cache

* feature upload remove all temp file

* bugfix Controller SaveToFile remove all temp file

* rft: motify BeeLogger signalChan (#5139)

* add non-block write log in asynchronous mode (#5150)

* add non-block write log in asynchronous mode

---------

Co-authored-by: chenhaokun <chenhaokun@itiger.com>

* fix the docsite URL (#5173)

* Unified gopkg.in/yaml version to v2 (#5169)

* Unified gopkg.in/yaml version to v2 and go mod tidy

* update CHANGELOG

* bugfix: protect field access with lock to avoid possible data race (#5211)

* fix some comments (#5194)

Signed-off-by: cui fliter <imcusg@gmail.com>

* build(deps): bump github.com/prometheus/client_golang (#5213)

Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.14.0 to 1.15.1.
- [Release notes](https://github.com/prometheus/client_golang/releases)
- [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md)
- [Commits](https://github.com/prometheus/client_golang/compare/v1.14.0...v1.15.1)

---
updated-dependencies:
- dependency-name: github.com/prometheus/client_golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump go.etcd.io/etcd/client/v3 from 3.5.4 to 3.5.9 (#5209)

Bumps [go.etcd.io/etcd/client/v3](https://github.com/etcd-io/etcd) from 3.5.4 to 3.5.9.
- [Release notes](https://github.com/etcd-io/etcd/releases)
- [Commits](https://github.com/etcd-io/etcd/compare/v3.5.4...v3.5.9)

---
updated-dependencies:
- dependency-name: go.etcd.io/etcd/client/v3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* cache: fix typo and optimize the naming

* Release 2.1.0 change log

* bugfix: beegoAppConfig String and Strings function has bug

* httplib: fix unstable test, do not use httplib.org

* chore: pkg imported more than once

* chore: fmt modify

* chore: Use github.com/go-kit/log

* chore: unnecessary use of fmt.Sprintf

* fix: golangci-lint error

* orm: refactor ORM introducing internal/models pkg

* remove adapter package

* build(deps): bump github.com/bits-and-blooms/bloom/v3

Bumps [github.com/bits-and-blooms/bloom/v3](https://github.com/bits-and-blooms/bloom) from 3.3.1 to 3.5.0.
- [Release notes](https://github.com/bits-and-blooms/bloom/releases)
- [Commits](https://github.com/bits-and-blooms/bloom/compare/v3.3.1...v3.5.0)

---
updated-dependencies:
- dependency-name: github.com/bits-and-blooms/bloom/v3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* feat: add write-delete cache mode

* fix: unnecessary assignment to the blank identifier

* fix: add change into .CHANGELOG file

* build(deps): bump golang.org/x/sync from 0.1.0 to 0.3.0

Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.1.0 to 0.3.0.
- [Commits](https://github.com/golang/sync/compare/v0.1.0...v0.3.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* build(deps): bump golang.org/x/crypto

Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.0.0-20220315160706-3147a52a75dd to 0.10.0.
- [Commits](https://github.com/golang/crypto/commits/v0.10.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* remove golang--lint-ci

* Beego web.Run() runs the server twice

* fix 5255: Check the rows.Err() if rows.Next() is false

* closes 5254: %COL% should be a common placeholder

* build(deps): bump github.com/prometheus/client_golang

Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.15.1 to 1.16.0.
- [Release notes](https://github.com/prometheus/client_golang/releases)
- [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md)
- [Commits](https://github.com/prometheus/client_golang/compare/v1.15.1...v1.16.0)

---
updated-dependencies:
- dependency-name: github.com/prometheus/client_golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: use of ioutil package (#5261)

* fix ioutil.NopCloser

* fix ioutil.ReadAll

* fix ioutil.ReadFile

* fix ioutil.WriteFile

* run goimports -w -format-only ./

* update CHANGELOG.md

* feature: add write-double-delete cache mode (#5263)

* cache/redis: support skipEmptyPrefix option (#5264)

* fix: refactor InsertValue method (#5267)

* fix: refactor insertValue method and add the test

* fix: exec goimports and add Licence file header

* fix: modify construct method of dbBase

* fix: add modify record into CHANGELOG

* fix: modify InsertOrUpdate method (#5269)

* fix: modify InsertOrUpdate method, Remove the isMulti variable and its associated code

* fix: Delete unnecessary judgment branches

* fix: add modify record into CHANGELOG

* cache/redis: use redisConfig to receive incoming JSON (previously using a map) (#5268)

* refactor cache/redis: Use redisConfig to receive incoming JSON (previously using a map).

* refactor cache/redis: Use the string type to receive JSON parameters.

---------

Co-authored-by: Tan <tanqianheng@gmail.com>

* fix: refactor Delete method (#5271)

* fix: refactor Delete method and add test

* fix: add modify record into CHANGELOG

* fix: refactor update sql (#5274)

* fix: refactor UpdateSQL method and add test

* fix: add modify record into CHANGELOG

* fix: modify url in the CHANGELOG

* fix: modify pr url in the CHANGELOG

* Fix setPK function for table without primary key (#5276)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: cui fliter <imcusg@gmail.com>
Co-authored-by: Stone-afk <73482944+Stone-afk@users.noreply.github.com>
Co-authored-by: hookokoko <hooko@tju.edu.cn>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: hookokoko <648646891@qq.com>
Co-authored-by: Stone-afk <1711865140@qq.com>
Co-authored-by: chenhaokun <chenhaokun@itiger.com>
Co-authored-by: Xuing <admin@xuing.cn>
Co-authored-by: cui fliter <imcusg@gmail.com>
Co-authored-by: guoguangwu <guoguangwu@magic-shield.com>
Co-authored-by: uzziah <uzziahlin@gmail.com>
Co-authored-by: Hanjiang Yu <delacroix.yu@gmail.com>
Co-authored-by: Kota <mdryzk64smsh@gmail.com>
Co-authored-by: Uzziah <120019273+uzziahlin@users.noreply.github.com>
Co-authored-by: Handkerchiefs-t <59816423+Handkerchiefs-t@users.noreply.github.com>
Co-authored-by: Tan <tanqianheng@gmail.com>
Co-authored-by: mlgd <mlgd17@gmail.com>
2023-07-31 23:00:02 +08:00

394 lines
9.3 KiB
Go

// Copyright 2014 beego Author. All Rights Reserved.
//
// 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.
// nolint
package cache
import (
"context"
"errors"
"fmt"
"testing"
"time"
"github.com/stretchr/testify/require"
"github.com/stretchr/testify/assert"
"github.com/beego/beego/v2/core/berror"
)
func TestWriteDoubleDeleteCache_Set(t *testing.T) {
mockDbStore := make(map[string]any)
cancels := make([]func(), 0)
defer func() {
for _, cancel := range cancels {
cancel()
}
}()
timeout := time.Second * 3
testCases := []struct {
name string
cache Cache
storeFunc func(ctx context.Context, key string, val any) error
ctx context.Context
interval time.Duration
sleepSecond time.Duration
key string
value any
wantErr error
}{
{
name: "store key/value in db fail",
interval: time.Second,
cache: NewMemoryCache(),
storeFunc: func(ctx context.Context, key string, val any) error {
return errors.New("failed")
},
ctx: context.TODO(),
wantErr: berror.Wrap(errors.New("failed"), PersistCacheFailed,
fmt.Sprintf("key: %s, val: %v", "", nil)),
},
{
name: "store key/value success",
interval: time.Second * 2,
sleepSecond: time.Second * 3,
cache: func() Cache {
cache := NewMemoryCache()
err := cache.Put(context.Background(), "hello", "world", time.Second*2)
require.NoError(t, err)
return cache
}(),
storeFunc: func(ctx context.Context, key string, val any) error {
mockDbStore[key] = val
return nil
},
ctx: context.TODO(),
key: "hello",
value: "world",
},
{
name: "store key/value timeout",
interval: time.Second * 2,
sleepSecond: time.Second * 3,
cache: func() Cache {
cache := NewMemoryCache()
err := cache.Put(context.Background(), "hello", "hello", time.Second*2)
require.NoError(t, err)
return cache
}(),
storeFunc: func(ctx context.Context, key string, val any) error {
select {
case <-ctx.Done():
return ctx.Err()
case <-time.After(3 * time.Second):
mockDbStore[key] = val
return nil
}
},
ctx: func() context.Context {
ctx, cancel := context.WithTimeout(context.Background(), time.Second)
cancels = append(cancels, cancel)
return ctx
}(),
key: "hello",
value: "hello",
},
}
for _, tt := range testCases {
t.Run(tt.name, func(t *testing.T) {
cache := tt.cache
c, err := NewWriteDoubleDeleteCache(cache, tt.interval, timeout, tt.storeFunc)
if err != nil {
assert.EqualError(t, tt.wantErr, err.Error())
return
}
err = c.Set(tt.ctx, tt.key, tt.value)
if err != nil {
assert.EqualError(t, tt.wantErr, err.Error())
return
}
_, err = c.Get(tt.ctx, tt.key)
assert.Equal(t, ErrKeyNotExist, err)
err = cache.Put(tt.ctx, tt.key, tt.value, tt.interval)
require.NoError(t, err)
val, err := c.Get(tt.ctx, tt.key)
require.NoError(t, err)
assert.Equal(t, tt.value, val)
time.Sleep(tt.sleepSecond)
_, err = c.Get(tt.ctx, tt.key)
assert.Equal(t, ErrKeyNotExist, err)
})
}
}
func TestNewWriteDoubleDeleteCache(t *testing.T) {
underlyingCache := NewMemoryCache()
storeFunc := func(ctx context.Context, key string, val any) error { return nil }
type args struct {
cache Cache
interval time.Duration
fn func(ctx context.Context, key string, val any) error
}
timeout := time.Second * 3
tests := []struct {
name string
args args
wantRes *WriteDoubleDeleteCache
wantErr error
}{
{
name: "nil cache parameters",
args: args{
cache: nil,
fn: storeFunc,
},
wantErr: berror.Error(InvalidInitParameters, "cache or storeFunc can not be nil"),
},
{
name: "nil storeFunc parameters",
args: args{
cache: underlyingCache,
fn: nil,
},
wantErr: berror.Error(InvalidInitParameters, "cache or storeFunc can not be nil"),
},
{
name: "init write-though cache success",
args: args{
cache: underlyingCache,
fn: storeFunc,
interval: time.Second,
},
wantRes: &WriteDoubleDeleteCache{
Cache: underlyingCache,
storeFunc: storeFunc,
interval: time.Second,
},
},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
_, err := NewWriteDoubleDeleteCache(tt.args.cache, tt.args.interval, timeout, tt.args.fn)
assert.Equal(t, tt.wantErr, err)
if err != nil {
return
}
})
}
}
func ExampleWriteDoubleDeleteCache() {
c := NewMemoryCache()
wtc, err := NewWriteDoubleDeleteCache(c, 1*time.Second, 3*time.Second, func(ctx context.Context, key string, val any) error {
fmt.Printf("write data to somewhere key %s, val %v \n", key, val)
return nil
})
if err != nil {
panic(err)
}
err = wtc.Set(context.Background(),
"/biz/user/id=1", "I am user 1")
if err != nil {
panic(err)
}
// Output:
// write data to somewhere key /biz/user/id=1, val I am user 1
}
func TestWriteDeleteCache_Set(t *testing.T) {
mockDbStore := make(map[string]any)
cancels := make([]func(), 0)
defer func() {
for _, cancel := range cancels {
cancel()
}
}()
testCases := []struct {
name string
cache Cache
storeFunc func(ctx context.Context, key string, val any) error
ctx context.Context
key string
value any
wantErr error
before func(Cache)
after func()
}{
{
name: "store key/value in db fail",
cache: NewMemoryCache(),
storeFunc: func(ctx context.Context, key string, val any) error {
return errors.New("failed")
},
ctx: context.TODO(),
wantErr: berror.Wrap(errors.New("failed"), PersistCacheFailed,
fmt.Sprintf("key: %s, val: %v", "", nil)),
before: func(cache Cache) {},
after: func() {},
},
{
name: "store key/value success",
cache: NewMemoryCache(),
storeFunc: func(ctx context.Context, key string, val any) error {
mockDbStore[key] = val
return nil
},
ctx: context.TODO(),
key: "hello",
value: "world",
before: func(cache Cache) {
_ = cache.Put(context.Background(), "hello", "testVal", 10*time.Second)
},
after: func() {
delete(mockDbStore, "hello")
},
},
{
name: "store key/value timeout",
cache: NewMemoryCache(),
storeFunc: func(ctx context.Context, key string, val any) error {
select {
case <-ctx.Done():
return ctx.Err()
case <-time.After(3 * time.Second):
mockDbStore[key] = val
return nil
}
},
ctx: func() context.Context {
ctx, cancel := context.WithTimeout(context.Background(), 1*time.Second)
cancels = append(cancels, cancel)
return ctx
}(),
key: "hello",
value: nil,
before: func(cache Cache) {
_ = cache.Put(context.Background(), "hello", "testVal", 10*time.Second)
},
after: func() {},
},
}
for _, tt := range testCases {
t.Run(tt.name, func(t *testing.T) {
w, err := NewWriteDeleteCache(tt.cache, tt.storeFunc)
if err != nil {
assert.EqualError(t, tt.wantErr, err.Error())
return
}
tt.before(tt.cache)
defer func() {
tt.after()
}()
err = w.Set(tt.ctx, tt.key, tt.value)
if err != nil {
assert.EqualError(t, tt.wantErr, err.Error())
return
}
_, err = w.Get(tt.ctx, tt.key)
assert.Equal(t, ErrKeyNotExist, err)
vv := mockDbStore[tt.key]
assert.Equal(t, tt.value, vv)
})
}
}
func TestNewWriteDeleteCache(t *testing.T) {
underlyingCache := NewMemoryCache()
storeFunc := func(ctx context.Context, key string, val any) error { return nil }
type args struct {
cache Cache
fn func(ctx context.Context, key string, val any) error
}
tests := []struct {
name string
args args
wantRes *WriteDeleteCache
wantErr error
}{
{
name: "nil cache parameters",
args: args{
cache: nil,
fn: storeFunc,
},
wantErr: berror.Error(InvalidInitParameters, "cache or storeFunc can not be nil"),
},
{
name: "nil storeFunc parameters",
args: args{
cache: underlyingCache,
fn: nil,
},
wantErr: berror.Error(InvalidInitParameters, "cache or storeFunc can not be nil"),
},
{
name: "init write-though cache success",
args: args{
cache: underlyingCache,
fn: storeFunc,
},
wantRes: &WriteDeleteCache{
Cache: underlyingCache,
storeFunc: storeFunc,
},
},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
_, err := NewWriteDeleteCache(tt.args.cache, tt.args.fn)
assert.Equal(t, tt.wantErr, err)
if err != nil {
return
}
})
}
}
func ExampleNewWriteDeleteCache() {
c := NewMemoryCache()
wtc, err := NewWriteDeleteCache(c, func(ctx context.Context, key string, val any) error {
fmt.Printf("write data to somewhere key %s, val %v \n", key, val)
return nil
})
if err != nil {
panic(err)
}
err = wtc.Set(context.Background(),
"/biz/user/id=1", "I am user 1")
if err != nil {
panic(err)
}
// Output:
// write data to somewhere key /biz/user/id=1, val I am user 1
}