search

apache / shenyu

Apache ShenYu is a Java native API Gateway for service proxy, protocol conversion and API governance.
https://shenyu.apache.org/
Apache License 2.0
8.41k stars 2.92k forks source link

[Task] ShenYu client core test case #4544

Open moremind opened 1 year ago

moremind commented 1 year ago

Description

Add test case for shenyu-client-core

Task List

  1. [ ] Add test case for AbstractContextRefreshedEventListener @strawberry-crisis
  2. [ ] Add test case for RegisterClientConsumerExecutor
  3. [x] Add test case for ShenyuClientApiDocExecutorSubscriber @omernaci
  4. [x] Add test case for ShenyuClientMetadataExecutorSubscriber @yunuskarakaya
  5. [ ] Add test case for ShenyuClientURIExecutorSubscriber
  6. [ ] Add test case for ShenyuClientRegisterEventPublisher
  7. [x] Add test case for ShenyuClientIllegalArgumentException
  8. [ ] Add test case for ShenyuClientRegisterRepositoryFactory
  9. [ ] Add test case for ShenyuClientShutdownHook
  10. [x] Add test case for PortUtils @VampireAchao
strawberry-crisis commented 1 year ago

Can I help with AbstractContextRefreshedEventListener? Thanks.

VampireAchao commented 1 year ago

add test case for PortUtils.findPort

omernaci commented 1 year ago

Hi @moremind, I would happy to contribute for ShenyuClientApiDocExecutorSubscriber class test.

moremind commented 1 year ago

Hi @moremind, I would happy to contribute for ShenyuClientApiDocExecutorSubscriber class test.

assigned

strawberry-crisis commented 1 year ago

Can I help with AbstractContextRefreshedEventListener? Thanks.

Sorry for the delay. Actually, I am not familiar with writing JUnit tests for an abstract class. Are there any examples that I can reference?

moremind commented 1 year ago

Can I help with AbstractContextRefreshedEventListener? Thanks.

Sorry for the delay. Actually, I am not familiar with writing JUnit tests for an abstract class. Are there any examples that I can reference?

yes, of course, please see shenyu-client

yunuskarakaya commented 10 months ago

hi @moremind, I would like write a test for this class ShenyuClientMetadataExecutorSubscriber.

Divyansh200102 commented 7 months ago

Hi, @moremind I would like to writ the test for ShenyuClientMetadataExecutorSubscriber

Divyansh200102 commented 7 months ago

@dragon-zhang @moremind please review #5404 #5406 #5408

Aias00 commented 7 months ago

Hi, @moremind I would like to write the test for ShenyuClientURIExecutorSubscriber

Aias00 commented 7 months ago

Hi, @moremind I would like to write the test for ShenyuClientURIExecutorSubscriber

@moremind please review https://github.com/apache/shenyu/pull/5413