Should be fairly simple to add some basic integration tests to ensure the module behaves as properly.

- Create a new normal user on the site
- Add three fields in user profiles
- Add two user profile as pcp user field and show pcp block on page
- Check page have pcp block
- Initial pcp block must show 0% Complete
- Fill one pcp user field
- Check pcp block must show 50% Complete
- Fill one non pcp user field
- Check pcp block still show 50% Complete
- Fill another pcp user field
- Check pcp block must show 100% Complete

CommentFileSizeAuthor
#1 pcp-add_simpletest_integration-2478539-1.patch4.59 KBperennial.sky
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

perennial.sky’s picture

perennial.sky CreditAttribution: perennial.sky at Innoraft commented
Assigned: perennial.sky » Unassigned
Status: Active » Needs review
FileSize
pcp-add_simpletest_integration-2478539-1.patch4.59 KB

Add a patch having simple test for pcp module

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented

Good idea! Some tests would be really helpful.
I've enabled automated testing for the project. Let's see what the bot thinks.

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Needs review » Active
BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Active » Needs review
perennial.sky’s picture

perennial.sky CreditAttribution: perennial.sky at Innoraft commented

Bot tested this patch and passed all its test,
https://qa.drupal.org/pifr/test/1035373

I think we should commit this patch.

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Needs review » Reviewed & tested by the community

Agreed!

perennial.sky’s picture

perennial.sky CreditAttribution: perennial.sky at Innoraft commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.