[libcalamares] Refactor test to be data-driven

- this test is going to get a lot more cases, so prepare for that

src/libcalamares/locale/Tests.cpp

@@ -52,6 +52,7 @@ private Q_SLOTS:
     void testComplexZones();
     void testTZLookup();
     void testTZIterator();
+    void testLocationLookup_data();
     void testLocationLookup();
 };
 
@@ -391,23 +392,30 @@ LocaleTests::testTZIterator()
 }
 
 void
-LocaleTests::testLocationLookup()
+LocaleTests::testLocationLookup_data()
 {
-    const CalamaresUtils::Locale::ZonesModel zones;
+    QTest::addColumn< double >( "latitude" );
+    QTest::addColumn< double >( "longitude" );
+    QTest::addColumn< QString >( "name" );
 
-    const auto* zone = zones.find( 50.0, 0.0 );
+    QTest::newRow( "London" ) << 50.0 << 0.0 << QString( "London" );
-    QVERIFY( zone );
+    QTest::newRow( "Tarawa E" ) << 0.0 << 179.0 << QString( "Tarawa" );
-    QCOMPARE( zone->zone(), QStringLiteral( "London" ) );
+    QTest::newRow( "Tarawa W" ) << 0.0 << -179.0 << QString( "Tarawa" );
 
+}
 
-    // Tarawa is close to "the other side of the world" from London
+void
-    zone = zones.find( 0.0, 179.0 );
+LocaleTests::testLocationLookup()
-    QVERIFY( zone );
+{
-    QCOMPARE( zone->zone(), QStringLiteral( "Tarawa" ) );
+    const CalamaresUtils::Locale::ZonesModel zones;
+
+    QFETCH( double, latitude );
+    QFETCH( double, longitude );
+    QFETCH( QString, name );
 
-    zone = zones.find( 0.0, -179.0 );
+    const auto* zone = zones.find( latitude, longitude );
     QVERIFY( zone );
-    QCOMPARE( zone->zone(), QStringLiteral( "Tarawa" ) );
+    QCOMPARE( zone->zone(), name );
 }