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

Fixed handful of test namespaces #15230

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

marcusmoore
Copy link
Collaborator

Description

This PR fixes a handful of incorrect namespaces that I noticed warnings for when running composer install.

Type of change

  • Chore(ish)

Copy link

what-the-diff bot commented Aug 6, 2024

PR Summary

  • Organized Test Files Better
    The labels of some groups of test files were updated to better represent what they're testing. This is equivalent to renaming folders in a filing cabinet to accurately represent what's inside. Thus, enhancing maintainability and understandability of the test suite.
    • The tests for working with "Assets" were moved from the 'Feature/Assets/Ui' group to the 'Tests/Features/Assets/Ui'.
    • The tests for "Checkouts" moved from the 'Feature/Checkins/Ui' group to the 'Tests/Feature/Checkouts/Ui'.
    • The tests for creating and viewing "Licenses" moved from 'Feature/Consumables/Ui' to 'Tests/Features/Licenses/Ui'.
    • The tests for handling locations in the API moved from 'Feature/Consumables/Api' to 'Tests/Feature/Locations/Api'.
      These changes contribute to making the test files easier to find and work with in the future.

@snipe snipe merged commit da33277 into snipe:develop Aug 6, 2024
9 checks passed
@marcusmoore marcusmoore deleted the fixes/test-namespaces branch August 6, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants