Skip to content

Commit

Permalink
dev-php/pecl-http: Fix tests so they don't SKIP everything
Browse files Browse the repository at this point in the history
The basic testing command ignores the requirements of other
extensions needed to run.  This adds the required ones.

Package-Manager: Portage-2.3.38, Repoman-2.3.9
  • Loading branch information
Brian Evans committed May 18, 2018
1 parent f90db98 commit d676eb3
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 0 deletions.
13 changes: 13 additions & 0 deletions dev-php/pecl-http/pecl-http-2.6.0-r2.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,19 @@ src_install() {
fi
}

src_test() {
for slot in $(php_get_slots); do
php_init_slot_env "${slot}"
ln -s "${EXT_DIR}/propro.so" "modules/propro.so" || die
ln -s "${EXT_DIR}/raphf.so" "modules/raphf.so" || die
sed -i \
's/PHP_TEST_SHARED_EXTENSIONS)/PHP_TEST_SHARED_EXTENSIONS) -d extension=propro.so -d extension=raphf.so/' \
Makefile || die
NO_INTERACTION="yes" emake test
rm modules/propro.so modules/raphf.so || die
done
}

pkg_postinst() {
ewarn "This API has drastically changed and is not compatible with the 1.x syntax."
ewarn "Please review the documentation and update your code."
Expand Down
17 changes: 17 additions & 0 deletions dev-php/pecl-http/pecl-http-3.1.0-r2.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,23 @@ src_install() {
fi
}

src_test() {
# Cannot use eclass function due to required modules
# All tests SKIP otherwise
for slot in $(php_get_slots); do
php_init_slot_env "${slot}"
# Link in required modules for testing
ln -s "${EXT_DIR}/propro.so" "modules/propro.so" || die
ln -s "${EXT_DIR}/raphf.so" "modules/raphf.so" || die
sed -i \
's/PHP_TEST_SHARED_EXTENSIONS)/PHP_TEST_SHARED_EXTENSIONS) -d extension=propro.so -d extension=raphf.so/' \
Makefile || die
NO_INTERACTION="yes" emake test
# Clean up testing links
rm modules/propro.so modules/raphf.so || die
done
}

pkg_postinst() {
ewarn "This API has drastically changed and is not compatible with the 1.x syntax."
ewarn "Please review the documentation and update your code."
Expand Down

0 comments on commit d676eb3

Please sign in to comment.