test: add entry controller tests

backend/src/controllers/entry/entry.ts

@@ -31,7 +31,7 @@ export const getAllEntries = async (
 ) => {
   const { journalId } = req.params;
 
-  const entries = await EntryServices.getAllEntriesInJournal(journalId);
+  const entries = await EntryServices.getAllEntries(journalId);
 
   if (entries.length === 0) {
     req.flash('info', 'Submit your first entry to get started.');
@@ -158,7 +158,7 @@ export const getEntryAnalysis = async (
     return next(new ExpressError('Entry analysis not found.', 404));
   }
 
-  res.status(200).json(entryAnalysis.toObject());
+  res.status(200).json(entryAnalysis);
 };
 
 /**
@@ -183,7 +183,7 @@ export const updateEntryAnalysis = async (
     res
       .status(200)
       .json({
-        ...entryAnalysis.toObject(),
+        ...entryAnalysis,
         entry: entry.toObject(),
         flash: req.flash(),
       });
@@ -204,7 +204,7 @@ export const getEntryConversation = async (req: Request, res: Response) => {
 
   const response = await EntryServices.getEntryConversation(entryId);
   const entryConversation = response
-    ? { ...response.toObject(), chatId: response.id }
+    ? { ...response, chatId: response.id }
     : {};
 
   res.status(200).json(entryConversation);
@@ -224,13 +224,12 @@ export const createEntryConversation = async (
   try {
     const configId = await verifyJournalExists(journalId);
 
-    const response = await EntryServices.createEntryConversation(
+    const entryConversation = await EntryServices.createEntryConversation(
       entryId,
       configId,
       messageData
     );
 
-    const entryConversation = response ? response.toObject() : {};
     req.flash('success', 'Successfully created conversation.');
     res.status(201).json({ ...entryConversation, flash: req.flash() });
   } catch (error) {
@@ -253,7 +252,11 @@ export const updateEntryConversation = async (
     const configId = await verifyJournalExists(journalId);
 
     const response = await EntryServices.updateEntryConversation(chatId, configId, messageData);
-    res.status(200).json({ ...response.toObject(), flash: req.flash() });
+
+    /* FIXME: No flash messages are attached to prevent over-displaying--this
+     * seems like something the frontend should handle 
+     */
+    res.status(200).json({ ...response, flash: req.flash() });
   } catch (error) {
     return next(error);
   }

backend/src/models/services/entry/entry.ts

@@ -24,7 +24,7 @@ interface EntryResponse {
  * @param journalId Journal._id as string
  * @returns array of documents in journal with journalId
  */
-export async function getAllEntriesInJournal(
+export async function getAllEntries(
   journalId: string
 ): Promise<HydratedDocument<EntryType>[]> {
   return await Entry.find({ journal: journalId });