feat: add tests, refactor

Author: martinjagodic

dev-test/config.yml

@@ -5,9 +5,6 @@ site_url: https://example.com
 
 publish_mode: editorial_workflow
 media_folder: assets/uploads
-pagination:
-  enabled: false
-  per_page: 5
 
 collections: # A list of collections the CMS should be able to edit
   - name: 'posts' # Used in routes, ie.: /admin/collections/:slug/edit
@@ -22,7 +19,7 @@ collections: # A list of collections the CMS should be able to edit
     create: true # Allow users to create new documents in this collection
     pagination:
       enabled: true
-      per_page: 8
+      per_page: 10
     editor:
       visualEditing: true
     view_filters:

docs/PAGINATION_IMPLEMENTATION_PLAN.md

@@ -251,8 +251,9 @@ pagination?: boolean | {
 **Files to Modify:**
 - `/packages/decap-cms-core/src/actions/entries.ts` - All entry action creators
 - `/packages/decap-cms-core/src/backend.ts` - Check for incompatible features
-### Phase 8: Testing & Documentation
-**Goal:** Ensure quality and provide documentation
+
+### Phase 8: Testing
+**Goal:** Ensure quality
 
 **Tasks:**
 1. Unit tests:
@@ -271,18 +272,10 @@ pagination?: boolean | {
    - User workflows with pagination
    - Large collections (1000+ entries)
    - i18n collections with many locales/entries
-4. Documentation:
-   - Configuration guide
-   - Backend implementation guide for other backends
-   - Migration guide for users
 5. Performance testing:
    - Measure improvement with large collections
    - Memory usage comparison
 
-**Files to Create:**
-- `/docs/pagination.md` - User documentation
-- `/docs/pagination-backend-implementation.md` - Developer guide
-
 ### Phase 7: User Preferences & Polish
 **Goal:** Make pagination user-friendly and persistent
 

packages/decap-cms-backend-github/src/implementation.tsx

@@ -435,7 +435,7 @@ export default class GitHub implements Implementation {
         depth,
       }).then(files => {
         const filtered = files.filter(file => filterByExtension(file, extension));
-        
+
         if (usePagination) {
           // Paginated: return only the requested page
           const result = this.getCursorAndFiles(filtered, page, pageSize);

packages/decap-cms-backend-test/src/implementation.ts

@@ -176,7 +176,7 @@ export default class TestBackend implements Implementation {
     };
     const meta = cursor.meta!;
     const currentPageSize = meta.get('pageSize', DEFAULT_PAGE_SIZE);
-    
+
     const newIndex = (() => {
       if (action === 'next') {
         return (index as number) + 1;
@@ -198,7 +198,10 @@ export default class TestBackend implements Implementation {
       data: f.content as string,
       file: { path: f.path, id: f.path },
     }));
-    const entries = allEntries.slice(newIndex * currentPageSize, newIndex * currentPageSize + currentPageSize);
+    const entries = allEntries.slice(
+      newIndex * currentPageSize,
+      newIndex * currentPageSize + currentPageSize,
+    );
     const newCursor = getCursor(folder, extension, allEntries, newIndex, depth, currentPageSize);
     return Promise.resolve({ entries, cursor: newCursor });
   }
@@ -221,14 +224,12 @@ export default class TestBackend implements Implementation {
     const pageSize = options?.pageSize ?? DEFAULT_PAGE_SIZE;
     const page = options?.page ?? 1;
     const usePagination = options?.pagination ?? false;
-    
+
     const cursor = getCursor(folder, extension, entries, page - 1, depth, pageSize);
-    
+
     // If pagination is enabled, return only the requested page
     // Otherwise, return all entries (for backward compatibility)
-    const ret = usePagination 
-      ? take(entries.slice((page - 1) * pageSize), pageSize)
-      : entries;
+    const ret = usePagination ? take(entries.slice((page - 1) * pageSize), pageSize) : entries;
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
     ret[CURSOR_COMPATIBILITY_SYMBOL] = cursor;