chore: rename more files from .js to .ts (#2842)

* chore: rename more files from .js to .ts

* fix: fix some little issues caught by TS

* fix: fix type of 'certificateId'

Author: bradenmacdonald

…d-settings/__mocks__/advancedSettings.js …d-settings/__mocks__/advancedSettings.tssrc/advanced-settings/__mocks__/advancedSettings.js renamed to src/advanced-settings/__mocks__/advancedSettings.ts src/advanced-settings/__mocks__/advancedSettings.js renamed to src/advanced-settings/__mocks__/advancedSettings.ts

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   advancedModules: {
     deprecated: false,
     displayName: 'Advanced Module List',

src/advanced-settings/__mocks__/index.js src/advanced-settings/__mocks__/index.tssrc/advanced-settings/__mocks__/index.js renamed to src/advanced-settings/__mocks__/index.ts

@@ -1,4 +1,4 @@
-module.exports = [
+export default [
   {
     id: 1,
     courseTitle: 'Course Title 1',

src/advanced-settings/index.js src/advanced-settings/index.tssrc/advanced-settings/index.js renamed to src/advanced-settings/index.ts

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   certificateActivationHandlerUrl: '/certificates/activation/course-v1:org+101+101/',
   certificateWebViewUrl: '//certificates/course/course-v1:org+101+101?preview=honor',
   certificates: [

…c/certificates/__mocks__/certificates.js …c/certificates/__mocks__/certificates.tssrc/certificates/__mocks__/certificates.js renamed to src/certificates/__mocks__/certificates.ts src/certificates/__mocks__/certificates.js renamed to src/certificates/__mocks__/certificates.ts

@@ -1,4 +1,4 @@
-module.exports = [
+export default [
   {
     id: '1', name: 'John Doe', title: 'CEO', organization: 'Company', signatureImagePath: '/path/to/signature1.png',
   },

…rtificates/__mocks__/certificatesData.js …rtificates/__mocks__/certificatesData.tssrc/certificates/__mocks__/certificatesData.js renamed to src/certificates/__mocks__/certificatesData.ts src/certificates/__mocks__/certificatesData.js renamed to src/certificates/__mocks__/certificatesData.ts

@@ -5,17 +5,15 @@ import { prepareCertificatePayload } from '../utils';
 
 const getApiBaseUrl = () => getConfig().STUDIO_BASE_URL;
 
-export const getCertificatesApiUrl = (courseId) => `${getApiBaseUrl()}/api/contentstore/v1/certificates/${courseId}`;
-export const getCertificateApiUrl = (courseId) => `${getApiBaseUrl()}/certificates/${courseId}`;
-export const getUpdateCertificateApiUrl = (courseId, certificateId) => `${getCertificateApiUrl(courseId)}/${certificateId}`;
-export const getUpdateCertificateActiveStatusApiUrl = (path) => `${getApiBaseUrl()}${path}`;
+export const getCertificatesApiUrl = (courseId: string) => `${getApiBaseUrl()}/api/contentstore/v1/certificates/${courseId}`;
+export const getCertificateApiUrl = (courseId: string) => `${getApiBaseUrl()}/certificates/${courseId}`;
+export const getUpdateCertificateApiUrl = (courseId: string, certificateId: number) => `${getCertificateApiUrl(courseId)}/${certificateId}`;
+export const getUpdateCertificateActiveStatusApiUrl = (path: string) => `${getApiBaseUrl()}${path}`;
 
 /**
  * Gets certificates for a course.
- * @param {string} courseId
- * @returns {Promise<Object>}
  */
-export async function getCertificates(courseId) {
+export async function getCertificates(courseId: string): Promise<Record<string, any>> {
   const { data } = await getAuthenticatedHttpClient()
     .get(getCertificatesApiUrl(courseId));
 
@@ -24,12 +22,11 @@ export async function getCertificates(courseId) {
 
 /**
  * Create course certificate.
- * @param {string} courseId
- * @param {object} certificatesData
- * @returns {Promise<Object>}
  */
-
-export async function createCertificate(courseId, certificatesData) {
+export async function createCertificate(
+  courseId: string,
+  certificatesData: Record<string, any>,
+): Promise<Record<string, any>> {
   const { data } = await getAuthenticatedHttpClient()
     .post(
       getCertificateApiUrl(courseId),
@@ -41,11 +38,11 @@ export async function createCertificate(courseId, certificatesData) {
 
 /**
  * Update course certificate.
- * @param {string} courseId
- * @param {object} certificateData
- * @returns {Promise<Object>}
  */
-export async function updateCertificate(courseId, certificateData) {
+export async function updateCertificate(
+  courseId: string,
+  certificateData: Record<string, any>,
+): Promise<Record<string, any>> {
   const { data } = await getAuthenticatedHttpClient()
     .post(
       getUpdateCertificateApiUrl(courseId, certificateData.id),
@@ -57,11 +54,8 @@ export async function updateCertificate(courseId, certificateData) {
 
 /**
  * Delete course certificate.
- * @param {string} courseId
- * @param {object} certificateId
- * @returns {Promise<Object>}
  */
-export async function deleteCertificate(courseId, certificateId) {
+export async function deleteCertificate(courseId: string, certificateId: number): Promise<Record<string, any>> {
   const { data } = await getAuthenticatedHttpClient()
     .delete(
       getUpdateCertificateApiUrl(courseId, certificateId),
@@ -71,11 +65,8 @@ export async function deleteCertificate(courseId, certificateId) {
 
 /**
  * Activate/deactivate course certificate.
- * @param {string} courseId
- * @param {object} activationStatus
- * @returns {Promise<Object>}
  */
-export async function updateActiveStatus(path, activationStatus) {
+export async function updateActiveStatus(path: string, activationStatus: unknown): Promise<Record<string, any>> {
   const body = {
     is_active: activationStatus,
   };

src/certificates/__mocks__/index.js src/certificates/__mocks__/index.tssrc/certificates/__mocks__/index.js renamed to src/certificates/__mocks__/index.ts

@@ -4,7 +4,7 @@ export const MODE_STATES = {
   view: 'view',
   editAll: 'edit_all',
   create: 'create',
-};
+} as const;
 
 export const ACTIVATION_MESSAGES = {
   activating: 'Activating',