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] Dev Branch #471

Closed
8 of 12 tasks
kjnanda opened this issue Feb 5, 2024 · 17 comments
Closed
8 of 12 tasks

[QA] Dev Branch #471

kjnanda opened this issue Feb 5, 2024 · 17 comments
Assignees

Comments

@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

Issue with GSI feature on CNN site

  1. False positive results on cnn.com ( though we had already mentioned in the PRD that it will only work for 80% websites and may show false positive results )

Screenshot 2024-02-05 at 12 31 24 (1)

  1. Infinite loading ✅

When I opened the CNN website with single tab mode, it kept processing but the GSI feature was not found. When I inspect the tool it threw the below errors on console.

Screenshot 2024-02-05 at 10 27 35

--

GSI.Issue.mov

This started working as expected when I opened one more new chrome profile.

@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

Uncaught (in promise) Error: No tab with id: 848821757.

Tab.ID.issue.mov

@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

CDP switch issue

Steps:

  1. Open chrome://extension tab
  2. Now, open 2-3 tabs with different URLs
  3. Enable CDP and you must see the debug bar
  4. Anlyse the different tabs in single tab environment and observe correct set of data should appear as CDP is enable. E.g. Blocked cookie, Yellow BG color etc
  5. Now, Keep checking the chrome://extensions tab for any errors
  6. Close each tab one by one and you will notice the error as mentioned here.
  7. Now, repeat step 2 and observe that
    - the debug bar is not reappearing
    - No blocked cookies
    - No other cookies getting loaded
    - Cookies landing page shows 0 cookies
    - CDP switch is still on but you won't get above thing

CDP.Switch.issue.mov

Screenshot 2024-02-05 at 11 58 07

@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

Error: Uncaught (in promise) TypeError: Cannot set properties of undefined (setting 'newUpdates')

Screenshot 2024-02-05 at 11 02 47

@amovar18 amovar18 self-assigned this Feb 5, 2024
@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

Blank Table and blank screen issue

When I toggle the setting - multi tab and/or CDP; the blank table and blank screen issues are still visible. Especially with multi tab environment.


Multi tab -

Blank.table.and.blank.screen.issue.mov

Single Tab -

Single.tab.-.blank.tab.issue.mov

@kjnanda
Copy link
Author

kjnanda commented Feb 5, 2024

Regression bug : Existing core functionality is breaking

As per the expected behaviour if the URLs in next to chrome://extensions page or if it is tab num - 2 then it should by default load the cookies. However, wit this branch this functionality is breaking.

Steps:

  1. Checkout to develop branch > Pull the code > Run npm start
  2. Open a new chrome profile and install the extension
  3. Open a new tab with any URL
  4. Observe that it loads cookies as soon as you hit the URL
  5. Now, close the tab and again open the new tab with any URL
  6. Observe that this time it does not load the cookie and shows "Analyse this tab" button instead
feature.mov

@mohdsayed
Copy link
Collaborator

@kjnanda

The infinite loading issue was fixed in #478 and QA tested by Pavan.

We cannot replicate the false positive issue on CNN after several attempts so we will handle in later.

@pavanpatil1
Copy link
Collaborator

pavanpatil1 commented Feb 7, 2024

CLI: The Format of the expire date is not properly visible in the cookie table.

image

Note: After the report is downloaded, the report shows blank value for which the expiry date is not in expected format.

image

@pavanpatil1
Copy link
Collaborator

CLI: In the "Technologies" frame, the description is not clearly visible. If the content overflows, it should display "..." to indicate there is more content. Currently, the content appears to be hidden

image

@pavanpatil1
Copy link
Collaborator

CLI: If there is no description present for any technology then the proper message should be visible.

image

@kjnanda
Copy link
Author

kjnanda commented Feb 7, 2024

PSAT CLI - Samesite does not show any value. It is empty. But after I download the report the file contains the correct information.

PSAT CLI PSAT CLI - Download Report PSAT Extension
image image image

@pavanpatil1
Copy link
Collaborator

PSAT Extension - The "Unknown frames" is visible with URL rewrite.

Steps to reproduce the issue:

  1. Open the bbc.com site and observe the unknown frames will be visible. open the unknown frames tab.
  2. Now, rewrite the URL to travelopia.com.
  3. See the "Unknown frames" on the bbc.com are still visible on travelopia.com as well. Eventhough on travelopia.com there is no unknown frames.
URL.rewrite.issue.with.unknown.frames.1.1.mp4

@kjnanda
Copy link
Author

kjnanda commented Feb 7, 2024

@ayushnirwal Blocked cookies are not appearing Yellow BG color in CLI Dash board.

Screenshot 2024-02-07 at 18 38 27

@pavanpatil1
Copy link
Collaborator

#471 (comment)

Hi @amovar18, It is still reproducible on the develop branch.

Check the below steps:

  1. Go to bbc.com and open the cookies frame.
  2. Now, rewrite the URL to example.com and check the unknown frames will be shown. It should not be displayed.
  3. Now, refresh the page the page will be refreshed and unknown frame frame will be gone but cookies will be displayed.
screencast-bpconcjcammlapcogcnnelfmaeghhagj-2024.02.13-13_46_32.webm

@pavanpatil1
Copy link
Collaborator

Hi @amovar18, still able to replicate this issue.

image

@amovar18
Copy link
Collaborator

@pavanpatil1 the issue is replicated is different than what you have shown, because the issue that was listed shows unknown frames in the sidebar which have been removed in 0.6.0.

@pavanpatil1
Copy link
Collaborator

The unknown frames have been removed and other issues have been resolved. Hence closing this issue

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

4 participants