Skip to content

Conversation

@jeremykendall
Copy link
Owner

These tests are based on the canonical TEST DATA. The failing tests here were changed to match the reference tests. All tests are now passing.

For reference: this is the section that changed (.cy changed to .mm):

// TLD with only 1 (wildcard) rule.
checkPublicSuffix('mm', null);
checkPublicSuffix('c.mm', null);
checkPublicSuffix('b.c.mm', 'b.c.mm');
checkPublicSuffix('a.b.c.mm', 'b.c.mm');

These tests are based on the canonical
[TEST DATA](https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt).
The failing tests here were changed to match the reference tests. All
tests are now passing.

For reference this is the section that changed (`.cy` changed to `.mm`):
```
// TLD with only 1 (wildcard) rule.
checkPublicSuffix('mm', null);
checkPublicSuffix('c.mm', null);
checkPublicSuffix('b.c.mm', 'b.c.mm');
checkPublicSuffix('a.b.c.mm', 'b.c.mm');
```
@jeremykendall jeremykendall merged commit e968db6 into develop Jan 22, 2016
@nyamsprod nyamsprod deleted the fix/updated-psl-breaks-tests branch December 11, 2017 11:00
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 this pull request may close these issues.

2 participants