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

Duplicate fixtures in tests #7934

Open
jrfavre opened this issue Nov 26, 2024 · 2 comments · Fixed by #7935
Open

Duplicate fixtures in tests #7934

jrfavre opened this issue Nov 26, 2024 · 2 comments · Fixed by #7935

Comments

@jrfavre
Copy link

jrfavre commented Nov 26, 2024

There are many duplicate user agents in /Tests/fixtures/ folder. Here are just a few examples :

Mozilla/5.0 (Linux; Android 5.1; HiBox-hero) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.62 Safari/537.36

Mozilla/5.0 (Linux; Android 9; HLTE720T Build/PKQ1.190302.001; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/76.0.3809.89 Mobile Safari/537.36

Mozilla/5.0 (Linux; U; Android 4.4.4; en-US; SM-G5306W Build/KTU84P) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/57.0.2987.108 UCBrowser/12.10.3.1199 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 5.0.2; SM-S920L) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.92 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 5.1.1; SM-G530R7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.99 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 5.1.1; SAMSUNG-SM-G530A) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.93 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 5.1.1; SM-N920X Build/LMY47X) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.158 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N920R7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.157 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N930S) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.136 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N930T) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.162 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N9300) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.87 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N930V) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.83 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 6.0.1; SM-N930P) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.121 YaBrowser/19.3.3.285.00 Mobile Safari/537.36

Mozilla/5.0 (Linux; Android 5.1.1; Xperia Z Ultra) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.96 Mobile Safari/537.36

Microsoft-Windows-XP/2002, UPnP/1.1, J. River Internet Reader/2.0 (compatible; Windows-Media-Player/10)

RadioPublic/android-2.2

GooglePodcasts/2.0.2 iPod_touch/13.4.1 hw/iPod9_1

Spotify/120800923 OSX/0 (MacBookPro6,2)

AmazonMusic

KaiOS Downloader PodKast

Procast (iOS)

...

Maybe it would be valuable to write a test to check duplicate fixtures

liviuconcioiu added a commit to liviuconcioiu/device-detector that referenced this issue Nov 26, 2024
sanchezzzhak pushed a commit that referenced this issue Nov 27, 2024
* Remove duplicate tests
* Remove duplicate tests
* Remove duplicate tests
* Remove duplicate tests for same app

ref #7934
@jrfavre
Copy link
Author

jrfavre commented Dec 19, 2024

I have still detected the following duplicate regex in Tests/fixtures (one per line):
RadioPublic/android-2.2
GooglePodcasts/2.0.2 iPod_touch/13.4.1 hw/iPod9_1
Spotify/120800923 OSX/0 (MacBookPro6,2)
Procast (iOS)
GooglePodcasts/2.0.2 iPod_touch/13.4.1 hw/iPod9_1
Echo/1.0(APNG)
FlyCast/1.34 (BlackBerry; 8330/4.5.0.131 Profile/MIDP-2.0 Configuration/CLDC-1.1 VendorID/-1)
AndroidDownloadManager/4.1.1 (Linux; U; Android 4.1.1; MB886 Build/9.8.0Q-97_MB886_FFW-20)
Amazon;AFTTI43;756a522d9f1648b89e76e80be654456a;;tpapi;3.202.356
bPod
SubStream/0.7 CFNetwork/485.12.30 Darwin/10.4.0

I have not checked all of them. Just the first one and it is exactly similar (in 2 different files).
Is there a reason to keep this duplicates ?

@sanchezzzhak sanchezzzhak reopened this Dec 19, 2024
@liviuconcioiu
Copy link
Collaborator

Some were added in this PR #7481

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 a pull request may close this issue.

3 participants