[GitHub] [incubator-superset] DiggidyDave commented on issue #7319: Cache invalidation

[GitBox <gi...@apache.org>]

DiggidyDave commented on issue #7319: Cache invalidation
URL: https://github.com/apache/incubator-superset/pull/7319#issuecomment-485480344
 
 
   ```I tried adding Cypress integration tests, but for some reason I get an error when writing to the Cache API (it works fine in Chrome) from an integration test:```
   
   Instead of end-to-end UI test for this, can we add some real unit tests and just mock the relevant storage/cache APIs? I think this needs some pretty robust testing, I'm afraid it is pretty tightly coupled to how URLs are structured and other things that could someday change and silently break this feature in an undiscoverable, hard-to-debug way.

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org