diff --git a/src/main/java/com/jcabi/aspects/aj/ImmutabilityChecker.java b/src/main/java/com/jcabi/aspects/aj/ImmutabilityChecker.java index 947a891c..34c68d62 100644 --- 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 { * *

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 index 303b9fd4..0d0dd4bc 100644 --- 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. */