Skip to content

Conversation

@Ben-Hodgkiss
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Please replace this line with a brief description of the changes made.

Related Tickets & Documents

  • Ticket Link
  • Related Issue #
  • Closes #

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

[optional] Are there any dependencies on other PRs or Work?

eveleighoj and others added 2 commits January 22, 2026 13:08
* edit how lookups are created andd used to remove complication from the phases

* remove unneccessary step from lookup phase

* lock black version due to breaking change

* add init files so tests are imiported correctly

* fix transform tests  by correcting lookups

* remove old test location

* fix async_request_pocessor tests
* Refill Logs on Fetch

* Tests

* add test structure

* test for overwitting

* wip

* remove random file

* run black on new files

* fix test and mock responses

---------

Co-authored-by: eveleighoj <35256612+eveleighoj@users.noreply.github.com>
@Ben-Hodgkiss Ben-Hodgkiss merged commit 9ff6758 into new-dataset-mandatory-fields Jan 27, 2026
9 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.

4 participants