Bug 1251921 - Please include psl test data
Summary: Please include psl test data
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: publicsuffix-list
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
Assignee: Yanko Kaneti
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-10 10:02 UTC by Christopher Meng
Modified: 2015-09-23 04:13 UTC (History)
1 user (show)

Fixed In Version: 20150831-1.fc23
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-09-23 04:13:21 UTC


Attachments (Terms of Use)

Description Christopher Meng 2015-08-10 10:02:28 UTC
Hi,

libpsl needs test_psl.txt from publicsuffix to test the publicsuffix( ;-) ), few days ago I encountered the problem that dated test data(*.cy->several new TLDs) failed the tests of libpsl.

Source: https://publicsuffix.org/list/test.txt

(Moved to http://mxr.mozilla.org/mozilla-central/source/netwerk/test/unit/data/test_psl.txt?raw=1

Or this one could be better(?):

https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt
)

You should put it underneath %{_datadir}/publicsuffix, but you may also create a test subfolder and put it inside.

Let me know when you finish this and I will start a new libpsl build. Thank you!

Comment 1 Yanko Kaneti 2015-08-10 11:15:13 UTC
(In reply to Christopher Meng from comment #0)
....
> Or this one could be better(?):
> 
> https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt
> )

I'd rather use
https://github.com/publicsuffix/list/raw/master/tests/test_psl.txt
which is the same as the above. 

Unfortunately currently it has a syntax error, the fix for which isn't yet merged.  https://github.com/publicsuffix/list/pull/21  . I'd rather wait for that to get merged then update tхe list and add the testс.

I hope its not a big problem to wait a little...

Comment 2 Fedora Update System 2015-09-01 08:00:53 UTC
publicsuffix-list-20150831-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14732

Comment 3 Fedora Update System 2015-09-01 20:22:52 UTC
publicsuffix-list-20150831-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update publicsuffix-list'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14732

Comment 4 Fedora Update System 2015-09-23 04:13:20 UTC
publicsuffix-list-20150831-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.