874: Update comments

Author: az2924

src/main/java/org/patinanetwork/codebloom/common/db/repos/question/topic/QuestionTopicRepository.java

@@ -20,6 +20,7 @@ public interface QuestionTopicRepository {
      * @param questionTopic - required fields:
      *     <ul>
      *       <li>questionId
+     *       <li>topicTag
      *       <li>topic
      *       <li>topicSlug
      *     </ul>
@@ -31,6 +32,7 @@ public interface QuestionTopicRepository {
      * @param questionTopic - overridden fields:
      *     <ul>
      *       <li>questionId
+     *       <li>topicTag
      *       <li>topic
      *       <li>topicSlug
      *     </ul>

src/main/java/org/patinanetwork/codebloom/common/db/repos/question/topic/QuestionTopicSqlRepository.java

@@ -132,7 +132,7 @@ public Optional<QuestionTopic> findQuestionTopicByQuestionIdAndTopicEnum(
 
             return Optional.empty();
         } catch (Exception e) {
-            throw new RuntimeException("Failed to get question topic by question ID and topic enum", e);
+            throw new RuntimeException("Failed to get question topic by ID", e);
         }
     }
 
@@ -210,7 +210,7 @@ public boolean deleteQuestionTopicById(final String id) {
             stmt.setObject("id", UUID.fromString(id));
             return stmt.executeUpdate() > 0;
         } catch (SQLException e) {
-            throw new RuntimeException("Failed to delete question topic by ID", e);
+            throw new RuntimeException("Failed to delete tag by tag ID", e);
         }
     }
 }

src/test/java/org/patinanetwork/codebloom/common/db/repos/question/topic/QuestionTopicRepositoryTest.java

@@ -59,7 +59,7 @@ void cleanUp() {
         boolean isSuccessful = questionTopicRepository.deleteQuestionTopicById(testQuestionTopic.getId())
                 && questionTopicRepository.deleteQuestionTopicById(testQuestionBankTopic.getId());
         if (!isSuccessful) {
-            fail("Failed to delete test question topics");
+            fail("Failed to delete test question");
         }
     }
 
@@ -74,7 +74,7 @@ void testFindQuestionTopicById() {
         if (!possibleQuestionTopic.get().equals(testQuestionTopic)) {
             log.info("possibleQuestionTopic: {}", possibleQuestionTopic.get());
             log.info("testQuestionTopic: {}", testQuestionTopic);
-            fail("testFindQuestionTopicById failed: possibleQuestionTopic does not equal testQuestionTopic");
+            fail("testFindQuestionTopicById failed: possibleQuestionTopic does not equal to testQuestionTopic");
         }
     }
 
@@ -91,7 +91,7 @@ void testFindQuestionTopicByQuestionIdAndTopicEnum() {
             log.info("possibleQuestionTopic: {}", possibleQuestionTopic.get());
             log.info("testQuestionTopic: {}", testQuestionTopic);
             fail(
-                    "testFindQuestionTopicByQuestionIdAndTopicEnum failed: possibleQuestionTopic does not equal testQuestionTopic");
+                    "testFindQuestionTopicByQuestionIdAndTopicEnum failed: possibleQuestionTopic does not equal to testQuestionTopic");
         }
     }
 
@@ -129,7 +129,7 @@ void testUpdateQuestion() {
         if (!newQuestionTopic.get().equals(testQuestionTopic)) {
             log.info("newQuestionTopic: {}", newQuestionTopic.get());
             log.info("testQuestionTopic: {}", testQuestionTopic);
-            fail("testUpdateQuestion failed: newQuestionTopic does not equal testQuestionTopic");
+            fail("testUpdateQuestion failed: newQuestionTopic does not equal to testQuestionTopic");
         }
     }