From 819669c97c61a04722558295c512dad38974e8d9 Mon Sep 17 00:00:00 2001 From: Amitabh Tiwari Date: Tue, 3 Nov 2020 14:07:23 +0530 Subject: [PATCH] BAEL-4399: Added changes as per review comment --- .../nosuchfielderror/Dependent.java | 4 ++-- .../nosuchfielderror/NoSuchFieldError.java | 21 ---------------- .../NoSuchFieldErrorExample.java | 8 +++++++ .../NoSuchFieldErrorExampleUnitTest.java | 15 ++++++++++++ .../NoSuchFieldErrorTest.java | 24 ------------------- 5 files changed, 25 insertions(+), 47 deletions(-) delete mode 100644 core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldError.java create mode 100644 core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExample.java create mode 100644 core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExampleUnitTest.java delete mode 100644 core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorTest.java diff --git a/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/Dependent.java b/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/Dependent.java index e475b7dcf9..d838eafdd9 100644 --- a/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/Dependent.java +++ b/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/Dependent.java @@ -1,6 +1,6 @@ package com.baeldung.exceptions.nosuchfielderror; public class Dependent { - // This needed to be commented post compilation of NoSuchFielDError and Compile - public static String message = "Hello Baeldung!!"; + // This needed to be commented post compilation of NoSuchFielDError and Compile + public static String message = "Hello Baeldung!!"; } diff --git a/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldError.java b/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldError.java deleted file mode 100644 index 4939adac31..0000000000 --- a/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldError.java +++ /dev/null @@ -1,21 +0,0 @@ -package com.baeldung.exceptions.nosuchfielderror; - -import java.lang.reflect.Field; - -public class NoSuchFieldError { - // String message = "Hello Baeldung!!!"; - - public static String getMessage() throws ClassNotFoundException, NoSuchFieldException, SecurityException, - IllegalArgumentException, IllegalAccessException { - NoSuchFieldError clss = new NoSuchFieldError(); - Class aClass = Class.forName(clss.getClass().getName()); - Field field = aClass.getDeclaredField("message"); - field.setAccessible(true); - String msgStr = (String) field.get(clss); - return msgStr; - } - - public static String getDependentMessage() { - return Dependent.message; - } -} diff --git a/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExample.java b/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExample.java new file mode 100644 index 0000000000..727f10581d --- /dev/null +++ b/core-java-modules/core-java-exceptions-3/src/main/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExample.java @@ -0,0 +1,8 @@ +package com.baeldung.exceptions.nosuchfielderror; + +public class NoSuchFieldErrorExample { + + public static String getDependentMessage() { + return Dependent.message; + } +} diff --git a/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExampleUnitTest.java b/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExampleUnitTest.java new file mode 100644 index 0000000000..8f2dc45aa0 --- /dev/null +++ b/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorExampleUnitTest.java @@ -0,0 +1,15 @@ +package com.baeldung.exceptions.nosuchfielderror; + +import static org.junit.Assert.assertTrue; + +import org.junit.Test; + +public class NoSuchFieldErrorExampleUnitTest { + + @Test + public void whenDependentMessage_returnMessage() { + String dependentMessage = NoSuchFieldErrorExample.getDependentMessage(); + assertTrue("Hello Baeldung".equals(dependentMessage)); + } + +} diff --git a/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorTest.java b/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorTest.java deleted file mode 100644 index dd7947b207..0000000000 --- a/core-java-modules/core-java-exceptions-3/src/test/java/com/baeldung/exceptions/nosuchfielderror/NoSuchFieldErrorTest.java +++ /dev/null @@ -1,24 +0,0 @@ -package com.baeldung.exceptions.nosuchfielderror; - -import static org.junit.Assert.assertTrue; - -import org.junit.Test; - -public class NoSuchFieldErrorTest { - - @Test(expected = NoSuchFieldException.class) - public void whenFieldNotFound_thenThrowNoSuchFieldException() throws Exception { - NoSuchFieldError.getMessage(); - } - - @Test(expected = NoSuchFieldException.class) - public void whenDependentFieldNotFound_thenThrowNoSuchFieldException() { - NoSuchFieldError.getDependentMessage(); - } - - @Test - public void whenDependentFieldNotFound_returnMessage() { - String dependentMessage = NoSuchFieldError.getDependentMessage(); - assertTrue("Hello Baeldung".equals(dependentMessage)); - } -}