tests for WsdlDownloader and cs fixes

src/BeSimple/SoapClient/WsdlDownloader.php

@@ -54,26 +54,28 @@ class WsdlDownloader
 
     /**
      * Constructor.
+     *
+     * @param array $options
      */
-    public function __construct( $options )
+    public function __construct(array $options = array())
     {
         // get current WSDL caching config
-        $this->cacheEnabled = (bool)ini_get( 'soap.wsdl_cache_enabled' );
+        $this->cacheEnabled = (bool)ini_get('soap.wsdl_cache_enabled');
-        if ( $this->cacheEnabled === true
+        if ($this->cacheEnabled === true
-            && isset( $options['cache_wsdl'] )
+            && isset($options['cache_wsdl'])
-            && $options['cache_wsdl'] === WSDL_CACHE_NONE
+            && $options['cache_wsdl'] === WSDL_CACHE_NONE) {
-        )
-        {
             $this->cacheEnabled = false;
         }
-        $this->cacheDir = ini_get( 'soap.wsdl_cache_dir' );
+        $this->cacheDir = ini_get('soap.wsdl_cache_dir');
-        if ( !is_dir( $this->cacheDir ) )
+        if (!is_dir($this->cacheDir)) {
-        {
             $this->cacheDir = sys_get_temp_dir();
         }
-        $this->cacheDir = rtrim( $this->cacheDir, '/\\' );
+        $this->cacheDir = rtrim($this->cacheDir, '/\\');
-        $this->cacheTtl = ini_get( 'soap.wsdl_cache_ttl' );
+        $this->cacheTtl = ini_get('soap.wsdl_cache_ttl');
         $this->options = $options;
+        if (!isset($this->options['resolve_xsd_includes'])) {
+            $this->options['resolve_xsd_includes'] = true;
+        }
     }
 
     /**
@@ -82,62 +84,44 @@ class WsdlDownloader
      * @param string $wsdl
      * @return string
      */
-    public function download( $wsdl )
+    public function download($wsdl)
     {
         // download and cache remote WSDL files or local ones where we want to
         // resolve remote XSD includes
-        $isRemoteFile = $this->isRemoteFile( $wsdl );
+        $isRemoteFile = $this->isRemoteFile($wsdl);
-        if ( $isRemoteFile === true || $this->options['resolve_xsd_includes'] === true )
+        if ($isRemoteFile === true || $this->options['resolve_xsd_includes'] === true) {
-        {
+            $cacheFile = $this->cacheDir . DIRECTORY_SEPARATOR . 'wsdl_' . md5($wsdl) . '.cache';
-            $cacheFile = $this->cacheDir . DIRECTORY_SEPARATOR . 'wsdl_' . md5( $wsdl ) . '.cache';
+            if ($this->cacheEnabled === false
-            if ( $this->cacheEnabled === false
+                || !file_exists($cacheFile)
-                || !file_exists( $cacheFile )
+                || (filemtime($cacheFile) + $this->cacheTtl) < time()) {
-                || ( filemtime( $cacheFile ) + $this->cacheTtl ) < time()
+                if ($isRemoteFile === true) {
-            )
-            {
-                if ( $isRemoteFile === true )
-                {
                     // new curl object for request
-                    $curl = new Curl( $this->options );
+                    $curl = new Curl($this->options);
                     // execute request
-                    $responseSuccessfull = $curl->exec( $wsdl );
+                    $responseSuccessfull = $curl->exec($wsdl);
                     // get content
-                    if ( $responseSuccessfull === true )
+                    if ($responseSuccessfull === true) {
-                    {
                         $response = $curl->getResponseBody();
-                        if ( $this->options['resolve_xsd_includes'] === true )
+                        if ($this->options['resolve_xsd_includes'] === true) {
-                        {
+                            $this->resolveXsdIncludes($response, $cacheFile, $wsdl);
-                            $this->resolveXsdIncludes( $response, $cacheFile, $wsdl );
+                        } else {
-                        }
+                            file_put_contents($cacheFile, $response);
-                        else
-                        {
-                            file_put_contents( $cacheFile, $response );
                         }
+                    } else {
+                        throw new \ErrorException("SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'");
                     }
-                    else
+                } elseif (file_exists($wsdl)) {
-                    {
+                    $response = file_get_contents($wsdl);
-                        throw new \ErrorException( "SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'" );
+                    $this->resolveXsdIncludes($response, $cacheFile);
-                    }
+                } else {
-                }
+                    throw new \ErrorException("SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'");
-                elseif ( file_exists( $wsdl ) )
-                {
-                    $response = file_get_contents( $wsdl );
-                    $this->resolveXsdIncludes( $response, $cacheFile );
-                }
-                else
-                {
-                    throw new \ErrorException( "SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'" );
                 }
             }
             return $cacheFile;
-        }
+        } elseif (file_exists($wsdl)) {
-        elseif ( file_exists( $wsdl ) )
+            return realpath($wsdl);
-        {
+        } else {
-            return realpath( $wsdl );
+            throw new \ErrorException("SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'");
-        }
-        else
-        {
-            throw new \ErrorException( "SOAP-ERROR: Parsing WSDL: Couldn't load from '" . $wsdl ."'" );
         }
     }
 
@@ -147,14 +131,12 @@ class WsdlDownloader
      * @param string $file
      * @return boolean
      */
-    private function isRemoteFile( $file )
+    private function isRemoteFile($file)
     {
         $isRemoteFile = false;
         // @parse_url to suppress E_WARNING for invalid urls
-        if ( ( $url = @parse_url( $file ) ) !== false )
+        if (($url = @parse_url($file)) !== false) {
-        {
+            if (isset($url['scheme']) && substr($url['scheme'], 0, 4) == 'http') {
-            if ( isset( $url['scheme'] ) && substr( $url['scheme'], 0, 4 ) == 'http' )
-            {
                 $isRemoteFile = true;
             }
         }
@@ -169,33 +151,28 @@ class WsdlDownloader
      * @param unknown_type $parentIsRemote
      * @return string
      */
-    private function resolveXsdIncludes( $xml, $cacheFile, $parentFile = null )
+    private function resolveXsdIncludes($xml, $cacheFile, $parentFile = null)
     {
         $doc = new \DOMDocument();
-        $doc->loadXML( $xml );
+        $doc->loadXML($xml);
-        $xpath = new \DOMXPath( $doc );
+        $xpath = new \DOMXPath($doc);
-        $xpath->registerNamespace( Helper::PFX_XML_SCHEMA, Helper::NS_XML_SCHEMA );
+        $xpath->registerNamespace(Helper::PFX_XML_SCHEMA, Helper::NS_XML_SCHEMA);
         $query = './/' . Helper::PFX_XML_SCHEMA . ':include';
-        $nodes = $xpath->query( $query );
+        $nodes = $xpath->query($query);
-        if ( $nodes->length > 0 )
+        if ($nodes->length > 0) {
-        {
+            foreach ($nodes as $node) {
-            foreach ( $nodes as $node )
+                $schemaLocation = $node->getAttribute('schemaLocation');
-            {
+                if ($this->isRemoteFile($schemaLocation)) {
-                $schemaLocation = $node->getAttribute( 'schemaLocation' );
+                    $schemaLocation = $this->download($schemaLocation);
-                if ( $this->isRemoteFile( $schemaLocation ) )
+                    $node->setAttribute('schemaLocation', $schemaLocation);
-                {
+                } elseif (!is_null($parentFile)) {
-                    $schemaLocation = $this->download( $schemaLocation );
+                    $schemaLocation = $this->resolveRelativePathInUrl($parentFile, $schemaLocation);
-                    $node->setAttribute( 'schemaLocation', $schemaLocation );
+                    $schemaLocation = $this->download($schemaLocation);
-                }
+                    $node->setAttribute('schemaLocation', $schemaLocation);
-                elseif ( !is_null( $parentFile ) )
-                {
-                    $schemaLocation = $this->resolveRelativePathInUrl( $parentFile, $schemaLocation );
-                    $schemaLocation = $this->download( $schemaLocation );
-                    $node->setAttribute( 'schemaLocation', $schemaLocation );
                 }
             }
         }
-        $doc->save( $cacheFile );
+        $doc->save($cacheFile);
     }
 
     /**
@@ -205,36 +182,44 @@ class WsdlDownloader
      * @param string $relative
      * @return string
      */
-    private function resolveRelativePathInUrl( $base, $relative )
+    private function resolveRelativePathInUrl($base, $relative)
     {
-        $urlParts = parse_url( $base );
+        $urlParts = parse_url($base);
         // combine base path with relative path
-        if ( strrpos( '/', $urlParts['path'] ) === ( strlen( $urlParts['path'] ) - 1 ) )
+        if (isset($urlParts['path']) && strpos($relative, '/') === 0) {
-        {
+            // $relative is absolute path from domain (starts with /)
-            $path = trim( $urlParts['path'] . $relative );
+            $path = $relative;
-        }
+        } elseif (isset($urlParts['path']) && strrpos($urlParts['path'], '/') === (strlen($urlParts['path']) )) {
-        else
+            // base path is directory
-        {
+            $path = $urlParts['path'] . $relative;
-            $path = trim( dirname( $urlParts['path'] ) . '/' . $relative );
+        } elseif (isset($urlParts['path'])) {
+            // strip filename from base path
+            $path = substr($urlParts['path'], 0, strrpos($urlParts['path'], '/')) . '/' . $relative;
+        } else {
+            // no base path
+            $path = '/' . $relative;
         }
         // foo/./bar ==> foo/bar
-        $path = preg_replace( '~/\./~', '/', $path );
+        $path = preg_replace('~/\./~', '/', $path);
         // remove double slashes
-        $path = preg_replace( '~/+~', '/', $path );
+        $path = preg_replace('~/+~', '/', $path);
         // split path by '/'
-        $parts = explode( '/', $path );
+        $parts = explode('/', $path);
         // resolve /../
-        foreach ( $parts as $key => $part )
+        foreach ($parts as $key => $part) {
-        {
+            if ($part == "..") {
-            if ( $part == ".." )
+                $keyToDelete = $key-1;
-            {
+                while ($keyToDelete > 0) {
-                if ( $key-1 >= 0 )
+                    if (isset($parts[$keyToDelete])) {
-                {
+                        unset($parts[$keyToDelete]);
-                    unset( $parts[$key-1] );
+                        break;
+                    } else {
+                        $keyToDelete--;
+                    }
                 }
-                unset( $parts[$key] );
+                unset($parts[$key]);
             }
         }
-        return $urlParts['scheme'] . '://' . $urlParts['host'] . implode( '/', $parts );
+        return $urlParts['scheme'] . '://' . $urlParts['host'] . implode('/', $parts);
     }
 }

tests/BeSimple/Tests/SoapClient/Fixtures/type_include.xsd

@@ -0,0 +1,15 @@
+<?xml version="1.0"?>
+<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema">
+
+<xs:element name="note">
+  <xs:complexType>
+    <xs:sequence>
+      <xs:element name="to" type="xs:string"/>
+      <xs:element name="from" type="xs:string"/>
+      <xs:element name="heading" type="xs:string"/>
+      <xs:element name="body" type="xs:string"/>
+    </xs:sequence>
+  </xs:complexType>
+</xs:element>
+
+</xs:schema>

tests/BeSimple/Tests/SoapClient/Fixtures/xsdinclude/xsdinctest_absolute.xml

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<wsdl:definitions xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:xs="http://www.w3.org/2001/XMLSchema">
+    <wsdl:documentation>xsdinctest</wsdl:documentation>
+    <wsdl:types>
+        <xs:schema attributeFormDefault="qualified" elementFormDefault="qualified" targetNamespace="http://test.sample">
+            <xs:include schemaLocation="http://localhost:8000/type_include.xsd"/>
+        </xs:schema>
+    </wsdl:types>
+</wsdl:definitions>

tests/BeSimple/Tests/SoapClient/Fixtures/xsdinclude/xsdinctest_relative.xml

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<wsdl:definitions xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:xs="http://www.w3.org/2001/XMLSchema">
+    <wsdl:documentation>xsdinctest</wsdl:documentation>
+    <wsdl:types>
+        <xs:schema attributeFormDefault="qualified" elementFormDefault="qualified" targetNamespace="http://test.sample">
+            <xs:include schemaLocation="../type_include.xsd"/>
+        </xs:schema>
+    </wsdl:types>
+</wsdl:definitions>

tests/BeSimple/Tests/SoapClient/WsdlDownloaderTest.php

@@ -0,0 +1,205 @@
+<?php
+/**
+ * This file is part of the BeSimpleSoapClient.
+ *
+ * (c) Christian Kerl <christian-kerl@web.de>
+ * (c) Francis Besset <francis.besset@gmail.com>
+ * (c) Andreas Schamberger <mail@andreass.net>
+ *
+ * This source file is subject to the MIT license that is bundled
+ * with this source code in the file LICENSE.
+ *
+ * @link https://github.com/BeSimple/BeSimpleSoapClient
+ */
+
+namespace BeSimple\SoapClient;
+
+use BeSimple\SoapClient\WsdlDownloader;
+
+class WsdlDownloaderTest extends \PHPUnit_Framework_TestCase
+{
+    protected $webserverProcessId;
+
+    protected function startPhpWebserver()
+    {
+        if ('Windows' == substr(php_uname('s'), 0, 7 )) {
+            $powershellCommand = "\$app = start-process php.exe -ArgumentList '-S localhost:8000 -t ".__DIR__.DIRECTORY_SEPARATOR."Fixtures' -WindowStyle 'Hidden' -passthru; Echo \$app.Id;";
+            $shellCommand = 'powershell -command "& {'.$powershellCommand.'}"';
+        } else {
+            $shellCommand = "nohup php -S localhost:8000 -t ".__DIR__.DIRECTORY_SEPARATOR."Fixtures &";
+        }
+        $output = array();
+        exec($shellCommand, $output);
+        $this->webserverProcessId = $output[0]; // pid is in first element
+    }
+
+    protected function stopPhpWebserver()
+    {
+        if (!is_null($this->webserverProcessId)) {
+            if ('Windows' == substr(php_uname('s'), 0, 7 )) {
+                exec('TASKKILL /F /PID ' . $this->webserverProcessId);
+            } else {
+                exec('kill ' . $this->webserverProcessId);
+            }
+            $this->webserverProcessId = null;
+        }
+    }
+
+    public function testDownload()
+    {
+        $this->startPhpWebserver();
+
+        $options = array(
+            'resolve_xsd_includes' => true,
+        );
+        $wd = new WsdlDownloader($options);
+
+        $cacheDir = ini_get('soap.wsdl_cache_dir');
+        if (!is_dir($cacheDir)) {
+            $cacheDir = sys_get_temp_dir();
+            $cacheDirForRegExp = preg_quote( $cacheDir );
+        }
+
+        $tests = array(
+            'localWithAbsolutePath' => array(
+                 'source' => __DIR__.DIRECTORY_SEPARATOR.'Fixtures/xsdinclude/xsdinctest_absolute.xml',
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+            'localWithRelativePath' => array(
+                 'source' => __DIR__.DIRECTORY_SEPARATOR.'Fixtures/xsdinclude/xsdinctest_relative.xml',
+                 'assertRegExp' => '~.*\.\./type_include\.xsd.*~',
+            ),
+            'remoteWithAbsolutePath' => array(
+                 'source' => 'http://localhost:8000/xsdinclude/xsdinctest_absolute.xml',
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+            'remoteWithAbsolutePath' => array(
+                 'source' => 'http://localhost:8000/xsdinclude/xsdinctest_relative.xml',
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+        );
+
+        foreach ($tests as $name => $values) {
+            $cacheFileName = $wd->download($values['source']);
+            $result = file_get_contents($cacheFileName);
+            $this->assertRegExp($values['assertRegExp'],$result,$name);
+            unlink($cacheFileName);
+        }
+
+        $this->stopPhpWebserver();
+    }
+
+    public function testIsRemoteFile()
+    {
+        $wd = new WsdlDownloader();
+
+        $class = new \ReflectionClass($wd);
+        $method = $class->getMethod('isRemoteFile');
+        $method->setAccessible(true);
+
+        $this->assertEquals(true, $method->invoke($wd, 'http://www.php.net/'));
+        $this->assertEquals(true, $method->invoke($wd, 'http://localhost/'));
+        $this->assertEquals(true, $method->invoke($wd, 'http://mylocaldomain/'));
+        $this->assertEquals(true, $method->invoke($wd, 'http://www.php.net/dir/test.html'));
+        $this->assertEquals(true, $method->invoke($wd, 'http://localhost/dir/test.html'));
+        $this->assertEquals(true, $method->invoke($wd, 'http://mylocaldomain/dir/test.html'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://www.php.net/'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://localhost/'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://mylocaldomain/'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://www.php.net/dir/test.html'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://localhost/dir/test.html'));
+        $this->assertEquals(true, $method->invoke($wd, 'https://mylocaldomain/dir/test.html'));
+        $this->assertEquals(false, $method->invoke($wd, 'c:/dir/test.html'));
+        $this->assertEquals(false, $method->invoke($wd, '/dir/test.html'));
+        $this->assertEquals(false, $method->invoke($wd, '../dir/test.html'));
+    }
+
+    public function testResolveXsdIncludes()
+    {
+        $this->startPhpWebserver();
+
+        $options = array(
+            'resolve_xsd_includes' => true,
+        );
+        $wd = new WsdlDownloader($options);
+
+        $class = new \ReflectionClass($wd);
+        $method = $class->getMethod('resolveXsdIncludes');
+        $method->setAccessible(true);
+
+        $cacheDir = ini_get('soap.wsdl_cache_dir');
+        if (!is_dir($cacheDir)) {
+            $cacheDir = sys_get_temp_dir();
+            $cacheDirForRegExp = preg_quote( $cacheDir );
+        }
+
+        $remoteUrlAbsolute = 'http://localhost:8000/xsdinclude/xsdinctest_absolute.xml';
+        $remoteUrlRelative = 'http://localhost:8000/xsdinclude/xsdinctest_relative.xml';
+        $tests = array(
+            'localWithAbsolutePath' => array(
+                 'source' => __DIR__.DIRECTORY_SEPARATOR.'Fixtures/xsdinclude/xsdinctest_absolute.xml',
+                 'cacheFile' => $cacheDir.'/cache_local_absolute.xml',
+                 'remoteParentUrl' => null,
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+            'localWithRelativePath' => array(
+                 'source' => __DIR__.DIRECTORY_SEPARATOR.'Fixtures/xsdinclude/xsdinctest_relative.xml',
+                 'cacheFile' => $cacheDir.'/cache_local_relative.xml',
+                 'remoteParentUrl' => null,
+                 'assertRegExp' => '~.*\.\./type_include\.xsd.*~',
+            ),
+            'remoteWithAbsolutePath' => array(
+                 'source' => $remoteUrlAbsolute,
+                 'cacheFile' => $cacheDir.'/cache_remote_absolute.xml',
+                 'remoteParentUrl' => $remoteUrlAbsolute,
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+            'remoteWithAbsolutePath' => array(
+                 'source' => $remoteUrlRelative,
+                 'cacheFile' => $cacheDir.'/cache_remote_relative.xml',
+                 'remoteParentUrl' => $remoteUrlRelative,
+                 'assertRegExp' => '~.*'.$cacheDirForRegExp.'\\\wsdl_.*\.cache.*~',
+            ),
+        );
+
+        foreach ($tests as $name => $values) {
+            $wsdl = file_get_contents( $values['source'] );
+            $method->invoke($wd, $wsdl, $values['cacheFile'],$values['remoteParentUrl']);
+            $result = file_get_contents($values['cacheFile']);
+            $this->assertRegExp($values['assertRegExp'],$result,$name);
+            unlink($values['cacheFile']);
+        }
+
+        $this->stopPhpWebserver();
+    }
+
+    public function testResolveRelativePathInUrl()
+    {
+        $wd = new WsdlDownloader();
+
+        $class = new \ReflectionClass($wd);
+        $method = $class->getMethod('resolveRelativePathInUrl');
+        $method->setAccessible(true);
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub', '/test'));
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/', '/test'));
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost', './test'));
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/', './test'));
+
+        $this->assertEquals('http://localhost/sub/test', $method->invoke($wd, 'http://localhost/sub/sub', './test'));
+        $this->assertEquals('http://localhost/sub/sub/test', $method->invoke($wd, 'http://localhost/sub/sub/', './test'));
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/sub', '../test'));
+        $this->assertEquals('http://localhost/sub/test', $method->invoke($wd, 'http://localhost/sub/sub/', '../test'));
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/sub/sub', '../../test'));
+        $this->assertEquals('http://localhost/sub/test', $method->invoke($wd, 'http://localhost/sub/sub/sub/', '../../test'));
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/sub/sub/sub', '../../../test'));
+        $this->assertEquals('http://localhost/sub/test', $method->invoke($wd, 'http://localhost/sub/sub/sub/sub/', '../../../test'));
+
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/sub/sub', '../../../test'));
+        $this->assertEquals('http://localhost/test', $method->invoke($wd, 'http://localhost/sub/sub/sub/', '../../../test'));
+    }
+}