Make City.is_good a property

process_subways.py

@@ -336,7 +336,7 @@ if __name__ == '__main__':
             c.error("Validation logic error: {}".format(str(e)))
         else:
             c.validate()
-            if c.is_good():
+            if c.is_good:
                 good_cities.append(c)
 
     logging.info('Finding transfer stations')

processors/gtfs.py

@@ -171,7 +171,7 @@ def process(cities, transfers, filename, cache_path):
     )
 
     all_stops = {}  # stop (stop area center or station) el_id -> stop data
-    good_cities = [c for c in cities if c.is_good()]
+    good_cities = [c for c in cities if c.is_good]
 
     def add_stop_gtfs(route_stop):
         """Add stop to all_stops.

processors/mapsme.py

@@ -87,7 +87,7 @@ class MapsmeCache:
         # One stoparea may participate in routes of different cities
         self.stop_cities = defaultdict(set)  # stoparea id -> city names
         self.city_dict = {c.name: c for c in cities}
-        self.good_city_names = {c.name for c in cities if c.is_good()}
+        self.good_city_names = {c.name for c in cities if c.is_good}
 
     def _is_cached_city_usable(self, city):
         """Check if cached stations still exist in osm data and
@@ -118,7 +118,7 @@ class MapsmeCache:
         """Put stops and networks for bad cities into containers
         passed as arguments."""
         for city in self.city_dict.values():
-            if not city.is_good() and city.name in self.cache:
+            if not city.is_good and city.name in self.cache:
                 city_cached_data = self.cache[city.name]
                 if self._is_cached_city_usable(city):
                     stops.update(city_cached_data['stops'])
@@ -212,7 +212,7 @@ def process(cities, transfers, filename, cache_path):
     stop_areas = {}  # stoparea el_id -> StopArea instance
     stops = {}  # stoparea el_id -> stop jsonified data
     networks = []
-    good_cities = [c for c in cities if c.is_good()]
+    good_cities = [c for c in cities if c.is_good]
     platform_nodes = {}
     cache.provide_stops_and_networks(stops, networks)
 

subway_io.py

@@ -282,7 +282,7 @@ def write_recovery_data(path, current_data, cities):
 
     data = current_data
     for city in cities:
-        if city.is_good():
+        if city.is_good:
             data[city.name] = make_city_recovery_data(city)
 
     try:

subway_structure.py

@@ -1477,6 +1477,7 @@ class RouteMaster:
 
 class City:
     def __init__(self, city_data, overground=False):
+        self.validate_called = False
         self.errors = []
         self.warnings = []
         self.notices = []
@@ -1734,7 +1735,12 @@ class City:
     def __iter__(self):
         return iter(self.routes.values())
 
+    @property
     def is_good(self):
+        assert self.validate_called, (
+            "You mustn't refer to City.is_good property before calling "
+            "the City.validate() method."
+        )
         return len(self.errors) == 0
 
     def get_validation_result(self):
@@ -1972,6 +1978,8 @@ class City:
             )
             self.notice('More than one network: {}'.format(n_str))
 
+        self.validate_called = True
+
 
 def find_transfers(elements, cities):
     transfers = []

tests/test_build_tracks.py

@@ -43,7 +43,7 @@ class TestOneRouteTracks(unittest.TestCase):
         city.extract_routes()
         city.validate()
 
-        self.assertTrue(city.is_good())
+        self.assertTrue(city.is_good)
 
         route_master = list(city.routes.values())[0]
         variants = route_master.routes