dfe-analytical-services / dfe-analytical-services/dfe-published-data-qa

Add handling for blank metadata files

Open
#139 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
R
Stars
4
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Currently you can crash the app if you upload a blank file.

We should make it so this fails more gracefully for users, and we should add test data and a UI test for this to make sure that it runs the initial data validation checks but fails.

Contributor guide

Open the contributing guide

Research direction

Start by locating the upload handling and initial data validation entry points, then review the existing UI test structure. Add test data for a blank metadata file and a UI test that confirms validation runs before the upload fails gracefully. Done means the app no longer crashes and the test covers the expected failure.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
testing
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.