Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: automatiseer feature files #132

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

MelvLee
Copy link
Collaborator

@MelvLee MelvLee commented Oct 7, 2021

Als POC zijn twee feature files (zoeken en fout-meldingen) geautomatiseerd.

De automation code is te vinden onder features/step_definitions. De automation code files heten resp. zoeken.js en fout-meldingen.js. world.js bevat code om variabelen voor de testen te configureren. Op dit moment baseurl en apikey.

Om zoeken.feature te automaten zijn wat wijzigingen doorgevoerd. Ook zijn hier duplicates verwijderd door het gebruiken van Abstract Scenario. Ook zijn wat voorbeelden/scenario's uitgecommentarieerd omdat de bijbehorende gegeven context niet kloppen.

Ook is het mogelijk om een html rapport te genereren met het testresultaat. Het laatste rapport voor de zoeken en fout-meldingen features is te vinden onder docs/features/test-report.html. Om dit rapport te zien moet de file naar lokaal worden gedownload.

Het draaien van de features wordt gedaan met npm run specs:verify en het genereren van de report met npm run specs:report

# Gegeven de LV WOZ kent een WOZ-object met objectnummer "000500000001"
# En de aanduiding WOZ-object heeft postcode "8000GB" en huisnummer 1 en huisletter "a" en huisnummertoevoeging "bis"
# Als ik een WOZ-object zoek met /wozobjecten?postcode=8000GB&huisnummer=1&huisnummertoevoeging=andere
# Dan bevat het antwoord niet het WOZ-object met identificatie "000500000001"

Voorbeeld: zoek WOZ-object met nummeraanduidingIdentificatie van de aanduiding
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moet hier 'Voorbeeld' dan niet ook 'Abstract Scenario' worden?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dit kan worden verplaatst naar de Abstract Scenario hierboven


Regel: Zoeken van WOZ-objecten levert lege property _embedded wanneer er niks gevonden wordt
Rule: Zoeken van WOZ-objecten levert lege property _embedded wanneer er niks gevonden wordt

Voorbeeld: zoeken met adresseerbaar object identificatie vindt geen WOZ-object
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Idem.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deze heb ik vergeten te verwijderen. Hij staat al in onderstaande Abstract Scenario

@melsk-r
Copy link
Contributor

melsk-r commented Oct 11, 2021

Het wil bij mij lokaal nog niet werken. npm run specs:verify geeft bij mij als resultaat npm ERR! missing script: specs:verify.

Moet ik misschien nog e.e.a. installeren?

@MelvLee
Copy link
Collaborator Author

MelvLee commented Oct 11, 2021

Het wil bij mij lokaal nog niet werken. npm run specs:verify geeft bij mij als resultaat npm ERR! missing script: specs:verify.

Moet ik misschien nog e.e.a. installeren?

ja, je moet npm install aanroepen om de packages te installeren

@melsk-r
Copy link
Contributor

melsk-r commented Oct 11, 2021

Het werkt nu bij mij wel, althans... er wordt wat verwerkt en het html report wordt gegenereerd.
Echter alle test staan op 'failed', waar zou dat aan kunnen liggen? In de api key wordt toch vanuit world.js. voorzien dus aan de auorisaties kan het niet liggen, toch?

@MelvLee
Copy link
Collaborator Author

MelvLee commented Oct 11, 2021

Het werkt nu bij mij wel, althans... er wordt wat verwerkt en het html report wordt gegenereerd. Echter alle test staan op 'failed', waar zou dat aan kunnen liggen? In de api key wordt toch vanuit world.js. voorzien dus aan de auorisaties kan het niet liggen, toch?

Heb je jouw api-key toegevoegd? Dat moet nog gebeuren, omdat je geen apikey wil/mag committen in de repo (security breach)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants