Redesigned Bytes.put(index, byte) logic

Author: Artem Labazin

CHANGELOG.md

@@ -12,6 +12,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Add more tests.
 - Add `JavaDoc`.
 
+## [1.3.2](https://github.com/appulse-projects/utils-java/releases/tag/1.3.2) - 2018-02-14
+
+### Changed
+
+- Redesigned `Bytes.put(index, byte)` logic, now it moves `limit`.
+
 ## [1.3.1](https://github.com/appulse-projects/utils-java/releases/tag/1.3.1) - 2018-02-09
 
 Minor release with unsigned numbers support.

pom.xml

@@ -24,7 +24,7 @@ limitations under the License.
 
   <groupId>io.appulse</groupId>
   <artifactId>utils-java</artifactId>
-  <version>1.3.1</version>
+  <version>1.3.2</version>
   <packaging>jar</packaging>
 
   <properties>
@@ -62,7 +62,7 @@ limitations under the License.
     <url>https://github.com/appulse-projects/utils-java</url>
     <connection>scm:git:https://github.com/appulse-projects/utils-java.git</connection>
     <developerConnection>scm:git:https://github.com/appulse-projects/utils-java.git</developerConnection>
-    <tag>1.3.1</tag>
+    <tag>1.3.2</tag>
   </scm>
 
   <distributionManagement>

src/main/java/io/appulse/utils/Bytes.java

@@ -86,7 +86,11 @@ public Bytes put (byte value) {
   }
 
   public Bytes put (int index, byte value) {
-    buffer.put(index, value);
+    if (index < limit) {
+      buffer.put(index, value);
+    } else {
+      put(value);
+    }
     return this;
   }
 
@@ -100,7 +104,7 @@ public Bytes put (@NonNull byte[] bytes) {
   public Bytes put (int index, @NonNull byte[] bytes) {
     checkCapacity(index, bytes.length);
     IntStream.range(index, index + bytes.length).forEach(it -> {
-      buffer.put(it, bytes[it - index]);
+      put(it, bytes[it - index]);
     });
     return this;
   }

src/test/java/io/appulse/utils/BytesTest.java

@@ -103,8 +103,15 @@ public void limit () {
     bytes.put4B(4);
     assertThat(bytes.limit()).isEqualTo(4);
 
-    Bytes wrapped = Bytes.wrap(new byte[] { 1 });
-    assertThat(wrapped.limit()).isEqualTo(1);
+    Bytes wrapped = Bytes.wrap(new byte[] { 1, 0 });
+    assertThat(wrapped.array()).isEqualTo(new byte[] { 1, 0 });
+    assertThat(wrapped.limit()).isEqualTo(2);
+
+    wrapped.position(wrapped.limit());
+    wrapped.put(1, new byte[] { 2, 3 });
+    assertThat(wrapped.array()).isEqualTo(new byte[] { 1, 2, 3 });
+    assertThat(wrapped.limit()).isEqualTo(3);
+    assertThat(wrapped.position()).isEqualTo(3);
   }
 
   @Test