makes students use core.

Author: filiplajszczak

pythonanywhere/api/__init__.py

@@ -9,7 +9,7 @@
 
 from snakesay import snakesay
 
-from pythonanywhere.api.students_api import StudentsAPI
+from pythonanywhere_core.students import StudentsAPI
 
 logger = logging.getLogger("pythonanywhere")
 

pythonanywhere/api/students_api.py

@@ -8,7 +8,7 @@ pytest==8.2.1
 pytest-cov==5.0.0
 pytest-mock==3.14.0
 pytest-mypy==0.10.3
-pythonanywhere_core==0.1.7
+pythonanywhere_core==0.1.8
 requests==2.32.3
 responses==0.25.0
 schema==0.7.2

pythonanywhere/api/students_api.pyi

@@ -28,7 +28,7 @@
         "Programming Language :: Python :: 3.8",
     ],
     keywords="pythonanywhere api cloud web hosting",
-    packages=["cli", "pythonanywhere", "pythonanywhere.api"],
+    packages=["cli", "pythonanywhere", ],
     install_requires=[
         "docopt",
         "packaging",

pythonanywhere/students.py

@@ -1,21 +1,21 @@
 import pytest
 from unittest.mock import call
 
-from pythonanywhere.api.students_api import StudentsAPI
+from pythonanywhere_core.students import StudentsAPI
 from pythonanywhere.students import Students
 
 
 @pytest.mark.students
 class TestStudentsInit:
     def test_instantiates_correctly(self):
         students = Students()
-        assert type(students.api) == StudentsAPI
+        assert isinstance(students.api, StudentsAPI)
 
 
 @pytest.mark.students
 class TestStudentsGet:
     def test_returns_list_of_usernames_when_found_in_api_response(self, mocker):
-        mock_students_api_get = mocker.patch("pythonanywhere.api.students_api.StudentsAPI.get")
+        mock_students_api_get = mocker.patch("pythonanywhere.students.StudentsAPI.get")
         student_usernames = ["student1", "student2"]
         mock_students_api_get.return_value = {
             "students": [{"username": s} for s in student_usernames]
@@ -27,7 +27,7 @@ def test_returns_list_of_usernames_when_found_in_api_response(self, mocker):
         assert result == student_usernames
 
     def test_returns_empty_list_when_no_usernames_found_in_api_response(self, mocker):
-        mock_students_api_get = mocker.patch("pythonanywhere.api.students_api.StudentsAPI.get")
+        mock_students_api_get = mocker.patch("pythonanywhere.students.StudentsAPI.get")
         mock_students_api_get.return_value = {"students": []}
 
         result = Students().get()
@@ -43,7 +43,7 @@ def test_returns_empty_list_when_no_usernames_found_in_api_response(self, mocker
         ]
     )
     def test_uses_correct_grammar_in_log_messages(self, mocker, api_response, expected_wording):
-        mock_students_api_get = mocker.patch("pythonanywhere.api.students_api.StudentsAPI.get")
+        mock_students_api_get = mocker.patch("pythonanywhere.students.StudentsAPI.get")
         mock_students_api_get.return_value = api_response
         mock_snake = mocker.patch("pythonanywhere.students.snakesay")
         mock_info =  mocker.patch("pythonanywhere.students.logger.info")
@@ -57,7 +57,7 @@ def test_uses_correct_grammar_in_log_messages(self, mocker, api_response, expect
 @pytest.mark.students
 class TestStudentsDelete:
     def test_returns_true_and_informs_when_student_removed(self, mocker):
-        mock_students_api_delete = mocker.patch("pythonanywhere.api.students_api.StudentsAPI.delete")
+        mock_students_api_delete = mocker.patch("pythonanywhere.students.StudentsAPI.delete")
         mock_students_api_delete.return_value = True
         mock_snake = mocker.patch("pythonanywhere.students.snakesay")
         mock_info =  mocker.patch("pythonanywhere.students.logger.info")
@@ -70,7 +70,7 @@ def test_returns_true_and_informs_when_student_removed(self, mocker):
         assert result is True
 
     def test_returns_false_and_warns_when_student_not_removed(self, mocker):
-        mock_students_api_delete = mocker.patch("pythonanywhere.api.students_api.StudentsAPI.delete")
+        mock_students_api_delete = mocker.patch("pythonanywhere.students.StudentsAPI.delete")
         mock_students_api_delete.side_effect = Exception("error msg")
         mock_students_api_delete.return_value = False
         mock_snake = mocker.patch("pythonanywhere.students.snakesay")