Skip to content

Conversation

@sujata-m
Copy link
Contributor

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"
  }
]

sujata-m and others added 2 commits January 15, 2026 18:39
- 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)
[0.3.3] Fixed Task-2817 & Issue-2844
@sujata-m sujata-m requested review from MashB and susrisha January 16, 2026 08:23
@sujata-m sujata-m merged commit b08fc0b into main Jan 16, 2026
7 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