shenyu icon indicating copy to clipboard operation
shenyu copied to clipboard

[Task] ShenYu client core test case

Open moremind opened this issue 1 year ago • 11 comments

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

moremind avatar Apr 10 '23 12:04 moremind

Can I help with AbstractContextRefreshedEventListener? Thanks.

strawberry-crisis avatar Apr 18 '23 02:04 strawberry-crisis

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

omernaci avatar May 10 '23 11:05 omernaci

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

assigned

moremind avatar May 10 '23 13:05 moremind

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?

strawberry-crisis avatar May 12 '23 02:05 strawberry-crisis

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

moremind avatar May 18 '23 01:05 moremind

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

yunuskarakaya avatar Nov 03 '23 12:11 yunuskarakaya

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

Divyansh200102 avatar Jan 17 '24 05:01 Divyansh200102

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

Divyansh200102 avatar Jan 18 '24 00:01 Divyansh200102

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

Aias00 avatar Jan 20 '24 06:01 Aias00

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

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

Aias00 avatar Jan 20 '24 07:01 Aias00