ShiningRush commented on pull request #1318:
URL: https://github.com/apache/apisix-dashboard/pull/1318#issuecomment-761993164


   > Original mock doesn't have the store ability, so we need to provide mock 
data in every unit test.
   
   Yes, this is the job that stub module should do, if you want to reuse stub 
data, you should just extract those data instead of the logic.
   Put the store logic into mock object will cause `Shotgun Surgery` that when 
we change the logic of store, we need also change the mock object's loigc.


----------------------------------------------------------------
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:
[email protected]


Reply via email to