You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/10/29 15:57:30 UTC

[GitHub] [apisix-dashboard] nic-chen opened a new issue #638: requirement: add e2e test for config route with management fileds

nic-chen opened a new issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638


   detail:
   
   
    - [ ]  add test case for config route with name and desc
   
    - [ ]  verify the route just configured with correct name and desc, and has create_time and update_time
   
    - [ ]  add test case for update route  with other name and desc
   
    - [ ]  verify the route just configured with correct name and desc, update_time has changed.
   
    - [ ]  add test case for config route with labels and filter_func
   
    - [ ]  add test case for config route with  non-existent fields
   
   
   For background and description, please see:
   
   https://github.com/apache/apisix-dashboard/issues/607
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] idbeta commented on issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
idbeta commented on issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638#issuecomment-735660427


   The frequency of using `filter_func` is low, it is recommended to move to the 2.2 milestone. What do you think? cc @nic-chen 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] idbeta commented on issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
idbeta commented on issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638#issuecomment-735812137


   ` verify routing correctness under multi routes`, this test point need to 
   Now, there are few effective uris provided by the upstream service docker, can you provide an upstream, similar to nginx `location /`? In that case, no matter how many routes are created, they can all use different valid uris.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] idbeta edited a comment on issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
idbeta edited a comment on issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638#issuecomment-735812137


   Now, there are few effective uris provided by the upstream service docker, can you provide an upstream, similar to nginx `location /`? In that case, no matter how many routes are created, they can all use different valid uris.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] nic-chen commented on issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
nic-chen commented on issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638#issuecomment-735661421


   agree with that.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] gxthrj closed issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
gxthrj closed issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-dashboard] idbeta commented on issue #638: add e2e test for config route with management fileds

Posted by GitBox <gi...@apache.org>.
idbeta commented on issue #638:
URL: https://github.com/apache/apisix-dashboard/issues/638#issuecomment-735558767


   Hi, please assign it to me.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org