Remove redundant type checks

TypeScript already checks the types, so no need to explicitly re-check
them with `assert.ok` or `assertString`.

src/lib/api/ApiClient.ts

@@ -74,9 +74,6 @@ export default class ApiClient {
         baseUrl = "https://api.appearin.net",
         fetchDeviceCredentials = noCredentials,
     }: ApiClientOptions = {}) {
-        assertString(baseUrl, "baseUrl");
-        assert.ok(typeof fetchDeviceCredentials === "function", "fetchDeviceCredentials<Function> is required");
-
         this.authenticatedHttpClient = new AuthenticatedHttpClient({
             httpClient: new HttpClient({
                 baseUrl,

src/lib/api/HttpClient.ts

@@ -10,7 +10,6 @@ export interface IHttpClient {
 }
 
 function _getAbsoluteUrl({ baseUrl, url }: { baseUrl?: string; url: string }): string {
-    assert.ok(typeof url === "string", "url<String> is required");
     return baseUrl ? baseUrl + url : url;
 }
 

src/lib/api/OrganizationApiClient.ts

@@ -18,8 +18,9 @@ export default class OrganizationApiClient {
     /**
      * Create an OrganizationApiClient instance.
      *
-     * @param {ApiClient} [apiClient] - The apiClient to use.
-     * @param {Function} [fetchOrganization] - function that returns a promise with the organization.
+     * @param {Object} options - The options for the OrganizationApiClient.
+     * @param {ApiClient} [options.apiClient] - The apiClient to use.
+     * @param {Function} [options.fetchOrganization] - function that returns a promise with the organization.
      */
     constructor({
         apiClient,
@@ -29,8 +30,6 @@ export default class OrganizationApiClient {
         fetchOrganization?: FetchOrganizationFunction;
     }) {
         this._apiClient = apiClient;
-        assert.ok(typeof fetchOrganization === "function", "fetchOrganization<Function> is required");
-
         this._fetchOrganization = fetchOrganization;
         this._apiClient = apiClient;
     }

src/lib/api/test/ApiClient.spec.ts

@@ -16,11 +16,6 @@ describe("ApiClient", () => {
         it("should not throw an error if no constructor params are passed through", () => {
             expect(() => new ApiClient()).not.toThrowError();
         });
-
-        //@ts-expect-error
-        itShouldThrowIfInvalid("baseUrl", () => new ApiClient({ baseUrl: null }));
-        //@ts-expect-error
-        itShouldThrowIfInvalid("fetchDeviceCredentials", () => new ApiClient({ fetchDeviceCredentials: null }));
     });
 
     describe("request", () => {
@@ -48,9 +43,6 @@ describe("ApiClient", () => {
             expect(() => apiClient.request("some-url", {})).toThrowError();
         });
 
-        //@ts-expect-error
-        itShouldThrowIfInvalid("fetchDeviceCredentials", () => new ApiClient({ fetchDeviceCredentials: null }));
-
         it("should run `this.fetchDeviceCredentials`", () => {
             const url = "/some/path";
             const fetchOptions = {};

src/lib/api/test/parameterAssertUtils.spec.ts

@@ -1,4 +1,3 @@
-import assert from "@whereby/jslib-media/src/utils/assert";
 import {
     assertBoolean,
     assertRoomName,
@@ -13,8 +12,6 @@ function forEachObject(object: Record<string, unknown>, func: (value: unknown, k
 }
 describe("parameterAssertUtils", () => {
     function itShouldThrowForValues(testValues: Record<string, unknown>, test: (desc: string, value: unknown) => void) {
-        assert.ok(typeof test === "function", "test must be a function");
-
         forEachObject(testValues, (value, descriptionOfValue) => {
             it(`should throw for ${descriptionOfValue}`, () => {
                 test(descriptionOfValue, value);

src/lib/utils/__tests__/debounce.spec.ts

@@ -16,13 +16,6 @@ describe("debounce", () => {
         jest.clearAllMocks();
     });
 
-    it("should throw if no function is provided", () => {
-        expect(() => {
-            // @ts-expect-error
-            debounce();
-        }).toThrow("fn<function> is required");
-    });
-
     it("should set the timer with the specified delay", () => {
         const delay = Math.floor(Math.random() * 2000);
 

src/lib/utils/debounce.ts

@@ -1,5 +1,3 @@
-import assert from "@whereby/jslib-media/src/utils/assert";
-
 interface Options {
     delay?: number;
     edges?: boolean;
@@ -21,8 +19,6 @@ interface DebouncedFunction {
  * @returns {Function} Debounced function.
  */
 export default function debounce(fn: DebouncedFunction, { delay = 500, edges }: Options = {}): DebouncedFunction {
-    assert.ok(typeof fn === "function", "fn<function> is required");
-
     let timeout: NodeJS.Timeout | undefined;
     let nCalls = 0;