zervo/seb-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge remote-tracking branch 'origin/dev-1.5' into development

Browse source
This commit is contained in:
anhefti 2023-06-15 15:09:56 +02:00
commit 6573b1438b
3 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/main/java/ch/ethz/seb/sebserver/gui/service/page/impl/PageAction.java
View file

@ -185,6 +185,9 @@ public final class PageAction {
} catch (final PageMessageException pme) {
PageAction.this.pageContext.publishPageMessage(pme);
return;
} catch (final Exception e) {
this.pageContext.notifyUnexpectedError(e);
return;
}
} else {
callback.accept(exec());

6
src/main/java/ch/ethz/seb/sebserver/gui/service/remote/webservice/api/RestCall.java
View file

@ -180,7 +180,7 @@ public abstract class RestCall<T> {
e,
"NO RESPONSE AVAILABLE",
String.valueOf(builder)));
return Result.ofError(e);
return Result.ofError(restCallError);
}
}
@ -216,6 +216,10 @@ public abstract class RestCall<T> {
} else {
restCallError.errors.add(APIMessage.ErrorMessage.GENERIC.of(responseEntity.getBody()));
}
} catch (final Exception e) {
final String body = responseEntity.getBody();
log.error("Failed to parse rest response error message: {}", body);
throw e;
}
log.debug(

4
src/test/java/ch/ethz/seb/sebserver/gui/integration/UseCasesIntegrationTest.java
View file

@ -1578,7 +1578,7 @@ public class UseCasesIntegrationTest extends GuiIntegrationTest {
fail("Exception expected here");
} catch (final Exception e) {
assertEquals("argument \"content\" is null", e.getMessage());
assertEquals("Unexpected error while rest call", e.getMessage());
}
// test follow-up integrity violation
@ -1596,7 +1596,7 @@ public class UseCasesIntegrationTest extends GuiIntegrationTest {
fail("Exception expected here");
} catch (final Exception e) {
assertEquals("argument \"content\" is null", e.getMessage());
assertEquals("Unexpected error while rest call", e.getMessage());
}
final ConfigurationTableValues newTableValue = new ConfigurationTableValues(