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

kk_KZ Company/person identification numbers unit tests #916

Merged
merged 2 commits into from
Nov 29, 2016

Conversation

YerlenZhubangaliyev
Copy link
Contributor

Some unit tests

Company/person identification number unit tests
added .idea folder to .gitignore
@YerlenZhubangaliyev YerlenZhubangaliyev changed the title Company/person identification numbers unit tests kk_KZ Company/person identification numbers unit tests May 18, 2016
@@ -1,2 +1,3 @@
.idea
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, please remove

@@ -52,7 +52,7 @@ public static function companyNameSuffix()
/**
* National Business Identification Numbers
*
* @link http://egov.kz/wps/portal/!utWCM/p/b1/04_Sj9Q1MjAwsDQ1s9CP0I_KSyzLTE8syczPS8wB8aPM4oO8PE2cnAwdDSxMw4wMHE08nZ2CA0KDXcwMgQoikRUYWIY4gxS4hwU4mRkbGBgTp98AB3A0IKQ_XD8KVQkWF4AV4LHCzyM_N1U_uKhUPzcqx83SU9cRANth_Rk!/dl4/d5/L0lHSkovd0RNQU5rQUVnQSEhLzRKVUUvZW4!/
* @link http://egov.kz/wps/portal/Content?contentPath=%2Fegovcontent%2Fbus_business%2Ffor_businessmen%2Farticle%2Fbusiness_identification_number&lang=en
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you explain this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Old link is outdated and wrong

use Faker\Provider\kk_KZ\Person;

/**
* Person test
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless comment

use Faker\Provider\kk_KZ\Company;

/**
* Company test
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless comment

}

/**
* @see \Faker\Provider\kk_KZ\Person::businessIdentificationNumber
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless comment

}

/**
* @see \Faker\Provider\kk_KZ\Person::individualIdentificationNumber
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless comment

@fzaninotto fzaninotto merged commit 9722bbe into fzaninotto:master Nov 29, 2016
@fzaninotto
Copy link
Owner

Thanks!

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