Apisix-dashboard: Feat: remove the etcd dependency in the unit test

Created on 30 Dec 2020  路  9Comments  路  Source: apache/apisix-dashboard

Feature request

Please describe your feature

Now, route, ssl services and upstream module are use etcd to store info.
I think we should remove the etcd dependency in the unit test.

Describe the solution you'd like

Use store.MockInterface{} instead of using the etcd.

  • [x] route
  • [x] ssl
  • [x] services
  • [x] upstream
backend testcase

All 9 comments

Agree +1.

If we need to resolve this issue in 2.3 or 2.4?

If we need to resolve this issue in 2.3 or 2.4?

I think it's a feature. So we may resolve it in 2.4.

This is a big project, I will finish part of them in M2.4

finish test cases of route, service in M2.4

@starsz
Hi,I have finished writing the SSL unit test, and if you do not start writing the upstream unit test, I can do so.

@starsz
Hi,I have finished writing the SSL unit test, and if you do not start writing the upstream unit test, I can do so.

Oh, good.Thank you very much.

Because the test cases of the route, service had finished, so I think we can change the milestone to M2.5.

This problem was solved. Thank you for every contributor.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

moonming picture moonming  路  6Comments

gxthrj picture gxthrj  路  5Comments

stu01509 picture stu01509  路  3Comments

bisakhmondal picture bisakhmondal  路  5Comments

juzhiyuan picture juzhiyuan  路  5Comments