Skip to content

Conversation

@alexjustesen
Copy link
Owner

@alexjustesen alexjustesen commented Mar 19, 2025

📃 Description

This PR lays the groundwork for a testing suite and updates the CI pipeline to require tests to pass. Much more work is needed to consider the current functionality "covered". With a test suite in place I can start looking at #1942.

🪵 Changelog

➕ Added

  • getting-started middleware to handle redirects instead of in the controller
  • test existing routes
  • test creating a Result model factory

✏️ Changed

  • reorganized home and getting-started routes into the PagesController
  • tuned phpunit to use in-memory sqlite to run tests

🔧 Fixed

  • /getting-started not rendering when PUBLIC_DASHBOARD=false which is the default behavior

📷 Screenshots

image

@alexjustesen alexjustesen added the feature New feature or request label Mar 19, 2025
@alexjustesen alexjustesen self-assigned this Mar 19, 2025
@alexjustesen alexjustesen merged commit 8c0758c into main Mar 19, 2025
2 checks passed
@alexjustesen alexjustesen deleted the tests-and-ci-pipeline branch March 19, 2025 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants