Skip to content
This repository has been archived by the owner on Jun 27, 2023. It is now read-only.

/api/teams is O(n) again! #53

Closed
Matir opened this issue Jun 12, 2016 · 2 comments
Closed

/api/teams is O(n) again! #53

Matir opened this issue Jun 12, 2016 · 2 comments
Assignees

Comments

@Matir
Copy link
Member

Matir commented Jun 12, 2016

/api/teams has fallen back to O(n) somehow.

@k9delight
Copy link
Contributor

Will no one rid me of the turbulent priest^W^WO(n) /api/teams

Matir added a commit that referenced this issue Jun 12, 2016
@Matir
Copy link
Member Author

Matir commented Jun 12, 2016

At least we will now have a regression test. :)

@Matir Matir self-assigned this Jun 12, 2016
@Matir Matir closed this as completed Jun 20, 2016
k9delight pushed a commit that referenced this issue Jun 20, 2016
…ting. (#74)

* Improve infrastructure for counting queries in tests.

* Add comprehensive query count testing to avoid regressions.

* Standardize on a single way to test query counts.

* Cover more edge cases of updating users.

* Monkey patch pbkdf2.crypt.

This reduces current test time by about 30%.
It is only done for REST API tests which should not depend on
pbkdf2 for correctness, but do require existing users for clients.

* Test /api/teams as well.

* Make /api/teams O(1) again.

Resolves issue #53.

* Session GET tests.

* Add session login/logout tests.

* /api/challenges testing.

* Make test clients work even for inline use.

* Basic testing for /api/scoreboard.
k9delight pushed a commit that referenced this issue Jun 21, 2016
* Improve infrastructure for counting queries in tests.

* Add comprehensive query count testing to avoid regressions.

* Standardize on a single way to test query counts.

* Cover more edge cases of updating users.

* Monkey patch pbkdf2.crypt.

This reduces current test time by about 30%.
It is only done for REST API tests which should not depend on
pbkdf2 for correctness, but do require existing users for clients.

* Test /api/teams as well.

* Make /api/teams O(1) again.

Resolves issue #53.

* Session GET tests.

* Add session login/logout tests.

* /api/challenges testing.

* Make test clients work even for inline use.

* Basic testing for /api/scoreboard.

* Basic /api/categories tests.

Also introduces new decorators and helper functions to make
tests cleaner and more reusable.

* Refactor user tests to use new utility functions.

* Remove redundant line.

* Cleanup team tests.

* Complete updating tests to use decorator pattern.

* Also refactor the UpdateTeam tests (still broken).

* Refactor how we launch tests to avoid config.py being loaded.
Matir added a commit that referenced this issue Jun 21, 2016
* Improve infrastructure for counting queries in tests.

* Add comprehensive query count testing to avoid regressions.

* Standardize on a single way to test query counts.

* Cover more edge cases of updating users.

* Monkey patch pbkdf2.crypt.

This reduces current test time by about 30%.
It is only done for REST API tests which should not depend on
pbkdf2 for correctness, but do require existing users for clients.

* Test /api/teams as well.

* Make /api/teams O(1) again.

Resolves issue #53.

* Session GET tests.

* Add session login/logout tests.

* /api/challenges testing.

* Make test clients work even for inline use.

* Basic testing for /api/scoreboard.

* Basic /api/categories tests.

Also introduces new decorators and helper functions to make
tests cleaner and more reusable.

* Refactor user tests to use new utility functions.

* Remove redundant line.

* Cleanup team tests.

* Complete updating tests to use decorator pattern.

* Also refactor the UpdateTeam tests (still broken).

* Refactor how we launch tests to avoid config.py being loaded.

* Fix logic in testing cookie expiration.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants