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

Improved en_AU provider and added related unit tests #858

Merged
merged 4 commits into from
Mar 22, 2016
Merged

Improved en_AU provider and added related unit tests #858

merged 4 commits into from
Mar 22, 2016

Conversation

jremes-foss
Copy link
Contributor

Hi.

Looks like en_AU provider needed some attention.

  1. Added some provider methods for src/Faker/Provider/en_AU/Address.php file.
  2. Created provider test class to test/Faker/Provider/en_AU/AddressTest.php
  3. Created test methods to test/Faker/Provider/en_AU/AddressTest.php in order to test that the results are sane.

Thank you.

@jremes-foss
Copy link
Contributor Author

Looks like there is a space issue with CodeSniffer:

FILE: ...home/travis/build/fzaninotto/Faker/src/Faker/Provider/en_AU/Address.php
--------------------------------------------------------------------------------
FOUND 4 ERROR(S) AFFECTING 4 LINE(S)
--------------------------------------------------------------------------------
  81 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
  90 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
  99 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
 108 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
--------------------------------------------------------------------------------
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY
--------------------------------------------------------------------------------

fzaninotto added a commit that referenced this pull request Mar 22, 2016
Improved en_AU provider and added related unit tests
@fzaninotto fzaninotto merged commit bb09cf8 into fzaninotto:master Mar 22, 2016
@fzaninotto
Copy link
Owner

Thanks!

@jremes-foss jremes-foss deleted the en_AU_AddressTest branch March 23, 2016 00:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants