Skip to content

Conversation

@sujata-m
Copy link
Contributor

@sujata-m sujata-m commented Jan 15, 2026

Dev Board Ticket

Changes

  • Updated validation to surface uploaded ZIP names (not temp extract dirs) when structure errors occur.
  • Clarified unsupported GeoJSON message to list allowed patterns as *.{edges, nodes, points, lines, zones, polygons}.geojson; synced error template and tests.
  • Added unit coverage for additional-properties hint and structure error filenames; refreshed unit_test_overview.md.
  • Bumped package version to 0.3.3 and documented changes in CHANGELOG.md.

Testing

  • Added 4 new test cases (all in tests/unit_tests/test_helpers.py and tests/unit_tests/test_osw_validation_extras.py).

  • Task-2817 - would return the error in below format

[
  {
    "filename":"graph.points.geojson",
    "feature_index":1,
    "error_message":[
      "Additional properties are not allowed ('color' was unexpected). If you want to carry this tag, change it to ext:color"
    ]
  }
]
  • Task-2844 - would return the error in below format
[
  {
    "filename":"valid 2.zip",
    "feature_index":"None",
    "error_message":"Unsupported .geojson files present: wa.microsoft.graph.OSW.geojson. Allowed file names are *.{edges, nodes, points, lines, zones, polygons}.geojson"
  }
]

- Updated validation to surface uploaded ZIP names (not temp extract dirs) when structure errors occur.
- Clarified unsupported GeoJSON message to list allowed patterns as *.{edges, nodes, points, lines, zones, polygons}.geojson; synced error template and tests.
- Added unit coverage for additional-properties hint and structure error filenames; refreshed unit_test_overview.md.
- Bumped package version to 0.3.3 and documented changes in CHANGELOG.md.
- Fixed [Task-2817](https://dev.azure.com/TDEI-UW/TDEI/_workitems/edit/2817)
- Fixed [Issue-2844](https://dev.azure.com/TDEI-UW/TDEI/_workitems/edit/2844)
@sujata-m sujata-m requested review from MashB and susrisha January 15, 2026 13:16
@sujata-m sujata-m merged commit 0f43241 into develop Jan 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants