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

Tests before 1st release #11

Open
Real-John-Cheung opened this issue Jul 24, 2021 · 8 comments
Open

Tests before 1st release #11

Real-John-Cheung opened this issue Jul 24, 2021 · 8 comments
Assignees

Comments

@Real-John-Cheung
Copy link
Collaborator

@dhowe I think now we are ready for the before release test. I have tested in a few websites on firefox and chrome (mainly on a win64 machine). Could you create a list here for what tests should be done? and maybe @MelodyneedsMoney can also help with this.

@dhowe
Copy link
Owner

dhowe commented Jul 25, 2021

I think I can do the first round of testing, as I know what the desired behavior is. After that we can create a list of things to check for a new release (which I'll start here):

-- Verify all tests pages (with ads) show at least one poem replacement
-- Verify correct font, font-size, leading
-- Verify grammar is correct for generated text
-- Verify no text (or fragment) is repeated on a page
-- Verify padding around border of rectangle
-- Verify no more than 5 poems appear on a given page

... other things?

@dhowe dhowe self-assigned this Jul 25, 2021
@Real-John-Cheung
Copy link
Collaborator Author

Maybe also this two?
-- Verify that custom fliters works (replace, not hide)
-- Verify that temporary replacement (the strike button on the pop up page) works

@dhowe dhowe assigned Real-John-Cheung and unassigned dhowe Aug 21, 2021
@dhowe
Copy link
Owner

dhowe commented Aug 21, 2021

  • Verify all tests pages (with ads) show at least one poem replacement
  • Verify correct font, font-size, leading
  • Verify grammar is correct for generated text
  • Verify no text (or fragment) is repeated on a page
  • Verify padding around border of rectangle
  • Verify no more than 5 poems appear on a given page/visible at once
  • Verify that custom fliters works (replace, not hide)
  • Verify that temporary replacement (the strike button on the popup page) works

@Real-John-Cheung
Copy link
Collaborator Author

Real-John-Cheung commented Sep 18, 2021

Time: 18/9/21
Platform: Win64&mac OS 10.15 Firefox 92.0

  • Verify all tests pages (with ads) show at least one poem replacement
  • Verify correct font, font-size, leading
  • Verify grammar is correct for generated text [ok]
  • Verify no text (or fragment) is repeated on a page
  • Verify padding around border of rectangle
  • Verify no more than 5 poems visible at once - on hold
  • Verify that custom filters works (replace, not hide)
  • Verify that temporary replacement (the strike button on the popup page) works

Platform: Win64&mac OS 10.15 Chrome 93.0.

  • Verify all tests pages (with ads) show at least one poem replacement
  • Verify correct font, font-size, leading
  • Verify grammar is correct for generated text [ok]
  • Verify no text (or fragment) is repeated on a page
  • Verify padding around border of rectangle
  • Verify no more than 5 poems visible at once - on hold
  • Verify that custom filters works (replace, not hide) - occasionally fails in chrome
  • Verify that temporary replacement (the strike button on the popup page) works

Noticeable things
圖片
These two ads have the same background and are next to each other...

@dhowe
Copy link
Owner

dhowe commented Sep 18, 2021

I will check grammar more closely later, so I've marked it as done

  • Verify that custom filters works (replace, not hide) - occasionally fails in chrome

can you give an example?

These two ads have the same background and are next to each other...

Ideally we would never have two ads directly adjacent, but not critical for launch (if not very easy, create new post-launch ticket)

@Real-John-Cheung
Copy link
Collaborator Author

Verify that custom filters works (replace, not hide) - occasionally fails in chrome
can you give an example?

It happens sometime when using the "replace this element" option from the right-click menu. An example is to replace the Yahoo logo on https://hk.yahoo.com

Will create tickets for both the problems

@dhowe
Copy link
Owner

dhowe commented Sep 19, 2021

And what is the status on > Verify no more than 5 poems visible at once - on hold

Is your suggestion doable? If not (or it is difficult), then perhaps a per-page limit is enough for launch, as long as ads over the limit are still blocked (and just made invisible)

@Real-John-Cheung
Copy link
Collaborator Author

Is your suggestion doable? If not (or it is difficult), then perhaps a per-page limit is enough for launch, as long as ads over the limit are still blocked (and just made invisible)

I think it is doable, but not easy... I added a per-page limit in f25c7f4

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

No branches or pull requests

2 participants