Added Issue and Releases Field (#625)

* added field

* changed in frontend

* changed

* fixed

* added dark theme support

* Update code

* Update code

* fetched from graphql

* fixed graphql error

* added test for both backend and frontend

* fixed linting and test

* fixed linting and test

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Kate Golovanova <[email protected]>

Author: 3 people

backend/apps/github/graphql/nodes/user.py

@@ -1,18 +1,92 @@
 """GitHub user GraphQL node."""
 
+import graphene
+
 from apps.common.graphql.nodes import BaseNode
 from apps.github.models.user import User
 
 
+class RepositoryType(graphene.ObjectType):
+    """Repository type for nested objects."""
+
+    key = graphene.String()
+    owner_key = graphene.String()
+
+
+class IssueType(graphene.ObjectType):
+    """GitHub issue type."""
+
+    created_at = graphene.Float()
+    comments_count = graphene.Int()
+    number = graphene.Int()
+    repository = graphene.Field(RepositoryType)
+    title = graphene.String()
+
+
+class ReleaseType(graphene.ObjectType):
+    """GitHub release type."""
+
+    is_pre_release = graphene.Boolean()
+    name = graphene.String()
+    published_at = graphene.Float()
+    repository = graphene.Field(RepositoryType)
+    tag_name = graphene.String()
+
+
 class UserNode(BaseNode):
     """GitHub user node."""
 
+    # Existing fields
+    created_at = graphene.Float()
+    updated_at = graphene.Float()
+    url = graphene.String()
+
+    # New fields for issues and releases
+    issues = graphene.List(IssueType)
+    issues_count = graphene.Int()
+    releases = graphene.List(ReleaseType)
+    releases_count = graphene.Int()
+
     class Meta:
         model = User
         fields = (
             "avatar_url",
+            "bio",
+            "company",
             "email",
+            "followers_count",
+            "following_count",
             "id",
+            "location",
             "login",
             "name",
+            "public_repositories_count",
         )
+
+    def resolve_created_at(self, info):
+        """Resolve user created at."""
+        return self.idx_created_at
+
+    def resolve_updated_at(self, info):
+        """Resolve user updated at."""
+        return self.idx_updated_at
+
+    def resolve_url(self, info):
+        """Resolve user URL."""
+        return self.url
+
+    def resolve_issues(self, info):
+        """Resolve user issues."""
+        return self.idx_issues
+
+    def resolve_issues_count(self, info):
+        """Resolve user issues count."""
+        return self.idx_issues_count
+
+    def resolve_releases(self, info):
+        """Resolve user releases."""
+        return self.idx_releases
+
+    def resolve_releases_count(self, info):
+        """Resolve user releases count."""
+        return self.idx_releases_count

backend/apps/github/graphql/queries/__init__.py

@@ -1,7 +1,8 @@
 """GitHub GraphQL queries."""
 
 from apps.github.graphql.queries.repository import RepositoryQuery
+from apps.github.graphql.queries.user import UserQuery
 
 
-class GithubQuery(RepositoryQuery):
+class GithubQuery(RepositoryQuery, UserQuery):
     """GitHub query."""

backend/apps/github/graphql/queries/user.py

@@ -0,0 +1,20 @@
+"""OWASP github GraphQL queries."""
+
+import graphene
+
+from apps.common.graphql.queries import BaseQuery
+from apps.github.graphql.nodes.user import UserNode
+from apps.github.models.user import User
+
+
+class UserQuery(BaseQuery):
+    """User queries."""
+
+    user = graphene.Field(UserNode, login=graphene.String(required=True))
+
+    def resolve_user(root, info, login):
+        """Resolve user by login."""
+        try:
+            return User.objects.get(login=login)
+        except User.DoesNotExist:
+            return None

backend/tests/github/graphql/nodes/user_test.py

@@ -14,12 +14,26 @@ def test_user_node_inheritance(self):
 
     def test_meta_configuration(self):
         """Test if Meta is properly configured."""
-        assert UserNode._meta.model == User
-        expected_fields = {
-            "avatar_url",
-            "email",
-            "id",
-            "login",
-            "name",
-        }
-        assert set(UserNode._meta.fields) == expected_fields
+
+    assert UserNode._meta.model == User
+    expected_fields = {
+        "avatar_url",
+        "bio",
+        "company",
+        "created_at",
+        "email",
+        "followers_count",
+        "following_count",
+        "id",
+        "issues_count",
+        "issues",
+        "location",
+        "login",
+        "name",
+        "public_repositories_count",
+        "releases_count",
+        "releases",
+        "updated_at",
+        "url",
+    }
+    assert set(UserNode._meta.fields) == expected_fields

backend/tests/github/graphql/queries/user_test.py

@@ -0,0 +1,61 @@
+"""Test cases for UserQuery."""
+
+from unittest.mock import Mock, patch
+
+import pytest
+from graphene import Field, NonNull, String
+
+from apps.common.graphql.queries import BaseQuery
+from apps.github.graphql.nodes.user import UserNode
+from apps.github.graphql.queries.user import UserQuery
+from apps.github.models.user import User
+
+
+class TestUserQuery:
+    """Test cases for UserQuery class."""
+
+    @pytest.fixture()
+    def mock_info(self):
+        """GraphQL info mock fixture."""
+        return Mock()
+
+    @pytest.fixture()
+    def mock_user(self):
+        """User mock fixture."""
+        return Mock(spec=User)
+
+    def test_user_query_inheritance(self):
+        """Test if UserQuery inherits from BaseQuery."""
+        assert issubclass(UserQuery, BaseQuery)
+
+    def test_resolve_user_existing(self, mock_user, mock_info):
+        """Test resolving an existing user."""
+        with patch("apps.github.models.user.User.objects.get") as mock_get:
+            mock_get.return_value = mock_user
+
+            result = UserQuery.resolve_user(None, mock_info, login="test-user")
+
+            assert result == mock_user
+            mock_get.assert_called_once_with(login="test-user")
+
+    def test_resolve_user_not_found(self, mock_info):
+        """Test resolving a non-existent user."""
+        with patch("apps.github.models.user.User.objects.get") as mock_get:
+            mock_get.side_effect = User.DoesNotExist
+
+            result = UserQuery.resolve_user(None, mock_info, login="non-existent")
+
+            assert result is None
+            mock_get.assert_called_once_with(login="non-existent")
+
+    def test_user_field_configuration(self):
+        """Test if user field is properly configured."""
+        user_field = UserQuery._meta.fields.get("user")
+
+        assert isinstance(user_field, Field)
+        assert user_field.type == UserNode
+        assert "login" in user_field.args
+
+        login_arg = user_field.args["login"]
+        assert isinstance(login_arg.type, NonNull)
+        assert login_arg.type.of_type == String

frontend/__tests__/src/data/mockUserData.ts

@@ -1,16 +1,18 @@
 export const mockUserData = {
   users: [
     {
+      avatar_url: 'https://example.com/avatar1.jpg',
+      company: 'OWASP',
       key: 'user_1',
+      login: 'user_1',
       name: 'John Doe',
-      company: 'OWASP',
-      avatar_url: 'https://example.com/avatar1.jpg',
     },
     {
+      avatar_url: 'https://example.com/avatar2.jpg',
+      company: 'Security Co',
       key: 'user_2',
+      login: 'user_2',
       name: 'Jane Smith',
-      company: 'Security Co',
-      avatar_url: 'https://example.com/avatar2.jpg',
     },
   ],
 }

frontend/__tests__/src/data/mockUserDetails.ts

@@ -0,0 +1,40 @@
+export const mockUserDetailsData = {
+  user: {
+    login: 'testuser',
+    name: 'Test User',
+    avatarUrl: 'https://example.com/avatar.jpg',
+    url: 'https://github.com/testuser',
+    bio: 'This is a test user',
+    company: 'Test Company',
+    location: 'Test Location',
+    email: '[email protected]',
+    followersCount: 10,
+    followingCount: 5,
+    publicRepositoriesCount: 3,
+    createdAt: 1723002473,
+    issues: [
+      {
+        number: 1,
+        title: 'Test Issue',
+        createdAt: 1723002473,
+        commentsCount: 5,
+        repository: {
+          key: 'test-repo',
+          ownerKey: 'testuser',
+        },
+      },
+    ],
+    releases: [
+      {
+        name: 'v1.0.0',
+        tagName: '1.0.0',
+        isPreRelease: false,
+        publishedAt: 1723002473,
+        repository: {
+          key: 'test-repo',
+          ownerKey: 'testuser',
+        },
+      },
+    ],
+  },
+}