Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Ruleset form sections and resource imports #358

Merged
merged 7 commits into from
Feb 11, 2025

Conversation

Jack-Golding
Copy link
Contributor

Presently we have all components used within the ruleset form being stored at the top level of the directory, without them needing to be shared with multiple components, this PR moves the field components to within their respective field array components to improve readability

Additionally sorted imports of the mock refdata resource

Renamed export in resource index from mockRefdata to just refdata, in cases where we need to mock variables, we can rename the import at the test level, rather than the export as these resources will not always need to be mocked
Fixed imports for ruleset view test
Shifted around the file structure for the enumeration field array compoenents so that they can be more easily readable within the repository
Mored chronology field intop the chronology field array file
Moved combination field into combination field array file
moved omission field to omission field array component
Moved issue publication field compoenent to its associated field array
Copy link

License CLA Stuck? (Developer should make sure that it is really stuck before clicking)

Copy link

Jest Unit Test Results

  1 files  ±0   55 suites  ±0   1m 52s ⏱️ -10s
392 tests ±0  392 ✅ ±0  0 💤 ±0  0 ❌ ±0 
425 runs  ±0  425 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit fb95d0e. ± Comparison against base commit 1315a44.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
7.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@Jack-Golding Jack-Golding merged commit 9dbd79a into master Feb 11, 2025
18 of 19 checks passed
@Jack-Golding Jack-Golding deleted the test-resource-import-tweaks branch February 11, 2025 14:55
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.

1 participant