Skip to content
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

enhance slime test: plugin and limiter #508

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

believening
Copy link
Contributor

The testing of the slime project has been enhanced in the following three points:

  1. add integration test for plugin module (kubebuilder envtest)
  2. add integration test for limiter module (kubebuilder envtest)
  3. add test target in Makefile

And current test result:

$ make test
?       slime.io/slime/modules/limiter  [no test files]
?       slime.io/slime/modules/limiter/api/config       [no test files]
?       slime.io/slime/modules/limiter/api/v1alpha2     [no test files]
?       slime.io/slime/modules/limiter/controllers/multicluster [no test files]
?       slime.io/slime/modules/limiter/model    [no test files]
?       slime.io/slime/modules/limiter/module   [no test files]
?       slime.io/slime/modules/meshregistry     [no test files]
?       slime.io/slime/modules/meshregistry/api/v1alpha1        [no test files]
?       slime.io/slime/modules/meshregistry/controllers [no test files]
?       slime.io/slime/modules/meshregistry/model       [no test files]
?       slime.io/slime/modules/meshregistry/module      [no test files]
?       slime.io/slime/modules/meshregistry/pkg/features        [no test files]
?       slime.io/slime/modules/meshregistry/pkg/mcpoverxds      [no test files]
?       slime.io/slime/modules/meshregistry/pkg/monitoring      [no test files]
?       slime.io/slime/modules/meshregistry/pkg/multicluster    [no test files]
?       slime.io/slime/modules/meshregistry/pkg/server  [no test files]
ok      slime.io/slime/modules/limiter/controllers      26.827s coverage: 46.0% of statements
ok      slime.io/slime/modules/limiter/test/e2e 0.468s  coverage: [no statements]
ok      slime.io/slime/modules/meshregistry/pkg/bootstrap       0.278s  coverage: 35.6% of statements
ok      slime.io/slime/modules/meshregistry/pkg/source  0.349s  coverage: 47.6% of statements
?       slime.io/slime/modules/meshregistry/pkg/source/k8s      [no test files]
?       slime.io/slime/modules/meshregistry/pkg/source/k8s/fs   [no test files]
ok      slime.io/slime/modules/meshregistry/pkg/source/eureka   0.490s  coverage: 45.0% of statements
?       slime.io/slime/modules/meshregistry/pkg/util/cache      [no test files]
ok      slime.io/slime/modules/meshregistry/pkg/source/nacos    0.655s  coverage: 26.5% of statements
ok      slime.io/slime/modules/meshregistry/pkg/source/sourcetest       0.354s  coverage: 13.3% of statements
ok      slime.io/slime/modules/meshregistry/pkg/source/zookeeper        0.765s  coverage: 37.3% of statements
ok      slime.io/slime/modules/meshregistry/pkg/util    0.354s  coverage: 5.4% of statements
?       slime.io/slime/modules/plugin   [no test files]
?       slime.io/slime/modules/plugin/api/config        [no test files]
?       slime.io/slime/modules/plugin/api/v1alpha1      [no test files]
?       slime.io/slime/modules/plugin/model     [no test files]
?       slime.io/slime/modules/plugin/module    [no test files]
ok      slime.io/slime/modules/plugin/controllers       16.296s coverage: 37.2% of statements

@believening
Copy link
Contributor Author

part of #487

@wbpcode wbpcode merged commit a6fff3a into slime-io:master Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants