Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

Test failure on BigEndian #1346

Closed
remicollet opened this issue Nov 10, 2017 · 3 comments
Closed

Test failure on BigEndian #1346

remicollet opened this issue Nov 10, 2017 · 3 comments

Comments

@remicollet
Copy link
Contributor

See Fedora scratch build (version 1.7.1 against PHP 7.2.0RC6)
https://koji.fedoraproject.org/koji/taskinfo?taskID=23038022

BigEndian (ppc64 and s390x):

There was 1 failure:
1) Faker\Test\Provider\UuidTest::testUuidExpectedSeed
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'8e2e0c84-50dd-367c-9e66-f3ab455c78d6'
+'840c2e8e-dd50-3c46-9e66-f3ab455c78d6'
/builddir/build/BUILD/Faker-d3ed4cc37051c1ca52d22d76b437d14809fc7e0d/test/Faker/Provider/UuidTest.php:20
@remicollet remicollet changed the title Test failure on various BigEndian Test failure on BigEndian Nov 10, 2017
@fzaninotto
Copy link
Owner

OK, then let's just say that Faker doesn't support BigEndian OSes.

@remicollet
Copy link
Contributor Author

Only for a single test ?
Especially as there is code to handle bidendian ?
=> https://github.com/fzaninotto/Faker/blob/master/src/Faker/Provider/Uuid.php#L27

@fzaninotto
Copy link
Owner

yes, because I don't want to download an entire OS to test a patch.

@remicollet remicollet mentioned this issue Nov 29, 2017
Merged
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fzaninotto @remicollet