[PATCH v2 1/6] trivial: Rename 'CoverLetter' references to 'Cover'

Stephen Finucane stephen at that.guru
Fri Apr 10 03:12:57 AEST 2020


We're going to be doing some model surgery shortly. Do the necessary
renaming of variables ahead of this.

Signed-off-by: Stephen Finucane <stephen at that.guru>
---
 docs/api/schemas/latest/patchwork.yaml | 14 +++++++-------
 docs/api/schemas/patchwork.j2          | 14 +++++++-------
 docs/api/schemas/v1.0/patchwork.yaml   | 14 +++++++-------
 docs/api/schemas/v1.1/patchwork.yaml   | 14 +++++++-------
 docs/api/schemas/v1.2/patchwork.yaml   | 14 +++++++-------
 patchwork/api/cover.py                 | 24 ++++++++++++------------
 patchwork/api/embedded.py              |  2 +-
 patchwork/api/event.py                 |  4 ++--
 patchwork/api/filters.py               |  2 +-
 patchwork/api/series.py                |  4 ++--
 patchwork/tests/api/test_cover.py      |  2 +-
 patchwork/tests/test_detail.py         |  2 +-
 patchwork/urls.py                      |  4 ++--
 13 files changed, 57 insertions(+), 57 deletions(-)

diff --git docs/api/schemas/latest/patchwork.yaml docs/api/schemas/latest/patchwork.yaml
index 13cdc9cd..082248a3 100644
--- docs/api/schemas/latest/patchwork.yaml
+++ docs/api/schemas/latest/patchwork.yaml
@@ -257,7 +257,7 @@ paths:
               schema:
                 type: array
                 items:
-                  $ref: '#/components/schemas/CoverLetterList'
+                  $ref: '#/components/schemas/CoverList'
       tags:
         - covers
   /api/covers/{id}/:
@@ -278,7 +278,7 @@ paths:
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/CoverLetterDetail'
+                $ref: '#/components/schemas/CoverDetail'
         '404':
           description: Not found
           content:
@@ -1573,7 +1573,7 @@ components:
           items:
             type: string
           readOnly: true
-    CoverLetterList:
+    CoverList:
       type: object
       properties:
         id:
@@ -1635,9 +1635,9 @@ components:
           type: string
           format: uri
           readOnly: true
-    CoverLetterDetail:
+    CoverDetail:
       allOf:
-        - $ref: '#/components/schemas/CoverLetterList'
+        - $ref: '#/components/schemas/CoverList'
         - properties:
             headers:
               title: Headers
@@ -2169,7 +2169,7 @@ components:
           format: uri
           readOnly: true
         cover_letter:
-          $ref: '#/components/schemas/CoverLetterEmbedded'
+          $ref: '#/components/schemas/CoverEmbedded'
         patches:
           title: Patches
           type: array
@@ -2282,7 +2282,7 @@ components:
           maxLength: 255
           minLength: 1
           readOnly: true
-    CoverLetterEmbedded:
+    CoverEmbedded:
       type: object
       properties:
         id:
diff --git docs/api/schemas/patchwork.j2 docs/api/schemas/patchwork.j2
index bd714d5e..91faef88 100644
--- docs/api/schemas/patchwork.j2
+++ docs/api/schemas/patchwork.j2
@@ -262,7 +262,7 @@ paths:
               schema:
                 type: array
                 items:
-                  $ref: '#/components/schemas/CoverLetterList'
+                  $ref: '#/components/schemas/CoverList'
       tags:
         - covers
   /api/{{ version_url }}covers/{id}/:
@@ -283,7 +283,7 @@ paths:
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/CoverLetterDetail'
+                $ref: '#/components/schemas/CoverDetail'
         '404':
           description: Not found
           content:
@@ -1629,7 +1629,7 @@ components:
           items:
             type: string
           readOnly: true
-    CoverLetterList:
+    CoverList:
       type: object
       properties:
         id:
@@ -1699,9 +1699,9 @@ components:
           format: uri
           readOnly: true
 {% endif %}
-    CoverLetterDetail:
+    CoverDetail:
       allOf:
-        - $ref: '#/components/schemas/CoverLetterList'
+        - $ref: '#/components/schemas/CoverList'
         - properties:
             headers:
               title: Headers
@@ -2253,7 +2253,7 @@ components:
           format: uri
           readOnly: true
         cover_letter:
-          $ref: '#/components/schemas/CoverLetterEmbedded'
+          $ref: '#/components/schemas/CoverEmbedded'
         patches:
           title: Patches
           type: array
@@ -2368,7 +2368,7 @@ components:
           maxLength: 255
           minLength: 1
           readOnly: true
-    CoverLetterEmbedded:
+    CoverEmbedded:
       type: object
       properties:
         id:
diff --git docs/api/schemas/v1.0/patchwork.yaml docs/api/schemas/v1.0/patchwork.yaml
index 6c7c068a..642c7773 100644
--- docs/api/schemas/v1.0/patchwork.yaml
+++ docs/api/schemas/v1.0/patchwork.yaml
@@ -142,7 +142,7 @@ paths:
               schema:
                 type: array
                 items:
-                  $ref: '#/components/schemas/CoverLetterList'
+                  $ref: '#/components/schemas/CoverList'
       tags:
         - covers
   /api/1.0/covers/{id}/:
@@ -163,7 +163,7 @@ paths:
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/CoverLetterDetail'
+                $ref: '#/components/schemas/CoverDetail'
         '404':
           description: Not found
           content:
@@ -1389,7 +1389,7 @@ components:
           items:
             type: string
           readOnly: true
-    CoverLetterList:
+    CoverList:
       type: object
       properties:
         id:
@@ -1431,9 +1431,9 @@ components:
           items:
             $ref: '#/components/schemas/SeriesEmbedded'
           readOnly: true
-    CoverLetterDetail:
+    CoverDetail:
       allOf:
-        - $ref: '#/components/schemas/CoverLetterList'
+        - $ref: '#/components/schemas/CoverList'
         - properties:
             headers:
               title: Headers
@@ -1881,7 +1881,7 @@ components:
           format: uri
           readOnly: true
         cover_letter:
-          $ref: '#/components/schemas/CoverLetterEmbedded'
+          $ref: '#/components/schemas/CoverEmbedded'
         patches:
           title: Patches
           type: array
@@ -1967,7 +1967,7 @@ components:
           maxLength: 255
           minLength: 1
           readOnly: true
-    CoverLetterEmbedded:
+    CoverEmbedded:
       type: object
       properties:
         id:
diff --git docs/api/schemas/v1.1/patchwork.yaml docs/api/schemas/v1.1/patchwork.yaml
index 6b497aba..e91457e5 100644
--- docs/api/schemas/v1.1/patchwork.yaml
+++ docs/api/schemas/v1.1/patchwork.yaml
@@ -142,7 +142,7 @@ paths:
               schema:
                 type: array
                 items:
-                  $ref: '#/components/schemas/CoverLetterList'
+                  $ref: '#/components/schemas/CoverList'
       tags:
         - covers
   /api/1.1/covers/{id}/:
@@ -163,7 +163,7 @@ paths:
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/CoverLetterDetail'
+                $ref: '#/components/schemas/CoverDetail'
         '404':
           description: Not found
           content:
@@ -1399,7 +1399,7 @@ components:
           items:
             type: string
           readOnly: true
-    CoverLetterList:
+    CoverList:
       type: object
       properties:
         id:
@@ -1456,9 +1456,9 @@ components:
           type: string
           format: uri
           readOnly: true
-    CoverLetterDetail:
+    CoverDetail:
       allOf:
-        - $ref: '#/components/schemas/CoverLetterList'
+        - $ref: '#/components/schemas/CoverList'
         - properties:
             headers:
               title: Headers
@@ -1950,7 +1950,7 @@ components:
           format: uri
           readOnly: true
         cover_letter:
-          $ref: '#/components/schemas/CoverLetterEmbedded'
+          $ref: '#/components/schemas/CoverEmbedded'
         patches:
           title: Patches
           type: array
@@ -2036,7 +2036,7 @@ components:
           maxLength: 255
           minLength: 1
           readOnly: true
-    CoverLetterEmbedded:
+    CoverEmbedded:
       type: object
       properties:
         id:
diff --git docs/api/schemas/v1.2/patchwork.yaml docs/api/schemas/v1.2/patchwork.yaml
index db2ed122..ed1cb5af 100644
--- docs/api/schemas/v1.2/patchwork.yaml
+++ docs/api/schemas/v1.2/patchwork.yaml
@@ -257,7 +257,7 @@ paths:
               schema:
                 type: array
                 items:
-                  $ref: '#/components/schemas/CoverLetterList'
+                  $ref: '#/components/schemas/CoverList'
       tags:
         - covers
   /api/1.2/covers/{id}/:
@@ -278,7 +278,7 @@ paths:
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/CoverLetterDetail'
+                $ref: '#/components/schemas/CoverDetail'
         '404':
           description: Not found
           content:
@@ -1573,7 +1573,7 @@ components:
           items:
             type: string
           readOnly: true
-    CoverLetterList:
+    CoverList:
       type: object
       properties:
         id:
@@ -1635,9 +1635,9 @@ components:
           type: string
           format: uri
           readOnly: true
-    CoverLetterDetail:
+    CoverDetail:
       allOf:
-        - $ref: '#/components/schemas/CoverLetterList'
+        - $ref: '#/components/schemas/CoverList'
         - properties:
             headers:
               title: Headers
@@ -2169,7 +2169,7 @@ components:
           format: uri
           readOnly: true
         cover_letter:
-          $ref: '#/components/schemas/CoverLetterEmbedded'
+          $ref: '#/components/schemas/CoverEmbedded'
         patches:
           title: Patches
           type: array
@@ -2282,7 +2282,7 @@ components:
           maxLength: 255
           minLength: 1
           readOnly: true
-    CoverLetterEmbedded:
+    CoverEmbedded:
       type: object
       properties:
         id:
diff --git patchwork/api/cover.py patchwork/api/cover.py
index c0c27fe8..66d2146c 100644
--- patchwork/api/cover.py
+++ patchwork/api/cover.py
@@ -11,14 +11,14 @@ from rest_framework.reverse import reverse
 from rest_framework.serializers import SerializerMethodField
 
 from patchwork.api.base import BaseHyperlinkedModelSerializer
-from patchwork.api.filters import CoverLetterFilterSet
+from patchwork.api.filters import CoverFilterSet
 from patchwork.api.embedded import PersonSerializer
 from patchwork.api.embedded import ProjectSerializer
 from patchwork.api.embedded import SeriesSerializer
 from patchwork.models import CoverLetter
 
 
-class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
+class CoverListSerializer(BaseHyperlinkedModelSerializer):
 
     web_url = SerializerMethodField()
     project = ProjectSerializer(read_only=True)
@@ -43,7 +43,7 @@ class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
         # NOTE(stephenfin): This is here to ensure our API looks the same even
         # after we changed the series-patch relationship from M:N to 1:N. It
         # will be removed in API v2
-        data = super(CoverLetterListSerializer, self).to_representation(
+        data = super(CoverListSerializer, self).to_representation(
             instance)
         data['series'] = [data['series']] if data['series'] else []
         return data
@@ -63,7 +63,7 @@ class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
         }
 
 
-class CoverLetterDetailSerializer(CoverLetterListSerializer):
+class CoverDetailSerializer(CoverListSerializer):
 
     headers = SerializerMethodField()
 
@@ -83,18 +83,18 @@ class CoverLetterDetailSerializer(CoverLetterListSerializer):
 
     class Meta:
         model = CoverLetter
-        fields = CoverLetterListSerializer.Meta.fields + (
+        fields = CoverListSerializer.Meta.fields + (
             'headers', 'content')
         read_only_fields = fields
-        extra_kwargs = CoverLetterListSerializer.Meta.extra_kwargs
-        versioned_fields = CoverLetterListSerializer.Meta.versioned_fields
+        extra_kwargs = CoverListSerializer.Meta.extra_kwargs
+        versioned_fields = CoverListSerializer.Meta.versioned_fields
 
 
-class CoverLetterList(ListAPIView):
+class CoverList(ListAPIView):
     """List cover letters."""
 
-    serializer_class = CoverLetterListSerializer
-    filter_class = filterset_class = CoverLetterFilterSet
+    serializer_class = CoverListSerializer
+    filter_class = filterset_class = CoverFilterSet
     search_fields = ('name',)
     ordering_fields = ('id', 'name', 'date', 'submitter')
     ordering = 'id'
@@ -106,10 +106,10 @@ class CoverLetterList(ListAPIView):
             .defer('content', 'headers')
 
 
-class CoverLetterDetail(RetrieveAPIView):
+class CoverDetail(RetrieveAPIView):
     """Show a cover letter."""
 
-    serializer_class = CoverLetterDetailSerializer
+    serializer_class = CoverDetailSerializer
 
     def get_queryset(self):
         return CoverLetter.objects.all()\
diff --git patchwork/api/embedded.py patchwork/api/embedded.py
index 85a30cae..d2300dff 100644
--- patchwork/api/embedded.py
+++ patchwork/api/embedded.py
@@ -103,7 +103,7 @@ class CheckSerializer(SerializedRelatedField):
             }
 
 
-class CoverLetterSerializer(SerializedRelatedField):
+class CoverSerializer(SerializedRelatedField):
 
     class _Serializer(MboxMixin, WebURLMixin, BaseHyperlinkedModelSerializer):
 
diff --git patchwork/api/event.py patchwork/api/event.py
index d7a99c7d..7ed9efb1 100644
--- patchwork/api/event.py
+++ patchwork/api/event.py
@@ -11,7 +11,7 @@ from rest_framework.serializers import SerializerMethodField
 from rest_framework.serializers import SlugRelatedField
 
 from patchwork.api.embedded import CheckSerializer
-from patchwork.api.embedded import CoverLetterSerializer
+from patchwork.api.embedded import CoverSerializer
 from patchwork.api.embedded import PatchSerializer
 from patchwork.api.embedded import PatchRelationSerializer
 from patchwork.api.embedded import ProjectSerializer
@@ -27,7 +27,7 @@ class EventSerializer(ModelSerializer):
     actor = UserSerializer()
     patch = PatchSerializer(read_only=True)
     series = SeriesSerializer(read_only=True)
-    cover = CoverLetterSerializer(read_only=True)
+    cover = CoverSerializer(read_only=True)
     previous_state = SlugRelatedField(slug_field='slug', read_only=True)
     current_state = SlugRelatedField(slug_field='slug', read_only=True)
     previous_delegate = UserSerializer()
diff --git patchwork/api/filters.py patchwork/api/filters.py
index deb5ace1..80b17232 100644
--- patchwork/api/filters.py
+++ patchwork/api/filters.py
@@ -184,7 +184,7 @@ class SeriesFilterSet(TimestampMixin, BaseFilterSet):
         fields = ('submitter', 'project')
 
 
-class CoverLetterFilterSet(TimestampMixin, BaseFilterSet):
+class CoverFilterSet(TimestampMixin, BaseFilterSet):
 
     project = ProjectFilter(queryset=Project.objects.all(), distinct=False)
     # NOTE(stephenfin): We disable the select-based HTML widgets for these
diff --git patchwork/api/series.py patchwork/api/series.py
index e4cffaa3..106e60f0 100644
--- patchwork/api/series.py
+++ patchwork/api/series.py
@@ -10,7 +10,7 @@ from rest_framework.serializers import SerializerMethodField
 from patchwork.api.base import BaseHyperlinkedModelSerializer
 from patchwork.api.base import PatchworkPermission
 from patchwork.api.filters import SeriesFilterSet
-from patchwork.api.embedded import CoverLetterSerializer
+from patchwork.api.embedded import CoverSerializer
 from patchwork.api.embedded import PatchSerializer
 from patchwork.api.embedded import PersonSerializer
 from patchwork.api.embedded import ProjectSerializer
@@ -23,7 +23,7 @@ class SeriesSerializer(BaseHyperlinkedModelSerializer):
     project = ProjectSerializer(read_only=True)
     submitter = PersonSerializer(read_only=True)
     mbox = SerializerMethodField()
-    cover_letter = CoverLetterSerializer(read_only=True)
+    cover_letter = CoverSerializer(read_only=True)
     patches = PatchSerializer(read_only=True, many=True)
 
     def get_web_url(self, instance):
diff --git patchwork/tests/api/test_cover.py patchwork/tests/api/test_cover.py
index 5eeb1902..4f3ad841 100644
--- patchwork/tests/api/test_cover.py
+++ patchwork/tests/api/test_cover.py
@@ -21,7 +21,7 @@ if settings.ENABLE_REST_API:
 
 
 @unittest.skipUnless(settings.ENABLE_REST_API, 'requires ENABLE_REST_API')
-class TestCoverLetterAPI(utils.APITestCase):
+class TestCoverAPI(utils.APITestCase):
     fixtures = ['default_tags']
 
     @staticmethod
diff --git patchwork/tests/test_detail.py patchwork/tests/test_detail.py
index 92fe2d7e..ddc2b937 100644
--- patchwork/tests/test_detail.py
+++ patchwork/tests/test_detail.py
@@ -12,7 +12,7 @@ from patchwork.tests.utils import create_patch
 from patchwork.tests.utils import create_project
 
 
-class CoverLetterViewTest(TestCase):
+class CoverViewTest(TestCase):
 
     def test_redirect(self):
         patch = create_patch()
diff --git patchwork/urls.py patchwork/urls.py
index dcdcfb49..9c3b85f0 100644
--- patchwork/urls.py
+++ patchwork/urls.py
@@ -207,10 +207,10 @@ if settings.ENABLE_REST_API:
             api_person_views.PersonDetail.as_view(),
             name='api-person-detail'),
         url(r'^covers/$',
-            api_cover_views.CoverLetterList.as_view(),
+            api_cover_views.CoverList.as_view(),
             name='api-cover-list'),
         url(r'^covers/(?P<pk>[^/]+)/$',
-            api_cover_views.CoverLetterDetail.as_view(),
+            api_cover_views.CoverDetail.as_view(),
             name='api-cover-detail'),
         url(r'^patches/$',
             api_patch_views.PatchList.as_view(),
-- 
2.25.2



More information about the Patchwork mailing list