@insignia-education/api-sdk-js 0.9.34 → 0.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@insignia-education/api-sdk-js",
-  "version": "0.9.34",
+  "version": "0.12.0",
   "description": "JavaScript SDK for the Insignia Education API",
   "main": "index.js",
   "type": "module",
@@ -26,8 +26,7 @@
     "url": "https://github.com/insignia-education/api-sdk-js/issues"
   },
   "homepage": "https://github.com/insignia-education/api-sdk-js#readme",
-  "dependencies": {
-  },
+  "dependencies": {},
   "devDependencies": {
     "@babel/preset-env": "^7.26.0",
     "babel-jest": "^30.3.0",

package/src/Client.js

@@ -79,7 +79,7 @@ export default class InsigniaClient {
         if (!response.ok) {
             const err = new Error(`HTTP ${response.status}`);
             err.status = response.status;
-            try { err.data = await this.#parseResponse(response); } catch (_) {}
+            try { err.data = await this.#parseResponse(response); } catch { /* body wasn't parseable — leave err.data unset */ }
             throw err;
         }
         const data = await this.#parseResponse(response);
@@ -97,14 +97,22 @@ export default class InsigniaClient {
         if (!response.ok) {
             const err = new Error(`HTTP ${response.status}`);
             err.status = response.status;
-            try { err.data = await this.#parseResponse(response); } catch (_) {}
+            try { err.data = await this.#parseResponse(response); } catch { /* body wasn't parseable — leave err.data unset */ }
             throw err;
         }
         const data = await this.#parseResponse(response);
         return data?.success ? data.response : data;
     }
 
-    get(path)               { return this.#request('GET',    path); }
+    get(path, params = null) {
+        if (params && typeof params === 'object') {
+            const qs = new URLSearchParams(
+                Object.entries(params).filter(([, v]) => v !== undefined && v !== null)
+            ).toString();
+            if (qs) path += (path.includes('?') ? '&' : '?') + qs;
+        }
+        return this.#request('GET', path);
+    }
     post(path, body = null) { return this.#request('POST',   path, body); }
     put(path, body = null)  { return this.#request('PUT',    path, body); }
     patch(path, body = null){ return this.#request('PATCH',  path, body); }

package/src/api/v1/Courses.js

@@ -24,6 +24,9 @@ export default class Courses {
     edit(id, data)  { return this.#client.patch(`/courses/${id}`, data); }
     delete(id)      { return this.#client.del(`/courses/${id}`); }
 
+    /** Users enrolled on this course as a teacher (user_courses.teacher = 1). */
+    teachers(courseId) { return this.#client.get(`/courses/${courseId}/teachers`); }
+
     dates(courseId) {
         const base = `/courses/${courseId}/dates`;
         return {

package/src/api/v1/Files.js

@@ -14,8 +14,8 @@ export default class Files {
     /** Upload a file (FormData). Supports: file, folder, organization_id, filename, generate_webp, … */
     upload(formData)            { return this.#client.upload('/files', formData); }
 
-    /** Create a logical folder. body: { organization_id, path } */
-    createFolder(body)          { return this.#client.upload('/files/folder', body); }
+    /** Create a logical folder (JSON PUT, no file involved). body: { path } */
+    createFolder(body)          { return this.#client.put('/files/folder', body); }
 
     /** Update file description. body: { description } */
     update(id, body)            { return this.#client.patch(`/files/${id}`, body); }

package/src/api/v1/Users.js

@@ -9,6 +9,10 @@ export default class Users {
     getByUsername(username)   { return this.#client.get(`/users/username/${username}`); }
     findByUsername(username)  { return this.#client.get(`/users/find/${username}`); }
     cashReceivers()           { return this.#client.get('/users/cash-receivers'); }
+    /** Employee-only: users assignable as a course owner (super-admins, education centers, organizations). */
+    assignable()              { return this.#client.get('/users/assignable'); }
+    /** Employee-only: users assignable as a course teacher (super-admins, admins, employees, teachers). */
+    assignableTeachers()      { return this.#client.get('/users/assignable-teachers'); }
     edit(id, data)  { return this.#client.patch(`/users/${id}`, data); }
 
     #nested(userId, path) {

package/tests/client.test.js

@@ -3,7 +3,7 @@ import InsigniaClient from '../src/Client.js';
 const BASE = 'http://localhost:8000';
 
 function mockFetch(json = {}) {
-    return jest.fn().mockResolvedValue({ json: () => Promise.resolve(json) });
+    return jest.fn().mockResolvedValue({ ok: true, status: 200, json: () => Promise.resolve(json) });
 }
 
 beforeEach(() => {
@@ -100,7 +100,7 @@ describe('HTTP methods', () => {
     let client;
 
     beforeEach(() => {
-        global.fetch = mockFetch({ ok: true });
+        global.fetch = mockFetch();
         client = new InsigniaClient(BASE);
     });
 
@@ -181,12 +181,16 @@ describe('HTTP methods', () => {
     test('stores response cookies and sends them on later requests in Node', async () => {
         global.fetch = jest.fn()
             .mockResolvedValueOnce({
+                ok: true,
+                status: 200,
                 headers: {
                     getSetCookie: () => ['token=abc123; Path=/; HttpOnly; Secure; SameSite=Lax'],
                 },
                 json: () => Promise.resolve({ success: 'ok' }),
             })
             .mockResolvedValueOnce({
+                ok: true,
+                status: 200,
                 json: () => Promise.resolve({ id: 1, title: 'test' }),
             });
         client = new InsigniaClient(BASE);
@@ -200,6 +204,8 @@ describe('HTTP methods', () => {
 
     test('returns parsed JSON from response', async () => {
         global.fetch = jest.fn().mockResolvedValue({
+            ok: true,
+            status: 200,
             json: () => Promise.resolve({ id: 1, name: 'test' }),
         });
         client = new InsigniaClient(BASE);
@@ -209,6 +215,7 @@ describe('HTTP methods', () => {
 
     test('returns null for empty response text', async () => {
         global.fetch = jest.fn().mockResolvedValue({
+            ok: true,
             status: 200,
             text: () => Promise.resolve(''),
         });
@@ -219,6 +226,7 @@ describe('HTTP methods', () => {
 
     test('returns null for no-content responses', async () => {
         global.fetch = jest.fn().mockResolvedValue({
+            ok: true,
             status: 204,
             json: jest.fn(),
         });
@@ -229,6 +237,8 @@ describe('HTTP methods', () => {
 
     test('returns null when json parsing fails because the response is empty', async () => {
         global.fetch = jest.fn().mockResolvedValue({
+            ok: true,
+            status: 200,
             json: () => Promise.reject(new SyntaxError('Unexpected end of JSON input')),
         });
         client = new InsigniaClient(BASE);