IOException not catched

Signed-off-by: Florian Dupuy <florian.dupuy@rte-france.com>

iidm/iidm-geodata/src/main/java/com/powsybl/iidm/geodata/odre/OdreGeoDataAdder.java

@@ -28,19 +28,17 @@ public class OdreGeoDataAdder {
  protected OdreGeoDataAdder() {
  }
 
- public static void fillNetworkSubstationsGeoDataFromFile(Network network, Path path, OdreConfig odreConfig) {
+ public static void fillNetworkSubstationsGeoDataFromFile(Network network, Path path, OdreConfig odreConfig) throws IOException {
  fillNetworkSubstationsGeoData(network, OdreGeoDataCsvLoader.getSubstationsGeoData(path, odreConfig));
  }
 
  public static void fillNetworkGeoDataFromFiles(Network network, Path aerialLinesFilePath,
- Path undergroundLinesFilePath, Path substationPath, OdreConfig odreConfig) {
+ Path undergroundLinesFilePath, Path substationPath, OdreConfig odreConfig) throws IOException {
  try (Reader substationReader = InputUtils.toReader(substationPath)) {
  Map<String, SubstationGeoData> substations = GeographicDataParser.parseSubstations(substationReader, odreConfig);
  fillNetworkSubstationsGeoData(network, substations.values());
  fillNetworkLinesGeoData(network,
  OdreGeoDataCsvLoader.getLinesGeoData(aerialLinesFilePath, undergroundLinesFilePath, substations, odreConfig));
- } catch (IOException e) {
- throw new UncheckedIOException(e);
  }
  }
 }

iidm/iidm-geodata/src/main/java/com/powsybl/iidm/geodata/odre/OdreGeoDataAdderPostProcessor.java

@@ -16,6 +16,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.Map;
@@ -76,7 +77,7 @@ public String getName() {
  }
 
  @Override
- public void process(Network network, ComputationManager computationManager) {
+ public void process(Network network, ComputationManager computationManager) throws IOException {
  if (Files.exists(substationsFilePath)) {
  boolean aerialLinesPresent = Files.exists(aerialLinesFilePath);
  boolean undergroundLinesPresent = Files.exists(undergroundLinesFilePath);

iidm/iidm-geodata/src/main/java/com/powsybl/iidm/geodata/odre/OdreGeoDataCsvLoader.java

@@ -25,34 +25,26 @@ public class OdreGeoDataCsvLoader {
  protected OdreGeoDataCsvLoader() {
  }
 
- public static Collection<SubstationGeoData> getSubstationsGeoData(Path path, OdreConfig odreConfig) {
+ public static Collection<SubstationGeoData> getSubstationsGeoData(Path path, OdreConfig odreConfig) throws IOException {
  try (Reader reader = InputUtils.toReader(path)) {
  return GeographicDataParser.parseSubstations(reader, odreConfig).values();
- } catch (IOException e) {
- throw new UncheckedIOException(e);
  }
  }
 
  public static Collection<LineGeoData> getLinesGeoData(Path aerialLinesFilePath, Path undergroundLinesFilePath,
- Path substationPath, OdreConfig odreConfig) {
+ Path substationPath, OdreConfig odreConfig) throws IOException {
  try (Reader substationReader = InputUtils.toReader(substationPath)) {
  return getLinesGeoData(aerialLinesFilePath, undergroundLinesFilePath, GeographicDataParser.parseSubstations(substationReader, odreConfig), odreConfig);
- } catch (IOException e) {
- throw new UncheckedIOException(e);
  }
  }
 
  public static Collection<LineGeoData> getLinesGeoData(Path aerialLinesFilePath, Path undergroundLinesFilePath,
- Map<String, SubstationGeoData> substations, OdreConfig odreConfig) {
-
+ Map<String, SubstationGeoData> substations, OdreConfig odreConfig) throws IOException {
  try (Reader aerialReader = InputUtils.toReader(aerialLinesFilePath);
  Reader undergroundReader = InputUtils.toReader(undergroundLinesFilePath)) {
 
  Map<String, LineGeoData> lineGeoDataMap = GeographicDataParser.parseLines(aerialReader, undergroundReader, substations, odreConfig);
  return lineGeoDataMap.values();
-
- } catch (IOException e) {
- throw new UncheckedIOException(e);
  }
  }
 }

iidm/iidm-geodata/src/test/java/com/powsybl/iidm/geodata/odre/OdreGeoDataAdderTest.java

@@ -18,6 +18,7 @@
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.MethodSource;
 
+import java.io.IOException;
 import java.net.URISyntaxException;
 import java.nio.file.Path;
 import java.nio.file.Paths;
@@ -40,7 +41,7 @@ void setUp() {
 
  @ParameterizedTest
  @MethodSource("provideTestArguments")
- void addSubstationsGeoDataFromFile(String descr, String directory, OdreConfig config) throws URISyntaxException {
+ void addSubstationsGeoDataFromFile(String descr, String directory, OdreConfig config) throws URISyntaxException, IOException {
  Path substationsPath = Paths.get(getClass()
  .getClassLoader().getResource(directory + "substations.csv").toURI());
 
@@ -61,7 +62,7 @@ void addSubstationsGeoDataFromFile(String descr, String directory, OdreConfig co
 
  @ParameterizedTest
  @MethodSource("provideTestArguments")
- void addLinesGeoDataFromFile(String descr, String directory, OdreConfig config) throws URISyntaxException {
+ void addLinesGeoDataFromFile(String descr, String directory, OdreConfig config) throws URISyntaxException, IOException {
  Path substationsPath = Paths.get(getClass()
  .getClassLoader().getResource(directory + "substations.csv").toURI());
  Path aerialLinesFile = Paths.get(getClass()

iidm/iidm-geodata/src/test/java/com/powsybl/iidm/geodata/odre/OdreGeoDataCsvLoaderTest.java

@@ -10,6 +10,7 @@
 import com.powsybl.iidm.geodata.elements.LineGeoData;
 import org.junit.jupiter.api.Test;
 
+import java.io.IOException;
 import java.net.URISyntaxException;
 import java.nio.file.Path;
 import java.nio.file.Paths;
@@ -23,7 +24,7 @@
 class OdreGeoDataCsvLoaderTest {
 
  @Test
- void validSubstationsLineParsing() throws URISyntaxException {
+ void validSubstationsLineParsing() throws URISyntaxException, IOException {
  Path substationsPath = Paths.get(getClass()
  .getClassLoader().getResource("valid-line-name/substations.csv").toURI());
  Path undergroundLinesPath = Paths.get(getClass()