Skip to content

Test/add runtime manager tests #2175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 28 commits into from
Aug 29, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
65f6bd3
add tests for runtime manager
miledxz Jul 7, 2024
b98942e
adding required tests update
miledxz Jul 7, 2024
266fdc0
resolving conflicts
miledxz Jul 7, 2024
676548c
adding resolving conflicts
miledxz Jul 7, 2024
88f656f
adding additional updates with Reload fnction tests
miledxz Feb 26, 2024
93157b9
adding required updates
miledxz Mar 12, 2024
df5ccbe
adding fixes
miledxz May 6, 2024
3dc727a
adding processHandler struct update
miledxz May 20, 2024
a0c0228
adding cleanup fix update
miledxz May 20, 2024
596efd3
adding gofumpt fix update
miledxz May 20, 2024
54e1efe
adding additional fix updates
miledxz May 20, 2024
39cc3d7
adding tests for runtime manager
miledxz Jun 27, 2024
7f6d1bf
resolving partionally comments in pr
miledxz Jun 29, 2024
9777319
Merge branch 'nginxinc:main' into test/add-runtime-manager-tests
miledxz Jul 7, 2024
0ecf5d1
adding first part of resolving final comments
miledxz Jul 7, 2024
ecc39c5
experimental commit for NewProcessHandlerImpl
miledxz Jul 7, 2024
948b9be
Merge branch 'nginxinc:main' into test/add-runtime-manager-tests
miledxz Jul 13, 2024
f146435
updating test desc
miledxz Jul 13, 2024
b7884d0
adding required updates regarding readFile
miledxz Jul 13, 2024
04f6e30
adding additional experimental ReadFile update
miledxz Jul 13, 2024
6963fd4
adding general fix for manager
miledxz Jul 14, 2024
547e903
adding small fix update
miledxz Aug 10, 2024
27fed56
adding manager fix update
miledxz Aug 20, 2024
8b94a05
quick upstream test update
miledxz Aug 20, 2024
eda39b2
Merge branch 'main' into test/add-runtime-manager-tests
kate-osborn Aug 20, 2024
7be17ee
experimental fix
miledxz Aug 21, 2024
1a56441
adding lint fix update
miledxz Aug 29, 2024
8913e20
Merge branch 'main' into test/add-runtime-manager-tests
sjberman Aug 29, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
adding manager fix update
  • Loading branch information
miledxz committed Aug 20, 2024
commit 27fed56f18b5089fb378dd5e856f895fc2235415
32 changes: 16 additions & 16 deletions internal/mode/static/nginx/runtime/manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,22 +49,6 @@ type nginxPlusClient interface {
GetUpstreams() (*ngxclient.Upstreams, error)
}

//counterfeiter:generate . ProcessHandler

type ProcessHandler interface {
FindMainProcess(
ctx context.Context,
timeout time.Duration,
) (int, error)
ReadFile(file string) ([]byte, error)
Kill(pid int) error
}

type ProcessHandlerImpl struct {
readFile ReadFileFunc
checkFile CheckFileFunc
}

//counterfeiter:generate . Manager

// Manager manages the runtime of NGINX.
Expand Down Expand Up @@ -193,6 +177,22 @@ func (m *ManagerImpl) GetUpstreams() (ngxclient.Upstreams, error) {
return *upstreams, nil
}

//counterfeiter:generate . ProcessHandler

type ProcessHandler interface {
FindMainProcess(
ctx context.Context,
timeout time.Duration,
) (int, error)
ReadFile(file string) ([]byte, error)
Kill(pid int) error
}

type ProcessHandlerImpl struct {
readFile ReadFileFunc
checkFile CheckFileFunc
}

func NewProcessHandlerImpl(readFile ReadFileFunc, checkFile CheckFileFunc) *ProcessHandlerImpl {
return &ProcessHandlerImpl{
readFile: readFile,
Expand Down
6 changes: 3 additions & 3 deletions internal/mode/static/nginx/runtime/manager_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ var _ = Describe("NGINX Runtime Manager", func() {
})

It("successfully returns server upstreams", func() {
upstreams := ngxclient.Upstreams{
expUpstreams := ngxclient.Upstreams{
"upstream1": {
Zone: "zone1",
Peers: []ngxclient.Peer{
Expand All @@ -179,12 +179,12 @@ var _ = Describe("NGINX Runtime Manager", func() {
},
}

ngxPlusClient.GetUpstreamsReturns(&upstreams, nil)
ngxPlusClient.GetUpstreamsReturns(&expUpstreams, nil)

upstreams, err := manager.GetUpstreams()

Expect(err).NotTo(HaveOccurred())
Expect(upstreams).To(Equal(upstreams))
Expect(upstreams).To(Equal(expUpstreams))
})

It("returns an error when GetUpstreams fails", func() {
Expand Down