ISO19111 PROJ string parser: discard parameters not recognized by PROJ

3 files changed, 22 insertions, 23 deletions

diff --git a/test/unit/test_factory.cpp b/test/unit/test_factory.cpp
index 027c0188..a6f4ecf8 100644
--- a/test/unit/test_factory.cpp
+++ b/test/unit/test_factory.cpp
@@ -2324,8 +2324,7 @@ TEST_F(FactoryWithTmpDatabase, custom_projected_crs) {
         EXPECT_EQ(crs->identifiers().size(), 1);
         EXPECT_EQ(crs->derivingConversion()->targetCRS().get(), crs.get());
         EXPECT_EQ(crs->exportToPROJString(PROJStringFormatter::create().get()),
-                  "+proj=mbt_s +unused_flag +datum=WGS84 +units=m +no_defs "
-                  "+type=crs");
+                  "+proj=mbt_s +datum=WGS84 +units=m +no_defs +type=crs");
         EXPECT_TRUE(crs->canonicalBoundCRS() == nullptr);
     }
     {
@@ -2334,8 +2333,7 @@ TEST_F(FactoryWithTmpDatabase, custom_projected_crs) {
         EXPECT_EQ(crs->identifiers().size(), 1);
         EXPECT_EQ(crs->derivingConversion()->targetCRS().get(), crs.get());
         EXPECT_EQ(crs->exportToPROJString(PROJStringFormatter::create().get()),
-                  "+proj=mbt_s +unused_flag +datum=WGS84 +units=m +no_defs "
-                  "+type=crs");
+                  "+proj=mbt_s +datum=WGS84 +units=m +no_defs +type=crs");
         EXPECT_TRUE(crs->canonicalBoundCRS() != nullptr);
     }
 
diff --git a/test/unit/test_io.cpp b/test/unit/test_io.cpp
index 9bd3d70f..e4886553 100644
--- a/test/unit/test_io.cpp
+++ b/test/unit/test_io.cpp
@@ -7472,7 +7472,7 @@ TEST(io, projparse_cea_ellipsoidal) {
 
 TEST(io, projparse_geos_sweep_x) {
     auto obj = PROJStringParser().createFromPROJString(
-        "+proj=geos +sweep=x +type=crs");
+        "+proj=geos +sweep=x +h=1 +type=crs");
     auto crs = nn_dynamic_pointer_cast<ProjectedCRS>(obj);
     ASSERT_TRUE(crs != nullptr);
     WKTFormatterNNPtr f(WKTFormatter::create());
@@ -7488,7 +7488,8 @@ TEST(io, projparse_geos_sweep_x) {
 // ---------------------------------------------------------------------------
 
 TEST(io, projparse_geos_sweep_y) {
-    auto obj = PROJStringParser().createFromPROJString("+proj=geos +type=crs");
+    auto obj =
+        PROJStringParser().createFromPROJString("+proj=geos +h=1 +type=crs");
     auto crs = nn_dynamic_pointer_cast<ProjectedCRS>(obj);
     ASSERT_TRUE(crs != nullptr);
     WKTFormatterNNPtr f(WKTFormatter::create());
@@ -8100,7 +8101,7 @@ TEST(io, projparse_projected_vunits) {
 TEST(io, projparse_projected_unknown) {
     auto obj = PROJStringParser().createFromPROJString(
         "+proj=mbt_s +unused_flag +lat_0=45 +lon_0=0 +k=1 +x_0=10 +y_0=0 "
-        "+type=crs");
+        "+datum=WGS84 +type=crs");
     auto crs = nn_dynamic_pointer_cast<ProjectedCRS>(obj);
     ASSERT_TRUE(crs != nullptr);
 
@@ -8111,8 +8112,8 @@ TEST(io, projparse_projected_unknown) {
         crs->exportToWKT(f.get());
         auto wkt = f->toString();
         EXPECT_TRUE(
-            wkt.find("CONVERSION[\"unknown\",METHOD[\"PROJ mbt_s "
-                     "unused_flag\"],PARAMETER[\"lat_0\",45,ANGLEUNIT["
+            wkt.find("CONVERSION[\"unknown\",METHOD[\"PROJ mbt_s\"],"
+                     "PARAMETER[\"lat_0\",45,ANGLEUNIT["
                      "\"degree\",0.0174532925199433]],PARAMETER[\"lon_0\","
                      "0,ANGLEUNIT[\"degree\",0.0174532925199433]],"
                      "PARAMETER[\"k\",1,SCALEUNIT[\"unity\",1]],PARAMETER["
@@ -8130,8 +8131,8 @@ TEST(io, projparse_projected_unknown) {
         "\"9122\"]]],"
         "PROJECTION[\"custom_proj4\"],UNIT[\"metre\",1,AUTHORITY[\"EPSG\","
         "\"9001\"]],AXIS[\"Easting\",EAST],AXIS[\"Northing\",NORTH],EXTENSION["
-        "\"PROJ4\",\"+proj=mbt_s +datum=WGS84 +unused_flag +lat_0=45 "
-        "+lon_0=0 +k=1 +x_0=10 +y_0=0 +wktext +no_defs\"]]";
+        "\"PROJ4\",\"+proj=mbt_s +lat_0=45 "
+        "+lon_0=0 +k=1 +x_0=10 +y_0=0 +datum=WGS84\"]]";
 
     {
         WKTFormatterNNPtr f(
@@ -8144,8 +8145,8 @@ TEST(io, projparse_projected_unknown) {
     }
 
     EXPECT_EQ(crs->exportToPROJString(PROJStringFormatter::create().get()),
-              "+proj=mbt_s +unused_flag +lat_0=45 +lon_0=0 +k=1 +x_0=10 "
-              "+y_0=0 +datum=WGS84 +units=m +no_defs +type=crs");
+              "+proj=mbt_s +lat_0=45 +lon_0=0 +k=1 +x_0=10 "
+              "+y_0=0 +datum=WGS84 +type=crs");
 
     {
         auto obj2 = WKTParser().createFromWKT(expected_wkt1);
@@ -8158,8 +8159,8 @@ TEST(io, projparse_projected_unknown) {
         crs2->exportToWKT(f.get());
         auto wkt = f->toString();
         EXPECT_TRUE(
-            wkt.find("CONVERSION[\"unknown\",METHOD[\"PROJ mbt_s "
-                     "unused_flag\"],PARAMETER[\"lat_0\",45,ANGLEUNIT["
+            wkt.find("CONVERSION[\"unknown\",METHOD[\"PROJ mbt_s\"],"
+                     "PARAMETER[\"lat_0\",45,ANGLEUNIT["
                      "\"degree\",0.0174532925199433]],PARAMETER[\"lon_0\","
                      "0,ANGLEUNIT[\"degree\",0.0174532925199433]],"
                      "PARAMETER[\"k\",1,SCALEUNIT[\"unity\",1]],PARAMETER["
@@ -8274,7 +8275,7 @@ TEST(io, projparse_longlat_wktext) {
         crs->exportToPROJString(
             PROJStringFormatter::create(PROJStringFormatter::Convention::PROJ_4)
                 .get()),
-        input);
+        "+proj=longlat +datum=WGS84 +no_defs +type=crs");
 }
 
 // ---------------------------------------------------------------------------
@@ -8288,7 +8289,7 @@ TEST(io, projparse_geocent_wktext) {
         crs->exportToPROJString(
             PROJStringFormatter::create(PROJStringFormatter::Convention::PROJ_4)
                 .get()),
-        input);
+        "+proj=geocent +datum=WGS84 +units=m +no_defs +type=crs");
 }
 
 // ---------------------------------------------------------------------------
@@ -8302,7 +8303,8 @@ TEST(io, projparse_projected_wktext) {
         crs->exportToPROJString(
             PROJStringFormatter::create(PROJStringFormatter::Convention::PROJ_4)
                 .get()),
-        input);
+        "+proj=merc +lon_0=0 +k=1 +x_0=0 +y_0=0 +datum=WGS84 +units=m +no_defs "
+        "+type=crs");
 }
 
 // ---------------------------------------------------------------------------
@@ -8312,8 +8314,7 @@ TEST(io, projparse_ob_tran_longlat) {
         "+type=crs +proj=pipeline +step +proj=axisswap +order=2,1 +step "
         "+proj=unitconvert +xy_in=deg +xy_out=rad +step +proj=ob_tran "
         "+o_proj=longlat +o_lat_p=52 +o_lon_p=-30 +lon_0=-25 +ellps=WGS84 "
-        "+step +proj=unitconvert +xy_in=rad +xy_out=deg +step "
-        "+proj=axisswap +order=2,1");
+        "+step +proj=axisswap +order=2,1");
     auto obj = PROJStringParser().createFromPROJString(input);
     auto crs = nn_dynamic_pointer_cast<DerivedGeographicCRS>(obj);
     ASSERT_TRUE(crs != nullptr);
diff --git a/test/unit/test_operation.cpp b/test/unit/test_operation.cpp
index ed31f155..01c6a38f 100644
--- a/test/unit/test_operation.cpp
+++ b/test/unit/test_operation.cpp
@@ -6323,8 +6323,8 @@ TEST(operation, createOperation_fallback_to_proj4_strings) {
 TEST(operation, createOperation_on_crs_with_bound_crs_and_wktext) {
     auto objSrc = PROJStringParser().createFromPROJString(
         "+proj=utm +zone=55 +south +ellps=GRS80 +towgs84=0,0,0,0,0,0,0 "
-        "+units=m +no_defs +nadgrids=GDA94_GDA2020_conformal.gsb "
-        "+type=crs");
+        "+units=m +no_defs +nadgrids=@GDA94_GDA2020_conformal.gsb +ignored1 "
+        "+ignored2=val +wktext +type=crs");
     auto src = nn_dynamic_pointer_cast<CRS>(objSrc);
     ASSERT_TRUE(src != nullptr);
 
@@ -6340,7 +6340,7 @@ TEST(operation, createOperation_on_crs_with_bound_crs_and_wktext) {
     EXPECT_EQ(op->exportToPROJString(PROJStringFormatter::create().get()),
               "+proj=pipeline +step +inv +proj=utm +zone=55 +south "
               "+ellps=GRS80 +step +proj=hgridshift "
-              "+grids=GDA94_GDA2020_conformal.gsb +step +proj=utm +zone=55 "
+              "+grids=@GDA94_GDA2020_conformal.gsb +step +proj=utm +zone=55 "
               "+south +ellps=GRS80");
 }