fix: 🛠️ adds missing validation on Agreement form

frontend/src/components/Agreements/AgreementEditor/AgreementEditForm.jsx

@@ -328,8 +328,6 @@ export const AgreementEditForm = ({
                     handleConfirm={modalProps.handleConfirm}
                 />
             )}
-            <h2 className="font-sans-lg margin-top-3 margin-bottom-0">Agreement Type</h2>
-            <p className="margin-top-1">Select the agreement type to get started.</p>
             <AgreementTypeSelect
                 messages={res.getErrors("agreement_type")}
                 className={cn("agreement_type")}

frontend/src/components/Agreements/AgreementEditor/AgreementEditFormSuite.js

@@ -37,6 +37,7 @@ const suite = create((data = {}, fieldName) => {
     });
     test("contract-type", "This is required information", () => {
         enforce(data.contract_type).notEquals("-Select an option-");
+        enforce(data.contract_type).isNotEmpty();
     });
     test("team-members", "This is required information", () => {
         enforce(data.team_members).lengthNotEquals(0);

frontend/src/components/BudgetLineItems/CreateBLIsAndSCs/CreateBLIsAndSCs.jsx

@@ -1,6 +1,5 @@
 import PropTypes from "prop-types";
 import React from "react";
-import { convertCodeForDisplay } from "../../../helpers/utils";
 import EditModeTitle from "../../../pages/agreements/EditModeTitle";
 import AgreementBudgetLinesHeader from "../../Agreements/AgreementBudgetLinesHeader";
 import AgreementTotalCard from "../../Agreements/AgreementDetailsCards/AgreementTotalCard";
@@ -61,7 +60,6 @@ export const CreateBLIsAndSCs = ({
     setIncludeDrafts
 }) => {
     const {
-        budgetLinePageErrorsExist,
         handleDeleteBudgetLine,
         handleDuplicateBudgetLine,
         handleEditBLI,
@@ -210,7 +208,7 @@ export const CreateBLIsAndSCs = ({
                 />
             )}
 
-            {budgetLinePageErrorsExist && (
+            {pageErrors && (
                 <ul
                     className="usa-list--unstyled font-12px text-error"
                     data-cy="error-list"
@@ -221,7 +219,6 @@ export const CreateBLIsAndSCs = ({
                             className="border-left-2px padding-left-1"
                             data-cy="error-item"
                         >
-                            <strong>{convertCodeForDisplay("validation", key)}: </strong>
                             {
                                 <span>
                                     {value.map((message, index) => (

frontend/src/components/BudgetLineItems/CreateBLIsAndSCs/suite.js

@@ -4,8 +4,8 @@ const suite = create((data) => {
     only(data);
 
     // test to ensure at least one budget line item exists
-    test("data", "Must have at least one budget line item", () => {
-        enforce(data.budgetLines).longerThan(0);
+    test("data", "Must have at least one budget line", () => {
+        enforce(data.budgetLines.length).greaterThan(0);
     });
     // test budget_line_items array
     each(data.budgetLines, (item) => {

frontend/src/pages/agreements/approve/ApproveAgreement.jsx

@@ -58,6 +58,12 @@ const ApproveAgreement = () => {
     if (!agreement) {
         return <div>No agreement data available.</div>;
     }
+
+    /*
+      TODO: Add FE validation to ensure that the user is authorized to approve the agreement
+      and if not, display an error page
+      to prevent unauthorized users from accessing this page
+    */
     const BeforeApprovalContent = React.memo(
         ({ groupedBudgetLinesByServicesComponent, servicesComponents, changeRequestTitle, urlChangeToStatus }) => (
             <>

frontend/src/pages/agreements/review/suite.js

@@ -32,7 +32,8 @@ const suite = create((fieldName) => {
         enforce(fieldName.project_officer_id).isNotBlank();
     });
     test("contract-type", "This is required information", () => {
-        enforce(fieldName.contract_type).isNotBlank();
+        enforce(fieldName.contract_type).notEquals("-Select an option-");
+        enforce(fieldName.contract_type).isNotEmpty();
     });
     test("team-members", "This is required information", () => {
         enforce(fieldName.team_members).longerThan(0);