checked the stop access column not just values (#2143)

Author: qcdyx

main/src/main/java/org/mobilitydata/gtfsvalidator/validator/StopAccessValidator.java

@@ -25,7 +25,7 @@
 public class StopAccessValidator extends SingleEntityValidator<GtfsStop> {
   @Override
   public void validate(GtfsStop entity, NoticeContainer noticeContainer) {
-    if (entity.stopAccess() == null) {
+    if (entity.stopAccess() == null || !entity.hasStopAccess()) {
       return;
     }
     if (entity.locationType() == GtfsLocationType.STOP) {

main/src/test/java/org/mobilitydata/gtfsvalidator/validator/StopAccessValidatorTest.java

@@ -61,4 +61,30 @@ public void nonStopLocationWithStopAccess_generatesIncorrectLocationNotice() {
                 GtfsStopAccess.NOT_ACCESSIBLE_VIA_PATHWAYS,
                 GtfsLocationType.STATION));
   }
+
+  @Test
+  public void stopLocationWithoutParentStation_withoutStopAccess_generatesNoNotice() {
+    GtfsStop stop =
+        new GtfsStop.Builder()
+            .setCsvRowNumber(10)
+            .setStopId("S3")
+            .setLocationType(GtfsLocationType.STOP)
+            .setStopName("Stop 3")
+            .build();
+
+    assertThat(generateNotices(stop)).isEmpty();
+  }
+
+  @Test
+  public void nonStopLocation_withoutStopAccess_generatesNoNotice() {
+    GtfsStop stop =
+        new GtfsStop.Builder()
+            .setCsvRowNumber(11)
+            .setStopId("S4")
+            .setLocationType(GtfsLocationType.STATION)
+            .setStopName("Stop 4")
+            .build();
+
+    assertThat(generateNotices(stop)).isEmpty();
+  }
 }