adsabs / adsws

ADS web services
Other
2 stars 15 forks source link

redis dependency for testing #80

Open romanchyla opened 8 years ago

romanchyla commented 8 years ago

It fails on localhost, succeeds on travis (when redis is not available)

setting OAUTH2_CACHE_TYPE='simple' triggers other error

self = <adsws.tests.test_accounts.TestAccounts testMethod=test_change_password>

    def get_csrf(self):
        """
            Returns a csrf token by visiting /bootstrap
            :return: string containing the csrf token bound to the current session
            """

        #  httpretty socket blocks if enabled before calling self.get_csrf() !
        r = self.client.get(url_for('csrfview'))
>       return r.json['csrf']
E       KeyError: 'csrf'
romanchyla commented 8 years ago

the flask-ratelimiter is actually deprecated in favor of https://github.com/alisaifee/flask-limiter

which can use local memory cache

we should move

jonnybazookatone commented 8 years ago

This issue is a duplicate of #67, so I've closed #67. Please re-open if you think I closed it incorrectly.

jonnybazookatone commented 8 years ago

close was accidental....