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

QA for 1.3.0 release #121

Closed
aviral-mittal opened this issue Sep 8, 2022 · 13 comments · Fixed by #123
Closed

QA for 1.3.0 release #121

aviral-mittal opened this issue Sep 8, 2022 · 13 comments · Fixed by #123
Assignees
Labels
priority/critical bugs that are inferring functionality, should be fixed right now. Ready for QA

Comments

@aviral-mittal
Copy link

aviral-mittal commented Sep 8, 2022

Issue Description

Please conduct a thorough testing of the plugin for upcoming 1.3.0 release. Below are the features, fixes and additions made to the plugin:

= V 1.3.0 =

@aviral-mittal aviral-mittal added priority/high something that should be prioritised when taking up next issues Ready for QA labels Sep 8, 2022
@Rink9
Copy link

Rink9 commented Sep 9, 2022

@abhishekfdd @aviral-mittal Performed functional and UI and regression test on Login with Google plugin.

Functional test cases can be found here:

Total scenario tested: 20

https://docs.google.com/spreadsheets/d/143XouHaIL0qf7RSbyfqXuZmL2tkFeJvxbfRTM6Db16E/edit#gid=250722887

Some feedbacks sharing below

  • 1. On the settings under the Display Logout we can put a text message

If the user is logged in, keeping this box unchecked will remove the Login with Google button from the page. If the box is checked, the button will show with title changed to ‘Logout’

image

  • 2. On logged out mode, after logged in from Login with Google button from frontend it's taking to the backend dashboard page.

Expected: After logged in with Login with Google button from frontend we can stay on the frontend page but the page will be logged in.

Login.with.google.test.Rinky.s.new.site.mp4

Note:

These issues are related to backend cannot verify from my end. Also it has test cases written and verified by @abhishekfdd

cc: @pooja-muchandikar

@aviral-mittal
Copy link
Author

@Rink9 In the current implementation, point no. 2 is not expected. We have provided hooks in case someone would like to build additional functionalities at authentication or login event.

@abhishek in point no.1, the text can be

If the box is unchecked the Login with Google button/block will not be visible to logged-in user. If the box is checked, the button/block will become a ‘Logout’ button

@abhishekfdd
Copy link
Contributor

PR for adding helper text is merged now

@aviral-mittal aviral-mittal assigned Rink9 and unassigned abhishekfdd Sep 15, 2022
@Rink9
Copy link

Rink9 commented Sep 15, 2022

@abhishekfdd I have downloaded the zip file from master branch and performed functional test. Also checked above points.

  1. On logged out mode, after logged in from Login with Google button from frontend it's taking to the backend dashboard page.

This scenario is not working please can you recheck it?

Rest scenarios are working as expected.

cc: @aviral-mittal

@Rink9 Rink9 assigned abhishekfdd and unassigned Rink9 Sep 15, 2022
@aviral-mittal aviral-mittal changed the title QA for 1.2.3 release QA for 1.3.0 release Sep 28, 2022
@aviral-mittal
Copy link
Author

@Souptik2001 @alvitazwar Also need to test the following:

  • Test plugin compatibility with WordPress 5.4.2

@aviral-mittal aviral-mittal added priority/critical bugs that are inferring functionality, should be fixed right now. and removed priority/high something that should be prioritised when taking up next issues labels Sep 28, 2022
@aviral-mittal
Copy link
Author

@alvitazwar Just checking if the findings are available.

@alvitazwar
Copy link

alvitazwar commented Sep 29, 2022

@aviral-mittal can you please elaborate? is there any deadline for this task? I am asking this because I am fully booked on the wp-com task for today.
CC: @pooja-muchandikar

@pooja-muchandikar
Copy link
Contributor

@alvitazwar @aviral-mittal

Test plugin compatibility with WordPress 5.4.2

Pavan will be working on testing v1.3.0 of Login with Google plugin with WordPress 5.4.2 and the latest stable release along with PHP 7.3/7.4 and quick sanity with PHP 8.1.

Let us know if any other specific version to be QA'ed.

Test cases/Checklists for same: https://drive.google.com/drive/folders/1x8Y_GZwZU_0GWNLYeU3R0tK4A1UESu00

cc: @pavanpatil1

@pavanpatil1
Copy link
Contributor

pavanpatil1 commented Sep 29, 2022

Tested the login with google v1.3.0 with the WP5.4.2 and the latest WP version. Verified with the PHP7 and 8. Added status below 👇

  1. PHP 7.4 - WP 6.0.2 latest. - Working fine
  2. PHP 7.3.5 - WP 6.0.2 latest - Working fine
  3. PHP 7.3.5 - WP5.4.2. Plugin functionality is working fine. But in the UI of login with google button added from Gutenberg block is showing as a link. It should be visible as a button with the image.
Actual result Expected result
image image

--

  1. Performed the quick sanity with the latest WP and PHP8, No blocker found plugin is working fine.

General issue observed for the newly added GB block -

  • While adding the login with google Gutenberg block, the warning is displayed on the console. Below warning is displayed.
    Warning: Invalid value for prop save on <div> tag. Either remove it from the element, or pass a string or number value to keep it in the DOM
console.mp4

cc: @aviral-mittal

@Souptik2001
Copy link
Member

Yes I also tested with WP version 5.4.2 and got the same styling problem.

Actually this is because the style key in the register_block_type function's argument list was most probably introduced in WP version 5.5.0.

So, either we can change the lower WP version range to 5.5.0 or we can enqueue the login.css style separately using wp_enqueue_style then we can keep the lower range to 5.4.2 only.

cc: @aviral-mittal @abhishekfdd

@Souptik2001
Copy link
Member

While adding the login with google Gutenberg block, the warning is displayed on the console.

@pavanpatil1 I tried to produce this console error. But I couldn't see it. If possible can you please provide the edit page link where you are getting this error, so that I can check it?

cc: @aviral-mittal

@aviral-mittal
Copy link
Author

@Souptik2001 Let's update the minimum WP version required in release details to WP5.5 as 5.5. at this time is already 2 years old.

Cc: @pavanpatil1

@pavanpatil1
Copy link
Contributor

pavanpatil1 commented Oct 3, 2022

While adding the login with google Gutenberg block, the warning is displayed on the console.

Discussed the issue with @Souptik2001. The issue is not reproducible on rt. gw(dev) sites and also on Souptik's local machine. it is reproducible only on my local setup. Skipping this for now. So if this issue is reproducible in the future then will try to debug it further.

cc: @aviral-mittal

@alvitazwar alvitazwar removed their assignment Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/critical bugs that are inferring functionality, should be fixed right now. Ready for QA
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants