From 1a44a1358c4910067a06f8a55c10dd732b09b0f5 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 08:46:34 -0600
Subject: [PATCH 01/10] previously could submit a description longer than 2000
 chars, but would get internal server error

---
 src/components/AddCourse.vue | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/components/AddCourse.vue b/src/components/AddCourse.vue
index a8ebba7..1787eeb 100644
--- a/src/components/AddCourse.vue
+++ b/src/components/AddCourse.vue
@@ -26,7 +26,16 @@
             hint="* Required"
             persistent-hint
             required
-            :rules="[inputRules.required]"
+            :rules="[
+              inputRules.characterCountWithinRange(
+                toAdd.name,
+                {
+                  min: 2,
+                  max: 256,
+                }
+              ),
+              inputRules.required
+            ]"
           ></v-text-field>
 
           <v-textarea
@@ -34,6 +43,15 @@
             counter="2000"
             rows="1"
             v-model="toAdd.description"
+            :rules="[
+              inputRules.characterCountWithinRange(
+                toAdd.description,
+                {
+                  min: 0,
+                  max: 2001,
+                }
+              ),
+            ]"
           ></v-textarea>
 
           <v-text-field
-- 
GitLab


From 9be5f39f371a170da6b37233ec3425b856bad0dc Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 08:56:54 -0600
Subject: [PATCH 02/10] adding standardization

---
 src/components/AddCourse.vue  | 24 +++++++++---------------
 src/views/TrainingDetails.vue | 27 +++++++++++++--------------
 2 files changed, 22 insertions(+), 29 deletions(-)

diff --git a/src/components/AddCourse.vue b/src/components/AddCourse.vue
index 1787eeb..d21fb82 100644
--- a/src/components/AddCourse.vue
+++ b/src/components/AddCourse.vue
@@ -27,14 +27,11 @@
             persistent-hint
             required
             :rules="[
-              inputRules.characterCountWithinRange(
-                toAdd.name,
-                {
-                  min: 2,
-                  max: 256,
-                }
-              ),
-              inputRules.required
+              inputRules.characterCountWithinRange(toAdd.name, {
+                min: 2,
+                max: 256,
+              }),
+              inputRules.required,
             ]"
           ></v-text-field>
 
@@ -44,13 +41,10 @@
             rows="1"
             v-model="toAdd.description"
             :rules="[
-              inputRules.characterCountWithinRange(
-                toAdd.description,
-                {
-                  min: 0,
-                  max: 2001,
-                }
-              ),
+              inputRules.characterCountWithinRange(toAdd.description, {
+                min: 0,
+                max: 2001,
+              }),
             ]"
           ></v-textarea>
 
diff --git a/src/views/TrainingDetails.vue b/src/views/TrainingDetails.vue
index 3e5dd4f..6a209eb 100644
--- a/src/views/TrainingDetails.vue
+++ b/src/views/TrainingDetails.vue
@@ -59,28 +59,27 @@
                       v-model="trainingCourseSnapshot.name"
                       label="Edit Course Name"
                       :rules="[
-                        inputRules.characterCountWithinRange(
-                          trainingCourseSnapshot.name,
-                          {
-                            min: 2,
-                            max: 256,
-                          }
-                        ),
+                        inputRules.characterCountWithinRange(trainingCourseSnapshot.name, {
+                          min: 2,
+                          max: 256,
+                        }),
+                        inputRules.required,
                       ]"
                     >
                     </v-text-field>
                     <!-- EDIT COURSE DESCRIPTION -->
-                    <v-text-field
+                    <v-textarea
                       v-model="trainingCourseSnapshot.description"
                       label="Edit Course Description"
+                      counter="2000"
+                      rows="1"
                       :rules="[
-                        inputRules.characterCountWithinRange(
-                          trainingCourseSnapshot.description,
-                          { min: 2, max: 2000 }
-                        ),
+                        inputRules.characterCountWithinRange(trainingCourseSnapshot.description, {
+                          min: 0,
+                          max: 2001,
+                        }),
                       ]"
-                    >
-                    </v-text-field>
+                    ></v-textarea>
 
                     <!-- EDIT COURSE START DATE -->
                     <v-menu
-- 
GitLab


From 0364ef7f7d85574d67a90da10c23dea99e54fd77 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 09:08:24 -0600
Subject: [PATCH 03/10] standardizing teams

---
 src/components/AddTeam.vue    | 16 ++++++++++++++--
 src/views/TrainingDetails.vue | 31 +++++++++++++++++++------------
 2 files changed, 33 insertions(+), 14 deletions(-)

diff --git a/src/components/AddTeam.vue b/src/components/AddTeam.vue
index 0f8903e..7fbcb1c 100644
--- a/src/components/AddTeam.vue
+++ b/src/components/AddTeam.vue
@@ -10,7 +10,13 @@
         hint="* Required"
         persistent-hint
         required
-        :rules="[inputRules.required]"
+        :rules="[
+          inputRules.characterCountWithinRange(toAdd.name, {
+            min: 2,
+            max: 256,
+          }),
+          inputRules.required,
+        ]"
       ></v-text-field>
 
       <v-textarea
@@ -21,7 +27,13 @@
         hint="* Required"
         persistent-hint
         required
-        :rules="[inputRules.required]"
+        :rules="[
+          inputRules.characterCountWithinRange(toAdd.description, {
+            min: 2,
+            max: 2001,
+          }),
+          inputRules.required,
+        ]"
       ></v-textarea>
 
       <v-text-field
diff --git a/src/views/TrainingDetails.vue b/src/views/TrainingDetails.vue
index 6a209eb..311b451 100644
--- a/src/views/TrainingDetails.vue
+++ b/src/views/TrainingDetails.vue
@@ -59,10 +59,13 @@
                       v-model="trainingCourseSnapshot.name"
                       label="Edit Course Name"
                       :rules="[
-                        inputRules.characterCountWithinRange(trainingCourseSnapshot.name, {
-                          min: 2,
-                          max: 256,
-                        }),
+                        inputRules.characterCountWithinRange(
+                          trainingCourseSnapshot.name,
+                          {
+                            min: 2,
+                            max: 256,
+                          }
+                        ),
                         inputRules.required,
                       ]"
                     >
@@ -74,10 +77,13 @@
                       counter="2000"
                       rows="1"
                       :rules="[
-                        inputRules.characterCountWithinRange(trainingCourseSnapshot.description, {
-                          min: 0,
-                          max: 2001,
-                        }),
+                        inputRules.characterCountWithinRange(
+                          trainingCourseSnapshot.description,
+                          {
+                            min: 0,
+                            max: 2001,
+                          }
+                        ),
                       ]"
                     ></v-textarea>
 
@@ -449,10 +455,11 @@ export default {
     },
     async fetchStudentData() {
       try {
-        const studentResponse = await TrainingService.getCourseRegistrationsById(
-          this.$route.params.trainingId,
-          { ...this.params, attendance: true }
-        );
+        const studentResponse =
+          await TrainingService.getCourseRegistrationsById(
+            this.$route.params.trainingId,
+            { ...this.params, attendance: true }
+          );
         this.listItems =
           studentResponse.registrations?.map((item) => {
             item.completed = { value: item.completed, loading: false };
-- 
GitLab


From 0faa3b4a33ba8d0a5c1c275256e8f435d3cf4313 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 09:14:20 -0600
Subject: [PATCH 04/10] linting

---
 src/components/UserBanner.vue | 35 +++++------------------------------
 src/views/TrainingDetails.vue |  9 ++++-----
 2 files changed, 9 insertions(+), 35 deletions(-)

diff --git a/src/components/UserBanner.vue b/src/components/UserBanner.vue
index def6983..8f2e7da 100644
--- a/src/components/UserBanner.vue
+++ b/src/components/UserBanner.vue
@@ -9,12 +9,7 @@
       >
         <div class="d-flex banner-row subhead">
           <div
-            class="
-              d-flex
-              align-items-center
-              flex-wrap flex-sm-nowrap
-              name-container
-            "
+            class="d-flex align-items-center flex-wrap flex-sm-nowrap name-container"
           >
             <v-avatar class="mr-2 mr-md-4">
               <img v-if="avatar" :src="avatar" :alt="name" />
@@ -24,23 +19,13 @@
           </div>
           <div
             v-if="$vuetify.breakpoint.mdAndUp"
-            class="
-              d-flex
-              align-items-center
-              justify-content-center
-              company-container
-            "
+            class="d-flex align-items-center justify-content-center company-container"
           >
             {{ company }}
           </div>
           <div
             v-if="$vuetify.breakpoint.mdAndUp"
-            class="
-              d-flex
-              align-items-center
-              justify-content-center
-              position-container
-            "
+            class="d-flex align-items-center justify-content-center position-container"
           >
             {{ position }}
           </div>
@@ -55,22 +40,12 @@
     >
       <div class="d-flex banner-row subhead">
         <div
-          class="
-            d-flex
-            align-items-center
-            justify-content-center
-            company-container
-          "
+          class="d-flex align-items-center justify-content-center company-container"
         >
           {{ company }}
         </div>
         <div
-          class="
-            d-flex
-            align-items-center
-            justify-content-center
-            position-container
-          "
+          class="d-flex align-items-center justify-content-center position-container"
         >
           {{ position }}
         </div>
diff --git a/src/views/TrainingDetails.vue b/src/views/TrainingDetails.vue
index 311b451..3d3e6e3 100644
--- a/src/views/TrainingDetails.vue
+++ b/src/views/TrainingDetails.vue
@@ -455,11 +455,10 @@ export default {
     },
     async fetchStudentData() {
       try {
-        const studentResponse =
-          await TrainingService.getCourseRegistrationsById(
-            this.$route.params.trainingId,
-            { ...this.params, attendance: true }
-          );
+        const studentResponse = await TrainingService.getCourseRegistrationsById(
+          this.$route.params.trainingId,
+          { ...this.params, attendance: true }
+        );
         this.listItems =
           studentResponse.registrations?.map((item) => {
             item.completed = { value: item.completed, loading: false };
-- 
GitLab


From ef5d043d10b6ba285b8e4b97739fc1bfefeb4a3e Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 09:17:56 -0600
Subject: [PATCH 05/10] adding resolution for trim newlines

---
 package-lock.json             | 12 ++++++------
 package.json                  |  1 +
 src/components/UserBanner.vue | 35 ++++++++++++++++++++++++++++++-----
 src/views/TrainingDetails.vue |  9 +++++----
 4 files changed, 42 insertions(+), 15 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 4961bd4..d3c9a74 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -13705,6 +13705,12 @@
             "lru-cache": "^6.0.0"
           }
         },
+        "trim-newlines": {
+          "version": "3.0.1",
+          "resolved": "https://registry.npmjs.org/trim-newlines/-/trim-newlines-3.0.1.tgz",
+          "integrity": "sha512-c1PTsA3tYrIsLGkJkzHF+w9F2EyxfXGo4UyJc4pFL++FMjnq0HJS69T3M7d//gKrFKwy429bouPescbjecU+Zw==",
+          "dev": true
+        },
         "type-fest": {
           "version": "0.18.1",
           "resolved": "https://registry.npmjs.org/type-fest/-/type-fest-0.18.1.tgz",
@@ -19936,12 +19942,6 @@
         "punycode": "^2.1.0"
       }
     },
-    "trim-newlines": {
-      "version": "3.0.0",
-      "resolved": "https://registry.npmjs.org/trim-newlines/-/trim-newlines-3.0.0.tgz",
-      "integrity": "sha512-C4+gOpvmxaSMKuEf9Qc134F1ZuOHVXKRbtEflf4NTtuuJDEIJ9p5PXsalL8SkeRw+qit1Mo+yuvMPAKwWg/1hA==",
-      "dev": true
-    },
     "trough": {
       "version": "1.0.5",
       "resolved": "https://registry.npmjs.org/trough/-/trough-1.0.5.tgz",
diff --git a/package.json b/package.json
index f1216d7..7882b03 100644
--- a/package.json
+++ b/package.json
@@ -161,6 +161,7 @@
     "serialize-javascript": "^3.1.0",
     "sockjs": "^0.3.21",
     "ssri": "^8.0.1",
+    "trim-newlines": "^3.0.0",
     "y18n": "^5.0.5",
     "yargs-parser": "^18.1.3",
     "ws": "^7.4.6",
diff --git a/src/components/UserBanner.vue b/src/components/UserBanner.vue
index 8f2e7da..def6983 100644
--- a/src/components/UserBanner.vue
+++ b/src/components/UserBanner.vue
@@ -9,7 +9,12 @@
       >
         <div class="d-flex banner-row subhead">
           <div
-            class="d-flex align-items-center flex-wrap flex-sm-nowrap name-container"
+            class="
+              d-flex
+              align-items-center
+              flex-wrap flex-sm-nowrap
+              name-container
+            "
           >
             <v-avatar class="mr-2 mr-md-4">
               <img v-if="avatar" :src="avatar" :alt="name" />
@@ -19,13 +24,23 @@
           </div>
           <div
             v-if="$vuetify.breakpoint.mdAndUp"
-            class="d-flex align-items-center justify-content-center company-container"
+            class="
+              d-flex
+              align-items-center
+              justify-content-center
+              company-container
+            "
           >
             {{ company }}
           </div>
           <div
             v-if="$vuetify.breakpoint.mdAndUp"
-            class="d-flex align-items-center justify-content-center position-container"
+            class="
+              d-flex
+              align-items-center
+              justify-content-center
+              position-container
+            "
           >
             {{ position }}
           </div>
@@ -40,12 +55,22 @@
     >
       <div class="d-flex banner-row subhead">
         <div
-          class="d-flex align-items-center justify-content-center company-container"
+          class="
+            d-flex
+            align-items-center
+            justify-content-center
+            company-container
+          "
         >
           {{ company }}
         </div>
         <div
-          class="d-flex align-items-center justify-content-center position-container"
+          class="
+            d-flex
+            align-items-center
+            justify-content-center
+            position-container
+          "
         >
           {{ position }}
         </div>
diff --git a/src/views/TrainingDetails.vue b/src/views/TrainingDetails.vue
index 3d3e6e3..311b451 100644
--- a/src/views/TrainingDetails.vue
+++ b/src/views/TrainingDetails.vue
@@ -455,10 +455,11 @@ export default {
     },
     async fetchStudentData() {
       try {
-        const studentResponse = await TrainingService.getCourseRegistrationsById(
-          this.$route.params.trainingId,
-          { ...this.params, attendance: true }
-        );
+        const studentResponse =
+          await TrainingService.getCourseRegistrationsById(
+            this.$route.params.trainingId,
+            { ...this.params, attendance: true }
+          );
         this.listItems =
           studentResponse.registrations?.map((item) => {
             item.completed = { value: item.completed, loading: false };
-- 
GitLab


From 577de2f9a6b3d5d480476896f27c7123c752eccb Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 09:31:10 -0600
Subject: [PATCH 06/10] resolutions css-what

---
 package-lock.json | 14 ++++++++------
 package.json      |  1 +
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index d3c9a74..71cb782 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -7376,6 +7376,14 @@
         "css-what": "^3.2.1",
         "domutils": "^1.7.0",
         "nth-check": "^1.0.2"
+      },
+      "dependencies": {
+        "css-what": {
+          "version": "3.4.2",
+          "resolved": "https://registry.npmjs.org/css-what/-/css-what-3.4.2.tgz",
+          "integrity": "sha512-ACUm3L0/jiZTqfzRM3Hi9Q8eZqd6IK37mMWPLz9PJxkLWllYeRf+EHUSHYEtFop2Eqytaq1FizFVh7XfBnXCDQ==",
+          "dev": true
+        }
       }
     },
     "css-select-base-adapter": {
@@ -7394,12 +7402,6 @@
         "source-map": "^0.6.1"
       }
     },
-    "css-what": {
-      "version": "3.4.2",
-      "resolved": "https://registry.npmjs.org/css-what/-/css-what-3.4.2.tgz",
-      "integrity": "sha512-ACUm3L0/jiZTqfzRM3Hi9Q8eZqd6IK37mMWPLz9PJxkLWllYeRf+EHUSHYEtFop2Eqytaq1FizFVh7XfBnXCDQ==",
-      "dev": true
-    },
     "cssesc": {
       "version": "3.0.0",
       "resolved": "https://registry.npmjs.org/cssesc/-/cssesc-3.0.0.tgz",
diff --git a/package.json b/package.json
index 7882b03..b3e0a13 100644
--- a/package.json
+++ b/package.json
@@ -149,6 +149,7 @@
   },
   "resolutions": {
     "ajv": "^6.12.6",
+    "css-what": "^3.4.2",
     "browserslist": "^4.16.5",
     "faye-websocket": "^0.11.3",
     "http-proxy": "^1.18.1",
-- 
GitLab


From de0ed86ec0fd36fe624b8c657d5bb44d8408bb30 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 09:57:50 -0600
Subject: [PATCH 07/10] resolution for css-what

---
 package-lock.json | 6 +++---
 package.json      | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 71cb782..643e93a 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -7379,9 +7379,9 @@
       },
       "dependencies": {
         "css-what": {
-          "version": "3.4.2",
-          "resolved": "https://registry.npmjs.org/css-what/-/css-what-3.4.2.tgz",
-          "integrity": "sha512-ACUm3L0/jiZTqfzRM3Hi9Q8eZqd6IK37mMWPLz9PJxkLWllYeRf+EHUSHYEtFop2Eqytaq1FizFVh7XfBnXCDQ==",
+          "version": "5.0.1",
+          "resolved": "https://registry.npmjs.org/css-what/-/css-what-5.0.1.tgz",
+          "integrity": "sha512-FYDTSHb/7KXsWICVsxdmiExPjCfRC4qRFBdVwv7Ax9hMnvMmEjP9RfxTEZ3qPZGmADDn2vAKSo9UcN1jKVYscg==",
           "dev": true
         }
       }
diff --git a/package.json b/package.json
index b3e0a13..24d49e2 100644
--- a/package.json
+++ b/package.json
@@ -149,7 +149,7 @@
   },
   "resolutions": {
     "ajv": "^6.12.6",
-    "css-what": "^3.4.2",
+    "css-what": "^5.0.1",
     "browserslist": "^4.16.5",
     "faye-websocket": "^0.11.3",
     "http-proxy": "^1.18.1",
-- 
GitLab


From 474cf08c02186b56a67dc743f91c4b86ef3e5349 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 11:26:13 -0600
Subject: [PATCH 08/10] courses input validation e2e tests

---
 tests/e2e/specs/superAdmin/courses.spec.js | 70 +++++++++++++++++++++-
 1 file changed, 69 insertions(+), 1 deletion(-)

diff --git a/tests/e2e/specs/superAdmin/courses.spec.js b/tests/e2e/specs/superAdmin/courses.spec.js
index e434c4d..e68c220 100644
--- a/tests/e2e/specs/superAdmin/courses.spec.js
+++ b/tests/e2e/specs/superAdmin/courses.spec.js
@@ -1,11 +1,24 @@
 describe("super admin user", () => {
   before(() => {
     cy.userFixture("users/superAdmin");
+    cy.visit("/training");
+  });
+  beforeEach(() => {
     cy.intercept("GET", "/api/courses?*", {
       fixture: "api/myCourses",
     });
-    cy.visit("/training");
   });
+
+  function randomStringLength(length) {
+    var text = "";
+    var possible = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
+
+    for (var i = 0; i < length; i++)
+      text += possible.charAt(Math.floor(Math.random() * possible.length));
+
+    return text;
+  }
+
   it("courses - should list course information", () => {
     cy.get("table tbody").within(() => {
       cy.get("tr")
@@ -15,4 +28,59 @@ describe("super admin user", () => {
         .should("have.length", 4);
     });
   });
+
+  it("courses - if no input should not be able to add a new course", () => {
+    cy.get("#training-table-header .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get('.v-dialog .v-card__actions').within(() => {
+      cy.get("button").eq(1).should('be.disabled');
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("courses - if title is too short should not be able to add a new course", () => {
+    cy.get("#training-table-header .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get('.v-dialog').within(() => {
+      cy.get("input").eq(0).type(randomStringLength(1));
+    });
+    cy.get('.v-dialog .v-card__actions').within(() => {
+      cy.get("button").eq(1).should('be.disabled');
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("courses - if title is too long should not be able to add a new course", () => {
+    cy.get("#training-table-header .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get('.v-dialog').within(() => {
+      cy.get("input").eq(0).type(randomStringLength(256));
+    });
+    cy.get('.v-dialog .v-card__actions').within(() => {
+      cy.get("button").eq(1).should('be.disabled');
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("courses - if description is too long should not be able to add a new course", () => {
+    cy.get("#training-table-header .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get('.v-dialog').within(() => {
+      cy.get("textarea").eq(0).type(randomStringLength(2010));
+    });
+    cy.get('.v-dialog .v-card__actions').within(() => {
+      cy.get("button").eq(1).should('be.disabled');
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+
 });
-- 
GitLab


From 1daf7bb20c81f8cf58a31b727a92ef3697f3b9a4 Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 11:34:57 -0600
Subject: [PATCH 09/10] teams e2e

---
 tests/e2e/specs/superAdmin/courses.spec.js | 27 ++++-----
 tests/e2e/specs/superAdmin/teams.spec.js   | 70 +++++++++++++++++++++-
 2 files changed, 82 insertions(+), 15 deletions(-)

diff --git a/tests/e2e/specs/superAdmin/courses.spec.js b/tests/e2e/specs/superAdmin/courses.spec.js
index e68c220..092b58a 100644
--- a/tests/e2e/specs/superAdmin/courses.spec.js
+++ b/tests/e2e/specs/superAdmin/courses.spec.js
@@ -11,7 +11,8 @@ describe("super admin user", () => {
 
   function randomStringLength(length) {
     var text = "";
-    var possible = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
+    var possible =
+      "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
 
     for (var i = 0; i < length; i++)
       text += possible.charAt(Math.floor(Math.random() * possible.length));
@@ -33,8 +34,8 @@ describe("super admin user", () => {
     cy.get("#training-table-header .text-right").within(() => {
       cy.get("button").eq(0).click();
     });
-    cy.get('.v-dialog .v-card__actions').within(() => {
-      cy.get("button").eq(1).should('be.disabled');
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
       cy.get("button").eq(0).click(); // reset to popup closed
       cy.wait(500);
     });
@@ -44,11 +45,11 @@ describe("super admin user", () => {
     cy.get("#training-table-header .text-right").within(() => {
       cy.get("button").eq(0).click();
     });
-    cy.get('.v-dialog').within(() => {
+    cy.get(".v-dialog").within(() => {
       cy.get("input").eq(0).type(randomStringLength(1));
     });
-    cy.get('.v-dialog .v-card__actions').within(() => {
-      cy.get("button").eq(1).should('be.disabled');
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
       cy.get("button").eq(0).click(); // reset to popup closed
       cy.wait(500);
     });
@@ -58,11 +59,11 @@ describe("super admin user", () => {
     cy.get("#training-table-header .text-right").within(() => {
       cy.get("button").eq(0).click();
     });
-    cy.get('.v-dialog').within(() => {
+    cy.get(".v-dialog").within(() => {
       cy.get("input").eq(0).type(randomStringLength(256));
     });
-    cy.get('.v-dialog .v-card__actions').within(() => {
-      cy.get("button").eq(1).should('be.disabled');
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
       cy.get("button").eq(0).click(); // reset to popup closed
       cy.wait(500);
     });
@@ -72,15 +73,13 @@ describe("super admin user", () => {
     cy.get("#training-table-header .text-right").within(() => {
       cy.get("button").eq(0).click();
     });
-    cy.get('.v-dialog').within(() => {
+    cy.get(".v-dialog").within(() => {
       cy.get("textarea").eq(0).type(randomStringLength(2010));
     });
-    cy.get('.v-dialog .v-card__actions').within(() => {
-      cy.get("button").eq(1).should('be.disabled');
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
       cy.get("button").eq(0).click(); // reset to popup closed
       cy.wait(500);
     });
   });
-
-
 });
diff --git a/tests/e2e/specs/superAdmin/teams.spec.js b/tests/e2e/specs/superAdmin/teams.spec.js
index 3c2142e..d8ba351 100644
--- a/tests/e2e/specs/superAdmin/teams.spec.js
+++ b/tests/e2e/specs/superAdmin/teams.spec.js
@@ -1,12 +1,26 @@
 describe("super admin user", () => {
   before(() => {
     cy.userFixture("users/superAdmin");
+    cy.visit("/teams");
+  });
+
+  beforeEach(() => {
     cy.intercept("GET", "api/teams?*", {
       fixture: "api/teamsList",
     });
-    cy.visit("/teams");
   });
 
+  function randomStringLength(length) {
+    var text = "";
+    var possible =
+      "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
+
+    for (var i = 0; i < length; i++)
+      text += possible.charAt(Math.floor(Math.random() * possible.length));
+
+    return text;
+  }
+
   it("teams - should list one team with names that contain the text 'Test'", () => {
     cy.get("table tbody").within(() => {
       cy.get("tr")
@@ -16,4 +30,58 @@ describe("super admin user", () => {
         .should("have.length", 2);
     });
   });
+
+  it("teams - if no input should not be able to add a new team", () => {
+    cy.get(".teams .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("teams - if title is too short should not be able to add a new team", () => {
+    cy.get(".teams .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get(".v-dialog").within(() => {
+      cy.get("input").eq(0).type(randomStringLength(1));
+    });
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("teams - if title is too long should not be able to add a new team", () => {
+    cy.get(".teams .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get(".v-dialog").within(() => {
+      cy.get("input").eq(0).type(randomStringLength(256));
+    });
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
+  it("teams - if description is too long should not be able to add a new team", () => {
+    cy.get(".teams .text-right").within(() => {
+      cy.get("button").eq(0).click();
+    });
+    cy.get(".v-dialog").within(() => {
+      cy.get("textarea").eq(0).type(randomStringLength(2010));
+    });
+    cy.get(".v-dialog .v-card__actions").within(() => {
+      cy.get("button").eq(1).should("be.disabled");
+      cy.get("button").eq(0).click(); // reset to popup closed
+      cy.wait(500);
+    });
+  });
+
 });
-- 
GitLab


From e561ce587c6311173b50bd6b3e15deb2356421fd Mon Sep 17 00:00:00 2001
From: ckwolff <cassandra.wolff@darkwolfsolutions.com>
Date: Mon, 7 Jun 2021 11:51:23 -0600
Subject: [PATCH 10/10] teams e2e

---
 tests/e2e/specs/superAdmin/teams.spec.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/e2e/specs/superAdmin/teams.spec.js b/tests/e2e/specs/superAdmin/teams.spec.js
index d8ba351..0b918f1 100644
--- a/tests/e2e/specs/superAdmin/teams.spec.js
+++ b/tests/e2e/specs/superAdmin/teams.spec.js
@@ -83,5 +83,4 @@ describe("super admin user", () => {
       cy.wait(500);
     });
   });
-
 });
-- 
GitLab