Fix build: use reflection for protected ConfigKey.valueOf() in test

- NASBackupProviderTest.overrideConfigValue() was calling
  configKey.valueOf() directly, but valueOf() has protected access.
  Use Method.setAccessible(true) to invoke it via reflection instead.
- nasbackup.sh: rewrite mount check as `if mount ...; then` to
  prevent set -e from bypassing custom error handling.

Author: jmsperu

plugins/backup/nas/src/test/java/org/apache/cloudstack/backup/NASBackupProviderTest.java

@@ -357,14 +357,19 @@ public void testGetVMHypervisorHostFallbackToZoneWideKVMHost() {
 
     private void overrideConfigValue(final ConfigKey configKey, final Object value) {
         try {
+            // Use reflection to invoke protected valueOf()
+            java.lang.reflect.Method valueOfMethod = ConfigKey.class.getDeclaredMethod("valueOf", String.class);
+            valueOfMethod.setAccessible(true);
+            Object typedValue = value != null ? valueOfMethod.invoke(configKey, String.valueOf(value)) : null;
+
             // Set _value for value() calls
             Field f = ConfigKey.class.getDeclaredField("_value");
             f.setAccessible(true);
-            f.set(configKey, value != null ? configKey.valueOf((String) value) : null);
+            f.set(configKey, typedValue);
 
             // Also set _defaultValue via Spring's ReflectionTestUtils (handles final fields)
             ReflectionTestUtils.setField(configKey, "_defaultValue", String.valueOf(value));
-        } catch (IllegalAccessException | NoSuchFieldException e) {
+        } catch (Exception e) {
             Assert.fail(e.getMessage());
         }
     }

scripts/vm/hypervisor/kvm/nasbackup.sh

@@ -314,8 +314,7 @@ mount_operation() {
   if [ ${NAS_TYPE} == "cifs" ]; then
     MOUNT_OPTS="${MOUNT_OPTS},nobrl"
   fi
-  mount -t ${NAS_TYPE} ${NAS_ADDRESS} ${mount_point} $([[ ! -z "${MOUNT_OPTS}" ]] && echo -o ${MOUNT_OPTS}) >> "$logFile" 2>&1
-  if [ $? -eq 0 ]; then
+  if mount -t ${NAS_TYPE} ${NAS_ADDRESS} ${mount_point} $([[ ! -z "${MOUNT_OPTS}" ]] && echo -o ${MOUNT_OPTS}) >> "$logFile" 2>&1; then
       log -ne "Successfully mounted ${NAS_TYPE} store"
   else
       echo "Failed to mount ${NAS_TYPE} store"