fix: remove passing tags

add command to readme close #84
StabiBerlin · Jun 14, 2024 · 40a38fa · 40a38fa
1 parent 8b5fc6d
commit 40a38fa
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 22 deletions.
diff --git a/README.md b/README.md
@@ -80,14 +80,31 @@ Mit GUI:
 npx cypress open
 ```
 
-Per Default laufen die Tests gegen den [Ranking1](http://stabikat-ranking1/). Die `searchspecs.yml` befindet sich im Verzeichnis: `/var/www/vufind/local/config/vufind`
+Per Default laufen die Tests gegen den [Ranking1](http://stabikat-ranking1/) testserver. Die `searchspecs.yml` befindet sich im Verzeichnis: `/var/www/vufind/local/config/vufind`
 
-### Tagging
+Für die Reproduktion der CI Testläufe gegen den produktiven Stabikat z.Bsp der `simple.cs.js` spec:
+
+- Ändere `BASE_URL` zum [Stabikat](https://stabikat.de)
+- Spezifiziere welche Testdatei via `--spec`, und
+- nur Tests ohne tag, via `--env`
 
-Zur besseren Organisation der Tests via tags:
+```bash
+CYPRESS_BASE_URL=https://stabikat.de/search/ npx cypress run --spec cypress/e2e/simple.cy.js --env grepUntagged=true   
+```
 
-- `@next` für Tests die in der Testinstanz laufen, aber noch nicht im produktivem Stabikat
+```powershell
+npx cypress open --config baseUrl=https://stabikat.de/search/ --env grepUntagged=true 
+```
+
+Für weitere Optionen siehe [Cypress command-line](https://docs.cypress.io/guides/guides/command-line)
+
+### Tagging
 
+Für offene issues, mit Test die in beiden Instanzen (noch) nicht laufen nutze: `.skip`.
+Tests die auf dem Testserver grün sind, aber im produktiven Stabikat rot werden durch den `@next` tag gekennzeichnet:
+
+- `@next`
+
   ```js
   // This works on test environment but not on stabi
   it('CJK author search should return translations', {tags: ['@next']}, () => {
@@ -102,25 +119,15 @@ Zur besseren Organisation der Tests via tags:
     })  
   ```
 
-Wenn die nötigen Änderungen der `searchpsec.yaml` im Stabikat live gegangen sind, müssen die entsprechenden `@next` Tags entfernt werden.
-
-Für Tests die in beiden Instanzen (noch) nicht laufen: `.skip`
-
-Für die Reproduktion der CI Testläufe gegen den produktiven Stabikat der `simple.cs.js` spec:
-
-- Ändere `BASE_URL` zum [Stabikat](https://stabikat.de)
-- Spezifiziere welche Testdatei via `--spec`, und
-- nur Tests ohne tag, via `--env`
+### Release
 
-```bash
-CYPRESS_BASE_URL=https://stabikat.de/search/ npx cypress run --spec cypress/e2e/simple.cy.js --env grepUntagged=true   
-```
+Wenn die nötigen Änderungen der `searchpsec.yaml` im Stabikat live gegangen sind, müssen die entsprechenden `@next` Tags entfernt werden. Folgender Befehl führ nur die tests mit tag aus:
 
-```powershell
-npx cypress open --config baseUrl=https://stabikat.de/search/ --env grepUntagged=true 
+```shell
+npx cypress run --config baseUrl=https://stabikat.de/search/ --env grepTags=@next
 ```
 
-Für weitere Optionen siehe [Cypress command-line](https://docs.cypress.io/guides/guides/command-line)
+Anschliessend bei allen grünen Tests den Tag entfernen.
 
 ## Yaml Prüfung
 
@@ -134,6 +141,8 @@ Dieser Test wird auf GitHub automatisch ausgeführt.
 
 ## Troubleshooting
 
+Die Datei `cypress/e2e/simple.cy.js` beinhaltet einen Selbsttest. Zur Prolembehebung können diese *unskipped* werden.
+
 ### Cypress Browser Warning
 
 Sollte ein eine Browser spezifische Warnung erscheinen, verhindern Windows Systemeinstellungen die automatisierte Nutzung des gewählten Browsers.

diff --git a/cypress/e2e/author.cy.js b/cypress/e2e/author.cy.js
@@ -12,7 +12,7 @@ describe('Author Search', () => {
         })
 
         // erster Treffer von Autorin
-        it('first hit should be by the author', {tags: ['@next']}, () => {
+        it('first hit should be by the author', () => {
             cy.get('#result0')
                 .find('.resultlist-data')
                 .contains(/(?=.*Karin)(?=.*Glaser)/)
@@ -32,7 +32,7 @@ describe('Author Search', () => {
         })
 
         // see #22
-        it('CJK author search should return translations', {tags: ['@next']}, () => {
+        it('CJK author search should return translations', () => {
             cy.get('.record-list')
               .contains('Yan, Lianke')
         })

diff --git a/cypress/e2e/chronology.cy.js b/cypress/e2e/chronology.cy.js
@@ -350,7 +350,7 @@ describe('Chronology', () => {
         })
     })
 
-    describe('future publication', {tags: ['@next']}, () => {
+    describe('future publication', () => {
         // see #58
         // run a query for publications 5 years from noww
         beforeEach(() => {