@eduware/oneroster 1.2.4 → 1.2.6

package/test/classlink.test.ts

@@ -199,7 +199,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('orgs', async () => {
                 try {
                     for await (const page of await client.organizationsManagement.getAllOrgs({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.orgs || [])).toBe(true);
                         recordResult(authType, 'orgs', 'passed');
@@ -214,7 +214,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('schools', async () => {
                 try {
                     for await (const page of await client.schoolsManagement.getAllSchools({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.orgs || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -231,7 +231,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('courses', async () => {
                 try {
                     for await (const page of await client.coursesManagement.getAllCourses({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.courses || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -248,7 +248,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('classes', async () => {
                 try {
                     for await (const page of await client.classesManagement.getAllClasses({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.classes || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -265,7 +265,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('users', async () => {
                 try {
                     for await (const page of await client.usersManagement.getAllUsers({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.users || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -282,7 +282,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('students', async () => {
                 try {
                     for await (const page of await client.studentsManagement.getAllStudents({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.users || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -299,7 +299,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('teachers', async () => {
                 try {
                     for await (const page of await client.teachersManagement.getAllTeachers({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.users || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -316,7 +316,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('enrollments', async () => {
                 try {
                     for await (const page of await client.enrollmentsManagement.getAllEnrollments({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.enrollments || [])).toBe(true);
                         recordResult(authType, 'enrollments', 'passed');
@@ -331,7 +331,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('terms', async () => {
                 try {
                     for await (const page of await client.termsManagement.getAllTerms({
-                        limit: 2,
+                        limit: 5,
                     })) {
                         const items = page.result.academicSessions || [];
                         expect(Array.isArray(items)).toBe(true);
@@ -370,7 +370,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForSchool({
                         schoolSourcedId: schoolId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'schools/{id}/classes', 'passed');
@@ -390,7 +390,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.schoolsManagement.getStudentsForSchool({
                         schoolSourcedId: schoolId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.users || [])).toBe(true);
                         recordResult(authType, 'schools/{id}/students', 'passed');
@@ -415,7 +415,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.schoolsManagement.getTeachersForSchool({
                         schoolSourcedId: schoolId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.users || [])).toBe(true);
                         recordResult(authType, 'schools/{id}/teachers', 'passed');
@@ -440,7 +440,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.schoolsManagement.getCoursesForSchool({
                         schoolSourcedId: schoolId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.courses || [])).toBe(true);
                         recordResult(authType, 'schools/{id}/courses', 'passed');
@@ -460,7 +460,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.schoolsManagement.getTermsForSchool({
                         schoolSourcedId: schoolId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.academicSessions || [])).toBe(true);
                         recordResult(authType, 'schools/{id}/terms', 'passed');
@@ -480,7 +480,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getStudentsForClass({
                         classSourcedId: classId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.users || [])).toBe(true);
                         recordResult(authType, 'classes/{id}/students', 'passed');
@@ -505,7 +505,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getTeachersForClass({
                         classSourcedId: classId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.users || [])).toBe(true);
                         recordResult(authType, 'classes/{id}/teachers', 'passed');
@@ -530,7 +530,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForUser({
                         userSourcedId: userId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'users/{id}/classes', 'passed');
@@ -550,7 +550,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForTeacher({
                         teacherSourcedId: teacherId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'teachers/{id}/classes', 'passed');
@@ -575,7 +575,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForStudent({
                         studentSourcedId: studentId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'students/{id}/classes', 'passed');
@@ -600,7 +600,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForTerm({
                         termSourcedId: termId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'terms/{id}/classes', 'passed');
@@ -620,7 +620,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.coursesManagement.getClassesForCourse({
                         courseSourcedId: courseId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'courses/{id}/classes', 'passed');
@@ -640,7 +640,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.termsManagement.getGradingPeriodsForTerm({
                         termSourcedId: termId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.academicSessions || [])).toBe(true);
                         recordResult(authType, 'terms/{id}/gradingPeriods', 'passed');
@@ -669,7 +669,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.classesManagement.getClassesForTerm({
                         termSourcedId: termId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'GET /terms/{id}/classes', 'passed');
@@ -699,7 +699,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.coursesManagement.getClassesForCourse({
                         courseSourcedId: courseId,
-                        limit: 2,
+                        limit: 5,
                     })) {
                         expect(Array.isArray(page.result.classes || [])).toBe(true);
                         recordResult(authType, 'GET /courses/{id}/classes', 'passed');
@@ -729,7 +729,7 @@ function createAPITestSuite(config: AuthConfig) {
                 try {
                     for await (const page of await client.termsManagement.getGradingPeriodsForTerm({
                         termSourcedId: termId,
-                        limit: 2,
+                        limit: 20,
                     })) {
                         expect(Array.isArray(page.result.academicSessions || [])).toBe(true);
                         recordResult(authType, 'GET /terms/{id}/gradingPeriods', 'passed');
@@ -751,7 +751,7 @@ function createAPITestSuite(config: AuthConfig) {
             it('should include totalCount from X-Total-Count header', async () => {
                 try {
                     for await (const page of await client.usersManagement.getAllUsers({
-                        limit: 1,
+                        limit: 100,
                     })) {
                         // totalCount is on the page object itself (from PaginationMeta), not on page.result
                         // It may be undefined if the server doesn't return the X-Total-Count header

package/vitest.config.ts

@@ -19,8 +19,8 @@ export default defineConfig({
             DOTENV_CONFIG_PATH: './test/.env',
         },
 
-        // Reporter options - use default + custom summary reporter
-        reporters: ['default', './test/summary-reporter.ts'],
+        // Reporter options
+        reporters: ['default'],
 
         // Globals (describe, it, expect, etc.)
         globals: true,

package/test/summary-reporter.ts

@@ -1,114 +0,0 @@
-/**
- * Custom Vitest Reporter that prints a summary of passed/failed endpoints
- */
-
-import type { Reporter, File, Task } from 'vitest';
-
-interface TestResult {
-    endpoint: string;
-    status: 'passed' | 'failed';
-    error?: string;
-}
-
-export default class SummaryReporter implements Reporter {
-    private results: Map<string, TestResult[]> = new Map();
-
-    onFinished(files?: File[]) {
-        if (!files) return;
-
-        // Process all test results
-        for (const file of files) {
-            this.processFile(file);
-        }
-
-        // Print summary
-        this.printSummary();
-    }
-
-    private processFile(file: File) {
-        for (const task of file.tasks) {
-            this.processTask(task, null);
-        }
-    }
-
-    private processTask(task: Task, authType: string | null) {
-        // Check if this is an auth type suite (e.g., "OAUTH1 API Operations")
-        const authMatch = task.name.match(/^(OAUTH1|OAUTH2|BEARER) API Operations$/);
-        if (authMatch) {
-            authType = authMatch[1]!;
-        }
-
-        if (task.type === 'suite' && 'tasks' in task && task.tasks) {
-            for (const child of task.tasks) {
-                this.processTask(child, authType);
-            }
-        } else if (task.type === 'test' && authType) {
-            const passed = task.result?.state === 'pass';
-            const errorMsg = this.getShortError(task.result?.errors?.[0]?.message);
-            this.recordResult(authType, task.name, passed ? 'passed' : 'failed', errorMsg);
-        }
-    }
-
-    private getShortError(error?: string): string | undefined {
-        if (!error) return undefined;
-        if (error.includes('404')) return '404 Not Found';
-        if (error.includes('401')) return '401 Unauthorized';
-        if (error.includes('403')) return '403 Forbidden';
-        if (error.includes('ResponseValidationError') || error.includes('Zod'))
-            return 'Response validation';
-        if (error.length > 40) return error.substring(0, 37) + '...';
-        return error;
-    }
-
-    private recordResult(
-        authType: string,
-        endpoint: string,
-        status: 'passed' | 'failed',
-        error?: string
-    ) {
-        if (!this.results.has(authType)) {
-            this.results.set(authType, []);
-        }
-
-        // Avoid duplicates
-        const existing = this.results.get(authType)!;
-        if (!existing.some((r) => r.endpoint === endpoint)) {
-            existing.push({ endpoint, status, error });
-        }
-    }
-
-    private printSummary() {
-        console.log('\n');
-        console.log('╔══════════════════════════════════════════════════════════════════╗');
-        console.log('║                    TEST RESULTS SUMMARY                          ║');
-        console.log('╚══════════════════════════════════════════════════════════════════╝');
-
-        for (const [authType, results] of this.results) {
-            const passed = results.filter((r) => r.status === 'passed');
-            const failed = results.filter((r) => r.status === 'failed');
-
-            console.log(`\n┌──────────────────────────────────────────────────────────────────┐`);
-            console.log(
-                `│ ${authType.padEnd(10)} - ${String(passed.length).padStart(2)} passed, ${String(failed.length).padStart(2)} failed`
-            );
-            console.log(`└──────────────────────────────────────────────────────────────────┘`);
-
-            if (passed.length > 0) {
-                console.log('\n  ✅ PASSED:');
-                for (const r of passed) {
-                    console.log(`     • ${r.endpoint}`);
-                }
-            }
-
-            if (failed.length > 0) {
-                console.log('\n  ❌ FAILED:');
-                for (const r of failed) {
-                    const reason = r.error ? ` (${r.error})` : '';
-                    console.log(`     • ${r.endpoint}${reason}`);
-                }
-            }
-        }
-
-        console.log('\n' + '═'.repeat(70) + '\n');
-    }
-}