fix: handle case when entire input is padding characters

TonyBogdanov · Dec 10, 2021 · c8d7329 · c8d7329
1 parent 7bc24ac
commit c8d7329
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/.phpunit.cache/test-results b/.phpunit.cache/test-results
diff --git a/src/Converter.php b/src/Converter.php
@@ -75,8 +75,11 @@ public static function convert( string $input, string $inputAlphabet, string $ou
         $ob = count( $os );
 
         $carry = preg_match( '/^' . preg_quote( $is[0], '/' ) . '+/', $input, $match ) ? strlen( $match[0] ) : 0;
-        $value = '0';
+        if ( $carry === strlen( $input ) ) {
+            return str_repeat( $os[0], $carry );
+        }
 
+        $value = '0';
         foreach ( array_reverse( str_split( substr( $input, $carry ) ) ) as $index => $symbol ) {
             $value = bcadd( $value, bcmul( $isf[ $symbol ], bcpow( $ib, $index ) ) );
         }

diff --git a/tests/ConvertTest.php b/tests/ConvertTest.php
@@ -60,6 +60,15 @@ public function testEmptyInput(): void {
         $this->assertSame( '', Converter::convert( '', 'ab', 'ab' ) );
     }
 
+    /**
+     * @return void
+     * @throws InvalidAlphabetException
+     * @throws InvalidInputException
+     */
+    public function testOnlyPaddingInput(): void {
+        $this->assertSame( 'qqq', Converter::convert( 'aaa', 'ab', 'qw' ) );
+    }
+
     /**
      * @return void
      * @throws InvalidAlphabetException