pokemon-world icon indicating copy to clipboard operation
pokemon-world copied to clipboard

test: add more ui test using POM

Open depapp opened this issue 2 years ago â€ĸ 1 comments

halo @afiiif salam kenal 🙇đŸģ

saya coba nambahin Playwright E2E Test nya sama pake Page-Object-Model

mohon review nya 📚

test result image

depapp avatar Aug 30 '22 13:08 depapp

Someone is attempting to deploy a commit to a Personal Account owned by @afiiif on Vercel.

@afiiif first needs to authorize it.

vercel[bot] avatar Aug 30 '22 13:08 vercel[bot]

Halo Mas @depapp, thank you PRnya. Aku udah cek changesnya..

Ada bbrp feedback dr aku,

1ī¸âƒŖ Kalo yg mas bikin itu independen ya antara ngetes filter sendiri, ngetes search sendiri, & ngetes compare sendiri. Kalo yg sebelumnya (https://github.com/afiiif/pokemon-world/blob/main/e2e/index.spec.ts), di situ aku berkesinambungan.

  1. Ngecek milih generasi, http://localhost:3000/?gen=1.
  2. Ngecek search by name (nah di sini URL param yg dari milih generasi gen=1 itu dicek expected buat tetep ada, http://localhost:3000/?q=pika&gen=1).

Nah menurutku bagusnya ada mengcover case yg saling dependen jg.

2ī¸âƒŖ Yg sebelumnya mengcover lebih byk case:

  1. Filter generation
  2. Search by name
  3. Dari hasil filter itu diklik cardnya (ke halaman detail pokemon)
  4. Dari halaman detail pokemon itu ke bagian compare with, masukin pokemon lain & redirect ke compare pokemon.
  5. Nambahin pokemon lagi buat dicompare (skrg compare pikachu-squirtle-charmander)
  6. Remove Squirtle dr comparison
  7. Dari halaman compare, bagian Charmander, klik detail & masuk ke halaman detail Charmander

Yg di PR baru mencakup poin 1,2,5

afiiif avatar Sep 02 '22 07:09 afiiif

ℹī¸ Btw aku sebelumnya langsung pake Playwright Test Generator:

npx playwright codegen http://localhost:3000/

afiiif avatar Sep 02 '22 07:09 afiiif

terima kasih mas @afiiif untuk review nya, next nya akan saya update sesuai dengan concern nya 🙇đŸģ

depapp avatar Sep 02 '22 08:09 depapp

@depapp tadi aku ada nambahin bbrp test case lagi. Cek branch main terbaru ya...

afiiif avatar Sep 02 '22 11:09 afiiif