smoketests

smoketests on push

cypress commands.js

cypress videos and screenshots

cypress videos and screenshots

wait

smoketest

Author: hollesse

.github/workflows/smoketest.yml

@@ -1,7 +1,8 @@
 on:
   schedule:
     - cron: '*/5 * * * *'
-
+  push:
+name: Smoketest
 jobs:
   smoketest:
     runs-on: ubuntu-latest
@@ -13,4 +14,16 @@ jobs:
       - name: Cypress run
         uses: cypress-io/github-action@v4
         with:
-          spec: cypress/cypress/e2e/smoketest.cy.js
+          working-directory: cypress
+          #spec: cypress/cypress/e2e/smoketest.cy.js
+      - uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: cypress-screenshots
+          path: cypress/cypress/screenshots
+      # Test run video was always captured, so this action uses "always()" condition
+      - uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: cypress-videos
+          path: cypress/cypress/videos

cypress/cypress/e2e/smoketest.cy.js

@@ -50,7 +50,7 @@ describe('Mob.sh Timer', () => {
     cy.wait(100)
     cy.get('.btn-close').click()
     let initialHistoryCountChainer = cy.get('#history-container').find('li').its('length')
-    cy.get('#timer-button').contains(1).click().wait(100)
+    cy.get('#timer-button').contains(1).click().wait(200)
     cy.get('#history-container').find('li').its('length').then(currentCount => {
       initialHistoryCountChainer.should('eq', currentCount - 1)
     })
@@ -63,7 +63,7 @@ describe('Mob.sh Timer', () => {
     cy.wait(100)
     cy.get('.btn-close').click()
     let initialHistoryCountChainer = cy.get('#history-container').find('li').its('length')
-    cy.get('#breaktimer-button').contains(1).click().wait(100)
+    cy.get('#breaktimer-button').contains(1).click().wait(200)
     cy.get('#history-container').find('li').its('length').then(currentCount => {
       initialHistoryCountChainer.should('eq', currentCount - 1)
     })

cypress/cypress/support/commands.js

@@ -0,0 +1,25 @@
+// ***********************************************
+// This example commands.js shows you how to
+// create various custom commands and overwrite
+// existing commands.
+//
+// For more comprehensive examples of custom
+// commands please read more here:
+// https://on.cypress.io/custom-commands
+// ***********************************************
+//
+//
+// -- This is a parent command --
+// Cypress.Commands.add('login', (email, password) => { ... })
+//
+//
+// -- This is a child command --
+// Cypress.Commands.add('drag', { prevSubject: 'element'}, (subject, options) => { ... })
+//
+//
+// -- This is a dual command --
+// Cypress.Commands.add('dismiss', { prevSubject: 'optional'}, (subject, options) => { ... })
+//
+//
+// -- This will overwrite an existing command --
+// Cypress.Commands.overwrite('visit', (originalFn, url, options) => { ... })

src/main/java/sh/mob/timer/web/RoomApiController.java

@@ -24,7 +24,7 @@
 @RequestMapping()
 public class RoomApiController {
 
-  private static final String SMOKETEST_ROOM_NAME = "testroom-310a9c47-515c-4ad7-a229-ae8efbab7387";
+  private static final String SMOKETEST_ROOM_NAME_PREFIX = "testroom-";
   private static final Logger log = LoggerFactory.getLogger(RoomApiController.class);
   private final RoomRepository roomRepository;
   private final Clock clock;
@@ -83,7 +83,7 @@ public void publishEvent(@PathVariable String roomId, @RequestBody PutTimerReque
           timerRequest.timer,
           timerRequest.user,
           room.name());
-      if (!Objects.equals(room.name(), SMOKETEST_ROOM_NAME)) {
+      if (!room.name().startsWith(SMOKETEST_ROOM_NAME_PREFIX)) {
         stats.incrementTimer(timer);
       }
     } else if (timerRequest.breaktimer() != null) {
@@ -94,7 +94,7 @@ public void publishEvent(@PathVariable String roomId, @RequestBody PutTimerReque
           timerRequest.breaktimer(),
           timerRequest.user,
           room.name());
-      if (!Objects.equals(room.name(), SMOKETEST_ROOM_NAME)) {
+      if (!room.name().startsWith(SMOKETEST_ROOM_NAME_PREFIX)) {
         stats.incrementBreaktimer(breaktimer);
       }
     } else {