Change type of statuses returned by getResponseCodes (#26)

* Change type of statuses returned by getResponseCodes

* Update CHANGELOG

* Remove allowance to failure on 7.2 on Travis

Author: slavcodev

.travis.yml

@@ -9,8 +9,6 @@ matrix:
       env:
         - EXECUTE_COVERAGE=true
     - php: 7.2
-  allow_failures:
-    - php: 7.2
 
 notifications:
   email: false

CHANGELOG.md

@@ -3,6 +3,18 @@ All notable changes to this project will be documented in this file
 using the [Keep a CHANGELOG](http://keepachangelog.com/) principles.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [0.4.0] 2018-12-01
+
+### Fixed
+
+- Change type of statuses returned by getResponseCodes from string to int 
+
+## [0.3.1] 2018-02-17
+
+### Changed
+
+- Add phpunit 6.0 fallback version
+
 ## [0.3.0] 2018-02-17
 
 ### Changed

src/PhpUnit/Asserts.php

@@ -145,7 +145,7 @@ final protected static function assertMethodAllowed(Spec $spec, $template, $meth
     final protected static function assertResponseDefined(Spec $spec, $template, $method, $status, $msg = '')
     {
         Assert::assertTrue(
-            in_array((string) $status, $spec->getResponseCodes($template, strtolower($method)), true),
+            in_array((int) $status, $spec->getResponseCodes($template, strtolower($method)), true),
             $msg ?: "Operation \"{$method} {$template}\" does not support response code \"{$status}\""
         );
     }

src/Schema.php

@@ -217,13 +217,16 @@ public function getRequestContentTypes($template, $method)
      * @param string $template
      * @param string $method
      *
-     * @return string[]
+     * @return int[]
      */
     public function getResponseCodes($template, $method)
     {
-        return array_filter(
-            array_keys((array) $this->fetch($this->schema, 'paths', $template, $method, 'responses')),
-            'is_numeric'
+        return array_map(
+            'intval',
+            array_filter(
+                array_keys((array) $this->fetch($this->schema, 'paths', $template, $method, 'responses')),
+                'is_numeric'
+            )
         );
     }
 

tests/SchemaTest.php

@@ -158,7 +158,7 @@ public function howToUseSwagger(Schema $spec)
         $this->assertSame(['application/json'], $produces);
 
         $statuses = $spec->getResponseCodes($template, 'get');
-        $this->assertSame(['200'], $statuses);
+        $this->assertSame([200], $statuses);
 
         $responseHeaders = $spec->getResponseHeaderSchemas($template, 'get', 200);
         $this->assertTrue(isset($responseHeaders['Content-Type']));