Issue jcabi#174 - Inserting correct version/buildnumber here and...

pecko · Sep 28, 2015 · c60618a · c60618a
1 parent 1bacbe4
commit c60618a
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 4 deletions.
diff --git a/src/main/java/com/jcabi/aspects/aj/ImmutabilityChecker.java b/src/main/java/com/jcabi/aspects/aj/ImmutabilityChecker.java
@@ -62,10 +62,6 @@ public final class ImmutabilityChecker {
      *
      * <p>Try NOT to change the signature of this method, in order to keep
      * it backward compatible.
-     * @todo #167:30min Inserting correct version/buildnumber
-     *  here and in other instances where Version.CURRENT is
-     *  used (not only in this class, but in every class that
-     *  uses Version.CURRENT) should be covered by a test.
      * @param point Joint point
      */
     @After("initialization((@com.jcabi.aspects.Immutable *).new(..))")

diff --git a/src/test/java/com/jcabi/aspects/ImmutableTest.java b/src/test/java/com/jcabi/aspects/ImmutableTest.java
@@ -30,6 +30,8 @@
 package com.jcabi.aspects;
 
 import java.util.regex.Pattern;
+import org.hamcrest.MatcherAssert;
+import org.hamcrest.Matchers;
 import org.junit.Test;
 
 /**
@@ -88,6 +90,35 @@ public void catchesTypesMutableByClassInheritance() {
         new MutableByInheritance();
     }
 
+    /**
+     * Test Version.
+     */
+    @Test
+    public void testVersion() {
+        try {
+            new Mutable();
+        } catch (final IllegalStateException ex) {
+            MatcherAssert.assertThat(
+                ex.getMessage(),
+                Matchers.not(
+                    Matchers.containsString(
+                        "${projectVersion}"
+                    )
+                )
+            );
+            MatcherAssert.assertThat(
+                ex.getMessage(),
+                Matchers.not(
+                    Matchers.containsString(
+                        "${buildNumber}"
+                    )
+                )
+            );
+            return;
+        }
+        MatcherAssert.assertThat("should not happen.", false);
+    }
+
     /**
      * Supposedly immutable class.
      */