fix(prepush): adjusts pre-commit to run tests in shell

.pre-commit-config.yaml

@@ -40,7 +40,7 @@ repos:
         stages: [pre-push]
     -   id: django-test
         name: 'Checking Tests'
-        entry: python manage.py test
+        entry: pipenv run python manage.py test
         always_run: true
         pass_filenames: false
         language: system

Pipfile

@@ -24,9 +24,9 @@ sentry-sdk = {extras = ["django"], version = "*"}
 pytz = "*"
 redis = "*"
 coverage = "*"
-celery = "*"
 django-celery-beat = "*"
 pre-commit = "*"
+celery = "*"
 
 [dev-packages]
 flake8 = "*"

README.md

@@ -335,11 +335,18 @@ Below are the commands and expected outputs:
 
 ```bash
 pre-commit install
-pre-commit installed at .git/hooks/pre-commit
+```
+`pre-commit installed at .git/hooks/pre-commit`
 
+```bash
 pre-commit install --hook-type commit-msg
-pre-commit installed at .git/hooks/commit-msg
 ```
+`pre-commit installed at .git/hooks/commit-msg`
+
+```bash
+pre-commit install --hook-type pre-push
+```
+`pre-commit installed at .git/hooks/pre-push`
 
 To run the pre-commit checks before making a commit, run the following command.
 
@@ -349,11 +356,9 @@ pre-commit run --all-files
 
 If no files need changes, the output should look like this:
 
-```bash
-isort....................................................................Passed
-black....................................................................Passed
-flake8...................................................................Passed
-```
+`isort....................................................................Passed` 
+`black....................................................................Passed` 
+`flake8...................................................................Passed` 
 
 If isort or black catch any errors, they will automatically alter the files to fix them.
 This will prevent making a commit, and you will need to stage the new changes.

team_production_system/admin.py

@@ -1,5 +1,6 @@
 from django.contrib import admin
 from django.contrib.auth.admin import UserAdmin
+
 from .models import (
     Availability,
     CustomUser,
@@ -18,7 +19,6 @@ class SessionAdmin(admin.ModelAdmin):
     readonly_fields = ('created_at', 'modified_at')
 
 
-# admin.site.register(UserAdmin)
 admin.site.register(CustomUser, UserAdmin)
 admin.site.register(Mentor)
 admin.site.register(Mentee)

team_production_system/tests/end_to_end_tests/test_user_profile_patch_upload.py

@@ -3,16 +3,15 @@
 from django.urls import reverse
 from rest_framework import status
 from rest_framework.test import APIClient
+
 from team_production_system.models import CustomUser
 
 
 class UserProfilePatchTestCase(TestCase):
     def setUp(self):
         self.client = APIClient()
         self.user = CustomUser.objects.create(
-          username="baby_yoda",
-          email="[email protected]",
-          password="badpassword"
+            username='baby_yoda', email='[email protected]', password='badpassword'
         )
 
     def test_new_user_gets_random_photo_assigned(self):
@@ -22,14 +21,12 @@ def test_new_user_gets_random_photo_assigned(self):
     def test_profile_photo_uploaded_first_time(self):
         # mock photo file
         photo = SimpleUploadedFile(
-          "photo.jpg",
-          b"this is a photo",
-          content_type="image/jpeg"
+            'photo.jpg', b'this is a photo', content_type='image/jpeg'
         )
         # Make a PATCH request to update the user profile with the new photo
-        url = reverse("my-profile")
+        url = reverse('my-profile')
         self.client.force_authenticate(user=self.user)
-        response = self.client.patch(url, {"profile_photo": photo})
+        response = self.client.patch(url, {'profile_photo': photo})
 
         # Check that the response status code is 200 OK
         self.assertEqual(response.status_code, status.HTTP_200_OK)
@@ -43,15 +40,13 @@ def test_previous_file_deleted_when_new_photo_saved(self):
         old_photo_name = self.user.profile_photo.name
         # Create a new profile photo file
         new_photo = SimpleUploadedFile(
-          "new_photo.jpg",
-          b"this is a new photo",
-          content_type="image/jpeg"
+            'new_photo.jpg', b'this is a new photo', content_type='image/jpeg'
         )
 
         # Make a PATCH request to update the user profile with the new photo
-        url = reverse("my-profile")
+        url = reverse('my-profile')
         self.client.force_authenticate(user=self.user)
-        response = self.client.patch(url, {"profile_photo": new_photo})
+        response = self.client.patch(url, {'profile_photo': new_photo})
 
         # Check that the response status code is 200 OK
         self.assertEqual(response.status_code, status.HTTP_200_OK)