add github workflows and refactoring code

.flake8

@@ -0,0 +1,14 @@
+[flake8]
+exclude = 
+    migrations,
+    venv,
+    env,
+    __pycache__,
+    .git,
+    .github,
+    appender.py,
+    work.py,
+    faker_data.py,
+max-line-length = 88
+ignore = E203, E266, E501, W503
+select = B,C,E,F,W

.github/workflows/ci.yml

@@ -0,0 +1,43 @@
+name: CI/CD Pipeline
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Python
+        uses: actions/setup-python@v2
+        with:
+          python-version: '3.10'
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+      
+      - name: Set up enviroment varibles
+        run: |
+          echo "SECRET_KEY"=${{ secrets.DJANGO_SECRET_KEY }} >> $GITHUB_ENV
+        
+      - name: Run linters
+        run: |
+          flake8
+
+      - name: Run tests
+        run: |
+          python manage.py test
+
+      - name: Deploy
+        run: |
+          echo "Deploying application..."

apps/shared/management/commands/createadmin.py

@@ -7,7 +7,7 @@ def handle(self, *args, **options):
         User = get_user_model()
         options.setdefault("is_active", True)
         self.create_superuser(User, "Akromjon9", "[email protected]", "2007")
-        
+
     def create_superuser(self, User, username, email, password):
         if not User.objects.filter(username=username).exists():
             user = User.objects.create_superuser(username, email, password)
@@ -17,6 +17,4 @@ def create_superuser(self, User, username, email, password):
                 self.style.SUCCESS(f"Superuser {username} created successfully.")
             )
         else:
-            self.stdout.write(
-                self.style.ERROR(f"Superuser {username} already exists.")
-            )
+            self.stdout.write(self.style.ERROR(f"Superuser {username} already exists."))

apps/spotify/api_endpoints/Album/AlbumCreate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Album/AlbumCreate/test.py

@@ -1,6 +1,6 @@
 from django.test import TestCase
 from django.urls import reverse
-from apps.spotify.models import Album, Artist
+from apps.spotify.models import Artist
 
 
 class AlbumCreateTest(TestCase):

apps/spotify/api_endpoints/Album/AlbumDestroy/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Album/AlbumList/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Album/AlbumRetrive/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Album/AlbumUpdate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Album/__init__.py

@@ -1,5 +1,5 @@
-from .AlbumCreate import *
-from .AlbumDestroy import *
-from .AlbumList import *
-from .AlbumRetrive import *
-from .AlbumUpdate import *
+from .AlbumCreate import *  # noqa
+from .AlbumDestroy import *  # noqa
+from .AlbumList import *  # noqa
+from .AlbumRetrive import *  # noqa
+from .AlbumUpdate import *  # noqa

apps/spotify/api_endpoints/Artist/ArtistCreate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Artist/ArtistCreate/test.py

@@ -1,5 +1,4 @@
 from django.test import TestCase
-from apps.spotify.models import Artist
 from django.urls import reverse
 from faker import Faker
 

apps/spotify/api_endpoints/Artist/ArtistDestroy/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Artist/ArtistList/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Artist/ArtistRetrive/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Artist/ArtistUpdate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Artist/__init__.py

@@ -1,5 +1,5 @@
-from .ArtistCreate import *
-from .ArtistDestroy import *
-from .ArtistList import *
-from .ArtistRetrive import *
-from .ArtistUpdate import *
+from .ArtistCreate import *  # noqa
+from .ArtistDestroy import *  # noqa
+from .ArtistList import *  # noqa
+from .ArtistRetrive import *  # noqa
+from .ArtistUpdate import *  # noqa

apps/spotify/api_endpoints/Song/SongCreate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Song/SongCreate/test.py

@@ -1,7 +1,5 @@
 from django.test import TestCase
-from apps.spotify.models import Genre, Album, Artist, Song
-import json
-from django.core.files import File
+from apps.spotify.models import Genre, Album, Artist
 from django.urls import reverse
 
 

apps/spotify/api_endpoints/Song/SongDestroy/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Song/SongDestroy/test.py

@@ -1,6 +1,5 @@
 from django.test import TestCase
 from apps.spotify.models import Album, Genre, Artist, Song
-from rest_framework.test import APITestCase
 from django.urls import reverse
 
 

apps/spotify/api_endpoints/Song/SongList/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Song/SongList/test.py

@@ -1,6 +1,6 @@
 from django.test import TestCase
 from django.urls import reverse
-from apps.spotify.models import Song, Genre, Artist, Album
+from apps.spotify.models import Genre, Artist, Album
 from faker import Faker
 import random
 

apps/spotify/api_endpoints/Song/SongRetrive/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Song/SongRetrive/test.py

@@ -1,15 +1,3 @@
-<<<<<<< HEAD
-from django.test import TestCase
-
-# url = "http://127.0.0.1:8000/api/v1/spotify/song-retrive/2"
-
-
-# class TestSongRetrive(TestCase):
-
-#     def test_list(self):
-#         response = self.client.get(url)
-#         print(response.status_code)
-=======
 # from django.test import TestCase
 # from django.urls import reverse
 
@@ -18,4 +6,3 @@
 #     def setUp(self):
 
 #         self.url = reverse("spotify:song-retrive", kwargs={'pk':1})
->>>>>>> 8ba28da (update dir in push)

apps/spotify/api_endpoints/Song/SongRetrive/views.py

@@ -2,7 +2,6 @@
 from apps.spotify.models import Song
 from .serializers import SongRetriveSerializer
 from django.db import transaction
-from rest_framework.response import Response
 
 
 class SongRetriveAPIView(RetrieveAPIView):

apps/spotify/api_endpoints/Song/SongUpdate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/spotify/api_endpoints/Song/SongUpdate/test.py

@@ -1,5 +1,5 @@
-from django.test import TestCase
-from django.urls import reverse
+# from django.test import TestCase
+# from django.urls import reverse
 
 
 # class TestSongUpdate(TestCase):

apps/spotify/api_endpoints/Song/__init__.py

@@ -1,5 +1,5 @@
-from .SongCreate import *
-from .SongDestroy import *
-from .SongList import *
-from .SongRetrive import *
-from .SongUpdate import *
+from .SongCreate import *  # noqa
+from .SongDestroy import *  # noqa
+from .SongList import *  # noqa
+from .SongRetrive import *  # noqa
+from .SongUpdate import *  # noqa

apps/users/api_endpoints/Follow/UserArtistFollow/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserArtistFollow/serializers.py

@@ -1,5 +1,4 @@
 from rest_framework import serializers
-from apps.spotify.models import Artist
 
 
 class UserArtisFollowSerializer(serializers.Serializer):

apps/users/api_endpoints/Follow/UserArtistFollow/test.py

@@ -1,9 +1,7 @@
 from django.test import TestCase
-from apps.users.models import User
 from apps.spotify.models import Artist
 from django.urls import reverse
 from faker import Faker
-from rest_framework.authtoken.models import Token
 from apps.users.tests import AuthenticationTest
 
 fake = Faker()

apps/users/api_endpoints/Follow/UserArtistUnfollow/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserArtistUnfollow/test.py

@@ -1,7 +1,6 @@
 from django.test import TestCase
 from django.urls import reverse
 from apps.users.models import User
-from apps.spotify.models import Artist
 import faker
 from apps.users.tests import AuthenticationTest
 

apps/users/api_endpoints/Follow/UserFollow/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserFollow/views.py

@@ -1,5 +1,6 @@
 from rest_framework import status
-from rest_framework.authentication import TokenAuthentication
+
+# from rest_framework.authentication import TokenAuthentication
 from rest_framework.exceptions import APIException
 from rest_framework.permissions import IsAuthenticated
 from rest_framework.response import Response

apps/users/api_endpoints/Follow/UserFollowers/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserFollowers/test.py

@@ -10,13 +10,12 @@
 
 class UserUnfollowTest(AuthenticationTest, TestCase):
 
-
     def test_follow_user(self):
 
         for _ in range(5):
-            self.users = User.objects.create(username = fake.user_name())
-        self.user = User.objects.create(username = "Ahmadjon")
+            self.users = User.objects.create(username=fake.user_name())
+        self.user = User.objects.create(username="Ahmadjon")
         self.user.profiles.follow(User.objects.last())
-        url = reverse("users:followers-user", kwargs={"pk":self.user.id})
+        url = reverse("users:followers-user", kwargs={"pk": self.user.id})
         response = self.client.get(url, data={"userid": 1}, headers=self.headers)
         self.assertEqual(response.status_code, 200)

apps/users/api_endpoints/Follow/UserFollowing/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserUnfollow/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Follow/UserUnfollow/views.py

@@ -1,5 +1,4 @@
-from rest_framework import status
-from rest_framework.authentication import TokenAuthentication
+# from rest_framework.authentication import TokenAuthentication
 from rest_framework.exceptions import APIException
 from rest_framework.generics import get_object_or_404
 from rest_framework.permissions import IsAuthenticated

apps/users/api_endpoints/Follow/__init__.py

@@ -1,6 +1,6 @@
-from .UserFollow import *
-from .UserUnfollow import *
-from .UserFollowers import *
-from .UserFollowing import *
-from .UserArtistFollow import *
-from .UserArtistUnfollow import *
+from .UserFollow import *  # noqa
+from .UserUnfollow import *  # noqa
+from .UserFollowers import *  # noqa
+from .UserFollowing import *  # noqa
+from .UserArtistFollow import *  # noqa
+from .UserArtistUnfollow import *  # noqa

apps/users/api_endpoints/UserProfile/UserProfileDestroy/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/UserProfile/UserProfileDestroy/test.py

@@ -1,5 +1,5 @@
 from django.test import TestCase
-from apps.users.models import UserProfile, User
+from apps.users.models import User
 from django.urls import reverse
 
 

apps/users/api_endpoints/UserProfile/UserProfileUpdate/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/UserProfile/__init__.py

@@ -1,2 +1,2 @@
-from .UserProfileDestroy import *
-from .UserProfileUpdate import *
+from .UserProfileDestroy import *  # noqa
+from .UserProfileUpdate import *  # noqa

apps/users/api_endpoints/Users/ForgotPassword/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa

apps/users/api_endpoints/Users/ForgotPassword/views.py

@@ -1,20 +1,18 @@
+import os
+
+from django.views.decorators.cache import cache_page
+from django.utils.decorators import method_decorator
+from django.core.mail import send_mail
+
 from rest_framework.views import APIView
-from apps.users.models import User
 from rest_framework.response import Response
 from rest_framework.generics import get_object_or_404
 
+from apps.users.models import User
+
 from dotenv import load_dotenv
 
 load_dotenv()
-import os
-
-from django.core.mail import send_mail
-from apps.users.utils import generate_token
-
-from apps.users.models import User
-from celery import shared_task
-from django.views.decorators.cache import cache_page
-from django.utils.decorators import method_decorator
 
 
 class ForgotPasswordAPIView(APIView):

apps/users/api_endpoints/Users/UpdatePasswordByEmail/__init__.py

@@ -1 +1 @@
-from .views import *
+from .views import *  # noqa