aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Use direct database test intead of relying on GETGravatar Akarys422019-11-16-3/+3
|
* Write list test for the offensive message modelGravatar Akarys422019-11-16-0/+39
|
* Merge branch 'master' into #222-offensive-msg-autodeletionGravatar Akarys422019-11-16-34/+1053
|\
| * Add pipenv run start command (#308)Gravatar scragly2019-11-12-0/+1
| |\ | | | | | | Add pipenv run start command
| | * Add the `--debug` flag to the start commandGravatar Matteo Bertucci2019-11-12-1/+1
| | |
| | * Add pipenv run start commandGravatar Akarys422019-11-11-0/+1
| |/
| * Merge pull request #292 from python-discord/allauth-user-settingsGravatar scragly2019-11-11-34/+1052
| |\ | | | | | | Allauth improvements & GH
| | * Merge branch 'master' into allauth-user-settingsGravatar scragly2019-11-11-49/+1123
| | |\ | | |/ | |/|
| | * Merge branch 'master' into allauth-user-settingsGravatar Gareth Coles2019-10-28-35/+50
| | |\
| | * \ Merge branch 'master' into allauth-user-settingsGravatar Gareth Coles2019-10-23-1/+1
| | |\ \
| | * | | Full coverage and finish cleanupGravatar Gareth Coles2019-10-23-5/+96
| | | | |
| | * | | Add user settings modal, with connections management and account deletionGravatar Gareth Coles2019-10-23-13/+310
| | | | |
| | * | | Add role mapping migration I forgot to commitGravatar Gareth Coles2019-10-23-0/+18
| | | | |
| | * | | Finalize orange and purple custom coloursGravatar Gareth Coles2019-10-22-3/+3
| | | | |
| | * | | New colours, as discussed on DiscordGravatar Gareth Coles2019-10-22-7/+7
| | | | |
| | * | | Full test coverage, better redirect testingGravatar Gareth Coles2019-10-22-4/+110
| | | | |
| | * | | Prevent user signup when they've never joined+verifiedGravatar Gareth Coles2019-10-22-13/+110
| | | | |
| | * | | Merge branch 'master' into allauth-user-settingsGravatar Gareth Coles2019-10-22-1/+1
| | |\ \ \
| | * | | | Implement account deletion at /accounts/deleteGravatar Gareth Coles2019-10-22-3/+76
| | | | | |
| | * | | | Merge branch 'master' into allauth-user-settingsGravatar Gareth Coles2019-10-22-0/+0
| | |\ \ \ \
| | * | | | | Update Django users with Discord username+discrim.Gravatar Gareth Coles2019-10-20-3/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This sets both the Django User's `username` and `first_name` params.
| | * | | | | Signals: Complete coverage in testsGravatar Gareth Coles2019-10-20-0/+41
| | | | | | |
| | * | | | | Tests for the new signal listener is_staff changesGravatar Gareth Coles2019-10-20-5/+26
| | | | | | |
| | * | | | | Add is_staff to role mappings, and the logic to go with itGravatar Gareth Coles2019-10-20-11/+63
| | | | | | |
| | * | | | | Signals: Move group changes outside of the loopGravatar Gareth Coles2019-10-20-6/+6
| | | | | | |
| | * | | | | GH signup prevention, views and templates for settings pagesGravatar Gareth Coles2019-10-20-5/+126
| | | | | | |
| | * | | | | Allauth: Re-add GitHub provider, prevent GH signupsGravatar Gareth Coles2019-10-19-0/+48
| | | | | | |
| | * | | | | Bring navbar styling in line on mobile as wellGravatar Gareth Coles2019-10-19-0/+10
| | | | | | |
| | * | | | | Initial navbar change to add settings button next to logoutGravatar Gareth Coles2019-10-19-3/+9
| | | | | | |
* | | | | | | Make use of subtestsGravatar Akarys422019-11-12-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Sebastiaan Zeeff <[email protected]>
* | | | | | | Change setUpTestData to use aware datetimeGravatar Akarys422019-11-11-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This remove a warning during test run
* | | | | | | Add deletion testsGravatar Akarys422019-11-11-0/+24
| | | | | | |
* | | | | | | Add tests to make sure the view return 405 for PATCH and PUT requestsGravatar Akarys422019-11-11-0/+22
| | | | | | |
* | | | | | | Make sure the API return the appropriate response.Gravatar Akarys422019-11-10-0/+9
| | | | | | |
* | | | | | | Merge master migration filesGravatar Akarys422019-11-07-2/+2
| | | | | | |
* | | | | | | Merge branch 'master' into #222-offensive-msg-autodeletionGravatar Akarys422019-11-07-81/+1141
|\| | | | | |
| * | | | | | Merge pull request #278 from python-discord/active-infractions-validationGravatar Sebastiaan Zeeff2019-11-03-4/+1069
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Validation for active infractions
| | * | | | | | Resolve migration merge conflictsGravatar Sebastiaan Zeeff2019-11-03-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since other pull requests were merged that also included migrations for the API app, this PR needed to be updated to avoid conflicts in the migration history. In addition, the test files contained names of specific migration files that needed to be updated to the correct names after the merge resolution.
| | * | | | | | Merge branch 'master' into active-infractions-validationGravatar Sebastiaan Zeeff2019-11-03-82/+106
| | |\ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Merge pull request #293 from python-discord/update-dependency-pinningGravatar Leon Sandøy2019-10-28-45/+54
| |\ \ \ \ \ \ \ | | |_|_|_|_|_|/ | |/| | | | | | Update dependency pinning
| | * | | | | | Merge branch 'master' into update-dependency-pinningGravatar Leon Sandøy2019-10-28-37/+52
| | |\ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Merge pull request #301 from python-discord/new-rulesGravatar scragly2019-10-26-32/+18
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Add new rules.
| | * | | | | | | Add new rules.Gravatar scragly2019-10-26-32/+18
| |/ / / / / / /
| | * | / / / / Update dependency pinningGravatar sco12019-10-20-45/+54
| | | |/ / / / | | |/| | | |
| | | * | | | Make test less fragile and improve test nameGravatar Sebastiaan Zeeff2019-10-19-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `test_returns_400_for_active_infractions_of_type_that_cannot_be_ active` test relied on the order in which the validation was done since it contained incompatible combinations of arguments. The test has been changed to make sure the data is valid except for the thing we actually want to test. I have also tried to improve the name of the test that tests the `test_unique_constraint_accepts_active_infraction_after_inactive_ infraction` test. It now includes the logic of what it does, but not the entire name of the test it's testing.
| | | * | | | Fix indentation and missing word in docstringGravatar Sebastiaan Zeeff2019-10-19-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The last two lines in the docstring of `InfractionFactory` in the `test_active_infraction_migration.py` file were overindented by one space. I've removed the space. The docstring of the `test_loader_build_graph_gets_called_once` method in the `MigrationsTestCaseNoSideEffectsTests` calss of the `test_base.py` file was missing two words. Corrected.
| | | * | | | Apply feedback for pull request #278Gravatar Sebastiaan Zeeff2019-10-19-273/+373
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/site/pull/278 This commit applies the feedback given to PR #278. I have made a number of significant changes: - I have added tests for the `MigrationsTestCase` class, including a test to see if the test case travels to the specified points in the migration history. - The dictionary of infracted users in the active migration tests now uses informative strings as keys. This makes it easier to relate a specific test to the data it's testing. - I have simplified the migration query logic by replacing unneeded Q objects by simpler keyword arguments to `filter` method calls. - I have removed the custom validation logic for allowing only one active infraction of a given type per user and replaced it by a `UniqueTogetherValidator` with a custom validation error message. - `test_unique_constraint_accepts_active_infraction_after_inactive_ infraction` relied on an Error results instead of a Fail result to indicate an issue with the database constraint. Since a fail should lead to a Fail status, I've replaced it by a try-except block that signals a test failure if an `IntegrityError` was caught by it.
| | | * | | | Solve migration conflict by renaming migrationsGravatar Sebastiaan Zeeff2019-10-18-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The migration files were generated and named before the migrations added by other pull requests. This caused the migration path to diverge. Since the migrations did not touch the same models, the solution was to rename the migration files to place them at the end of the migration history.
| | | * | | | Merge branch 'master' into active-infractions-validationGravatar Sebastiaan Zeeff2019-10-18-124/+1389
| | | |\ \ \ \ | | | |/ / / / | | |/| | | |
| | | * | | | Prevent double active infractions with constraintGravatar Sebastiaan Zeeff2019-10-08-2/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/site/issues/273 This commits adds a UniqueConstraint for active infractions on a combination of the `user` and `type` field. This means that a user can only have one active infraction of a given type in the database at any time. I've also added tests to make sure that this behaves as expected.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-19 01:59:45 +0000